Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 559520 - =x11-proto/xcb-proto-1.11 fails to build on musl with python3_4 bindings
Summary: =x11-proto/xcb-proto-1.11 fails to build on musl with python3_4 bindings
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Unspecified (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo musl team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: musl-porting 546890
  Show dependency tree
 
Reported: 2015-09-03 12:40 UTC by tt_1
Modified: 2015-10-25 14:42 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
config-error.log from autoconf (config-error.log,6.03 KB, text/plain)
2015-09-03 12:40 UTC, tt_1
Details
emerge.info (emerge.info.log,4.82 KB, text/plain)
2015-09-03 12:41 UTC, tt_1
Details
Patch for musl (untested) (musl-1.1.11-utf-8-code-units.patch,371 bytes, patch)
2015-09-03 21:03 UTC, Felix Janda
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description tt_1 2015-09-03 12:40:37 UTC
Created attachment 410922 [details]
config-error.log from autoconf

a workaround is to put this into you package.use 

=x11-proto/xcb-proto-1.11::gentoo -python_targets_python3_4 

it seems to be a musl exclusive problem, as I am able to compile the python3_4 bindings on my regular machine without any problems.
Comment 1 tt_1 2015-09-03 12:41:53 UTC
Created attachment 410924 [details]
emerge.info
Comment 2 Felix Janda 2015-09-03 18:15:58 UTC
This is related to the new C locale of musl-1.11. There is a thread

http://www.openwall.com/lists/musl/2015/09/01/3

on the musl list.

So downgrading to musl-1.10 should also be a workaround.
Comment 3 Anthony Basile gentoo-dev 2015-09-03 20:32:35 UTC
(In reply to Felix Janda from comment #2)
> This is related to the new C locale of musl-1.11. There is a thread
> 
> http://www.openwall.com/lists/musl/2015/09/01/3
> 
> on the musl list.
> 
> So downgrading to musl-1.10 should also be a workaround.

Darn, a regression.  Can we isolate a patch against 1.11 and I'll push out -r1?
Comment 4 Felix Janda 2015-09-03 21:03:43 UTC
Created attachment 410954 [details, diff]
Patch for musl (untested)
Comment 5 tt_1 2015-09-03 21:52:44 UTC
I could test the patch tomorrow for this specific case and for @system. By the way, why is musl not included in the @system template? 

Could you explain the patch a bit for the non professional?
Comment 6 tt_1 2015-09-04 07:31:29 UTC
@ felix 

I guess your patch does break quite a few things, causing file collisions to be more precise. have a look here 

https://bugs.gentoo.org/show_bug.cgi?id=545502#c5

should we open a new bug about the regression and close this one to make things less confusing?
Comment 7 Felix Janda 2015-09-04 17:37:35 UTC
The patch tries to implement the suggestion of Rich Felker: Make
nl_langinfo(CODESET) always return "UTF-8".

@ tt_1:
I can reproduce (in the example of sys-apps/sed) the installation of
charset.alias despite the INSTALL_MASK in make.defaults. However it
seems to happen no matter whether the patch against musl is applied or
not. Are you sure that during your tests you have changed nothing other
than whether to patch musl?
Comment 8 tt_1 2015-09-05 13:13:51 UTC
I cannot reproduce the file collisions on a clean stage 3. Neither with the utf-8 patch for musl being applied, nor the 213-posix_memalign.patch for gcc-4.8.5. The results of emerge --info I attached in this bug are from the suspicious chroot where I cannot reproduce the file collision.  

@ Felix: 

what about you?
Comment 9 Felix Janda 2015-09-05 15:10:27 UTC
I've found the cause for the INSTALL_MASK issue:

If you use INSTALL_MASK="..." in make.conf, it will override the
INSTALL_MASK from the profile. With INSTALL_MASK="${INSTALL_MASK} ..."
it is still honored.
Comment 10 tt_1 2015-09-05 15:51:56 UTC
correct - I had #INSTALL_MASK="locale.alias" in my make.conf due to #545502. 

with INSTALL_MASK="${INSTALL_MASK} locale.alias" , no more file collisions.
Comment 11 Felix Janda 2015-09-07 17:44:53 UTC
Alpine Linux uses now the same patch:

http://git.alpinelinux.org/cgit/aports/commit/?id=3e4dc327b193eb85e43c466be9f13d3a82101f1a
Comment 12 Felix Janda 2015-09-09 18:20:40 UTC
And an equivalent patch has been committed to musl git master:

http://git.musl-libc.org/cgit/musl/commit/?id=844212d94f582c4e3c5055e0a1524931e89ebe76
Comment 13 tt_1 2015-10-25 14:42:33 UTC
fixed with musl-1.1.12