All of lore.kernel.org
 help / color / mirror / Atom feed
From: Adrian Bunk <bunk@stusta.de>
To: Khem Raj <raj.khem@gmail.com>
Cc: Patches and discussions about the oe-core layer
	<openembedded-core@lists.openembedded.org>
Subject: Re: musl thoughts
Date: Sat, 23 Mar 2019 19:30:11 +0200	[thread overview]
Message-ID: <20190323173011.GA10315@localhost> (raw)
In-Reply-To: <CAMKF1sqJe3G3rC4U8S-gZHDCrjuBgBPpXh-hohWJH2PL_+i-ww@mail.gmail.com>

On Fri, Mar 22, 2019 at 03:20:19PM -0700, Khem Raj wrote:
> On Fri, Mar 22, 2019 at 1:03 PM Adrian Bunk <bunk@stusta.de> wrote:
> >
> > On Fri, Mar 22, 2019 at 02:11:35AM +0100, Andreas Müller wrote:
> > >...
> > > 3. Change our musl slightly: Many patches we currently have are
> > > trivial. Missing headers or #defines for missing functions... So if we
> > > add few headers
> > >   * Empty chunks for e.g <net/ethernet.h>
> > >...
> > > Some other ideas?
> >
> > Not upstreamable hacks aren't a long-term sustainable way,
> > no matter where you do the patching.
> >
> > net/ethernet.h header problems are requiring 2 patches in NM.
> >
> > It would be good to get whatever is the actual root cause fixed properly,
> > and that fix upstreamed.
> >
> > E.g. I wonder whether this was supposed to be fixed by
> > 0001-if_ether-move-muslc-ethhdr-protection-to-uapi-file.patch
> >
> > Or is this patch in linux-libc-headers even the cause of the problem?
> >
> > linux-libc-headers applying 6 patches just for musl is not a good sign
> > in any case.
> > Is that much patching also done by other distributions that use musl?
> > Why are these patches Upstream-Status: Pending/Submitted for a  long
> > time without having been applied upstream?
> >
> > Long-term it would be less work if everything would get fixed properly
> > with fixes reviewed and applied by upstream.
> 
> agreed, meanwhile we have to also ensure that musl users keep going and make it
> possible, if OE devs are fixing musl related issues in upstream
> packages then it is also good reputation for the project.
>...

Some of the NM patches are obviously just quick hacks to workaround bugs 
that are elsewhere. Trying to push such stuff upstream would not exactly
improve the reputation.

A problem with a "keep going" approach is that you might end up spending 
much work on working around symptoms, or even working around OE-specific 
bugs without noticing that they are OE-specific.

Building an image (with things like NM included) for musl builds for me, 
and it also builds for me with all 6 musl patches not applied to the 
kernel headers.

Are these patches still required at all?
Could they be the cause of other musl related issues in OE?

Patching leaf packages can create a huge mess if the core packages are 
not in a good state.

cu
Adrian

-- 

       "Is there not promise of rain?" Ling Tan asked suddenly out
        of the darkness. There had been need of rain for many days.
       "Only a promise," Lao Er said.
                                       Pearl S. Buck - Dragon Seed



  reply	other threads:[~2019-03-23 17:30 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-03-22  1:11 musl thoughts Andreas Müller
2019-03-22 14:53 ` akuster808
2019-03-22 19:35   ` Adrian Bunk
2019-03-22 20:03 ` Adrian Bunk
2019-03-22 22:20   ` Khem Raj
2019-03-23 17:30     ` Adrian Bunk [this message]
2019-03-22 22:18 ` Khem Raj
2019-03-23 21:16   ` Adrian Bunk
2019-03-23 21:22     ` Andreas Müller
2019-03-23 21:53       ` Adrian Bunk
2019-03-23 22:00         ` Andreas Müller
2019-03-25 15:36           ` Andrea Adami
2019-03-25 16:10             ` Adrian Bunk
2019-03-25 16:26               ` Andreas Müller
2019-03-25 17:15                 ` Andreas Müller
2019-03-25 17:36                   ` Andreas Müller
2019-03-25 18:03                     ` Adrian Bunk
2019-03-25 19:46                       ` Andreas Müller
2019-03-25 17:46                   ` Adrian Bunk
2019-03-25 21:11                     ` Andreas Müller
2019-03-25 22:38                       ` Khem Raj
2019-03-25 16:31               ` Andrea Adami
2019-03-25 17:07                 ` Adrian Bunk
2019-03-25 17:44     ` Khem Raj
2019-03-25 18:23       ` Adrian Bunk
2019-03-25 15:46 ` Mark Hatle

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=20190323173011.GA10315@localhost \
    --to=bunk@stusta.de \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=raj.khem@gmail.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.