public inbox for gentoo-dev@lists.gentoo.org
 help / color / mirror / Atom feed
From: Alexis Ballier <aballier@gentoo.org>
To: gentoo-dev@lists.gentoo.org
Subject: Re: [gentoo-dev] [RFC] Master plan for fixing elibtoolize
Date: Sat, 18 Mar 2017 11:31:05 +0100	[thread overview]
Message-ID: <20170318113105.1d912494@gentoo.org> (raw)
In-Reply-To: <1489832518.1289.9.camel@gentoo.org>

On Sat, 18 Mar 2017 11:21:58 +0100
Michał Górny <mgorny@gentoo.org> wrote:

> On sob, 2017-03-18 at 11:18 +0100, Alexis Ballier wrote:
> > On Sat, 18 Mar 2017 07:53:31 +0100
> > Michał Górny <mgorny@gentoo.org> wrote:  
> > > > > 3. copy elibtoolize logic to Portage, and make it apply
> > > > > implicitly on econf [do we need to apply it elsewhere?];
> > > > > disable explicit libtoolize when Portage supports that.    
> > > > 
> > > > Related to the above point, if you make it part of econf then it
> > > > needs to be part of PMS and that's quite a complex beast to
> > > > have in the spec. It has been suggested twice on this list
> > > > (once quite recently) that the script itself should put into a
> > > > separate package for this reason. Then PMS just needs to say
> > > > "install and use this script" without any further detail.    
> > > 
> > > Strictly speaking, you don't have to have it in the PMS. This can
> > > be left purely as Portage extension, much like gnuconfig hacking
> > > is right now.  
> > 
> > Having different portage versions or different PM behaving
> > differently for the same ebuild and portage tree, producing
> > different binaries, definitely defeats PMS goals. If such things do
> > not need to be in PMS then I don't know why we even have PMS in the
> > first place. 
> 
> If elibtoolize results in different binaries being produced, then it's
> done wrong in the first place. AFAIU the primary goal of elibtoolize
> logic is to fix issues on recent systems with outdated build systems.
> Which is certainly not something that needs to be done for every user
> out there.

You probably didn't have a look at what the patches fix. Having a
quick look at patches there, I could fine one fixing relink to
old libs (from / instead of $D), another one fixing parallel install.
The former produces broken binaries, the latter none at all.

I seriously doubt this shouldn't be fixed for every user.


  reply	other threads:[~2017-03-18 10:31 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-03-17 17:14 [gentoo-dev] [RFC] Master plan for fixing elibtoolize Michał Górny
2017-03-17 17:17 ` Alexis Ballier
2017-03-17 23:38 ` James Le Cuirot
2017-03-18  6:53   ` Michał Górny
2017-03-18 10:18     ` Alexis Ballier
2017-03-18 10:21       ` Michał Górny
2017-03-18 10:31         ` Alexis Ballier [this message]
2017-03-18 19:29           ` Peter Stuge
2017-03-22  9:38             ` Alexis Ballier

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=20170318113105.1d912494@gentoo.org \
    --to=aballier@gentoo.org \
    --cc=gentoo-dev@lists.gentoo.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox