Gentoo Archives: gentoo-portage-dev

From: Zac Medico <zmedico@g.o>
To: gentoo-portage-dev@l.g.o
Subject: Re: [gentoo-portage-dev] Re: Patch problem
Date: Thu, 03 Mar 2011 18:23:33
Message-Id: 4D6FDC8B.4090608@gentoo.org
In Reply to: [gentoo-portage-dev] Re: Patch problem by Michael
1 On 03/03/2011 09:47 AM, Michael wrote:
2 > Zac Medico wrote:
3 >>
4 >> It seems like you wouldn't have to modify the code that your patch
5 >> modifies, since essentially what you want is --autounmask except that
6 >> you want to skip the part about editing config files. Also note that
7 >> there is an existing feature request to edit config files automatically:
8 >>
9 >> http://bugs.gentoo.org/show_bug.cgi?id=345775
10 >
11 > Interesting, especially #258371 as referenced by the above - it seems to be
12 > exactly what I was after.
13 >
14 > If I'm way off even in terms of code location, where would a more
15 > appropriate place for me to hack this be?
16
17 Inside depgraph._resolve(), it returns False if
18 self._dynamic_config._needed_use_config_changes is non-empty. You want
19 to add an option that causes it to return True instead. Also, you'll
20 need to propagate these changes to the config.setcpv() method somehow,
21 so that the changes will be applied by the Scheduler at build time.
22 Normally, the config.setcpv() method calculates USE based on config
23 files, but you want it override the config files with whatever values
24 the depgraph's _needed_use_config_changes contains.
25 --
26 Thanks,
27 Zac

Replies

Subject Author
[gentoo-portage-dev] Re: Re: Patch problem Michael <kensington@×××××××××××.net>
[gentoo-portage-dev] Re: Re: Patch problem Michael <kensington@×××××××××××.net>