Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#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)

Portfile-fix-ticket-40004.patch (396 bytes) - added by gthb (Gunnlaugur Thor Briem) 11 years ago.
Portfile patch fixing accidental wxWidgets30 dependency
poedit-1.5.7-and-wxwidgets30.patch (2.9 KB) - added by mojca (Mojca Miklavec) 11 years ago.
Another suggestion to fix the problem + update to 1.5.7

Download all attachments as: .zip

Change History (10)

Changed 11 years ago by gthb (Gunnlaugur Thor Briem)

Portfile patch fixing accidental wxWidgets30 dependency

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)

Another suggestion to fix the problem + update to 1.5.7

comment:3 Changed 11 years ago by raimue (Rainer Müller)

Status: newassigned

comment:4 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:5 Changed 11 years ago by cooljeanius (Eric Gallager)

Cc: egall@… added

Cc Me!

comment:6 in reply to:  4 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: assignedclosed

Should be fixed in r110243.

comment:8 Changed 11 years ago by raimue (Rainer Müller)

Thanks for taking care of the wxWidgets dependency!

Note: See TracTickets for help on using tickets.