I have a small nitpick bout this patch.

-=| Thiemo Nagel, Tue, Jul 21, 2009 at 10:10:34PM +0200 |=-
> [...]
> diff --git a/etc/acpi/actions/gsm.sh b/etc/acpi/actions/gsm.sh
> new file mode 100644
> index 0000000..a16f428
> --- /dev/null
> +++ b/etc/acpi/actions/gsm.sh
> [...]
> +    if echo `cat $RFKILL` > $RFKILL 2> /dev/zero; then

How about changing /dev/zero to /dev/null? I somehow always associate 
/dev/zero with reading, and /dev/null with a black hole.

Really a nitpick, the rest of the patch looks fine to me.

-- 
dam

Attachment: signature.asc
Description: Digital signature

_______________________________________________
Debian-eeepc-devel mailing list
[email protected]
http://lists.alioth.debian.org/mailman/listinfo/debian-eeepc-devel

Reply via email to