[Yum-devel] [PATCH] Persistent enable/disable of repositories.

[Jeremy Katz]

On Wed, 2007-07-04 at 17:45 -0400, seth vidal wrote:
> On Thu, 2007-07-05 at 03:04 +0530, Debarshi 'Rishi' Ray wrote:
> > Please find attached a patch to incorporate the persistent
> > enable/disable of repositories by editing the .repo files.
> > Repositories mentioned in yum.conf are still not supported.
> > 
> > This is done using the following two methods:
> > YumRepository.enablePersistent(self)
> > YumRepository.disablePersistent(self)
> > 
> > This is just a proof of concept and your comments are awaited.
> 
> the only thing I've noticed from your patch is that the following
> behavior, which works now will no longer work:
[snip]
> currently we concatenate all the .repo files then parse them (also we
> end up with a single parser instance)
> 
> In your patch we'd have a parser instance for each of them and they
> would not be concatenated. now, I don't really have a problem with
> losing that behavior b/c I'm betting it is next to never used since most
> people who had not looked at that section of the code would never know
> how it happened anyway. :)
> 
> However, What does everyone else think?

I think I just lost my lunch ;-)

Seriously, if someone is counting on this behavior, then they're
completely and utterly insane

Jeremy