On Fri, Jun 06, 2014 at 12:06:29PM +0100, Gianfranco Costamagna wrote:
> I did a quick check of the poedit trac system.
> 
> Seems that ftl is needed to build poedit with wx-3.0
> http://trac.poedit.net/ticket/568
> 
> Do you plan to enable ftl in wx?
> 
> Attaching a log file with the debdiff of the trivial patch
> diff -Nru wxwidgets3.0-3.0.0/debian/rules wxwidgets3.0-3.0.0/debian/rules
> --- wxwidgets3.0-3.0.0/debian/rules     2014-05-21 06:08:05.000000000 +0200
> +++ wxwidgets3.0-3.0.0/debian/rules     2014-06-06 09:52:52.000000000 +0200
> @@ -81,6 +81,7 @@
> 
>  GTK_CONFIGURE_OPTIONS = $(COMMON_CONFIGURE_OPTIONS) \
>      --enable-display \
> +    --enable-stl \
>      --enable-geometry \
>      --enable-graphics_ctx \
>      --enable-mediactrl \

What --enable-stl does is make wxString derive from std::string, and
similarly for various wx container classes which have STL equivalents.
That means that making this change is inevitably going to break ABI
compatibility, so changing this is a lot more work than that one line
change might suggest; it would require rebuilding all applications using
wxwidgets3.0, and making sure they all still work.

I also worry that if one application needs wx built with --enable-stl,
others might fail with wx built with --enable-stl - the comment in
include/wx/setup_inc.h says "even at the cost of backwards
compatibility":

// Set wxUSE_STL to 1 to enable maximal interoperability with the standard
// library, even at the cost of backwards compatibility.
//
// Default is 0
//
// Recommended setting: 0 as the options below already provide a relatively
// good level of interoperability and changing this option arguably isn't worth
// diverging from the official builds of the library.

It also recommends leaving it set to 0.

So I'm afraid I don't think this is a change we can sanely make at this
point for wx3.0 in Debian.  Back in November before the transition had
started it might have been feasible, but now there's already ~40
packages using wx3.0 in Debian, plus another 5 delayed NMUs and another
~10 tested patches filed in the BTS.  That's a lot of testing to have to
repeat, and a lot of packages to potentially have to fix for
compatibility with --enable-stl.

What's the reason why poedit requires --enable-stl?  Can it be patched
to work with a standard wx build?

Cheers,
    Olly


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to