Closed
Bug 1048687
Opened 10 years ago
Closed 10 years ago
Handle move of dependentlibs.list from Contents/MacOS to Contents/Resources
Categories
(Firefox :: General, defect)
Tracking
()
RESOLVED
FIXED
Firefox 35
People
(Reporter: spohl, Assigned: spohl)
References
Details
Attachments
(1 file, 5 obsolete files)
2.74 KB,
patch
|
spohl
:
review+
|
Details | Diff | Splinter Review |
dependentlibs.list now lives under Contents/Resources instead of Contents/MacOS since it is a data file. We need to fix up our XPCOM glue code to pick this up properly.
Assignee | ||
Comment 1•10 years ago
|
||
Will request reviews once we've fixed up all the other dependencies that have been broken by the new bundle structure.
Assignee: nobody → spohl.mozilla.bugs
Status: NEW → ASSIGNED
Assignee | ||
Updated•10 years ago
|
Assignee | ||
Updated•10 years ago
|
Group: mozilla-employee-confidential
Assignee | ||
Comment 2•10 years ago
|
||
Attachment #8467526 -
Attachment is obsolete: true
Attachment #8473015 -
Flags: feedback?(benjamin)
Comment 3•10 years ago
|
||
Comment on attachment 8473015 [details] [diff] [review] Patch This needs more comments about we're modifying tempBuffer and what the goal of this code is. Oh how we need a C++ library equivalent of python os.path.
Attachment #8473015 -
Flags: feedback?(benjamin) → feedback+
Assignee | ||
Comment 4•10 years ago
|
||
Just updated for current trunk, not addressed feedback yet.
Attachment #8473015 -
Attachment is obsolete: true
Attachment #8480257 -
Flags: feedback+
Comment 5•10 years ago
|
||
Comment 6•10 years ago
|
||
Comment on attachment 8485799 [details] [diff] [review] unbitrotted meh! The latest was already unbitrotted!
Attachment #8485799 -
Attachment is obsolete: true
Assignee | ||
Comment 7•10 years ago
|
||
Addressed feedback. Try push is almost completely green, so requesting review (updater xpcshell test failures and gaia python integration test suite failures are handled in separate bugs): https://tbpl.mozilla.org/?tree=Try&rev=7362867ad903
Attachment #8480257 -
Attachment is obsolete: true
Attachment #8487889 -
Flags: review?(benjamin)
Comment 8•10 years ago
|
||
Comment on attachment 8487889 [details] [diff] [review] Patch >diff --git a/xpcom/glue/standalone/nsXPCOMGlue.cpp b/xpcom/glue/standalone/nsXPCOMGlue.cpp > static GetFrozenFunctionsFunc > XPCOMGlueLoad(const char* aXPCOMFile) > { > char xpcomDir[MAXPATHLEN]; >-#if defined(XP_WIN) >+#if XP_WIN Why did this change from an #ifdef to an #if? I believe #ifdef is still the standard for these. Please put this back or explain why it had to change, r=me
Attachment #8487889 -
Flags: review?(benjamin) → review+
Assignee | ||
Comment 9•10 years ago
|
||
(In reply to Benjamin Smedberg [:bsmedberg] from comment #8) > Comment on attachment 8487889 [details] [diff] [review] > Patch > > >diff --git a/xpcom/glue/standalone/nsXPCOMGlue.cpp b/xpcom/glue/standalone/nsXPCOMGlue.cpp > > > static GetFrozenFunctionsFunc > > XPCOMGlueLoad(const char* aXPCOMFile) > > { > > char xpcomDir[MAXPATHLEN]; > >-#if defined(XP_WIN) > >+#if XP_WIN > > Why did this change from an #ifdef to an #if? I believe #ifdef is still the > standard for these. > > Please put this back or explain why it had to change, r=me Oops, I meant to change this from an #if defined() to an #ifdef, but forgot to add 'def' after removing the defined(). Thanks for catching this!
Assignee | ||
Comment 10•10 years ago
|
||
Changed #if to #ifdef. Carrying over r+.
Attachment #8487889 -
Attachment is obsolete: true
Attachment #8488644 -
Flags: review+
Assignee | ||
Comment 11•10 years ago
|
||
https://hg.mozilla.org/projects/oak/rev/e305e14d93cf
Comment 12•10 years ago
|
||
Pushed to fx-team https://hg.mozilla.org/integration/fx-team/rev/f877c9b66cec
Comment 13•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/f877c9b66cec
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 35
Comment 14•10 years ago
|
||
Landed on aurora in the Mac V2 signing combined patch in bug 1047584
status-firefox34:
--- → fixed
status-firefox35:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•