[Bug 632970] Review Request: rec-applet - An audio recording applet for the GNOME-desktop

bugzilla at redhat.com bugzilla at redhat.com
Sun Sep 12 10:13:53 UTC 2010

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


Mohamed El Morabity <pikachu.2014 at gmail.com> changed:

           What    |Removed                     |Added
             Status|NEW                         |ASSIGNED
                 CC|                            |pikachu.2014 at gmail.com
         AssignedTo|nobody at fedoraproject.org    |pikachu.2014 at gmail.com
               Flag|                            |fedora-review?

--- Comment #1 from Mohamed El Morabity <pikachu.2014 at gmail.com> 2010-09-12 06:13:51 EDT ---

some comments to start:

1) be careful with the release: since the sources are taken from a VCS, the
Release tag should look like:
   Release:   0.1.20100821bzr
("1" corresponds to the "release number increment", "20100821" to the date of
the last commit in the bazaar repo and "bzr" to the VCS used by upstream).

See for more details:

2) In Summary, the dash in "GNOME-desktop" is useless maybe

3) I don't think gstreamer-java is really required

4) In %description, maybe you should remove the reference to MP3 in the list of
supported formats... 
    Dont forget also a dot at the end of the description

5) DON'T use CFLAGS to set additional linker flags. You'd better use LDFLAGS:
   export LDFLAGS=" -lm"
   make %{?_smp_mflags}

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