[Bug 543425] Review Request: gource - Software version control visualization

bugzilla at redhat.com bugzilla at redhat.com
Thu Dec 3 06:30:25 UTC 2009


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=543425





--- Comment #4 from Rakesh Pandit <rpandit at redhat.com>  2009-12-03 01:30:24 EDT ---
""
autoreconf
%configure --prefix=/usr
make
""

needs to be replaced with

""
%configure
make %{?_smp_mflags}
""

%configure macro already has --prefix=/usr and other options and a lone make is
not advisable.

moreover timestamp needs to be saved, while copying files into BUILDROOT in
%install section.

I would also suggest to try and get rid of patch for version correction, as
running autoreconf is not advised. If you really care about package version,
try sed in configure script.

Thanks,

-- 
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.




More information about the Fedora-package-review mailing list