Re: Support for musl

2018-01-25 Thread Henri
On Fri, 2018-01-26 at 12:33 +1100, Ben Elliston wrote: > Hi Henri > > On Fri, Jan 26, 2018 at 11:28:31AM +1300, Henri wrote: > > > > > Sounds good to me.  A new patch is attached.  I had to pipe stderr > > to stdout though because musl ldd writes version info to stderr. > Here's a modified versi

Re: Support for musl

2018-01-25 Thread Ben Elliston
Hi Henri On Fri, Jan 26, 2018 at 11:28:31AM +1300, Henri wrote: > Sounds good to me.  A new patch is attached.  I had to pipe stderr > to stdout though because musl ldd writes version info to stderr. Here's a modified version of your patch. You shouldn't use command substitution (`...`) but just

Re: Support for musl

2018-01-25 Thread Henri
On Thu, 2018-01-25 at 19:29 +1100, Ben Elliston wrote: > On Thu, Jan 25, 2018 at 05:35:10PM +1300, Henri wrote: > > > > > > > > > Another option to determine the libc version would be to parse the > > > output of "ldd --version". > It's fine to assume the presence of ldd(1) since you know you ar

Re: Support for musl

2018-01-25 Thread Ben Elliston
On Thu, Jan 25, 2018 at 05:35:10PM +1300, Henri wrote: > > Another option to determine the libc version would be to parse the > > output of "ldd --version". It's fine to assume the presence of ldd(1) since you know you are on a Linux system. Within each case in the big case statement, you don't n

Re: Support for musl

2018-01-24 Thread Henri
On Thu, 2018-01-25 at 16:14 +1300, Henri wrote: > On Thu, 2018-01-25 at 13:58 +1100, Ben Elliston wrote: > > > > On Thu, Jan 25, 2018 at 03:46:59PM +1300, Henri wrote: > > > > > > > > > > > The config.guess script currently only checks for the glibc, uclibc, > > > and dietlibc libraries. The at

Re: Support for musl

2018-01-24 Thread Henri
On Thu, 2018-01-25 at 13:58 +1100, Ben Elliston wrote: > On Thu, Jan 25, 2018 at 03:46:59PM +1300, Henri wrote: > > > > > The config.guess script currently only checks for the glibc, uclibc, > > and dietlibc libraries. The attached patch adds support for the musl > > C library. Unfortunately, fea

Re: Support for musl

2018-01-24 Thread Ben Elliston
On Thu, Jan 25, 2018 at 03:46:59PM +1300, Henri wrote: > The config.guess script currently only checks for the glibc, uclibc, > and dietlibc libraries. The attached patch adds support for the musl > C library. Unfortunately, features.h in musl does not define a > __MUSL__ macro (or similar) which