Bug 488498 - Review Request: gnome-bluetooth2 - Bluetooth graphical utilities
Review Request: gnome-bluetooth2 - Bluetooth graphical utilities
Status: CLOSED WONTFIX
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Bill Nottingham
Fedora Extras Quality Assurance
:
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2009-03-04 12:15 EST by Bastien Nocera
Modified: 2014-03-16 23:17 EDT (History)
4 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2009-03-04 18:18:54 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)

  None (edit)
Description Bastien Nocera 2009-03-04 12:15:34 EST
Spec URL: http://people.fedoraproject.org/~hadess/gnome-bluetooth2/gnome-bluetooth2.spec
SRPM URL: http://people.fedoraproject.org/~hadess/gnome-bluetooth2/gnome-bluetooth2-2.27.0-1.fc10.src.rpm
Description: The gnome-bluetooth package contains graphical utilities to setup,
monitor and use Bluetooth devices.
Comment 1 Bastien Nocera 2009-03-04 12:27:07 EST
History:
http://svn.gnome.org/viewvc/gnome-bluetooth/trunk/README?view=markup

I didn't want to merge this into the existing gnome-bluetooth package, as it is still used in the repo. Clean slate.

Scratch build at:
http://koji.fedoraproject.org/koji/taskinfo?taskID=1220933
Comment 2 Bastien Nocera 2009-03-04 12:47:34 EST
SRPM updated:
http://people.fedoraproject.org/~hadess/gnome-bluetooth2/gnome-bluetooth2-2.27.1-1.fc10.src.rpm

This one doesn't conflict with the original gnome-bluetooth package for translations.
Comment 3 Bill Nottingham 2009-03-04 16:04:57 EST
Given that this is named 'gnome-bluetooth' upstream, and the old gnome-bluetooth is going away, it would seem simpler just to leave this named gnome-bluetooth, in which case it doesn't require a review.

That being said, since it's mostly new content:

MUST items:
- Package meets naming and packaging guidelines - OK
- Spec file matches base package name. - ***

See above about potentially keeping the gnome-bluetooth name.

- Spec has consistant macro usage. - OK
- Meets Packaging Guidelines. - OK
- License - GPLv2+ - ***

*** About half the source is LGPLv2+, which means the whole is GPLv2+.

- License field in spec matches - OK
- License file included in package - ***

It's not, it's only in the tarball.

- Spec in American English - OK
- Spec is legible. - OK
- Sources match upstream md5sum: - OK
- Package needs ExcludeArch - N/A
- BuildRequires correct - OK
- Spec handles locales/find_lang - OK
- Package is relocatable and has a reason to be. - N/A
- Package has %defattr and permissions on files is good. - OK
- Package has a correct %clean section. - OK
- Package has correct buildroot - OK
- Package is code or permissible content. - OK
- Doc subpackage needed/used. - N/A
- Packages %doc files don't affect runtime. - OK

- Headers/static libs in -devel subpackage. - OK
- Spec has needed ldconfig in post and postun - OK
- .pc files in -devel subpackage/requires pkgconfig - ***

The -devel package does not require pkgconfig.

- .so files in -devel subpackage. - OK
- -devel package Requires: %{name} = %{version}-%{release} - OK
- .la files are removed - OK

- Package is a GUI app and has a .desktop file - OK

- Package compiles and builds on at least one arch. - OK (tested x86_64)
- Package has no duplicate files in %files. - OK
- Package doesn't own any directories other packages own. - OK
- Package owns all the directories it creates. - ***

-devel package should own %{_includedir}/gnome-bluetooth

- No rpmlint output. - ***
gnome-bluetooth2.src:21: W: unversioned-explicit-obsoletes bluez-pin
gnome-bluetooth2.src:22: W: unversioned-explicit-provides dbus-bluez-pin-helper
gnome-bluetooth2.src:24: W: unversioned-explicit-obsoletes bluez-gnome

It's probably best to version these, unless the old bluez packages
are really dead forever and ever amen.

gnome-bluetooth2.src: W: mixed-use-of-spaces-and-tabs (spaces: line 89, tab: line 1)

Meh.

gnome-bluetooth2.x86_64: W: non-conffile-in-etc /etc/xdg/autostart/bluetooth-applet.desktop
gnome-bluetooth2.x86_64: W: non-conffile-in-etc /etc/gconf/schemas/bluetooth-manager.schemas

Both ignorable.

gnome-bluetooth2.x86_64: W: obsolete-not-provided bluez-pin
gnome-bluetooth2.x86_64: W: obsolete-not-provided bluez-gnome

Well, if you Provide it, it's going to conflict with itself from the Conflicts. 

- final provides and requires are sane: - ***

-libs package requires the main package. Does it need to? (Seems odd that it would require the applet.)

SHOULD Items:

- Should build in mock. - OK (tested x86_64)
- Should function as described. - didn't test
- Should have sane scriptlets. - ***

There's a new icon cache guideline - see 
https://fedoraproject.org/wiki/PackagingDrafts/Icon_Cache

- Should have subpackages require base package with fully versioned depend. - OK
- Should have dist tag - OK
- Should package latest version - OK
Comment 4 Bastien Nocera 2009-03-04 18:18:54 EST
> -libs package requires the main package. Does it need to? (Seems odd that it
would require the applet.)

Otherwise you don't get any translations...

I've fixed the rest in gnome-bluetooth CVS instead. But thanks for the review!

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