[Bug 678128] Review Request: cdw - Front-end for tools used for burning data CD/DVD

bugzilla at redhat.com bugzilla at redhat.com
Mon Feb 28 22:13:10 UTC 2011


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

--- Comment #17 from Sergio Belkin <sebelk at gmail.com> 2011-02-28 17:13:09 EST ---
(In reply to comment #16)

Hi Mamoru

Firstly the newly files URL's:
***
http://sebelk.fedorapeople.org/cdw-0.6.0-8.fc14.src.rpm
http://sebelk.fedorapeople.org/cdw.spec
***


> (In reply to comment #15)
> > (In reply to comment #14)
> > > * Timestamps
> > >   - Please consider to use
> > > ---------------------------------------------------------
> > > make install DESTDIR=$RPM_BUILD_ROOT INSTALL="install -p"
> > > ---------------------------------------------------------
> 
> The above line is one line (i.e. INSTALL="install -p" is the option
> of "make" command).

Shame on me :(

> 
> > >     to keep timestamps on installed man file.
> > 
> > > ? ${LIBS}
> > >   - It seems that before setting LIBS environ LIBS is null, and
> > >     LIBS="-lm" is sufficient.
> > 
> > Yup, it's null, fixed too!
> 
> This does not seem to be fixed yet.

Id. above.

> 
> By the way, now you can use fedorapeople.org site for hosting your srpm
> and so on. Please try it, see:
> https://fedoraproject.org/wiki/Fedorapeople.org

Now those mistakes are fixed and files are in fedorapeople.org :)

Thanks in advance

-- 
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 package-review mailing list