Bug 592628 - Review Request: key-mon - Utility to show live keyboard and mouse status
Summary: Review Request: key-mon - Utility to show live keyboard and mouse status
Keywords:
Status: CLOSED DUPLICATE of bug 1039138
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Rahul Sundaram
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2010-05-15 19:44 UTC by kushaldas@gmail.com
Modified: 2015-12-04 01:48 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2015-12-04 01:48:12 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
proposed changes to initially provided SPEC file (2.32 KB, patch)
2010-06-28 20:37 UTC, Moritz Barsnick
no flags Details | Diff
proposed changes to initially provided SPEC file (2.18 KB, patch)
2010-06-28 20:40 UTC, Moritz Barsnick
no flags Details | Diff
latest version of proposed changes to initially provided SPEC file (2.53 KB, patch)
2010-06-29 14:12 UTC, Moritz Barsnick
no flags Details | Diff
updated spec (1.75 KB, text/x-rpm-spec)
2011-06-03 13:30 UTC, Marc-Andre Lureau
no flags Details

Description kushaldas@gmail.com 2010-05-15 19:44:07 UTC
Spec URL: http://kushaldas.in/tmp/key-mon.spec
SRPM URL: http://kushaldas.in/tmp/key-mon-0.15-1.fc13.src.rpm
Description: Utility to show live keyboard and mouse status for teaching and screencasts.

Comment 1 kushaldas@gmail.com 2010-05-15 19:46:48 UTC
Forgot to add patch0 URL: http://kushaldas.in/tmp/su.patch

Koji scratch build http://koji.fedoraproject.org/koji/taskinfo?taskID=2190240

Comment 2 Rahul Sundaram 2010-05-29 08:51:39 UTC
You can drop the following:
- python macro definition on top
- Buildroot definition
- no need to clean the buildroot in install section
- clean section

The su patch requires a comment as per

http://fedoraproject.org/wiki/Packaging:Guidelines#All_patches_should_have_an_upstream_bug_link_or_comment

You could run sed in a loop 

License status is unclear.  Source files don't have a proper license attribution and does not specify the version.  LGPL also requires that you include a copy of the license along with the source.  Please contact upstream and ask them to fix these issues.

Comment 3 Moritz Barsnick 2010-06-28 20:35:06 UTC
In the meantime, 1.2.2 has been released. It uses Xlib to track the keys (which actually pics up the correct ones when multiple displays are active, unlike the hardcoded device before), so more need for beesu. Some other Requires (pygtk2?) need to be added though.

The license seems to be ASL 2.0 now (COPYING is provided).

Comment 4 Moritz Barsnick 2010-06-28 20:37:10 UTC
Created attachment 427507 [details]
proposed changes to initially provided SPEC file

This patch shows most of the potential changes to accommodate for key-mon-1.2.2, and also puts the requested loop around sed.

Comment 5 Moritz Barsnick 2010-06-28 20:40:22 UTC
Created attachment 427508 [details]
proposed changes to initially provided SPEC file

Oops, previous patch was against wrong old SPEC. :) This one should be correct.

Comment 6 Moritz Barsnick 2010-06-29 14:12:28 UTC
Created attachment 427695 [details]
latest version of proposed changes to initially provided SPEC file

Some fixes to the previous SPEC update suggestions:
- fix the sed loop to use both variants of the shebang
- add Requires: python-xlib
- add some comments

Comment 7 Golo Fuchert 2010-10-22 22:58:18 UTC
I get the following from rpmlint on the rpm:

key-mon.noarch: E: explicit-lib-dependency python-xlib
key-mon.noarch: W: spelling-error %description -l en_US screencasts -> screen casts, screen-casts, screenplays
key-mon.noarch: E: non-executable-script /usr/lib/python2.6/site-packages/keymon/config.py 0644L /usr/bin/env
key-mon.noarch: W: no-manual-page-for-binary key-mon
1 packages and 0 specfiles checked; 2 errors, 2 warnings.

Have you tried building the package without the "Requires: python-xlib"?
Concerning the non-executable-script error: I just took a quick look into the file and it looks like a module. Then the shebang should be unnecessary and should be removed. You made a comment on this in the SPEC file that it is skipped for some reasons, however, if I add it to the list in "for file in ..." it works fine here.
I think you should fix those or comment on them at least.

Comment 8 Marc-Andre Lureau 2011-06-03 13:30:58 UTC
Created attachment 502810 [details]
updated spec

$ rpmlint key-mon
key-mon.noarch: W: no-manual-page-for-binary key-mon
1 packages and 0 specfiles checked; 0 errors, 1 warnings.

$ rpmlint key-mon.spec 
0 packages and 1 specfiles checked; 0 errors, 0 warnings.

Comment 9 barsnick 2014-02-24 14:45:01 UTC
So, this ticket seems to have fallen asleep. ;-)

Superseded by #1039138. The (new) package has now appeared in F20 repos.

Comment 10 Upstream Release Monitoring 2015-09-24 15:05:07 UTC
jgrulich's scratch build of kdevelop?#c8e2b9bc57f11e41f3dc6612cdbcc591078d9062 for f22-candidate and git://pkgs.fedoraproject.org/kdevelop?#c8e2b9bc57f11e41f3dc6612cdbcc591078d9062 completed http://koji.fedoraproject.org/koji/taskinfo?taskID=11212117

Comment 11 James Hogarth 2015-12-04 01:48:12 UTC

*** This bug has been marked as a duplicate of bug 1039138 ***


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