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 [email protected]
with a subject of "unsubscribe". Trouble? Contact [email protected]
