Re: [gentoo-dev] Re: Removal of kdeprefix news item

2011-05-23 Thread Ciaran McCreesh
On Sun, 22 May 2011 16:52:55 -0400
Jonathan Callen a...@gentoo.org wrote:
 Here's the updated news item, which I will be committing very
 shortly. Please note that there is absolutely no version of portage
 that was not in package.mask that would have problems with an EAPI-2
 atom in a news item because EAPI-2 support was added to portage at
 the same time as news item support (they both appeared in portage
 2.1.6).

Please don't. This needs to be figured out properly.

-- 
Ciaran McCreesh


signature.asc
Description: PGP signature


Re: [gentoo-dev] Re: Removal of kdeprefix news item

2011-05-19 Thread Marc Schiffbauer
* Jonathan Callen schrieb am 19.05.11 um 11:49 Uhr:
 Ulrich Mueller wrote:
 
  On Wed, 18 May 2011, Jonathan Callen wrote:
  
  Display-If-Installed: kde-base/kdelibs[kdeprefix]
  
  I don't think that USE dependencies (or any other EAPI specific
  features) are allowed here.
 
 The GLEP does not make any meantion as to whether it is legal to use 
 USE dependencies here.  I have, however, tested with portage and 
 portage itself Does The Right Thing when faced with a USE dep in 
 Display-If-Installed.
 
  run: emerge --oneshot $(qlist -IC kde-base/)
  
  Is it guaranteed that your users have portage-utils installed?
  Otherwise the qlist command may not be available.
  
 
 I had thought that we depended on this, but I was mistaken (the 
 dependency was pulled in via pambase).  I guess we could just use 
 `emerge --update --deep --newuse @world` (and yes, @world is available 
 in stable portage).

test -x /usr/bin/qlist  emerge --oneshot $(qlist -IC kde-base/) \
|| emerge --update --deep --newuse @world

?

-Marc
-- 
8AAC 5F46 83B4 DB70 8317  3723 296C 6CCA 35A6 4134



Re: [gentoo-dev] Re: Removal of kdeprefix news item

2011-05-19 Thread Alec Warner
On Thu, May 19, 2011 at 2:49 AM, Jonathan Callen a...@gentoo.org wrote:
 Ulrich Mueller wrote:

 On Wed, 18 May 2011, Jonathan Callen wrote:

 Display-If-Installed: kde-base/kdelibs[kdeprefix]

 I don't think that USE dependencies (or any other EAPI specific
 features) are allowed here.

 The GLEP does not make any meantion as to whether it is legal to use
 USE dependencies here.  I have, however, tested with portage and
 portage itself Does The Right Thing when faced with a USE dep in
 Display-If-Installed.

reading the code for the version of portage I have installed:

The NewsManager will call Isvalid on each news item and each news item
will scan its restrictions for 'valid-ness' and report that upward. So
if we are lucky, on old versions of portage, portage.isvalidatom will
report the USE-dep as an invalid atom and portage will print mean
errors messages about 'invalid news items' but will not throw up (in
theory).


 run: emerge --oneshot $(qlist -IC kde-base/)

 Is it guaranteed that your users have portage-utils installed?
 Otherwise the qlist command may not be available.


 I had thought that we depended on this, but I was mistaken (the
 dependency was pulled in via pambase).  I guess we could just use
 `emerge --update --deep --newuse @world` (and yes, @world is available
 in stable portage).

 --
 Jonathan Callen