#38267 closed defect (fixed)
v8 @3.17.6 +universal needs its configure.universal_args nuked
Reported by: | cooljeanius (Eric Gallager) | Owned by: | macports-tickets@… |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | 2.1.3 |
Keywords: | haspatch | Cc: | frodwith@…, neverpanic (Clemens Lang) |
Port: | v8 |
Description
Its configure script doesn't recognize the --disable-dependency-tracking
flag.
Attachments (1)
Change History (6)
Changed 12 years ago by cooljeanius (Eric Gallager)
Attachment: | Portfile.diff added |
---|
comment:1 Changed 12 years ago by ryandesign (Ryan Carsten Schmidt)
Please just write:
configure.universal_args-delete --disable-dependency-tracking
like other ports do. It doesn't need to go inside the universal variant; I don't think any other ports do that.
comment:2 follow-up: 3 Changed 12 years ago by neverpanic (Clemens Lang)
Resolution: | → fixed |
---|---|
Status: | new → closed |
Fixed in r103654. What's the benefit of this flag anyway? Should we remove it in base?
comment:3 follow-up: 4 Changed 12 years ago by larryv (Lawrence Velázquez)
Replying to cal@…:
What's the benefit of this flag anyway?
http://www.gnu.org/software/automake/manual/html_node/Dependency-Tracking.html
comment:4 follow-up: 5 Changed 12 years ago by cooljeanius (Eric Gallager)
Replying to larryv@…:
Replying to cal@…:
What's the benefit of this flag anyway? Should we remove it in base?
http://www.gnu.org/software/automake/manual/html_node/Dependency-Tracking.html
So in other words we could remove it from everything that doesn't use automake
?
comment:5 Changed 12 years ago by larryv (Lawrence Velázquez)
Replying to egall@…:
So in other words we could remove it from everything that doesn't use
automake
?
No. The flag is necessary for universal builds because automatic dependency generation cannot be done when targeting multiple architectures. I presume that the configure scripts that don’t accept the flag have dependency generation disabled by default.
In my experience, most configure scripts that don’t accept the flag just ignore it. Unless the flag actually causes problems with configure, I don’t see why we should bother dealing with it.
diff to apply to v8's portfile