From: Peter Seiderer <ps.report@gmx.net>
To: buildroot@busybox.net
Subject: [Buildroot] [RFC v1] directfb: fix musl compile
Date: Thu, 19 Nov 2015 20:12:13 +0100 [thread overview]
Message-ID: <20151119201213.6e82f6b8@gmx.net> (raw)
In-Reply-To: <87vb8z9djd.fsf@dell.be.48ers.dk>
Hello Peter,
On Wed, 18 Nov 2015 22:57:10 +0100, Peter Korsgaard <peter@korsgaard.com> wrote:
> >>>>> "Peter" == Peter Seiderer <ps.report@gmx.net> writes:
>
> Hi,
>
> > - use 'union sigval' instead of sigval_t (same as suggested
> > for canfestival by Yann E. MORIN, see [2])
>
> > - delete DIRECT_RECURSIVE_MUTEX_INITIALIZER define (which uses unknown
> > PTHREAD_RECURSIVE_MUTEX_INITIALIZER_NP)
>
> > - use DIRECT_MUTEX_INITIALIZER instead for trace.c (no recursive
> > mutex needed)
>
> > - use pthread_once and direct_recursive_mutex_init instead for
> > directfb.c
>
> > Fixes [1]:
>
> > system.c:114:6: error: unknown type name 'sigval_t'
>
> > and
>
> > ../lib/direct/os/linux/glibc/mutex.h:49:53: error: ?PTHREAD_RECURSIVE_MUTEX_INITIALIZER_NP? undeclared (first use in this function)
>
> > [1] http://autobuild.buildroot.net/results/548/548bf7b800a946e97135c0ba66363f388e517d6f
> > [2] https://patchwork.ozlabs.org/patch/509731
>
> > Signed-off-by: Peter Seiderer <ps.report@gmx.net>
> > ---
> > RFC: not yet runtime tested
>
> Thanks, but I don't feel confident applying such a patch, especially as
> upstream seems dead so we cannot get their feedback on it / will have to
> carry it along ourselves - So I've instead marked directfb as !musl.
>
O.k.
According to [1] the directfb source code is (temporarily?) hosted on github [2]
by one of the developers...
If I find some more time to work on my patch I will try to submit it there...
Regards,
Peter
[1] http://www.phoronix.com/scan.php?page=news_item&px=DirectFB-On-GitHub
[2] https://github.com/deniskropp/DirectFB/
prev parent reply other threads:[~2015-11-19 19:12 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-11-09 23:05 [Buildroot] [RFC v1] directfb: fix musl compile Peter Seiderer
2015-11-18 21:57 ` Peter Korsgaard
2015-11-19 19:12 ` Peter Seiderer [this message]
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=20151119201213.6e82f6b8@gmx.net \
--to=ps.report@gmx.net \
--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