Closed
Bug 509788
Opened 15 years ago
Closed 15 years ago
Use dehydra-config.h instead of config.h to avoid confusion with GCC's config.h
Categories
(Developer Infrastructure :: Source Code Analysis, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: dnovillo, Assigned: taras.mozilla)
Details
Attachments
(2 files)
|
1.33 KB,
patch
|
Details | Diff | Splinter Review | |
|
1.06 KB,
patch
|
Details | Diff | Splinter Review |
This is a change that may not be applicable outside our build environment. We were getting confusion between GCC's config.h and dehydra's config.h. The attached patch renames config.h -> dehydra-config.h to avoid that and makes config.h a symlink.
|
Reporter | |
Comment 1•15 years ago
|
||
|
Assignee | |
Comment 2•15 years ago
|
||
Comment on attachment 393852 [details] [diff] [review] Rename config.h -> dehydra-config.h. >+ os.symlink ("dehydra-config.h", "config.h") Why is the symlink needed?
|
|
Reporter | |
Comment 3•15 years ago
|
||
|
|
Reporter | |
Comment 4•15 years ago
|
||
(In reply to comment #2) > (From update of attachment 393852 [details] [diff] [review]) > > >+ os.symlink ("dehydra-config.h", "config.h") > > Why is the symlink needed? It's not, actually. I don't recall why I added it (probably testing it without changing dehydra files). New patch attached.
|
|
Assignee | |
Comment 5•15 years ago
|
||
http://hg.mozilla.org/rewriting-and-analysis/dehydra/rev/600e6709d1bd
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
|
||
Updated•6 years ago
|
Product: Core → Firefox Build System
|
|
||
Updated•2 years ago
|
Product: Firefox Build System → Developer Infrastructure
You need to log in
before you can comment on or make changes to this bug.
Description
•