Change request: Another tmpfile fix

2009-02-02 Thread Ricky Zhou
Looks like there's a problem with the fullfilelist tmpfile removal for EPEL as well. Can I get some +1s for this patch? diff --git a/manifests/services/mirrormaster.pp b/manifests/services/mirrormaster.pp index ca01522..846d581 100644 --- a/manifests/services/mirrormaster.pp +++ b/manifests/serv

Re: Change request: Another tmpfile fix

2009-02-02 Thread Ricky Zhou
On 2009-02-02 09:31:08 AM, Ricky Zhou wrote: > On 2009-02-02 08:53:29 AM, Ricky Zhou wrote: > > On 2009-02-02 08:40:55 AM, Ricky Zhou wrote: > > > Looks like there's a problem with the fullfilelist tmpfile removal for > > > EPEL as well. Can I get some +1s for this patch? > > Actually, ignore that

Re: Change request: Another tmpfile fix

2009-02-02 Thread Ricky Zhou
On 2009-02-02 08:40:55 AM, Ricky Zhou wrote: > Looks like there's a problem with the fullfilelist tmpfile removal for > EPEL as well. Can I get some +1s for this patch? Actually, ignore that one. This might be a bit simpler: diff --git a/manifests/services/mirrormaster.pp b/manifests/services/m

Re: Change request: Another tmpfile fix

2009-02-02 Thread Jesse Keating
On Mon, 2009-02-02 at 09:35 -0500, Ricky Zhou wrote: > Sorry again. I just realized that I reversed the if statement. That's > what I get for not sleeping. Here the corrected version: +1, I'll change buildrawhide to match. -- Jesse Keating Fedora -- FreedomĀ² is a feature! identi.ca: http://i

Re: Change request: Another tmpfile fix

2009-02-02 Thread Nigel Jones
+1 - "Ricky Zhou" wrote: > On 2009-02-02 09:31:08 AM, Ricky Zhou wrote: > > On 2009-02-02 08:53:29 AM, Ricky Zhou wrote: > > > On 2009-02-02 08:40:55 AM, Ricky Zhou wrote: > > > > Looks like there's a problem with the fullfilelist tmpfile > removal for > > > > EPEL as well. Can I get some +1

Re: Change request: Another tmpfile fix

2009-02-02 Thread Ricky Zhou
On 2009-02-02 08:53:29 AM, Ricky Zhou wrote: > On 2009-02-02 08:40:55 AM, Ricky Zhou wrote: > > Looks like there's a problem with the fullfilelist tmpfile removal for > > EPEL as well. Can I get some +1s for this patch? > Actually, ignore that one. This might be a bit simpler: Oops, I forgot to f