Closed
Bug 486782
Opened 12 years ago
Closed 12 years ago
New variable: MOZ_FS_LAYOUT should be used instead of OS or Toolkit to determine file system layout
Categories
(Firefox Build System :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla1.9.2a1
People
(Reporter: jeremyhu, Assigned: jeremyhu)
References
Details
Attachments
(1 file, 2 obsolete files)
4.23 KB,
patch
|
jeremyhu
:
review+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Macintosh; U; PPC Mac OS X 10_5_6; en-us) AppleWebKit/530.5+ (KHTML, like Gecko) Version/3.2.1 Safari/525.27.1 Build Identifier: Darwin/gtk uses the traditional UNIX layout, but there are places in the build that assume Darwin = bundle. The cleanest solution is to create a single new variable to describe this that has options for: traditional : The traditional unix layout (placing the startup script in bin, libs, etc in mozilla_five_home, etc) bundle : Create a NextStep / Darwin / OSX style bundle (ie: Firefox.app) winnt : windows stuff (I have no real understanding of how that works, but I assume we'd replace most of the OS=WINNT with this check) Reproducible: Always
Updated•12 years ago
|
Status: UNCONFIRMED → NEW
Component: General → Build Config
Ever confirmed: true
Product: Firefox → Core
QA Contact: general → build-config
Assignee | ||
Updated•12 years ago
|
Assignee: nobody → jeremyhu
Hardware: PowerPC → All
Assignee | ||
Updated•12 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Comment 1•12 years ago
|
||
What should the values of this variable be for OS2 and BeOS?
Blocks: 487371
Comment 2•12 years ago
|
||
We would probably *not* replace OS=WINNT with this, since most of the windows checks are for which files to build (OS integration, etc) and not packaging details. I think for now we only need two values, "unix" and "bundle".
Assignee | ||
Comment 3•12 years ago
|
||
Here's a patch to consider. The one thing I'm not too sure about is the transition of -DNO_X11 from HOST_CFLAGS to TK_CFLAGS. I was having some trouble figuring out what HOST and TARGET were for since they seem to be breaking convention that I'm used to (BUILD = host you're building on, HOST = host the code will run on, TARGET = host built apps will generate code for). Therefore, I don't understand why there is anything to do with TARGET in configure.in, and it looks like TARGET is what HOST should be (the platform mozilla will run on) and HOST is what BUILD should be?
Attachment #372025 -
Flags: review?(benjamin)
Comment 4•12 years ago
|
||
Comment on attachment 372025 [details] [diff] [review] possible patch to add MOZILLA_FS_LAYOUT to configure.in Yeah, our configure uses host|target for build|host due to longstanding (1998) tradition.
Attachment #372025 -
Flags: review?(benjamin) → review+
Comment 5•12 years ago
|
||
The patch has been bitrotted by several intervening changes (removing the old mac-cairo codepaths, as well as removing all references to FlatCarbon). Also I've reverted the changes for -DNO_X11, because that is specific to HOST_CFLAGS: it's used in config/mkdepend/def.h and I think we can happily state that you don't need X11 to build mkdepend on Darwin, no matter whether you end up targeting darwin-cocoa or x11/gtk.
Attachment #372413 -
Flags: review?(jeremyhu)
Assignee | ||
Comment 6•12 years ago
|
||
Comment on attachment 372413 [details] [diff] [review] Updated to trunk, reverted NO_X11 changes, rev. 1 [Checkin: Comment 10] Thanks, that looks good. Is this and the related changes only going to be targeted at trunk? I'd like to get it into 1.9.1 as well, so I can reduce our patchset size.
Attachment #372413 -
Flags: review?(jeremyhu) → review+
Assignee | ||
Comment 7•12 years ago
|
||
Although in retrospect, we might want to name it MOZ_FS_LAYOUT instead of MOZILLA_FS_LAYOUT for better naming consistency with the other variables...
Assignee | ||
Comment 8•12 years ago
|
||
This is an updated patch against 1.9.1 which addresses changes your made in your trunk patch and changes the name to MOZ_FS_LAYOUT to match the naming style of similar variables
Attachment #372025 -
Attachment is obsolete: true
Assignee | ||
Updated•12 years ago
|
Summary: New variable: MOZILLA_FS_LAYOUT should be used instead of OS or Toolkit to determine file system layout → New variable: MOZ_FS_LAYOUT should be used instead of OS or Toolkit to determine file system layout
Comment 10•12 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/58460d86223d
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Comment 11•11 years ago
|
||
Comment on attachment 372571 [details] [diff] [review] updated 1.9.1 patch You should either request review+approval or obsolete this patch.
Updated•11 years ago
|
Flags: in-testsuite-
Target Milestone: --- → mozilla1.9.2a1
Version: unspecified → Trunk
Assignee | ||
Updated•11 years ago
|
Attachment #372571 -
Attachment is obsolete: true
Updated•11 years ago
|
Attachment #372413 -
Attachment description: Updated to trunk, reverted NO_X11 changes, rev. 1 → Updated to trunk, reverted NO_X11 changes, rev. 1
[Checkin: Comment 10]
Updated•11 years ago
|
Blocks: C192ConfSync
Updated•3 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•