[Bug 199484] New: Review Request: notify-python - python bindings for libnotify

[bugzilla at redhat.com]

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




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

           Summary: Review Request: notify-python - python bindings for
                    libnotify
           Product: Fedora Core
           Version: devel
          Platform: All
        OS/Version: Linux
            Status: NEW
          Severity: normal
          Priority: normal
         Component: Package Review
        AssignedTo: notting at redhat.com
        ReportedBy: jdennis at redhat.com
         QAContact: extras-qa at fedoraproject.org
                CC: fedora-package-review at redhat.com


Spec URL: ftp://people.redhat.com/jdennis/notify-python.spec
SRPM URL: ftp://people.redhat.com/jdennis/notify-python-0.1.0-1.src.rpm

Description:

These are the python bindings for libnotify. Not much to say here, this is a pretty simple package. Perhaps two items of note:

* The package name was set to match upstream, arguments could be made for either of the two following names: "python-notify" because that what the guildlines say, or "pynotify" because thats the python import name. I discussed this with Jeremy Katz and he felt making the package name match upstream was the best, sounds good to me too.

* The only thing the package installs is the bindings, no doc. There are some test python files which could be used as examples. Perhaps they should go into /usr/share/doc/notify-python-*/examples? Jeremy thought it was fine not to install the test examples and because this is an expedient exercise, other packages in FC6 need the bindings, being minimal for an initial package sounds like the right approach.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.