[gentoo-dev] fixing dev-libs/icu c++11 bustage, testers wanted

[gentoo-dev] fixing dev-libs/icu c++11 bustage, testers wanted

["Paweł Hajdan, Jr."]

[-- Attachment #1: Type: text/plain, Size: 668 bytes --]

tl;dr - please test dev-libs/icu-50.1-r2 and report if it works for you
or not; if it doesn't, please try dev-libs/icu-50.1-r1 and also report

I've done two version bumps of dev-libs/icu related to the c++11 bug
<https://bugs.gentoo.org/show_bug.cgi?id=439892> (feel free to add the
package to the chromium herd btw).

My _limited_ testing seems to indicate it's "working". But if you people
can just do your tests, that would be most welcome. Please report back
either success or failure.

I hope you like those little "drive-by" icu fixes I've done here. :)
Feedback welcome, and I'm willing to fix any breakage resulting from my
changes.

Paweł


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 203 bytes --]

Re: [gentoo-dev] fixing dev-libs/icu c++11 bustage, testers wanted

[Diego Elio Pettenò]

On 14/11/2012 21:02, "Paweł Hajdan, Jr." wrote:
> My _limited_ testing seems to indicate it's "working". But if you people
> can just do your tests, that would be most welcome. Please report back
> either success or failure.

What kind of testing are you thinking of? Execution or build? And I
assume this is still under the idea that C++11 support is kept
_disabled_ for now.

Anyway I'll run an icu tinderbox soon as the current one is done (libav-9).

-- 
Diego Elio Pettenò — Flameeyes
flameeyes@flameeyes.eu — http://blog.flameeyes.eu/

Re: [gentoo-dev] fixing dev-libs/icu c++11 bustage, testers wanted

["Paweł Hajdan, Jr."]

[-- Attachment #1: Type: text/plain, Size: 1098 bytes --]

On 11/14/12 9:09 PM, Diego Elio Pettenò wrote:
> On 14/11/2012 21:02, "Paweł Hajdan, Jr." wrote:
>> My _limited_ testing seems to indicate it's "working". But if you
>> people can just do your tests, that would be most welcome. Please
>> report back either success or failure.
> 
> What kind of testing are you thinking of? Execution or build?

Build. But also possibly the upgrade paths, e.g. make sure the ABI is
not changed on gcc upgrade to 4.7 and similar cases (well actually ABI
change with an SONAME change is fine; it's mostly changing ABI without
changing SONAME which is a very bad thing).

> And I assume this is still under the idea that C++11 support is kept 
> _disabled_ for now.

-r1 disables C++11 completely, while -r2 just allows reverse
dependencies of icu not to support C++11, while possibly making icu use
C++11 features internally (I was asked by an upstream icu developer to
test that; if this still causes problems, we should report them to him).

> Anyway I'll run an icu tinderbox soon as the current one is done
> (libav-9).

Sounds good!


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 203 bytes --]

Re: [gentoo-dev] fixing dev-libs/icu c++11 bustage, testers wanted

[Diego Elio Pettenò]

On 15/11/2012 21:13, "Paweł Hajdan, Jr." wrote:
> -r1 disables C++11 completely, while -r2 just allows reverse
> dependencies of icu not to support C++11, while possibly making icu use
> C++11 features internally (I was asked by an upstream icu developer to
> test that; if this still causes problems, we should report them to him).

That's extremely fishy because it means that you have to execute to be
sure that the symbols don't collide. As far as I know, if you build
something with C++11, you can't link it to C++98, but Ryan probably
knows this a bit better.

>> > Anyway I'll run an icu tinderbox soon as the current one is done
>> > (libav-9).
> Sounds good!

It's setting up... it takes a while because something was bringing in 49
even though the tinderbox has 50 unmasked... that's never good.

-- 
Diego Elio Pettenò — Flameeyes
flameeyes@flameeyes.eu — http://blog.flameeyes.eu/

Re: [gentoo-dev] fixing dev-libs/icu c++11 bustage, testers wanted

[Mike Gilbert]

On Thu, Nov 15, 2012 at 12:02 AM, "Paweł Hajdan, Jr."
<phajdan.jr@gentoo.org> wrote:
> tl;dr - please test dev-libs/icu-50.1-r2 and report if it works for you
> or not; if it doesn't, please try dev-libs/icu-50.1-r1 and also report
>
> I've done two version bumps of dev-libs/icu related to the c++11 bug
> <https://bugs.gentoo.org/show_bug.cgi?id=439892> (feel free to add the
> package to the chromium herd btw).
>
> My _limited_ testing seems to indicate it's "working". But if you people
> can just do your tests, that would be most welcome. Please report back
> either success or failure.
>
> I hope you like those little "drive-by" icu fixes I've done here. :)
> Feedback welcome, and I'm willing to fix any breakage resulting from my
> changes.
>

I'll do some run time testing (web browsing) with chromium-25 and
icu-50-r2 / gcc-4.7 this weekend.