Use dehydra-config.h instead of config.h to avoid confusion with GCC's config.h

RESOLVED FIXED

Status

()

Core
Rewriting and Analysis
RESOLVED FIXED
9 years ago
9 years ago

People

(Reporter: Diego Novillo, Assigned: (dormant account))

Tracking

Trunk
x86
Linux
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Reporter)

Description

9 years ago
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

9 years ago
Created attachment 393852 [details] [diff] [review]
Rename config.h -> dehydra-config.h.
(Assignee)

Comment 2

9 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

9 years ago
Created attachment 393866 [details] [diff] [review]
New version without symlink for config.h
(Reporter)

Comment 4

9 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

9 years ago
http://hg.mozilla.org/rewriting-and-analysis/dehydra/rev/600e6709d1bd
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.