[Bug 637868] Rename request: meego-panel-pasteboard - MeeGo Panel for Paste Board

bugzilla at redhat.com bugzilla at redhat.com
Fri Oct 22 15:37:00 UTC 2010

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


--- Comment #1 from Michel Alexandre Salim <fedora at michelsylvain.info> 2010-10-22 11:36:59 EDT ---
Two things to fix:
- license should probably be GPLv2+
- nobody owns the panels directory. mutter-meego should probably do this

* TODO Review [80%]
** DONE Names [2/2]
*** DONE Package name
*** DONE Spec name
** DONE Meets
** DONE source files match upstream
** TODO License [2/3]
*** DONE License is Fedora-approved
*** FAIL License field accurate
      License is GPLv2+ *not* LGPLv2+
*** DONE License included iff packaged by upstream
** DONE rpmlint [2/2]
*** DONE on src.rpm
    meego-panel-pasteboard.src:3: W: macro-in-comment %{version}
    meego-panel-pasteboard.src:3: W: macro-in-comment %{version}
    meego-panel-pasteboard.src:3: W: macro-in-comment %{name}
    meego-panel-pasteboard.src:3: W: macro-in-comment %{version}

    => harmless, but inconsistent: sometimes meego-panel-pasteboard is
    replaced with %{name}, sometimes not

    meego-panel-pasteboard.src: W: no-buildroot-tag

    => ignore; MeeGo only targeting recent Fedora versions

    meego-panel-pasteboard.src: W: invalid-url Source0:
    1 packages and 0 specfiles checked; 0 errors, 6 warnings.

    blame upstream :) hopefully soon we can use VCS URLs as source

*** DONE on x86_64.rpm
    2 packages and 0 specfiles checked; 0 errors, 0 warnings.
** DONE Language & locale [3/3]
*** DONE Spec in US English
*** DONE Spec legible
*** DONE Use %find_lang to handle locale files
** TODO Build [2/3]
*** DONE Koji results
*** DONE BRs complete
*** FAIL Directory ownership
    Nobody owns %{_datadir}/mutter-meego/panels -- should mutter-meego own it?
** DONE Spec inspection [9/9]
*** N/A ldconfig for libraries
*** DONE No duplicate files
*** DONE File permissions
*** DONE Filenames must be UTF-8
*** DONE Has %clean section
    note: %clean section can be removed on F-13+:
*** DONE %buildroot cleaned on %install
*** DONE Macro usage consistent
*** DONE Documentation [2/2]
**** N/A If large docs, separate -doc
**** DONE %doc files are non-essential
*** N/A Development
** N/A Desktop file validation
** N/A

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