Re: [CVS] RPM: rpm/build/ parsePrep.c

2007-07-20 Thread Ralf S. Engelschall
On Fri, Jul 20, 2007, Jeff Johnson wrote: [...] simplify macro undefined test, remove absolute path restriction. Ah, even better. Thanks, Jeff. Ralf S. Engelschall [EMAIL PROTECTED]

Re: [CVS] RPM: rpm/build/ parsePrep.c

2007-07-20 Thread Jeff Johnson
On Jul 20, 2007, at 9:02 AM, Jeff Johnson wrote: RPM Package Manager, CVS Repository http://rpm5.org/cvs/ __ __ Server: rpm5.org Name: Jeff Johnson Root: /v/rpm/cvs

Re: [CVS] RPM: rpm/build/ parsePrep.c

2007-07-20 Thread Ralf S. Engelschall
On Fri, Jul 20, 2007, Jeff Johnson wrote: On Jul 20, 2007, at 9:02 AM, Jeff Johnson wrote: grr, stoopid trailing '/' on directories. perhaps %{?foo/} expansion? Having embarrassed myself with Yet Another Corner Case, the attached patch permits the rather clunky

Re: [CVS] RPM: rpm/build/ parsePrep.c

2007-07-20 Thread Jeff Johnson
On Jul 20, 2007, at 11:56 AM, Ralf S. Engelschall wrote: On Fri, Jul 20, 2007, Jeff Johnson wrote: On Jul 20, 2007, at 9:02 AM, Jeff Johnson wrote: grr, stoopid trailing '/' on directories. perhaps %{?foo/} expansion? Having embarrassed myself with Yet Another Corner Case, the

Re: [CVS] RPM: rpm/build/ parsePrep.c

2007-07-20 Thread Jason Corley
But since everything else in rpm is on the table, why not rip the current macro implementation as well? I'd be in favor of something that a) has a defined and verifiable grammar, and b) doesn't hurt my eyes with complex operations. :-) Way back when I thought xmlspec was headed in that

Re: [CVS] RPM: rpm/build/ parsePrep.c

2007-07-20 Thread Jeff Johnson
On Jul 20, 2007, at 2:10 PM, Jason Corley wrote: But since everything else in rpm is on the table, why not rip the current macro implementation as well? I'd be in favor of something that a) has a defined and verifiable grammar, and b) doesn't hurt my eyes with complex operations. :-) Way

Re: [CVS] RPM: rpm/build/ parsePrep.c

2007-07-03 Thread Jeff Johnson
%__tar instead of %_tarbin. I fixed that but haven't checked in. 73 de Jeff On Jul 2, 2007, at 8:55 PM, Mark Hatle wrote: RPM Package Manager, CVS Repository http://rpm5.org/cvs/ __ __ Server: rpm5.org