Closed
Bug 465662
Opened 12 years ago
Closed 9 years ago
remove libreg from toolkit
Categories
(Toolkit Graveyard :: Build Config, defect)
Tracking
(Not tracked)
RESOLVED
INVALID
People
(Reporter: dougt, Unassigned)
References
Details
Attachments
(1 file, 1 obsolete file)
1.19 KB,
patch
|
benjamin
:
review+
|
Details | Diff | Splinter Review |
The only consumer of libreg is browser/migration. We should consider moving this functionality out of toolkit and basically make the browser makefiles.sh create the makefiles for libreg instead of toolkit. This is ideal since mobile doesn't need libreg and for windows mobile, we have some build bustages in that code.
Reporter | ||
Updated•12 years ago
|
Assignee: nobody → doug.turner
Reporter | ||
Comment 1•12 years ago
|
||
something like this.
Comment 2•12 years ago
|
||
(In reply to comment #0) > The only consumer of libreg is browser/migration. s/browser/browser,mail,suite,mailnews/
Reporter | ||
Comment 3•12 years ago
|
||
not mobile, songbird. it would be good to have a configure flag or something.
Comment 4•12 years ago
|
||
You could add an AC_SUBST(MOZ_LIBREG) in configure.in, and have apps opt in in their confvars.sh.
Comment 5•12 years ago
|
||
Plugins don't use libreg for pluginreg.dat any more?
Comment 6•12 years ago
|
||
(In reply to comment #5) > Plugins don't use libreg for pluginreg.dat any more? Doesn't appear to. It uses nsPluginManifestLineReader, which has no dependencies on libreg.
Reporter | ||
Comment 7•12 years ago
|
||
so, can we move this out of toolkit?
Comment 8•12 years ago
|
||
Yes, I think so. We should continue to build the static library always as part of the xulrunner build process, but there's no need to link it to libxul.
Reporter | ||
Comment 9•12 years ago
|
||
bsmedberg, can you take a look at the patch?
Updated•12 years ago
|
Attachment #348917 -
Flags: review?(benjamin)
Updated•12 years ago
|
Attachment #348917 -
Flags: review?(benjamin) → review-
Comment 10•12 years ago
|
||
Comment on attachment 348917 [details] [diff] [review] patch v.1 I think we should build libreg from the XR build system even when it might not be used.
Reporter | ||
Comment 11•12 years ago
|
||
yeah, that removes my motivation from fixing this. I do not want to build libreg on WINCE. There were a bunch of problems with libreg building on WINCE and given that we do not use it, I didn't look at fixing them. Why build something we don't use?
Comment 12•12 years ago
|
||
You don't have to build it on WinCE. I just want it to keep building otherwise because a fair number of apps use it, and getting them each to build it is pretty hard...
Reporter | ||
Updated•12 years ago
|
Flags: wanted1.9.1?
Reporter | ||
Comment 13•12 years ago
|
||
it doesn't look like we/fennec needs this any longer. we are successfully building as is.
Assignee: doug.turner → nobody
Here's a patch to disable it when building on CE. Doug, I think you're building with --disable-xpinstall, which causes it to not be needed... though now that I think about it, we don't need xpinstall for my use anyway. However, we may as well fix this so that things don't break if someone doesn't specify the disable. I don't really agree that we should keep building this for other platforms -- if other apps need it, they should take ownership of it; but disabling it here fixes the immediate problem.
Attachment #348917 -
Attachment is obsolete: true
Attachment #366249 -
Flags: review?(benjamin)
Updated•12 years ago
|
Attachment #366249 -
Flags: review?(benjamin) → review+
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → DUPLICATE
Duplicate of bug: 679352
Comment 18•9 years ago
|
||
This is not exactly a duplicate of that bug, however is useless now :)
Resolution: DUPLICATE → INVALID
Updated•2 years ago
|
Product: Toolkit → Toolkit Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•