[Bug 920844] Review Request: gnome-robots - GNOME Robots game
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=920844 Jon Ciesla changed: What|Removed |Added Flags|fedora-cvs? | -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=nvRPm36XV2&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 920844] Review Request: gnome-robots - GNOME Robots game
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=920844 --- Comment #8 from Jon Ciesla --- Clearing flag. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=pbDHNxLvTi&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 920844] Review Request: gnome-robots - GNOME Robots game
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=920844 Tanner Doshier changed: What|Removed |Added Status|NEW |CLOSED Resolution|--- |NEXTRELEASE Flags|fedora-cvs+ | Flags||fedora-cvs? Last Closed||2013-05-06 17:08:53 --- Comment #7 from Tanner Doshier --- Built in f19 and rawhide. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=LNmOZlcccV&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 920844] Review Request: gnome-robots - GNOME Robots game
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=920844 --- Comment #6 from Jon Ciesla --- Git done (by process-git-requests). -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=Dgh8FkKD3E&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 920844] Review Request: gnome-robots - GNOME Robots game
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=920844 Jon Ciesla changed: What|Removed |Added Flags|fedora-cvs? | Flags||fedora-cvs+ -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=Jdib06YXTs&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 920844] Review Request: gnome-robots - GNOME Robots game
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=920844 Tanner Doshier changed: What|Removed |Added Status|ASSIGNED|NEW Flags||fedora-cvs? --- Comment #5 from Tanner Doshier --- New Package SCM Request === Package Name: gnome-robots Short Description: Avoid the robots and make them crash into each other Owners: doshitan Branches: f19 InitialCC: -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=7eScSM0pgg&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 920844] Review Request: gnome-robots - GNOME Robots game
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=920844 Kalev Lember changed: What|Removed |Added Flags||fedora-review+ --- Comment #4 from Kalev Lember --- Fedora review gnome-robots-3.8.0-1.fc19.src.rpm 2013-05-06 + OK ! needs attention + Rpmlint is very noisy here, but most of the warnings / errors seem harmless. The unreadable /usr/bin/gnome-robots is a bit weird, but it's the same as in the old gnome-games and I suppose it didn't have the r bit for a reason. + The package is named according to Fedora packaging guidelines + The spec file name matches the base package name. + The package meets the Packaging Guidelines + The package is licensed with a Fedora approved license and meets the Licensing Guidelines. + The license field in the spec file matches the actual license + The package contains the license file (COPYING) + Spec file is written in American English + Spec file is legible + Upstream sources match sources in the srpm. md5sum: 05ea450e0449e6f810a7a975ac0ec48e gnome-robots-3.8.0.tar.xz 05ea450e0449e6f810a7a975ac0ec48e Download/gnome-robots-3.8.0.tar.xz + The package builds in koji n/a ExcludeArch bugs filed + BuildRequires look sane + The spec file handles locales properly n/a ldconfig in %post and %postun + Package does not bundle copies of system libraries n/a Package isn't relocatable + Package owns all the directories it creates + No duplicate files in %files + Permissions are properly set + Consistent use of macros + The package must contain code or permissible content n/a Large documentation files should go in -doc subpackage + Files marked %doc should not affect package n/a Header files should be in -devel n/a Static libraries should be in -static n/a Library files that end in .so must go in a -devel package n/a -devel must require the fully versioned base + Packages should not contain libtool .la files + Proper .desktop file handling + Doesn't own files or directories already owned by other packages + Filenames are valid UTF-8 Looks good! APPROVED -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=nCOB3BU2Gf&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 920844] Review Request: gnome-robots - GNOME Robots game
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=920844 Kalev Lember changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|nob...@fedoraproject.org|kalevlem...@gmail.com --- Comment #3 from Kalev Lember --- rpmlint output: $ rpmlint gnome-robots \ gnome-robots-debuginfo-3.8.0-1.fc20.x86_64.rpm \ gnome-robots-3.8.0-1.fc20.src.rpm gnome-robots.x86_64: W: spelling-error %description -l en_US teleportation -> deportation, exportation, importation gnome-robots.x86_64: W: obsolete-not-provided gnome-games-gnobots2 gnome-robots.x86_64: W: binaryinfo-readelf-failed /usr/bin/gnome-robots readelf: Error: Input file '/usr/bin/gnome-robots' is not readable. gnome-robots.x86_64: W: binaryinfo-tail-failed /usr/bin/gnome-robots: [Errno 13] Permission denied: '/usr/bin/gnome-robots' gnome-robots.x86_64: W: ldd-failed /usr/bin/gnome-robots gnome-robots.x86_64: E: zero-length /var/games/gnome-robots.classic_robots-safe.scores gnome-robots.x86_64: E: incorrect-fsf-address /usr/share/doc/gnome-robots-3.8.0/COPYING gnome-robots.x86_64: E: zero-length /var/games/gnome-robots.robots_with_safe_teleport-safe.scores gnome-robots.x86_64: E: zero-length /var/games/gnome-robots.classic_robots-super-safe.scores gnome-robots.x86_64: E: zero-length /var/games/gnome-robots.robots2.scores gnome-robots.x86_64: E: zero-length /var/games/gnome-robots.robots2-super-safe.scores gnome-robots.x86_64: E: zero-length /var/games/gnome-robots.nightmare-super-safe.scores gnome-robots.x86_64: E: zero-length /var/games/gnome-robots.robots2_easy-super-safe.scores gnome-robots.x86_64: E: zero-length /var/games/gnome-robots.robots_with_safe_teleport.scores gnome-robots.x86_64: E: zero-length /var/games/gnome-robots.robots2_easy-safe.scores gnome-robots.x86_64: E: zero-length /var/games/gnome-robots.nightmare.scores gnome-robots.x86_64: E: setgid-binary /usr/bin/gnome-robots games 02551L gnome-robots.x86_64: E: non-standard-executable-perm /usr/bin/gnome-robots 02551L gnome-robots.x86_64: E: non-standard-executable-perm /usr/bin/gnome-robots 02551L gnome-robots.x86_64: E: zero-length /var/games/gnome-robots.robots2_easy.scores gnome-robots.x86_64: E: zero-length /var/games/gnome-robots.nightmare-safe.scores gnome-robots.x86_64: E: zero-length /var/games/gnome-robots.robots2-safe.scores gnome-robots.x86_64: E: zero-length /var/games/gnome-robots.robots_with_safe_teleport-super-safe.scores gnome-robots.x86_64: E: zero-length /var/games/gnome-robots.classic_robots.scores gnome-robots.src: W: spelling-error %description -l en_US teleportation -> deportation, exportation, importation gnome-robots-debuginfo.x86_64: E: incorrect-fsf-address /usr/src/debug/gnome-robots-3.8.0/src/sound.c gnome-robots-debuginfo.x86_64: E: incorrect-fsf-address /usr/src/debug/gnome-robots-3.8.0/src/gameconfig.c gnome-robots-debuginfo.x86_64: E: incorrect-fsf-address /usr/src/debug/gnome-robots-3.8.0/src/game.c gnome-robots-debuginfo.x86_64: E: incorrect-fsf-address /usr/src/debug/gnome-robots-3.8.0/src/games-stock.h gnome-robots-debuginfo.x86_64: E: incorrect-fsf-address /usr/src/debug/gnome-robots-3.8.0/src/games-preimage.h gnome-robots-debuginfo.x86_64: E: incorrect-fsf-address /usr/src/debug/gnome-robots-3.8.0/src/menu.c gnome-robots-debuginfo.x86_64: E: incorrect-fsf-address /usr/src/debug/gnome-robots-3.8.0/src/games-scores.h gnome-robots-debuginfo.x86_64: E: incorrect-fsf-address /usr/src/debug/gnome-robots-3.8.0/src/graphics.c gnome-robots-debuginfo.x86_64: E: incorrect-fsf-address /usr/src/debug/gnome-robots-3.8.0/src/cursors.c gnome-robots-debuginfo.x86_64: E: incorrect-fsf-address /usr/src/debug/gnome-robots-3.8.0/src/find-file.h gnome-robots-debuginfo.x86_64: E: incorrect-fsf-address /usr/src/debug/gnome-robots-3.8.0/src/keyboard.c gnome-robots-debuginfo.x86_64: E: incorrect-fsf-address /usr/src/debug/gnome-robots-3.8.0/src/properties.c gnome-robots-debuginfo.x86_64: E: incorrect-fsf-address /usr/src/debug/gnome-robots-3.8.0/src/cursors.h gnome-robots-debuginfo.x86_64: E: incorrect-fsf-address /usr/src/debug/gnome-robots-3.8.0/src/find-file.c gnome-robots-debuginfo.x86_64: E: incorrect-fsf-address /usr/src/debug/gnome-robots-3.8.0/src/games-preimage.c gnome-robots-debuginfo.x86_64: E: incorrect-fsf-address /usr/src/debug/gnome-robots-3.8.0/src/gnobots.c gnome-robots-debuginfo.x86_64: E: incorrect-fsf-address /usr/src/debug/gnome-robots-3.8.0/src/games-stock.c gnome-robots-debuginfo.x86_64: E: incorrect-fsf-address /usr/src/debug/gnome-robots-3.8.0/src/games-scores.c gnome-robots-debuginfo.x86_64: E: incorrect-fsf-address /usr/src/debug/gnome-robots-3.8.0/src/games-file-list.c gnome-robots-debuginfo.x86_64: E: incorrect-fsf-address /usr/src/debug/gnome-robots-3.8.0/src/games-fullscreen-action.c gnome-robots-debuginfo.x86_64: E: incorrect-fsf-address /usr/src/debug/gnome-ro
[Bug 920844] Review Request: gnome-robots - GNOME Robots game
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=920844 --- Comment #2 from Tanner Doshier --- 3.8.0-1 - Update to 3.8.0 - Use setgid games 3.7.92-1 - Update to 3.7.92 - Use old desktop file name - Add high contrast icons Spec URL: http://doshitan.com/tmp/gnome-robots/gnome-robots.spec SRPM URL: http://doshitan.com/tmp/gnome-robots/gnome-robots-3.8.0-1.fc19.src.rpm -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=0EILZesePi&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 920844] Review Request: gnome-robots - GNOME Robots game
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=920844 Kalev Lember changed: What|Removed |Added CC||kalevlem...@gmail.com --- Comment #1 from Kalev Lember --- rpmlint has a lot of warnings about the /var/games/gnome-robots.* score files. I guess these are supposed to be writable by any user? The old gnome-games package handled this by setting the setgid on the executable and making the score file writable by the games group: %verify(not md5 size mtime) %config(noreplace) %attr(664, games, games) /var/lib/games/gnobots2* # this is a setgid game %attr(2551, root, games) %{_bindir}/gnobots2 Should we do the same thing here? -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=t2VKV4y0IY&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review