On Wed, Jan 04, 2006 at 09:33:07PM -0500, Alec Warner wrote: > Author: ferringb > Date: 2006-01-04 08:57:07 +0000 (Wed, 04 Jan 2006) > New Revision: 2522 > > Modified: > main/trunk/pym/portage_dep.py > Log: > el buggo pointed out via spyderous. > || ( a ( x? ( b ) y? ( c ) ) ) > -x -y , was resulting in || ( a () ) > > the main consumer of this, portage.dep_check is stupid, and was assuming > () was valid. > It's not, obviously. > > Long term bug, around in at least .51 . Should correct dep_check > handling of it also, but no reason to be > handing () in the result lists also. > > Sadly this breaks something else we apparently need and use ( or at > least Ciaran tells me so :) ) > > DEPEND="|| ( )" which should evaluate to DEPEND="" doesn't with this > patch. It dies, which is bad, because while stupid, it should work.
Corrected with a nasty recursive filter tagged into dep_check. I really hate that code on a side note. ~harring
pgpsy0XVePl1v.pgp
Description: PGP signature
