Re: [CVS] RPM: rpm/ CHANGES rpm/build/ parseFiles.c

2011-03-23 Thread Jeff Johnson
On Mar 23, 2011, at 9:44 PM, Per Øyvind Karlsen wrote: >>> >> >> I will look for you. todo++. >> Google "Michael Jennings RPM %clean" quickly finds one (of several) threads I remember: http://www.spinics.net/lists/rpm/msg05401.html As you can quickly see, the issue or removing %clean

Re: [CVS] RPM: rpm/ CHANGES rpm/build/ parseFiles.c

2011-03-23 Thread Jeff Johnson
On Mar 23, 2011, at 9:44 PM, Per Øyvind Karlsen wrote: >> > No, I'm not saying that I'm even planning on actively ripping out %clean > everywhere, but I do have started ripping out other things like > %defattr, BuildRoot: etc. every now and then. > I'm just arguing that's why I haven't ripped it

Re: [CVS] RPM: rpm/ CHANGES rpm/build/ parseFiles.c

2011-03-23 Thread Per Øyvind Karlsen
2011/3/24 Jeff Johnson : > > On Mar 23, 2011, at 5:19 PM, Per Øyvind Karlsen wrote: > >> 2011/3/23 Jeff Johnson : >>> >>> On Mar 23, 2011, at 5:01 PM, Per Øyvind Karlsen wrote: >>> > Yes, but this isn't run unless %clean exists in the spec file.. >>> >>> Not true (from 4+ year old mem

Re: [CVS] RPM: rpm/ CHANGES rpm/build/ parseFiles.c

2011-03-23 Thread Jeff Johnson
On Mar 23, 2011, at 5:19 PM, Per Øyvind Karlsen wrote: > 2011/3/23 Jeff Johnson : >> >> On Mar 23, 2011, at 5:01 PM, Per Øyvind Karlsen wrote: >> >>> Yes, but this isn't run unless %clean exists in the spec file.. >>> >> >> Not true (from 4+ year old memory). But I've forgot. > Well, yo

Re: [CVS] RPM: rpm/ CHANGES rpm/build/ parseFiles.c

2011-03-23 Thread Per Øyvind Karlsen
2011/3/23 Jeff Johnson : > > On Mar 23, 2011, at 5:01 PM, Per Øyvind Karlsen wrote: > >>> >> Yes, but this isn't run unless %clean exists in the spec file.. >> > > Not true (from 4+ year old memory). But I've forgot. Well, your memory seems to be wrong then.. ;) This is the reason why I've not yet

Re: [CVS] RPM: rpm/ CHANGES rpm/build/ parseFiles.c

2011-03-23 Thread Jeff Johnson
On Mar 23, 2011, at 5:01 PM, Per Øyvind Karlsen wrote: >> > Yes, but this isn't run unless %clean exists in the spec file.. > Not true (from 4+ year old memory). But I've forgot. PART_CLEAN (because its compiled in) is my major objection. I also see no need for overrides and all the usual stu

Re: [CVS] RPM: rpm/ CHANGES rpm/build/ parseFiles.c

2011-03-23 Thread Per Øyvind Karlsen
2011/3/23 Jeff Johnson : > > On Mar 23, 2011, at 4:51 PM, Jeff Johnson wrote: > >> This is a reversion from what was intended and already implemented >> almost 4 years ago. >> >> Under #ifdef RPM_VENDOR_MANDRIVA please. >> > > Since your first question is likely: >        OK where is it? > > The mu

Re: [CVS] RPM: rpm/ CHANGES rpm/build/ parseFiles.c

2011-03-23 Thread Per Øyvind Karlsen
2011/3/23 Jeff Johnson : > This is a reversion from what was intended and already implemented > almost 4 years ago. > > Under #ifdef RPM_VENDOR_MANDRIVA please. So %clean shouldn't be run by default if not added to .spec? As %clean is a preferred default, it would seem backwards not having it invo

Re: [CVS] RPM: rpm/ CHANGES rpm/build/ parseFiles.c

2011-03-23 Thread Jeff Johnson
On Mar 23, 2011, at 4:51 PM, Jeff Johnson wrote: > This is a reversion from what was intended and already implemented > almost 4 years ago. > > Under #ifdef RPM_VENDOR_MANDRIVA please. > Since your first question is likely: OK where is it? The murky path to automated (and configurable

Re: [CVS] RPM: rpm/ CHANGES rpm/build/ parseFiles.c

2011-03-23 Thread Jeff Johnson
This is a reversion from what was intended and already implemented almost 4 years ago. Under #ifdef RPM_VENDOR_MANDRIVA please. 73 de Jeff On Mar 23, 2011, at 4:48 PM, Per Øyvind Karlsen wrote: > RPM Package Manager, CVS Repository > http://rpm5.org/cvs/ > ___