All of lore.kernel.org
 help / color / mirror / Atom feed
From: Adrian Bunk <bunk@stusta.de>
To: Jean-Francois Dagenais <jeff.dagenais@gmail.com>
Cc: paul.eggleton@linux.intel.com, Poky Project <poky@yoctoproject.org>
Subject: Re: /usr/include/libnl3 or /usr/include
Date: Mon, 4 Mar 2019 20:59:37 +0200	[thread overview]
Message-ID: <20190304185937.GC1170@localhost> (raw)
In-Reply-To: <D67743D4-7A14-4AB5-AAC4-CC5E4999EB6D@gmail.com>

On Mon, Mar 04, 2019 at 11:50:43AM -0500, Jean-Francois Dagenais wrote:
> Hi guys,
> 
> There's been a couple of time when I've hit snags where packages use their version numbers in the paths of stuff they provide. This poses a small challenge where dependent recipes might have to specifically account for this "special" path.
> 
> For example, libnl is using /usr/include/libnl3 and wpa_supplicant has to specifically "hack" its configure process to be able to find the libnl headers.
> 
> Since libnl does explicitly that it cannot co-exist with it's previous versions:
> RREPLACES_${PN} = "libnl2"
> RCONFLICTS_${PN} = "libnl2"
> 
> what is the point of the sub-path?

It's where upstream installs their headers.

> Would you receive a patchset which moves libnl's headers files back at /usr/include (or rather: ${includedir}) ?
> 
> I suspect this would also set a precedent.

Diverging libnl from upstream does not sound good to me.

The correct fix would be for wpa_supplicant to use
"pkg-config --cflags libnl-3.0", and the upstream
sources are already doing that.

If this is not working without the hack in wpa_supplicant then it should 
be fixed there, but it is also possible that this hack is just a leftover
from ancient wpa_supplicant versions and can simply be removed.

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-04 19:06 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-03-04 16:50 /usr/include/libnl3 or /usr/include Jean-Francois Dagenais
2019-03-04 18:59 ` Adrian Bunk [this message]
2019-03-05 15:32   ` Jean-Francois Dagenais
2019-03-05 15:38     ` Jean-Francois Dagenais
2019-03-05 17:45       ` Adrian Bunk

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=20190304185937.GC1170@localhost \
    --to=bunk@stusta.de \
    --cc=jeff.dagenais@gmail.com \
    --cc=paul.eggleton@linux.intel.com \
    --cc=poky@yoctoproject.org \
    /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.