Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
Summary: Review Request: spnavcfg - Spacenav daemon interactive configuration program
https://bugzilla.redhat.com/show_bug.cgi?id=731525
Summary: Review Request: spnavcfg - Spacenav daemon interactive configuration program Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: hobbes1069@gmail.com QAContact: extras-qa@fedoraproject.org CC: notting@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: ---
Spec URL: http://hobbes1069.fedorapeople.org/spnavcfg/spnavcfg.spec SRPM URL: http://hobbes1069.fedorapeople.org/spnavcfg/spnavcfg-0.2.1-1.fc15.src.rpm Description: Spacenav daemon interactive configuration program.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=731525
Richard Shaw hobbes1069@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Alias| |spnavcfg
--- Comment #1 from Richard Shaw hobbes1069@gmail.com 2011-08-17 15:28:23 EDT --- 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.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=731525
Rex Dieter rdieter@math.unl.edu changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |rdieter@math.unl.edu AssignedTo|nobody@fedoraproject.org |rdieter@math.unl.edu Flag| |fedora-review?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=731525
Rex Dieter rdieter@math.unl.edu changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review? |fedora-review+
--- Comment #2 from Rex Dieter rdieter@math.unl.edu 2011-08-19 10:15:24 EDT --- 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.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=731525
--- Comment #3 from Richard Shaw hobbes1069@gmail.com 2011-08-19 10:57:23 EDT --- (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
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=731525
--- Comment #4 from Rex Dieter rdieter@math.unl.edu 2011-08-19 11:03:36 EDT --- 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).
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=731525
Richard Shaw hobbes1069@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |MODIFIED Flag| |fedora-cvs?
--- Comment #5 from Richard Shaw hobbes1069@gmail.com 2011-08-19 13:42:10 EDT --- New Package SCM Request ======================= Package Name: spnavcfg Short Description: Spacenav daemon interactive configuration program Owners: hobbes1069 Branches: f15 f16 InitialCC:
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=731525
--- Comment #6 from Jon Ciesla limb@jcomserv.net 2011-08-19 13:46:43 EDT --- Git done (by process-git-requests).
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=731525
--- Comment #7 from Fedora Update System updates@fedoraproject.org 2011-08-25 15:18:00 EDT --- 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
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=731525
--- Comment #8 from Fedora Update System updates@fedoraproject.org 2011-08-25 15:18:09 EDT --- 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
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=731525
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|MODIFIED |ON_QA
--- Comment #9 from Fedora Update System updates@fedoraproject.org 2011-08-26 10:18:25 EDT --- spnavcfg-0.2.1-1.fc16 has been pushed to the Fedora 16 testing repository.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=731525
--- Comment #10 from Fedora Update System updates@fedoraproject.org 2011-09-06 20:25:41 EDT --- spnavcfg-0.2.1-1.fc15 has been pushed to the Fedora 15 stable repository.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=731525
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ON_QA |CLOSED Fixed In Version| |spnavcfg-0.2.1-1.fc15 Resolution| |ERRATA Last Closed| |2011-09-06 20:25:46
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=731525
--- Comment #11 from Fedora Update System updates@fedoraproject.org 2011-09-06 23:32:03 EDT --- spnavcfg-0.2.1-1.fc16 has been pushed to the Fedora 16 stable repository.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=731525
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Fixed In Version|spnavcfg-0.2.1-1.fc15 |spnavcfg-0.2.1-1.fc16
https://bugzilla.redhat.com/show_bug.cgi?id=731525
John Morris john@zultron.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |john@zultron.com Flags|fedora-cvs+ | Flags| |fedora-cvs?
--- Comment #12 from John Morris john@zultron.com --- 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.
https://bugzilla.redhat.com/show_bug.cgi?id=731525
--- Comment #13 from Jon Ciesla limburgher@gmail.com --- Git done (by process-git-requests).
package-review@lists.fedoraproject.org