Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: "Jörg Krause" <joerg.krause@embedded.rocks>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 2/2 v2] package/libtirpc: does not build with musl libc
Date: Wed, 08 Jul 2015 06:54:31 +0200	[thread overview]
Message-ID: <1436331271.3691.4.camel@embedded.rocks> (raw)
In-Reply-To: <20150614231725.6865e1a3@free-electrons.com>

Dear Thomas, Yann,

On So, 2015-06-14 at 23:17 +0200, Thomas Petazzoni wrote:
> Dear Yann E. MORIN,
> 
> On Sun, 14 Jun 2015 16:29:06 +0200, Yann E. MORIN wrote:
> > Previously-packaged libtirpc version, 0.2.4, did not build with 
> > musl on
> > ARM (see build failures, below) because of missing header 
> > sys/cdefs.h
> > 
> > Newly-packaged version, 0.3.1, does not build on musl, on ARM for 
> > the
> > same reason as 0.2.4, and on whatever the architecture (ARM 
> > included)
> > because of the extra-missing header sys/queue.h .
> 
> At
> http://git.free-electrons.com/users/thomas
> -petazzoni/buildroot/log/?h=libtirpc-musl-fix,
> I have a patch that makes libtirpc 0.2.4 build fine with musl.
> Unfortunately, it breaks glibc build :)
> 
> I think the patch set can be simplified on top of libtirpc 0.3.1,
> because:
> 
>  * the cdefs.h issue is fixed upstream already:
>    http://git.linux
> -nfs.org/?p=cel/libtirpc.git;a=commit;h=7228a9482b92ff0305f7e7ccd0276
> c638bee1884
> 
>  * the IPPORT_RESERVED issue is also fixed upstream:
>    http://git.linux
> -nfs.org/?p=cel/libtirpc.git;a=commit;h=4d3263ca3b64de8acdcaf71d36bfb
> 0eda06b8710
> 
> So maybe that leaves only the sys/queue.h issue, for which my patch
> contains a proposal.
> 

I submitted a patchset for libtirpc 0.2.4 too:
http://patchwork.ozlabs.org/patch/468161/

I tested this patchset with eglibc and uClibc.

Note there is also another patch for libtirpc in the queue:
http://patchwork.ozlabs.org/patch/468160/

Best regards
J?rg Krause

  reply	other threads:[~2015-07-08  4:54 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-06-14 14:29 [Buildroot] [PATCH 0/2 v2] package/libtirpc: bump and add new dependencies (branch yem/libtirpc) Yann E. MORIN
2015-06-14 14:29 ` [Buildroot] [PATCH 1/2 v2] package/libtirpc: bump version Yann E. MORIN
2015-06-14 21:31   ` Thomas Petazzoni
2015-06-14 14:29 ` [Buildroot] [PATCH 2/2 v2] package/libtirpc: does not build with musl libc Yann E. MORIN
2015-06-14 21:17   ` Thomas Petazzoni
2015-07-08  4:54     ` Jörg Krause [this message]
2015-07-08  7:31       ` Thomas Petazzoni

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1436331271.3691.4.camel@embedded.rocks \
    --to=joerg.krause@embedded.rocks \
    --cc=buildroot@busybox.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox