Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
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/

      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