public inbox for gentoo-dev@lists.gentoo.org
 help / color / mirror / Atom feed
* [gentoo-dev] [GLEP 56] metadata.xml USE flag descriptions [Clarifications]
@ 2008-08-13 17:11 Doug Goldstein
  2008-08-13 18:07 ` Santiago M. Mola
                   ` (2 more replies)
  0 siblings, 3 replies; 10+ messages in thread
From: Doug Goldstein @ 2008-08-13 17:11 UTC (permalink / raw
  To: gentoo-dev, gentoo-dev-announce

Howdy all,

Further questions regarding use.desc have come up with regard to this 
GLEP. My proposed solution would be a potential amendment to the GLEP to 
state that

<flag name='png' />

Would be allowed. This syntax is not actually disallowed or allowed by 
the current GLEP, but mentioning it would allow a metadata.xml contain 
all the USE flags that appear in IUSE, even the global ones. By using 
the above syntax, it would simply state that there is no additional 
descriptions or details but to just use the use.desc description.

Further more, it would allow us in the future to make that mandatory and 
repoman would only have to check metadata.xml for your USE flag.

Comments, Suggestions, Input are all welcome.

--
Doug Goldstein
Gentoo Developer



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

* Re: [gentoo-dev] [GLEP 56] metadata.xml USE flag descriptions [Clarifications]
  2008-08-13 17:11 [gentoo-dev] [GLEP 56] metadata.xml USE flag descriptions [Clarifications] Doug Goldstein
@ 2008-08-13 18:07 ` Santiago M. Mola
  2008-08-13 20:13   ` Doug Goldstein
  2008-08-13 18:59 ` Josh Saddler
  2008-08-18 15:16 ` Donnie Berkholz
  2 siblings, 1 reply; 10+ messages in thread
From: Santiago M. Mola @ 2008-08-13 18:07 UTC (permalink / raw
  To: gentoo-dev

On Wed, Aug 13, 2008 at 7:11 PM, Doug Goldstein <cardoe@gentoo.org> wrote:
> Howdy all,
>
> Further questions regarding use.desc have come up with regard to this GLEP.
> My proposed solution would be a potential amendment to the GLEP to state
> that
>
> <flag name='png' />
>
> Would be allowed. This syntax is not actually disallowed or allowed by the
> current GLEP, but mentioning it would allow a metadata.xml contain all the
> USE flags that appear in IUSE, even the global ones. By using the above
> syntax, it would simply state that there is no additional descriptions or
> details but to just use the use.desc description.
>
> Further more, it would allow us in the future to make that mandatory and
> repoman would only have to check metadata.xml for your USE flag.
>
> Comments, Suggestions, Input are all welcome.
>

What is the benefit?

Regards,
-- 
Santiago M. Mola
Jabber ID: cooldwind@gmail.com



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

* Re: [gentoo-dev] [GLEP 56] metadata.xml USE flag descriptions [Clarifications]
  2008-08-13 17:11 [gentoo-dev] [GLEP 56] metadata.xml USE flag descriptions [Clarifications] Doug Goldstein
  2008-08-13 18:07 ` Santiago M. Mola
@ 2008-08-13 18:59 ` Josh Saddler
  2008-08-13 19:15   ` Mark Loeser
  2008-08-14 11:21   ` Jan Kundrát
  2008-08-18 15:16 ` Donnie Berkholz
  2 siblings, 2 replies; 10+ messages in thread
From: Josh Saddler @ 2008-08-13 18:59 UTC (permalink / raw
  To: gentoo-dev

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

Doug Goldstein wrote:
> Howdy all,
> 
> Further questions regarding use.desc have come up with regard to this 
> GLEP. My proposed solution would be a potential amendment to the GLEP to 
> state that
> 
> <flag name='png' />

> Comments, Suggestions, Input are all welcome.

If you're going for proper XML, then it should look like this:

<flag name="png"/>

XML doesn't put a space between the attribute and the closing slash -- 
XHTML does. Common mistake. Also, use " for attributes, rather than '.


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 197 bytes --]

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

* Re: [gentoo-dev] [GLEP 56] metadata.xml USE flag descriptions [Clarifications]
  2008-08-13 18:59 ` Josh Saddler
@ 2008-08-13 19:15   ` Mark Loeser
  2008-08-14 11:21   ` Jan Kundrát
  1 sibling, 0 replies; 10+ messages in thread
From: Mark Loeser @ 2008-08-13 19:15 UTC (permalink / raw
  To: gentoo-dev

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

Josh Saddler <nightmorph@gentoo.org> said:
> If you're going for proper XML, then it should look like this:
>
> <flag name="png"/>
>
> XML doesn't put a space between the attribute and the closing slash -- 
> XHTML does. Common mistake. Also, use " for attributes, rather than '.

Well, if we want to nitpick, adding a space, tab, or newline is
perfectly valid, as is using ' to quote the attribute. :)

http://www.w3.org/TR/2006/REC-xml11-20060816/#NT-EmptyElemTag
http://www.w3.org/TR/2006/REC-xml11-20060816/#NT-AttValue

-- 
Mark Loeser
email         -   halcy0n AT gentoo DOT org
email         -   mark AT halcy0n DOT com
web           -   http://www.halcy0n.com

[-- Attachment #2: Type: application/pgp-signature, Size: 197 bytes --]

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

* Re: [gentoo-dev] [GLEP 56] metadata.xml USE flag descriptions [Clarifications]
  2008-08-13 18:07 ` Santiago M. Mola
@ 2008-08-13 20:13   ` Doug Goldstein
  2008-08-15 13:35     ` Jeroen Roovers
  2008-08-18 21:48     ` Tobias Scherbaum
  0 siblings, 2 replies; 10+ messages in thread
From: Doug Goldstein @ 2008-08-13 20:13 UTC (permalink / raw
  To: gentoo-dev

Santiago M. Mola wrote:
> On Wed, Aug 13, 2008 at 7:11 PM, Doug Goldstein <cardoe@gentoo.org> wrote:
>   
>> Howdy all,
>>
>> Further questions regarding use.desc have come up with regard to this GLEP.
>> My proposed solution would be a potential amendment to the GLEP to state
>> that
>>
>> <flag name='png' />
>>
>> Would be allowed. This syntax is not actually disallowed or allowed by the
>> current GLEP, but mentioning it would allow a metadata.xml contain all the
>> USE flags that appear in IUSE, even the global ones. By using the above
>> syntax, it would simply state that there is no additional descriptions or
>> details but to just use the use.desc description.
>>
>> Further more, it would allow us in the future to make that mandatory and
>> repoman would only have to check metadata.xml for your USE flag.
>>
>> Comments, Suggestions, Input are all welcome.
>>
>>     
>
> What is the benefit?
>
> Regards,
>   
There is none really. Allow all use flags to exist in metadata.xml. It's 
really more of a clarification to the GLEP if this is allowed.



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

* Re: [gentoo-dev] [GLEP 56] metadata.xml USE flag descriptions [Clarifications]
  2008-08-13 18:59 ` Josh Saddler
  2008-08-13 19:15   ` Mark Loeser
@ 2008-08-14 11:21   ` Jan Kundrát
  1 sibling, 0 replies; 10+ messages in thread
From: Jan Kundrát @ 2008-08-14 11:21 UTC (permalink / raw
  To: gentoo-dev

Josh Saddler wrote:
> XML doesn't put a space between the attribute and the closing slash --
> XHTML does. Common mistake. Also, use " for attributes, rather than '.

Nope, both is perfectly legal in XML (and illegal per the GDP coding
style, which certainly doesn't apply to metadata.xml) :p.

Cheers,
-jkt



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

* Re: [gentoo-dev] [GLEP 56] metadata.xml USE flag descriptions [Clarifications]
  2008-08-13 20:13   ` Doug Goldstein
@ 2008-08-15 13:35     ` Jeroen Roovers
  2008-08-18 13:54       ` Doug Goldstein
  2008-08-18 21:48     ` Tobias Scherbaum
  1 sibling, 1 reply; 10+ messages in thread
From: Jeroen Roovers @ 2008-08-15 13:35 UTC (permalink / raw
  To: gentoo-dev

On Wed, 13 Aug 2008 16:13:26 -0400
Doug Goldstein <cardoe@gentoo.org> wrote:

> > What is the benefit?

> There is none really. Allow all use flags to exist in metadata.xml.
> It's really more of a clarification to the GLEP if this is allowed.

I personally think that this would facilitate a lot of duplication that
would ultimately amount to bitrot. Of course having, say, repoman
suggest cleanups would be one solution to the problem of bitrot, but
then I still fail to see the advantage over having both an IUSE per
ebuild as well as maintaining <gentoo-x86/profiles/use.desc>.

In my opinion the GLEP needs clarification on this matter. It should
forbid the use of these empty flag tags, and the DTD should reflect
that[1].


Kind regards,
     JeR


[1] Come to think of it, in the recent metadata.xml / no-herd debate,
wasn't having an empty herd tag ever suggested? <herd />



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

* Re: [gentoo-dev] [GLEP 56] metadata.xml USE flag descriptions  [Clarifications]
  2008-08-15 13:35     ` Jeroen Roovers
@ 2008-08-18 13:54       ` Doug Goldstein
  0 siblings, 0 replies; 10+ messages in thread
From: Doug Goldstein @ 2008-08-18 13:54 UTC (permalink / raw
  To: gentoo-dev

Jeroen Roovers wrote:
> On Wed, 13 Aug 2008 16:13:26 -0400
> Doug Goldstein <cardoe@gentoo.org> wrote:
>
>   
>>> What is the benefit?
>>>       
>
>   
>> There is none really. Allow all use flags to exist in metadata.xml.
>> It's really more of a clarification to the GLEP if this is allowed.
>>     
>
>   
<snip>
> [1] Come to think of it, in the recent metadata.xml / no-herd debate,
> wasn't having an empty herd tag ever suggested? <herd />
>
>   
I've been championing that for what feels like 3 years now. The problem 
is it breaks backwards compat with all tools out there..




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

* Re: [gentoo-dev] [GLEP 56] metadata.xml USE flag descriptions [Clarifications]
  2008-08-13 17:11 [gentoo-dev] [GLEP 56] metadata.xml USE flag descriptions [Clarifications] Doug Goldstein
  2008-08-13 18:07 ` Santiago M. Mola
  2008-08-13 18:59 ` Josh Saddler
@ 2008-08-18 15:16 ` Donnie Berkholz
  2 siblings, 0 replies; 10+ messages in thread
From: Donnie Berkholz @ 2008-08-18 15:16 UTC (permalink / raw
  To: gentoo-dev

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

On 13:11 Wed 13 Aug     , Doug Goldstein wrote:
> Further questions regarding use.desc have come up with regard to this  
> GLEP. My proposed solution would be a potential amendment to the GLEP to  
> state that
>
> <flag name='png' />
>
> Would be allowed. This syntax is not actually disallowed or allowed by  
> the current GLEP, but mentioning it would allow a metadata.xml contain  
> all the USE flags that appear in IUSE, even the global ones. By using  
> the above syntax, it would simply state that there is no additional  
> descriptions or details but to just use the use.desc description.
>
> Further more, it would allow us in the future to make that mandatory and  
> repoman would only have to check metadata.xml for your USE flag.

It seems like this doesn't have much benefit and is a bit confusing to 
me. You now need to know which flags in metadata.xml are global so you 
don't allow descriptions for them. You also need to verify the globals 
between the two places they'll be specified (metadata.xml and use.desc 
for the description) so you don't have things claiming they're global 
but aren't. The benefit here doesn't end up saving anything at all once 
you have a consistency check anyway.

Halcy0n also mentioned that this gets really annoying when USE flags 
inherited from eclasses change. You'd need to edit every metadata.xml of 
all inheriting packages.

-- 
Thanks,
Donnie

Donnie Berkholz
Developer, Gentoo Linux
Blog: http://dberkholz.wordpress.com

[-- Attachment #2: Type: application/pgp-signature, Size: 197 bytes --]

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

* Re: [gentoo-dev] [GLEP 56] metadata.xml USE flag descriptions [Clarifications]
  2008-08-13 20:13   ` Doug Goldstein
  2008-08-15 13:35     ` Jeroen Roovers
@ 2008-08-18 21:48     ` Tobias Scherbaum
  1 sibling, 0 replies; 10+ messages in thread
From: Tobias Scherbaum @ 2008-08-18 21:48 UTC (permalink / raw
  To: gentoo-dev

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

Doug Goldstein wrote:
> > What is the benefit?
> >
> > Regards,
> >   
> There is none really. Allow all use flags to exist in metadata.xml. It's 
> really more of a clarification to the GLEP if this is allowed.

Agreed, it has no benefit at all plus would lead to some kind of useless
duplication of information. Stating <flag name='png' /> in metadata.xml
for global use flags makes basically no difference to IUSE="png", except
that we already have the latter one.

  Tobias

[-- Attachment #2: Dies ist ein digital signierter Nachrichtenteil --]
[-- Type: application/pgp-signature, Size: 197 bytes --]

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

end of thread, other threads:[~2008-08-18 21:48 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-08-13 17:11 [gentoo-dev] [GLEP 56] metadata.xml USE flag descriptions [Clarifications] Doug Goldstein
2008-08-13 18:07 ` Santiago M. Mola
2008-08-13 20:13   ` Doug Goldstein
2008-08-15 13:35     ` Jeroen Roovers
2008-08-18 13:54       ` Doug Goldstein
2008-08-18 21:48     ` Tobias Scherbaum
2008-08-13 18:59 ` Josh Saddler
2008-08-13 19:15   ` Mark Loeser
2008-08-14 11:21   ` Jan Kundrát
2008-08-18 15:16 ` Donnie Berkholz

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