Opened 14 years ago

Closed 11 years ago

Last modified 11 years ago

#22870 closed enhancement (wontfix)

muniversal group: merge .la dependency_libs value

Reported by: pguyot (Paul Guyot) Owned by: macports-tickets@…
Priority: Normal Milestone:
Component: base Version: 1.8.1
Keywords: Cc: ryandesign (Ryan Carsten Schmidt), MarcusCalhoun-Lopez (Marcus Calhoun-Lopez), cooljeanius (Eric Gallager)
Port: muniversal

Description

.la files are automatically generated text files that contain linking information. muniversal group fails if two .la files differ.

However, it seems we could apply some heuristics, for example with the dependency_libs key.

i386 and x86_64 libgettextlib.la files are identical except for the following line:

# Libraries that this one depends upon.
dependency_libs=' /opt/local/lib/libintl.la -L/opt/local/lib -lc -ltermcap'
# Libraries that this one depends upon.
dependency_libs=' -R/opt/local/lib /opt/local/lib/libintl.la -L/opt/local/lib -lc /opt/local/lib/libiconv.la -lncurses'

That these lines could probably be merged by putting every item in each, and that wouldn't create a problem. The attached patch does just this and allows gettext +universal to be built on 10.6.

The patch could be extended to process inherited_linker_flags as well (see ticket #22801).

What do you think?

Attachments (1)

muniversal-1.0.tcl.diff (4.2 KB) - added by pguyot (Paul Guyot) 14 years ago.
Patch to merge .la files with muniversal group

Download all attachments as: .zip

Change History (6)

Changed 14 years ago by pguyot (Paul Guyot)

Attachment: muniversal-1.0.tcl.diff added

Patch to merge .la files with muniversal group

comment:1 in reply to:  description Changed 14 years ago by ryandesign (Ryan Carsten Schmidt)

Cc: mcalhoun@… added
Port: muniversal added

Replying to pguyot@…:

i386 and x86_64 libgettextlib.la files are identical except for the following line:

They are completely identical for me on Snow Leopard. Are you certain your ncurses and ncursesw are built universal for the same architectures as those for which you are now trying to build gettext?

comment:2 Changed 14 years ago by pguyot (Paul Guyot)

Indeed, ncurses and ncursesw were not built +universal.

comment:3 in reply to:  2 Changed 14 years ago by ryandesign (Ryan Carsten Schmidt)

Replying to pguyot@…:

Indeed, ncurses and ncursesw were not built +universal.

That specific problem should be addressed by #22876. I am worried about applying your proposed change to the muniversal portgroup, because differences in .la files are often caused by dependencies that are not built universal, and we don't want to mask that problem.

comment:4 Changed 11 years ago by ryandesign (Ryan Carsten Schmidt)

Resolution: wontfix
Status: newclosed

This issue should be solved due to #38010 (dependency_libs will now be removed from all ports).

comment:5 Changed 11 years ago by cooljeanius (Eric Gallager)

Cc: egall@… added

Cc Me!

Note: See TracTickets for help on using tickets.