On Sun, Dec 13, 2009 at 10:07:05AM +0100, Andreas Vögele wrote:
> Cesare Gargano writes:
> 
> > Oh, I agree... I forgot to apply your patch, sorry :(
> > This could be ok?
> > 
> > Cesare
> > 
> > On Tue, Dec 08, 2009 at 06:49:42PM +0100, Landry Breuil wrote:
> >> On Tue, Dec 08, 2009 at 05:13:21PM +0000, Cesare Gargano wrote:
> >>> Hi!
> >>> This updates p5-Alien-wxWidgets to 0.47
> >>>
> >>> I replaced a 1 with a 2 in the patch,
> >>> I don't like their method to get our wxWidgets version ;(
> >> No, sorry, this won't work. Please keep the patch as is.
> >>
> >> There is a reason i did it that way : we want to be able to update
> >> wxwidgets without breaking/patching alien-wxwidgets.
> >>
> >> I don't care if you like it or not, but hardcoding so versions like this
> >> in patches is a sure way to break things at one moment.
> >>
> >> Landry
> 
> BTW, upstream is interested in getting Alien::wxWidgets fixed on
> OpenBSD. Some time ago James Wright created a bug report, which is
> available at L<https://rt.cpan.org/Public/Bug/Display.html?id=41678>.

Yeah, i was aware of the RT ticket, i just forgot about it lately - i
just doubt upstream will like the way i fixed the issue (ie look for all
libxxx.so.x.y files, and for each xxx lib take the biggest number).

If you have a rt account you can still link mattia barbon to the patch
(ports/devel/p5-Alien-wxWidgets/patches/patch-inc_My_Build_Any_wx_config_Bakefile_pm)
and see what he thinks about it, i won't have time to handle this before
some weeks.

Landry

Reply via email to