[Bug 715127] Review Request: abcMIDI - ABC to/from MIDI conversion utilities

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

 



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

Volker Fröhlich <volker27@xxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |volker27@xxxxxx

--- Comment #1 from Volker Fröhlich <volker27@xxxxxx> 2011-06-26 18:32:48 EDT ---
Please append a slash to the URL, otherwise you gets a 503.

If you don't want to maintain this package in EPEL5 or older, remove Buildroot,
the first rm in the install section.

The FSF address is not up to date. This is not a blocker, but please inform
upstream.

Please use -p with install to preserve timestamps.

The defattr line is no longer needed.

Rather use %{_mandir}/man1/*, as you only install man1 pages. You can also be
more specific in the files section, so you don't package stuff inadvertently.
But this is rather unlikely in the case of this package.

Is the content of doc/programming of no interest?

"# remove executable flag from text files" -- There are none, so you can remove
that.

doc/CHANGES has Unix EOL. So has VERSION.

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