Bug 167544 - Review Request: netgo : Networking profile manager
Summary: Review Request: netgo : Networking profile manager
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Christian Iseli
QA Contact: David Lawrence
URL: http://netgo.hjolug.org/
Depends On:
TreeView+ depends on / blocked
Reported: 2005-09-05 00:24 UTC by Tom "spot" Callaway
Modified: 2007-11-30 22:11 UTC (History)
1 user (show)

Clone Of:
Last Closed: 2005-09-16 23:30:16 UTC

Attachments (Terms of Use)

Description Tom "spot" Callaway 2005-09-05 00:24:24 UTC
Spec Name or Url: http://www.auroralinux.org/people/spot/review/netgo.spec
SRPM Name or Url: http://www.auroralinux.org/people/spot/review/netgo-0.5-1.src.rpm
netGo is a distribution-independent networking tool (written in Qt) for
changing network settings quickly and easily. It is intended for laptop
owners, who often need to change network settings when relocating.
It allows you to create profiles that contain the network settings such as the
IP, netmask, gateway, name servers and wireless options. After creating a
profile it can be executed with a single click.

Comment 1 Christian Iseli 2005-09-16 16:39:41 UTC
I started to review netgo, and I think there is a problem... but I'm not too
sure how to handle this.

The upstream netgo package puts its real binary as /usr/share/netgo/netgo-bin and
puts a wrapper in /usr/bin to gain root access.

The proposed package uses consolehelper to replace the wrapper.

However, the real bin is still in /usr/share, which leads to rpmlint
E: netgo arch-dependent-file-in-usr-share /usr/share/netgo/netgo-bin
W: netgo no-dependency-on usermode-consoleonly

I think the real bin should be named: /usr/sbin/netgo

But I'm not too familiar with consolehelper...

Please propose a fix, and I'll complete the review.

Comment 2 Tom "spot" Callaway 2005-09-16 17:30:34 UTC
Fixed in -2 (you're right, it should be in %{_sbindir}):

SRPM: http://www.auroralinux.org/people/spot/review/netgo-0.5-2.src.rpm
SPEC: http://www.auroralinux.org/people/spot/review/netgo.spec

Comment 3 Christian Iseli 2005-09-16 22:56:20 UTC
- rpmlint is pretty happy:
W: netgo no-dependency-on usermode-consoleonly
- name ok
- spec name ok
- packaging guidelines met
- license ok
- license matches actual license in package
- license file in %doc
- spec file in American English and legible
- source matches upstream
- builds, installs, runs, and uninstalls cleanly
- BuildRequires ok
- no locale to worry about
- no shared libs to worry about
- no relocation to worry about
- no duplicated files
- files perms ok
- clean section ok
- macro useage ok
- package contains code
- no large doc
- %doc is only doc
- desktop file present
- builds in mock

- owns all dirs
Patch below.

Minor nit:
I'd add the README file to %doc...



--- netgo.spec.orig     2005-09-17 00:51:00.000000000 +0200
+++ netgo.spec  2005-09-17 00:52:57.000000000 +0200
@@ -1,6 +1,6 @@
 Name:          netgo
 Version:       0.5
-Release:       2%{?dist}
+Release:       3%{?dist}
 License:       GPL
 Group:         Applications/System
 Summary:       Networking profile manager
@@ -73,10 +73,13 @@ rm -rf $RPM_BUILD_ROOT
 %config(noreplace) %{_sysconfdir}/security/console.apps/netgo

+* Sat Sep 17 2005 Christian Iseli <Christian.Iseli@licr.org> 0.5-3
+- fix unowned dir
 * Fri Sep 16 2005 Tom "spot" Callaway <tcallawa@redhat.com> 0.5-2
 - move netgo-bin to %%{_sbindir}/netgo

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