Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#40290 closed defect (fixed)

wxgtk-2.8: wx-config contains -arch flags

Reported by: ryandesign (Ryan Carsten Schmidt) Owned by: mojca (Mojca Miklavec)
Priority: Normal Milestone:
Component: ports Version: 2.2.0
Keywords: Cc: jyrkiwahlstedt
Port: wxgtk-2.8

Description

wxgtk-2.8's wx-config script contains -arch flags, e.g.:

$ /opt/local/Library/Frameworks/wxWidgets.framework/Versions/wxGTK/2.8/bin/wx-config --libs
-L/opt/local/Library/Frameworks/wxWidgets.framework/Versions/wxGTK/2.8/lib  -arch x86_64 -arch i386 -L/opt/local/lib -Wl,-headerpad_max_install_names -arch x86_64 -arch i386   -L/opt/local/lib -framework IOKit -framework CoreServices -framework System -framework ApplicationServices  -lwx_gtk2u_richtext-2.8 -lwx_gtk2u_aui-2.8 -lwx_gtk2u_xrc-2.8 -lwx_gtk2u_qa-2.8 -lwx_gtk2u_html-2.8 -lwx_gtk2u_adv-2.8 -lwx_gtk2u_core-2.8 -lwx_baseu_xml-2.8 -lwx_baseu_net-2.8 -lwx_baseu-2.8 

They must be removed.

wxgtk-3.0 and wxWidgets-3.0 do not appear to be affected. I have not tested wxWidgets-2.8 because it cannot be installed on Mountain Lion.

Change History (4)

comment:1 Changed 11 years ago by mojca (Mojca Miklavec)

The wx-config seems to contain:

echo $_ldflags " -arch x86_64 -L/opt/local/lib -Wl,-headerpad_max_install_names -arch x86_64   -L/opt/local/lib -framework IOKit -framework CoreServices -framework System -framework ApplicationServices " $_rpath $wx_libs ""

and the source for it is:

echo $_ldflags "@LDFLAGS@" $_rpath $wx_libs "@DMALLOC_LIBS@"

I don't think I see a straight solution to modify sources in such a way that they would produce the right output, so I probably need to modify wx-config with a reinplace change in the post-build step?

I need to learn how regular expressions work in Tcl, but if you have it at your fingertips, I would be grateful for hints. I probably need a global replacement (that can be applied multiple times in a line) for something like

echo $_ldflags "<anything but newline>(-arch<space>*([letter|number|_]*))

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

The recipe I referred you to above contains the code you need.

comment:3 Changed 11 years ago by mojca (Mojca Miklavec)

Resolution: fixed
Status: newclosed

I committed r110395. I suspect this might not be the only problem with the port (for example the missing empty folders might be needed as well), so maybe I'll need to submit another patch within a day. I postponed increasing the revision for that reason, but if you believe it has to be increased ASAP, feel free to do it.

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

Replying to mojca@…:

I postponed increasing the revision for that reason

Oh ok, then ignore my mail to the mailing list about this.

Note: See TracTickets for help on using tickets.