#40004 closed defect (fixed)
poedit +wxwidgets_devel declares dependency on wxWidgets30 too
Reported by: | gthb (Gunnlaugur Thor Briem) | Owned by: | raimue (Rainer Müller) |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | |
Keywords: | haspatch | Cc: | mojca (Mojca Miklavec), cooljeanius (Eric Gallager) |
Port: | poedit |
Description
Installing poedit with the wxwidgets_devel variant fails due to an extra dependency on wxWidgets30:
$ sudo port install poedit +wxwidgets_devel ---> Computing dependencies for poedit ---> Dependencies to be installed: wxWidgets30 ---> Activating wxWidgets30 @2.9.5_1+sdl Error: org.macports.activate for port wxWidgets30 returned: Image error: /opt/local/bin/wx-config is being used by the active wxWidgets-devel port. Please deactivate this port first, or use 'port -f activate wxWidgets30' to force the activation. Error: Failed to install wxWidgets30 Please see the log file for port wxWidgets30 for details: /opt/local/var/macports/logs/_opt_local_var_macports_sources_rsync.macports.org_release_tarballs_ports_graphics_wxWidgets30/wxWidgets30/main.log Error: The following dependencies were not installed: wxWidgets30 To report a bug, follow the instructions in the guide: http://guide.macports.org/#project.tickets Error: Processing of port poedit failed
Indeed both dependencies are present:
$ sudo port deps poedit +wxwidgets_devel Full Name: poedit @1.5.5_1+wxwidgets_devel Build Dependencies: grep, pkgconfig, boost Library Dependencies: wxWidgets30, gettext, wxWidgets-devel
Attached portfile patch fixes this and makes the build and install work for me.
Attachments (2)
Change History (10)
Changed 11 years ago by gthb (Gunnlaugur Thor Briem)
Attachment: | Portfile-fix-ticket-40004.patch added |
---|
comment:1 Changed 11 years ago by ryandesign (Ryan Carsten Schmidt)
Cc: | mojca@… added |
---|---|
Keywords: | haspatch added |
comment:2 Changed 11 years ago by mojca (Mojca Miklavec)
Cc: | raimue@… removed |
---|---|
Owner: | changed from macports-tickets@… to raimue@… |
Version: | 2.2.0 |
The port could also be upgraded to 1.5.7. I would prefer if the maintainer would take a look first, but here is what I would suggest for a fix. (Please note that I didn't test the patch yet as it would take me hours to get wxWidgets
back to the original state ;)
This allows specifying dependency on wxWidgets30
also on 10.6 or 10.7 and should work on 10.9 (currently it only replaces wxWidgets
on 10.8).
Changed 11 years ago by mojca (Mojca Miklavec)
Attachment: | poedit-1.5.7-and-wxwidgets30.patch added |
---|
Another suggestion to fix the problem + update to 1.5.7
comment:3 Changed 11 years ago by raimue (Rainer Müller)
Status: | new → assigned |
---|
comment:4 follow-up: 6 Changed 11 years ago by raimue (Rainer Müller)
I am not sure whether it is the correct way to change the default dependencies based on the active ports. Is this also how it is done in other ports?
comment:6 Changed 11 years ago by mojca (Mojca Miklavec)
Replying to raimue@…:
I am not sure whether it is the correct way to change the default dependencies based on the active ports. Is this also how it is done in other ports?
I don't track other ports, but I don't see what would be wrong about activating the variant based on active ports. If it's not done, it only means that user will be faced with an error and will have to manually resolve something that could be automatically resolved for him.
The elisp
portgroup sets the path to emacs
binary based on which port is active.
I find it more "problematic" to simply replace dependency on wxWidgets
with wxWidgets30
, but only on OS X >= 10.8, without a way to depend on wxWidgets 2.9 on Lion.
I hope that wxWidgets mess will be cleaned up soon, so the port would need another round of fixes with dependencies anyway, but I would need some help with that.
comment:7 Changed 11 years ago by mojca (Mojca Miklavec)
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Should be fixed in r110243.
comment:8 Changed 11 years ago by raimue (Rainer Müller)
Thanks for taking care of the wxWidgets dependency!
Portfile patch fixing accidental wxWidgets30 dependency