public inbox for gentoo-dev@lists.gentoo.org
 help / color / mirror / Atom feed
From: Joshua Kinard <kumba@gentoo.org>
To: gentoo-dev@lists.gentoo.org
Subject: Re: [gentoo-dev] Re: [PATCH] sys-devel/autoconf: Convert from eblits into an eclass, #586424
Date: Tue, 21 Mar 2017 17:54:30 -0400	[thread overview]
Message-ID: <6caf3cb4-10ce-35c8-0045-5b497af20ef0@gentoo.org> (raw)
In-Reply-To: <263d36db-1a1f-3570-951c-de3312756ed0@gentoo.org>

On 03/21/2017 04:43, Kristian Fiskerstrand wrote:
> On 03/21/2017 09:28 AM, Joshua Kinard wrote:
>> In general, the concept of code-sharing common blocks of logic between multiple
>> ebuilds in a specific package directory that is not a top-level eclass is not
>> entirely without merit.  But the only way this idea can be realized in a
>> suitable manner and be used by far more consumers than today's eblits are, is
>> to either find and finish the old elibs GLEP or start one over from scratch,
>> submit whatever needs submitting via patches to at least PMS and Portage, work
>> through whatever processes are required for approval, and then deploy it in the
>> next EAPI.
>>
>> If anyone is game for working something up or discussing further, let me know.
> 
> I personally fail to see good reasons to have a separate approach for
> this instead of putting it in the eclass framework. However this might
> simply mean I'm missing something in the discussion.
> 
> Before restarting such a GLEP process; maybe a simple pros and cons list
> of comparison of the future eblit use and existing eclass structure
> could be helpful? (along with more description of the differences)

Well, maybe it's a sign of my age and tenure, but I always thought one should
be conservative in the definition of new eclasses.  I may be wrong, but back in
the old days, to define a new toplevel eclass, I believe one had to demonstrate
that a number of different packages all had common ebuild logic that could all
merged into a single eclass.  Kinda like what we do now with new global USE
flags.  So throwing package-specific eclasses into the toplevel eclass
directory seems to run against this.  Additionally, repoman does not validate
the logic in eclasses currently (a long-time issue), which can lead to code rot
of such package-specific common code.

That said, I'm not wedded to the current implementation of eblits as they
exist, and IMHO, I do agree in principle with QA that any such feature like
that needs to be defined in PMS and employed by the package manager in some
capacity.  It is certainly doable right now with an eclass (I even have an
example eblit.eclass that demos this), but it would be better to leverage
existing loading and sourcing logic within the package managers for such a
capability, which means changing them and updating PMS, which effectively means
a new EAPI.

How we go about implementing this idea, or whether we even bother to do so, is
what needs to be discussed.  I certainly have some ideas, but I've largely
isolated myself to the MIPS world the last few years and my ideas might not be
the best when applied elsewhere in the tree.  As such, starting a GLEP is
probably the best approach, elsewise, this idea will just die on the mailing
lists yet again.  I'm just not sure when I'll have some free time to educate
myself on GLEPs and throw a draft together.  I've lately been trying to chase a
really obscure kernel bug down on one of my SGI systems in addition to other
life responsibilities, so a GLEP is somewhat low priority right now.

-- 
Joshua Kinard
Gentoo/MIPS
kumba@gentoo.org
6144R/F5C6C943 2015-04-27
177C 1972 1FB8 F254 BAD0 3E72 5C63 F4E3 F5C6 C943

"The past tempts us, the present confuses us, the future frightens us.  And our
lives slip away, moment by moment, lost in that vast, terrible in-between."

--Emperor Turhan, Centauri Republic


  parent reply	other threads:[~2017-03-21 21:54 UTC|newest]

Thread overview: 62+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-03-16  9:38 [gentoo-dev] [PATCH] sys-devel/autoconf: Convert from eblits into an eclass, #586424 Michał Górny
2017-03-16 10:07 ` Ulrich Mueller
2017-03-16 12:19   ` Alexis Ballier
2017-03-16 12:57     ` Ulrich Mueller
2017-03-16 18:42       ` Alexis Ballier
2017-03-16 18:58         ` Michał Górny
2017-03-16 22:48           ` Alexis Ballier
2017-03-16 20:14         ` Ulrich Mueller
2017-03-16 23:12           ` Alexis Ballier
2017-03-16 12:15 ` Alexis Ballier
2017-03-16 12:59   ` Michał Górny
2017-03-16 13:30 ` [gentoo-dev] [PATCH v2] " Michał Górny
2017-03-20  8:35 ` [gentoo-dev] Re: [PATCH] " Mike Frysinger
2017-03-20  9:49   ` Ulrich Mueller
2017-03-20 11:19     ` Alexis Ballier
2017-03-20 14:12       ` Ulrich Mueller
2017-03-20 17:01         ` Alexis Ballier
2017-03-20 17:40           ` Mike Gilbert
2017-03-20 18:15             ` Alexis Ballier
2017-03-20 18:26               ` Mike Gilbert
2017-03-20 18:24           ` Michał Górny
2017-03-20 21:12             ` Alexis Ballier
2017-03-20 21:19               ` Michał Górny
2017-03-20 21:30                 ` Alexis Ballier
2017-03-20 19:25           ` Ulrich Mueller
2017-03-20 21:14             ` Alexis Ballier
2017-03-21  8:28             ` Joshua Kinard
2017-03-21  8:43               ` Kristian Fiskerstrand
2017-03-21  9:17                 ` Alexis Ballier
2017-03-21  9:41                   ` Kristian Fiskerstrand
2017-03-21 10:00                     ` Alexis Ballier
2017-03-21 10:16                       ` Kristian Fiskerstrand
2017-03-21 21:54                 ` Joshua Kinard [this message]
2017-03-21 10:24   ` Andreas K. Huettel
2017-03-23  9:41     ` Andreas K. Huettel
2017-03-23  9:51       ` Alexis Ballier
2017-03-23 10:55         ` Andreas K. Huettel
2017-03-23 13:36           ` Alexis Ballier
2017-03-23 16:36             ` Michael Orlitzky
2017-03-23 18:41               ` Alexis Ballier
2017-03-23 18:46                 ` Ciaran McCreesh
2017-03-23 19:12                   ` Alexis Ballier
2017-03-23 19:17                     ` Ciaran McCreesh
2017-03-23 19:49                       ` Alexis Ballier
2017-03-23 19:53                         ` Ciaran McCreesh
2017-03-23 16:53         ` Michał Górny
2017-03-23 18:52           ` Alexis Ballier
2017-03-23 19:00             ` Ciaran McCreesh
2017-03-23 19:00             ` Michał Górny
2017-03-23 20:22               ` Alexis Ballier
2017-03-23 20:30                 ` Ciaran McCreesh
2017-03-23 21:37                   ` Alexis Ballier
2017-03-23 21:45                     ` Ciaran McCreesh
2017-03-24 10:23                       ` Alexis Ballier
2017-03-23 21:20                 ` Michael Orlitzky
2017-03-23 21:42                   ` Alexis Ballier
2017-03-23 21:45                     ` David Seifert
2017-03-24  8:57                       ` Alexis Ballier
2017-03-24 10:05                         ` Ulrich Mueller
2017-03-24 10:36                           ` Alexis Ballier
2017-03-24  1:58                     ` M. J. Everitt
2017-03-24 19:43 ` [gentoo-dev] " Michał Górny

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=6caf3cb4-10ce-35c8-0045-5b497af20ef0@gentoo.org \
    --to=kumba@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