Ticket #26410 (closed update: fixed)
wxwidgets-devel v2.9.1
| Reported by: | pixilla@… | Owned by: | jwa@… |
|---|---|---|---|
| Priority: | Normal | Milestone: | |
| Component: | ports | Version: | |
| Keywords: | Cc: | tvb377@…, michaelld@…, raimue@…, s@…, l2g@…, wmiler@…, mjhsieh@…, and.damore@… | |
| Port: | wxWidgets-devel |
Description
The port file was not making sense to me and the development sources appear to be on sourceforge so I made more then the minimal changes to Portfile.
If you only want to solve the install_name_tool issue just look at patch-configure.diff. It is a one liner.
Attachments
Change History
comment:2 Changed 5 years ago by tvb377@…
I would implement the correction of the library paths in the Portfile. See suggested diff attached. Anyway, I also made more than minimal changes; at least the variant changes should be implemented in the upgrade.
comment:4 Changed 5 years ago by ryandesign@…
- Owner changed from macports-tickets@… to jwa@…
- Cc mww@… added; jwa, mww removed
Cc field must contain complete email addresses.
comment:6 Changed 5 years ago by jwa@…
- Status changed from new to closed
- Resolution set to fixed
done in r71416
comment:7 Changed 5 years ago by tvb377@…
- Status changed from closed to reopened
- Resolution fixed deleted
You did not apply the patch Portfile-wxWidgets-devel.diff
--> The added variants are missing
--> The variant debug is still not corrected
--> The libraries dependencies do not work this way.
Why not simply apply the patch? It's not *that* bad.
Changed 5 years ago by tvb377@…
- Attachment Portfile-wxWidgets-devel.diff added
corrections necessary after r71416
comment:8 Changed 5 years ago by raimue@…
- Cc raimue@… added
I took a look at your patch, but actually I don't understand already why the unmodified Portfile is doing things in this way.
Usually *-devel ports use the same paths as the original ports, so they can be used as a immediate replacement for testing. What is really missing here are conflict statements in wxwidgets and wxwidgets-devel in my opinion and they should use the same paths.
Does this new version break API so that it won't be compatible at all with other ports or is there another reason to use different paths?
comment:9 Changed 5 years ago by tvb377@…
The released wxWidgets 2.9.0 was of no great use for production software, maybe that's why the original Portfile author went this way.
Currently, I'm not using much of wxWidgets in my projects, my code compiles and works with 2.9.1. Some time ago I ported CodeLite to a snapshot of wx > 2.9.0 just to see if it is useable and this was not too lengthy or difficult (see also wx change documentation).
In my opinion it would be good to endorse usage of wx > 2.8 for compatibility with x86_64 (and who knows when Apple decides to remove Carbon completely).
But I just spent some minutes trying to compile pgAdmin with wx 2.9.1 Besides some character type conversion errors which might be easy to fix this fails because it requires wxOGL. wxOGL is noted as dead on some wikis and was moved out of wxWidgets. It does not compile with 2.9.1.
All not too difficult, but IMO not the job of a macport.
--> I'd leave it as is.
comment:10 Changed 4 years ago by s@…
I made a different version of the wxWidgets-devel port that uses Cocoa always, removes SDL (is that actually needed for anything in wxWidgets?) and seems to work on Mac OS X 10.5 ppc and 10.6 x86_64.
The various patches I've seen to fix the install names for the libraries don't work correctly or completely. The patch to configure and configure.in leverages the change_install_names script that is run post install to change the install name for the two libraries and the binary.
Changed 4 years ago by s@…
- Attachment patch-configure-change_install_names.diff added
Patch to generated change_install_names script
comment:14 Changed 4 years ago by jmr@…
- Cc mww@…, brad@… removed
- Reporter changed from brad@… to pixilla@…
Brad, this is well and truly timed out; if you're sure your configure patch is correct you can commit it.
comment:17 Changed 4 years ago by and.damore@…
Using wxWidgets-devel-2.9.1-Portfile and patch-configure-change_install_names.diff I have an error in destroot phase for change-install-names script, here's the relevant output (lines broken at 80 chars):
/opt/local/var/macports/build/_opt_local_var_macports_sources_svn.macports. /org_trunk_dports_graphics_wxWidgets-devel/wxWidgets-devel/work/wxWidgets-2.9.1/ /build/change-install-names //opt/local/var/macports/build/_opt_local_var_macports_sources_svn.macports. /org_trunk_dports_graphics_wxWidgets-devel/wxWidgets-devel/work/destroot/opt/ /local/lib //opt/local/var/macports/build/_opt_local_var_macports_sources_svn.macports. /org_trunk_dports_graphics_wxWidgets-devel/wxWidgets-devel/work/destroot/opt/ /local/bin /opt/local/lib //opt/local/var/macports/build/_opt_local_var_macports_sources_svn.macports. /org_trunk_dports_graphics_wxWidgets-devel/wxWidgets-devel/work/wxWidgets-2.9.1/ /build/lib Usage: install_name_tool [-change old new] ... [-rpath old new] ... [-add_rpath new] ... [-delete_rpath old] ... [-id name] input make: *** [install] Error 1
comment:18 Changed 4 years ago by and.damore@…
Issue solved, the find instructions was using '+' and therefore needs -E in order to use extended regexp.
I'm attaching a combined patch for wxWidgets-devel port, Portfile has whitespace changes.
This is actual output after install:
$ otool -L libwx_osx_cocoau-2.9.1.0.0.dylib libwx_osx_cocoau-2.9.1.0.0.dylib: /opt/local/lib/libwx_osx_cocoau-2.9.1.0.0.dylib (compatibility version 1.0.0, current version 1.0.0) […] $ otool -L libwx_osx_cocoau_gl-2.9.1.0.0.dylib libwx_osx_cocoau_gl-2.9.1.0.0.dylib: /opt/local/lib/libwx_osx_cocoau_gl-2.9.1.0.0.dylib (compatibility version 1.0.0, current version 1.0.0)
comment:19 Changed 4 years ago by pixilla@…
- Status changed from reopened to closed
- Resolution set to fixed
Fixed r79389