Bug 731525 (spnavcfg)
| Summary: | Review Request: spnavcfg - Spacenav daemon interactive configuration program | ||
|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Richard Shaw <hobbes1069> |
| Component: | Package Review | Assignee: | Rex Dieter <rdieter> |
| Status: | CLOSED ERRATA | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
| Severity: | medium | Docs Contact: | |
| Priority: | medium | ||
| Version: | rawhide | CC: | john, notting, package-review, rdieter |
| Target Milestone: | --- | Flags: | rdieter:
fedora-review+
gwync: fedora-cvs+ |
| Target Release: | --- | ||
| Hardware: | All | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Fixed In Version: | spnavcfg-0.2.1-1.fc16 | Doc Type: | Bug Fix |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2011-09-07 00:25:46 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: | |||
|
Description
Richard Shaw
2011-08-17 19:23:05 UTC
rpmlint output:
$ ls *.rpm
spnavcfg-0.2.1-1.fc15.src.rpm spnavcfg-debuginfo-0.2.1-1.fc15.x86_64.rpm
spnavcfg-0.2.1-1.fc15.x86_64.rpm
$ rpmlint *.rpm
spnavcfg.src: W: spelling-error Summary(en_US) Spacenav -> Spacemen
spnavcfg.src: W: spelling-error %description -l en_US Spacenav -> Spacemen
spnavcfg.src:29: W: rpm-buildroot-usage %build %configure --prefix=%{buildroot}%{_prefix}
spnavcfg.x86_64: W: spelling-error Summary(en_US) Spacenav -> Spacemen
spnavcfg.x86_64: W: spelling-error %description -l en_US Spacenav -> Spacemen
spnavcfg.x86_64: E: setuid-binary /usr/bin/spnavcfg root 04775L
spnavcfg.x86_64: E: non-standard-executable-perm /usr/bin/spnavcfg 04775L
spnavcfg.x86_64: E: non-standard-executable-perm /usr/bin/spnavcfg 04775L
- They uses setuid in order to allow normal users to write to the config file in /etc. Supposedly it's secure because they have a frontend (gui) and backend part of the program so only the backend runs as root.
spnavcfg.x86_64: W: no-manual-page-for-binary spnavcfg
- This is a gui only app.
3 packages and 0 specfiles checked; 3 errors, 6 warnings.
naming: ok sources: ok d20cf09515d1e39fd04877a00a2023b5 spnavcfg-0.2.1.tar.gz scriptlets: ok licensing: ok Not entirely happy with the setuid thing though, there's more robust and elegant solutions to that (like policykit), but meh. not a blocker, imo. rest is fairly simple. APPROVED. (In reply to comment #2) > Not entirely happy with the setuid thing though, there's more robust and > elegant solutions to that (like policykit), but meh. not a blocker, imo. Upstream is willing to make any simple changes that are not Fedora specific. Is there a simple solution to this problem? Thanks, Richard policykit is not fedora specific, but it's also a wee-bit non-trivial implementation-wise. would require a bit of work/coding (which, for better or worse, I lack the expertise/experience/skill at the moment myself). New Package SCM Request ======================= Package Name: spnavcfg Short Description: Spacenav daemon interactive configuration program Owners: hobbes1069 Branches: f15 f16 InitialCC: Git done (by process-git-requests). spnavcfg-0.2.1-1.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/spnavcfg-0.2.1-1.fc15 spnavcfg-0.2.1-1.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/spnavcfg-0.2.1-1.fc16 spnavcfg-0.2.1-1.fc16 has been pushed to the Fedora 16 testing repository. spnavcfg-0.2.1-1.fc15 has been pushed to the Fedora 15 stable repository. spnavcfg-0.2.1-1.fc16 has been pushed to the Fedora 16 stable repository. Package Change Request ====================== Package Name: spnavcfg New Branches: el6 Owners: zultron hobbes1069 InitialCC: The owner of this package (hobbes1069) and I (zultron) are building this package for EPEL6. Git done (by process-git-requests). |