[Bug 495902] Review Request: olpc-kbdshim - grab key and better rotation support for the XO laptop
bugzilla at redhat.com
bugzilla at redhat.com
Fri Jun 5 14:23:09 UTC 2009
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=495902
--- Comment #14 from Christoph Wickert <fedora at christoph-wickert.de> 2009-06-05 10:23:06 EDT ---
(In reply to comment #13)
> i'm not sure what you mean. are you saying my line breaks shouldn't be there?
No, you should use the lines up to 80 characters, yours are shorter currently.
> i think the usefulness of timestamps
> is subverted almost entirely by making it appear as if files are older than
> they actually are. but i'll adapt, i guess. :-)
timstamps are important for rpm to see if a file has actually changed. This is
especially important on multiarch systems (so not on the XO), where you can
install two versions of a package in parallel. If two files have same size and
date, they will be treated as one file, if they differ, rpm will say they
conflict.
> i'll research this. i confess it's a little confusing as the maintainer to be
> doing a release based on a sandbox, rather than on a tarball. can you perhaps
> point me at a (simple) git-based package that does this correctly? a template
> would help me here.
Version is 6
Release is 2 (pls increase even during review)
To mark the git checkout: 20090506git
All together it becomes: olpc-kbdshim-6-2.20090506git
> there's no configuration: after installation and a reboot, the following
> things should work:
> - with any grab key pressed, both the touchpad and the arrow keys should
> cause scrolling. on a USB keyboard, the "windows" keys will act as grab
> keys.
> - the rotate and brightness keys should "just work". these keys cause the
> olpc-rotate and olpc-brightness scripts in /usr/bin to be invoked.
I don't have /usr/bin/olpc-brightness. What package is it from?
> that's it. you should feel free to test on the current package (URL above),
> since none of the current review comments have affected its operation. i'll do
> a new package when i've resolved the "description" and tagging issues still
> open above. okay?
Fine with me, will test tonight.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
More information about the Fedora-package-review
mailing list