Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=592670 --- Comment #8 from Chen Lei <[email protected]> 2010-05-18 10:40:00 EDT --- (In reply to comment #7) > (In reply to comment #6) > > 1.Patch1 is not enough, you should also remove some gcc switches in CFLAGS: > > -W > > -Wall -std=c99 -pedantic -Os > make CFLAGS="$RPM_OPT_FLAGS" linux > > 3. > > use install instead of %{__install} > > > > rpm --eval %__install > > /usr/bin/install > > > > rpm --eval %__make > > /usr/bin/make > > > > rpm --eval %__rm > > /usr/bin/rm > No idea why you are fighting these macros. These macros are correct, there is > nothing wrong in using them. Several months, there's a disscussion in fedora-packaging maillist most packagers object using those unesscessary macros. Actually for seasoned packagers, I think use those macros are acceptable if they can use them consistently throughout the spec. But, for new packagers, they can hardly use those macros consistently, taken this spec for a example: The packager choose %{__install}, but still use rm, make, mkdir. This breaks the principle of pick one packaging style and use it consistently. For me, I always suggest new packagers not to use those macros, few new packagers can remember those rpm macros which have shorten alternative commands. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug. _______________________________________________ package-review mailing list [email protected] https://admin.fedoraproject.org/mailman/listinfo/package-review
