[Bug 821094] Review Request: scamper - A network measurement tool

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=821094

--- Comment #2 from Fabian Affolter <mail@xxxxxxxxxxxxxxxxxx> ---
Thanks Volker.

(In reply to comment #1)
> The description of the devel package mentions "Static libraries", which is
> not correct. It also doesn't contain documentation, as advertised.

I took a standard text, changed. 

> Please use the name macro in a consistent way.

Added macro for version, too.

> The base package must be required in a architecture dependent way, i. e.
> using ?_isa:
> http://fedoraproject.org/wiki/Packaging:Guidelines#Requiring_Base_Package

Missed that, seams to be new in the guidelines.

> Please leave a comment on how to obtain the source tarball.

The tarball is provided by upstream.

> Please ask upstream to provide a license file, if you haven't already.

I asked upstream about the integration of the COPYING.

Updated files:
Spec URL: http://fab.fedorapeople.org/packages/SRPMS/scamper.spec
SRPM URL:
http://fab.fedorapeople.org/packages/SRPMS/scamper-0-0.2.20111202b.fc16.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review



[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]