public inbox for gentoo-dev@lists.gentoo.org
 help / color / mirror / Atom feed
* [gentoo-dev] Don't require assignment of empty variables in ebuilds?
@ 2012-07-18 17:53 Ulrich Mueller
  2012-07-18 18:02 ` Robin H. Johnson
  2012-07-25 16:00 ` Mike Frysinger
  0 siblings, 2 replies; 10+ messages in thread
From: Ulrich Mueller @ 2012-07-18 17:53 UTC (permalink / raw
  To: gentoo-dev

Our current policy [1] requires that ebuilds must assign the seven
variables DESCRIPTION, HOMEPAGE, SRC_URI, LICENSE, SLOT, KEYWORDS, and
IUSE, even if their value is empty.

Could we drop this requirement? Repoman already enforces that
DESCRIPTION, HOMEPAGE, LICENSE, SLOT, and KEYWORDS are non-empty
(with some exceptions for virtuals). I don't see why we need to
distinguish the "empty value" and "not assigned" cases.

For example, many live ebuilds already don't define SRC_URI. I'm not
aware of any problems caused by this.

Ulrich

[1] <http://devmanual.gentoo.org/ebuild-writing/variables/#required-variables>



^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [gentoo-dev] Don't require assignment of empty variables in ebuilds?
  2012-07-18 17:53 [gentoo-dev] Don't require assignment of empty variables in ebuilds? Ulrich Mueller
@ 2012-07-18 18:02 ` Robin H. Johnson
  2012-07-18 18:11   ` Davide Pesavento
  2012-07-25 16:00 ` Mike Frysinger
  1 sibling, 1 reply; 10+ messages in thread
From: Robin H. Johnson @ 2012-07-18 18:02 UTC (permalink / raw
  To: gentoo-dev

On Wed, Jul 18, 2012 at 07:53:37PM +0200, Ulrich Mueller wrote:
> Our current policy [1] requires that ebuilds must assign the seven
> variables DESCRIPTION, HOMEPAGE, SRC_URI, LICENSE, SLOT, KEYWORDS, and
> IUSE, even if their value is empty.
> 
> Could we drop this requirement? Repoman already enforces that
> DESCRIPTION, HOMEPAGE, LICENSE, SLOT, and KEYWORDS are non-empty
> (with some exceptions for virtuals). I don't see why we need to
> distinguish the "empty value" and "not assigned" cases.
> 
> For example, many live ebuilds already don't define SRC_URI. I'm not
> aware of any problems caused by this.
+1 from me.

-- 
Robin Hugh Johnson
Gentoo Linux: Developer, Trustee & Infrastructure Lead
E-Mail     : robbat2@gentoo.org
GnuPG FP   : 11ACBA4F 4778E3F6 E4EDF38E B27B944E 34884E85



^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [gentoo-dev] Don't require assignment of empty variables in ebuilds?
  2012-07-18 18:02 ` Robin H. Johnson
@ 2012-07-18 18:11   ` Davide Pesavento
  0 siblings, 0 replies; 10+ messages in thread
From: Davide Pesavento @ 2012-07-18 18:11 UTC (permalink / raw
  To: gentoo-dev

On Wed, Jul 18, 2012 at 11:02 AM, Robin H. Johnson <robbat2@gentoo.org> wrote:
> On Wed, Jul 18, 2012 at 07:53:37PM +0200, Ulrich Mueller wrote:
>> Our current policy [1] requires that ebuilds must assign the seven
>> variables DESCRIPTION, HOMEPAGE, SRC_URI, LICENSE, SLOT, KEYWORDS, and
>> IUSE, even if their value is empty.
>>
>> Could we drop this requirement? Repoman already enforces that
>> DESCRIPTION, HOMEPAGE, LICENSE, SLOT, and KEYWORDS are non-empty
>> (with some exceptions for virtuals). I don't see why we need to
>> distinguish the "empty value" and "not assigned" cases.
>>
>> For example, many live ebuilds already don't define SRC_URI. I'm not
>> aware of any problems caused by this.
> +1 from me.
>

++

Thanks,
Pesa



^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [gentoo-dev] Don't require assignment of empty variables in ebuilds?
  2012-07-18 17:53 [gentoo-dev] Don't require assignment of empty variables in ebuilds? Ulrich Mueller
  2012-07-18 18:02 ` Robin H. Johnson
@ 2012-07-25 16:00 ` Mike Frysinger
  2012-07-25 16:38   ` Ulrich Mueller
  1 sibling, 1 reply; 10+ messages in thread
From: Mike Frysinger @ 2012-07-25 16:00 UTC (permalink / raw
  To: gentoo-dev

[-- Attachment #1: Type: Text/Plain, Size: 877 bytes --]

On Wednesday 18 July 2012 13:53:37 Ulrich Mueller wrote:
> Our current policy [1] requires that ebuilds must assign the seven
> variables DESCRIPTION, HOMEPAGE, SRC_URI, LICENSE, SLOT, KEYWORDS, and
> IUSE, even if their value is empty.
> 
> Could we drop this requirement? Repoman already enforces that
> DESCRIPTION, HOMEPAGE, LICENSE, SLOT, and KEYWORDS are non-empty
> (with some exceptions for virtuals). I don't see why we need to
> distinguish the "empty value" and "not assigned" cases.

i think we should clarify and say that when an eclass provides these, the 
ebuild need not.  completely missing DESCRIPTION/HOMEPAGE should be a warning 
(and maybe KEYWORDS), and LICENSE should be an error.  there are plenty of 
examples of SRC_URI not being set and that's fine (live ebuilds, ebuilds that 
only install out of $FILESDIR, virtuals, etc...).
-mike

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [gentoo-dev] Don't require assignment of empty variables in ebuilds?
  2012-07-25 16:00 ` Mike Frysinger
@ 2012-07-25 16:38   ` Ulrich Mueller
  2012-07-25 16:56     ` Pacho Ramos
                       ` (3 more replies)
  0 siblings, 4 replies; 10+ messages in thread
From: Ulrich Mueller @ 2012-07-25 16:38 UTC (permalink / raw
  To: gentoo-dev

>>>>> On Wed, 25 Jul 2012, Mike Frysinger wrote:

>> Our current policy [1] requires that ebuilds must assign the seven
>> variables DESCRIPTION, HOMEPAGE, SRC_URI, LICENSE, SLOT, KEYWORDS,
>> and IUSE, even if their value is empty.
>> 
>> Could we drop this requirement? Repoman already enforces that
>> DESCRIPTION, HOMEPAGE, LICENSE, SLOT, and KEYWORDS are non-empty
>> (with some exceptions for virtuals). I don't see why we need to
>> distinguish the "empty value" and "not assigned" cases.

> i think we should clarify and say that when an eclass provides
> these, the ebuild need not. completely missing DESCRIPTION/HOMEPAGE
> should be a warning (and maybe KEYWORDS), and LICENSE should be an
> error. there are plenty of examples of SRC_URI not being set and
> that's fine (live ebuilds, ebuilds that only install out of
> $FILESDIR, virtuals, etc...).

I think we have to distinguish between PMS and tree policy here.
The package manager should be able to handle any empty or missing
variables (except for DESCRIPTION and SLOT). Otherwise we'd have to
complicate the spec with additional case distinctions, e.g. for
virtuals.

On the other hand, tree policy (as enforced by repoman) wouldn't
really change. In the cases you've mentioned above, it already
displays errors or warnings. Repoman also doesn't distinguish between
empty and unset variables. The single exception to this is IUSE, which
is required to be present in an ebuild even if it's empty. Maybe we
should drop this requirement, too.

Ulrich


^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [gentoo-dev] Don't require assignment of empty variables in ebuilds?
  2012-07-25 16:38   ` Ulrich Mueller
@ 2012-07-25 16:56     ` Pacho Ramos
  2012-07-25 17:05       ` Rick "Zero_Chaos" Farina
  2012-07-25 18:15     ` Michał Górny
                       ` (2 subsequent siblings)
  3 siblings, 1 reply; 10+ messages in thread
From: Pacho Ramos @ 2012-07-25 16:56 UTC (permalink / raw
  To: gentoo-dev

[-- Attachment #1: Type: text/plain, Size: 235 bytes --]

El mié, 25-07-2012 a las 18:38 +0200, Ulrich Mueller escribió:
[...]
> The single exception to this is IUSE, which
> is required to be present in an ebuild even if it's empty. Maybe we
> should drop this requirement, too.

+1

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [gentoo-dev] Don't require assignment of empty variables in ebuilds?
  2012-07-25 16:56     ` Pacho Ramos
@ 2012-07-25 17:05       ` Rick "Zero_Chaos" Farina
  0 siblings, 0 replies; 10+ messages in thread
From: Rick "Zero_Chaos" Farina @ 2012-07-25 17:05 UTC (permalink / raw
  To: gentoo-dev

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 07/25/2012 12:56 PM, Pacho Ramos wrote:
> El mié, 25-07-2012 a las 18:38 +0200, Ulrich Mueller escribió:
> [...]
>> The single exception to this is IUSE, which
>> is required to be present in an ebuild even if it's empty. Maybe we
>> should drop this requirement, too.
> 
> +1

So many pointless lines of code removed...

+1
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.19 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iQIcBAEBAgAGBQJQECdJAAoJEKXdFCfdEflKHbgP/30clLKhZOPB1A+2R64nYlQk
Nvt+YBHtN0kFcvKQMwFc9t7jGwhBiLPHfNBW4bfZnXuXA1PIl57/cx6EgmihYNVF
xwKkwCd43ri4IfBU8HDSM7hS0ZrOzy2hRoyEAE9G15ab7L71ZWb9wJoM41Zrweoa
IGHVBGFhv4MrFehRReC4Tt1MObFydlz4vofszh1/mpSPxSOx2124Mrmoe5WaWr2X
+KSEVtG7H8XM3U/eFiZNGDAtRc5lF1Qt4bxSop8FmTe3pCZTCdnUip+SydZ2nLc5
LVzc0YfFdbnasQ+ujO/sG8KVY/ScJzmJMSOrsUD+8oxDiLpAu3q9Dl3yqSYe44nz
P9njp6tubIuJ4DkbEn4wthcHW6OIatsxRM4rVSEOzQiWgTGGpOzyugRNkzYMJwdd
FcUGeHnwZ6QnfhDi1/MsT8XzykmuGuj2I6+Vr2V3NdXWk2qLUnqadElfxIKnqZA/
mIBTCbBue5R72gRHlPqoQHwbwIAatzU15d1Xo20YogOCeAAYNko7AVqF5Dc22yGZ
8MZzCZbPu03htfa5WABVT1tN5O8HoEeilAJAxTM/W6quYullo2yseuGInY0zU9BP
xf4uft+EHKCKBQF7+xYEpXdQicwtSMQSrnJgfX5r/QKWZpQDmEiyiYO9h2oO+Zxa
NRG/tAktlDVrVUvJ/kN1
=bmvc
-----END PGP SIGNATURE-----


^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [gentoo-dev] Don't require assignment of empty variables in ebuilds?
  2012-07-25 16:38   ` Ulrich Mueller
  2012-07-25 16:56     ` Pacho Ramos
@ 2012-07-25 18:15     ` Michał Górny
  2012-07-25 18:32     ` Mike Frysinger
  2012-08-09  2:04     ` Zac Medico
  3 siblings, 0 replies; 10+ messages in thread
From: Michał Górny @ 2012-07-25 18:15 UTC (permalink / raw
  To: gentoo-dev; +Cc: ulm

[-- Attachment #1: Type: text/plain, Size: 1451 bytes --]

On Wed, 25 Jul 2012 18:38:05 +0200
Ulrich Mueller <ulm@gentoo.org> wrote:

> >>>>> On Wed, 25 Jul 2012, Mike Frysinger wrote:
> 
> >> Our current policy [1] requires that ebuilds must assign the seven
> >> variables DESCRIPTION, HOMEPAGE, SRC_URI, LICENSE, SLOT, KEYWORDS,
> >> and IUSE, even if their value is empty.
> >> 
> >> Could we drop this requirement? Repoman already enforces that
> >> DESCRIPTION, HOMEPAGE, LICENSE, SLOT, and KEYWORDS are non-empty
> >> (with some exceptions for virtuals). I don't see why we need to
> >> distinguish the "empty value" and "not assigned" cases.
> 
> > i think we should clarify and say that when an eclass provides
> > these, the ebuild need not. completely missing DESCRIPTION/HOMEPAGE
> > should be a warning (and maybe KEYWORDS), and LICENSE should be an
> > error. there are plenty of examples of SRC_URI not being set and
> > that's fine (live ebuilds, ebuilds that only install out of
> > $FILESDIR, virtuals, etc...).
> 
> I think we have to distinguish between PMS and tree policy here.
> The package manager should be able to handle any empty or missing
> variables (except for DESCRIPTION and SLOT). Otherwise we'd have to
> complicate the spec with additional case distinctions, e.g. for
> virtuals.

PMS should be able to handle empty DESCRIPTION (i.e. for hand-written
dirty test ebuilds). And an empty SLOT has its meaning.

-- 
Best regards,
Michał Górny

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 316 bytes --]

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [gentoo-dev] Don't require assignment of empty variables in ebuilds?
  2012-07-25 16:38   ` Ulrich Mueller
  2012-07-25 16:56     ` Pacho Ramos
  2012-07-25 18:15     ` Michał Górny
@ 2012-07-25 18:32     ` Mike Frysinger
  2012-08-09  2:04     ` Zac Medico
  3 siblings, 0 replies; 10+ messages in thread
From: Mike Frysinger @ 2012-07-25 18:32 UTC (permalink / raw
  To: gentoo-dev

[-- Attachment #1: Type: Text/Plain, Size: 1606 bytes --]

On Wednesday 25 July 2012 12:38:05 Ulrich Mueller wrote:
> >>>>> On Wed, 25 Jul 2012, Mike Frysinger wrote:
> >> Our current policy [1] requires that ebuilds must assign the seven
> >> variables DESCRIPTION, HOMEPAGE, SRC_URI, LICENSE, SLOT, KEYWORDS,
> >> and IUSE, even if their value is empty.
> >> 
> >> Could we drop this requirement? Repoman already enforces that
> >> DESCRIPTION, HOMEPAGE, LICENSE, SLOT, and KEYWORDS are non-empty
> >> (with some exceptions for virtuals). I don't see why we need to
> >> distinguish the "empty value" and "not assigned" cases.
> > 
> > i think we should clarify and say that when an eclass provides
> > these, the ebuild need not. completely missing DESCRIPTION/HOMEPAGE
> > should be a warning (and maybe KEYWORDS), and LICENSE should be an
> > error. there are plenty of examples of SRC_URI not being set and
> > that's fine (live ebuilds, ebuilds that only install out of
> > $FILESDIR, virtuals, etc...).
> 
> I think we have to distinguish between PMS and tree policy here.

who said anything about PMS ? ;)  this thread was all about tree policy.

> The package manager should be able to handle any empty or missing
> variables (except for DESCRIPTION and SLOT). Otherwise we'd have to
> complicate the spec with additional case distinctions, e.g. for
> virtuals.

SLOT is about the only variable i could see possibly being required by PMS to 
be set somewhere (whether eclass or ebuild doesn't matter).  all the others we 
could (and should) do away with from PMS requirement list considering our PMs 
handle it fine.
-mike

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [gentoo-dev] Don't require assignment of empty variables in ebuilds?
  2012-07-25 16:38   ` Ulrich Mueller
                       ` (2 preceding siblings ...)
  2012-07-25 18:32     ` Mike Frysinger
@ 2012-08-09  2:04     ` Zac Medico
  3 siblings, 0 replies; 10+ messages in thread
From: Zac Medico @ 2012-08-09  2:04 UTC (permalink / raw
  To: gentoo-dev

On 07/25/2012 09:38 AM, Ulrich Mueller wrote:

> On the other hand, tree policy (as enforced by repoman) wouldn't
> really change. In the cases you've mentioned above, it already
> displays errors or warnings. Repoman also doesn't distinguish between
> empty and unset variables. The single exception to this is IUSE, which
> is required to be present in an ebuild even if it's empty. Maybe we
> should drop this requirement, too.

IUSE.undefined is removed in Portage git:

http://git.overlays.gentoo.org/gitweb/?p=proj/portage.git;a=commit;h=f7ec7a4db6543e1fd3a13c341a37ccb667956978
-- 
Thanks,
Zac


^ permalink raw reply	[flat|nested] 10+ messages in thread

end of thread, other threads:[~2012-08-09  2:05 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-07-18 17:53 [gentoo-dev] Don't require assignment of empty variables in ebuilds? Ulrich Mueller
2012-07-18 18:02 ` Robin H. Johnson
2012-07-18 18:11   ` Davide Pesavento
2012-07-25 16:00 ` Mike Frysinger
2012-07-25 16:38   ` Ulrich Mueller
2012-07-25 16:56     ` Pacho Ramos
2012-07-25 17:05       ` Rick "Zero_Chaos" Farina
2012-07-25 18:15     ` Michał Górny
2012-07-25 18:32     ` Mike Frysinger
2012-08-09  2:04     ` Zac Medico

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox