Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 167544
Summary: | Review Request: netgo : Networking profile manager | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Tom "spot" Callaway <tcallawa> |
Component: | Package Review | Assignee: | Christian Iseli <Christian.Iseli> |
Status: | CLOSED NEXTRELEASE | QA Contact: | David Lawrence <dkl> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | fedora-extras-list |
Target Milestone: | --- | ||
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
URL: | http://netgo.hjolug.org/ | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2005-09-16 23:30:16 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: | |||
Bug Depends On: | |||
Bug Blocks: | 163779 |
Description
Tom "spot" Callaway
2005-09-05 00:24:24 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 complaining: 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. 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 Good: - 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 Bad: - owns all dirs Patch below. Minor nit: I'd add the README file to %doc... Approved. Cheers, Christian --- 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 %{_datadir}/applications/fedora-netgo.desktop %{_datadir}/pixmaps/netgo.png -%{_datadir}/netgo/icons/globe_32x32.png +%{_datadir}/netgo %{_sbindir}/netgo %changelog +* Sat Sep 17 2005 Christian Iseli <Christian.Iseli> 0.5-3 +- fix unowned dir + * Fri Sep 16 2005 Tom "spot" Callaway <tcallawa> 0.5-2 - move netgo-bin to %%{_sbindir}/netgo |