[Bug 218678] Review Request: pybluez - python API for the bluez bluetooth stack

[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 report.

Summary: Review Request: pybluez - python API for the bluez bluetooth stack


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





------- Additional Comments From wwoods@xxxxxxxxxx  2006-12-29 13:28 EST -------
Thanks very much for the help! I don't currently have any other packages ready to review, but if you 
absolutely need to see some example specfiles I could dig some up from some internal packages I've 
made.

As for the issues you mentioned:

1) Heh! I just wasn't sure what to do with the BuildArch line, so I put the three arches that Fedora 
currently supports. I'll remove it so this package can be all-arch.

2) This package is *not* noarch, hence the CFLAGS part *is* needed. I've removed the comment to 
avoid confusion.

3) Whoops, there should have been a space between that initial dash and the version. Fixed.
Also, bluezchat.py is now chmod'd a-x before installation, which makes that rpmlint warning go away.

4) Why is that ugly? Isn't it normal for python modules which contain binary extensions to provide 
the .so file for that extension?

Here are the updated files:

Spec URL: http://homepage.mac.com/wgwoods/fedora/pybluez-0.9.1-3.spec
SRPM URL: http://homepage.mac.com/wgwoods/fedora/pybluez-0.9.1-3.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]