Closed
Bug 1430023
Opened 4 years ago
Closed 4 years ago
Get rid of the remaining code bits about localstore.rdf and rdf in general
Categories
(Core Graveyard :: RDF, enhancement, P3)
Tracking
(firefox62 fixed)
RESOLVED
FIXED
mozilla62
Tracking | Status | |
---|---|---|
firefox62 | --- | fixed |
People
(Reporter: mak, Assigned: mak)
References
Details
Attachments
(2 files)
localstore.rdf is gone from some time (Firefox 34), so it sounds like we could get rid of: 1. the migration code from xulstore and the code supporting it. 2. if we didn't yet also remove the file from the profile with a migration 3. There's also an animals.rdf file that I'm not sure what's about (probably testing) and can be removed. 4. I also think (to be verified) the rdf/datasource folder may not be needed anymore? 5. NS_APP_LOCALSTORE_50_FILE, NS_LOCALSTORE_UNSAFE_FILE 6. https://searchfox.org/mozilla-central/search?q=localstore.rdf&path=
Assignee | ||
Updated•4 years ago
|
Component: General → RDF
Product: Firefox → Core
Comment 1•4 years ago
|
||
bug 1425356 stopped using animals.rdf, but didn't remove the file. Not sure about the other stuff. Disclaimer, I'm not sure if this bug is well-served by being in the RDF component, as nobody works here. All lights are out. That said, if you know someone that intends to work on this, I don't mind this bug being here. I'd also appreciate help with the triage. RDF bugs are generally P5 or WONTFIX, unless there's an out-of-RDF-land priority.
Assignee | ||
Comment 2•4 years ago
|
||
(In reply to Axel Hecht from comment #1) > bug 1425356 stopped using animals.rdf, but didn't remove the file. good to know! > Disclaimer, I'm not sure if this bug is well-served by being in the RDF > component, as nobody works here. It blocks the kill rdf bug, that is all we need, people interested in that project will look at its dependencies, there's interest in removals of old tech like XBL/XUL/RDF in general, so I'm not worried. (In reply to Axel Hecht from comment #1) > I'd also appreciate help with the triage. RDF bugs are generally P5 or > WONTFIX, unless there's an out-of-RDF-land priority. I agree, removal bugs should likely be P3 (backlog), anything affecting TB or comm-central could be P5. Anything else wontfix.
Priority: -- → P3
Updated•4 years ago
|
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → WORKSFORME
Assignee | ||
Comment 3•4 years ago
|
||
Hi Kris, why was this resolved when some of the code points in comment 0 still exist?
Flags: needinfo?(kmaglione+bmo)
Comment 4•4 years ago
|
||
I deleted every reference to RDF I could find in bug 833098 and nothing seemed to break. Please reopen if I missed something, but perhaps remove it as a blocker.
Flags: needinfo?(kmaglione+bmo)
Assignee | ||
Comment 5•4 years ago
|
||
Ah yeah, not a blocker, it's mostly about unused leftovers referring to localstore.rdf (though I don't know if comm-central may use any of those). Anyway, I'm reopening and inverting the dependencies.
Assignee | ||
Updated•4 years ago
|
Status: REOPENED → NEW
Assignee | ||
Updated•4 years ago
|
Summary: Get rid of the remaining bits about localstore.rdf → Get rid of the remaining code bits about localstore.rdf and rdf in general
Assignee | ||
Updated•4 years ago
|
Assignee: nobody → mak77
Status: NEW → ASSIGNED
Assignee | ||
Comment 6•4 years ago
|
||
Attachment #8974321 -
Flags: review?(jorgk)
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment 9•4 years ago
|
||
Comment on attachment 8974321 [details] [diff] [review] Stop using LStoreS in comm-central Thanks, this looks good.
Attachment #8974321 -
Flags: review?(jorgk) → review+
Comment 10•4 years ago
|
||
Pushed by mak77@bonardo.net: https://hg.mozilla.org/comm-central/rev/c0b88c3cd2cd Stop using LStoreS in comm-central. r=jorgk
Status: ASSIGNED → RESOLVED
Closed: 4 years ago → 4 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•4 years ago
|
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment 11•4 years ago
|
||
mozreview-review |
Comment on attachment 8974322 [details] Bug 1430023 - Get rid of some remaining localstore.rdf code. https://reviewboard.mozilla.org/r/242650/#review250084
Attachment #8974322 -
Flags: review?(dtownsend) → review+
Comment 12•4 years ago
|
||
Pushed by mak77@bonardo.net: https://hg.mozilla.org/integration/autoland/rev/ba79a9e472c0 Get rid of some remaining localstore.rdf code. r=mossop
Comment 13•4 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/ba79a9e472c0
Status: REOPENED → RESOLVED
Closed: 4 years ago → 4 years ago
status-firefox62:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla62
Updated•4 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•