Bug 199484 - Review Request: notify-python - python bindings for libnotify
Summary: Review Request: notify-python - python bindings for libnotify
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: David Cantrell
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: FC-ACCEPT
TreeView+ depends on / blocked
 
Reported: 2006-07-19 20:07 UTC by John Dennis
Modified: 2013-01-10 01:27 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2006-07-22 12:13:44 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description John Dennis 2006-07-19 20:07:44 UTC
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.

Comment 1 Bill Nottingham 2006-07-19 20:19:49 UTC
Tech ack.

Comment 2 Jesse Keating 2006-07-19 23:21:31 UTC
NEEDSWORK:

Compiled modules are arch specific, you need to use sitearch rather than sitelib.

Thus, sitelib isn't used anywhere, why define it?

Don't need the requires on python-abi, thats automagic now.

Needed to add pkgconfig, libnotify-devel, pygtk2-devel, gtk2-devel,
dbus-glib-devel, dbus-devel as buildreqs.

No rpmlint errors or warnings to worry about.  Please fix above issues.


Comment 3 John Dennis 2006-07-20 14:32:05 UTC
Fixed above issues, except I could not find dbus-glib-devel in rawhide, so I
omitted that package.

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

Added Jeremy to CC, as he needs this as well.

Comment 4 Jesse Keating 2006-07-20 14:42:18 UTC
dbus-glib hit rawhide last night, I was testing in brew with the current
dist-fc6 stuff, which included dbus-glib.  Please add it back, this will not
compile w/out it.

Comment 5 John Dennis 2006-07-20 15:02:58 UTC
O.K. added dbus-glib-devel, but I can't test the build because I can't find this
package. Normally I get my rawhide packages off of bigpapi (formerly qafiler),
so I'm guessing the package wasn't there when the last sync occurred. Can you
point me to another location (e.g. something off of /mnt/redhat?)

Comment 6 John Dennis 2006-07-20 16:18:31 UTC
performed a brew scratch build on notify-python-0.1.0-2 with dbus-glib-devel
added, everything seems o.k. Note URL's for spec file and src rpm in #3 updated
as well.

Comment 7 Jesse Keating 2006-07-20 18:00:19 UTC
Looks good now.

Approving.

Does this need to be in comps anywhere, or will it be pulled in through a
dependancy of something else?

Comment 8 John Dennis 2006-07-20 20:34:46 UTC
It will be a dependency of other packages

Comment 9 Rahul Sundaram 2006-07-22 12:13:44 UTC

Package showed up in rawhide today. Closing


Note You need to log in before you can comment on or make changes to this bug.