Get rid of the remaining code bits about localstore.rdf and rdf in general

RESOLVED FIXED in Firefox 62

Status

enhancement
P3
normal
RESOLVED FIXED
2 years ago
Last year

People

(Reporter: mak, Assigned: mak)

Tracking

55 Branch
mozilla62
Dependency tree / graph

Firefox Tracking Flags

(firefox62 fixed)

Details

Attachments

(2 attachments)

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=
Component: General → RDF
Product: Firefox → Core
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.
(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
Status: NEW → RESOLVED
Closed: Last year
Resolution: --- → WORKSFORME
Hi Kris, why was this resolved when some of the code points in comment 0 still exist?
Flags: needinfo?(kmaglione+bmo)
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)
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.
No longer blocks: 833098
Status: RESOLVED → REOPENED
Depends on: 833098
Resolution: WORKSFORME → ---
Status: REOPENED → NEW
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: nobody → mak77
Status: NEW → ASSIGNED
Comment on attachment 8974321 [details] [diff] [review]
Stop using LStoreS in comm-central

Thanks, this looks good.
Attachment #8974321 - Flags: review?(jorgk) → review+
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: Last yearLast year
Resolution: --- → FIXED
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
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+
Pushed by mak77@bonardo.net:
https://hg.mozilla.org/integration/autoland/rev/ba79a9e472c0
Get rid of some remaining localstore.rdf code. r=mossop
https://hg.mozilla.org/mozilla-central/rev/ba79a9e472c0
Status: REOPENED → RESOLVED
Closed: Last yearLast year
Resolution: --- → FIXED
Target Milestone: --- → mozilla62
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.