Re: [PATCH] virtinst: also look at xorg.conf for keymap

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Cole Robinson wrote:
> Guido Günther wrote:
>> Hi,
>> attached patch looks at xorg.conf if no /etc/sysconfig/keyboard for the
>> default keyboard. This helps with distros that don't have
>> /etc/sysconfig/ like Debian.
>> Cheers,
>>  -- Guido
>>
> 
> Couple small nits:
> 
>>  HG changeset patch
>> # User Guido Günther <agx@xxxxxxxxxxx>
>> # Date 1227979760 -3600
>> # Node ID a292eb8430cd2d9544e89dba127323704f3963ce
>> # Parent  16e923676e84d81cf9bf6dc2cdd30d95bf65bbe1
>> parse xorg.conf for keymap
>>
>> diff -r 16e923676e84 -r a292eb8430cd virtinst/util.py
>> --- a/virtinst/util.py	Tue Nov 25 10:25:08 2008 -0500
>> +++ b/virtinst/util.py	Sat Nov 29 18:29:20 2008 +0100
>> @@ -33,6 +33,7 @@
>>  
>>  
>>  KEYBOARD_DIR = "/etc/sysconfig/keyboard"
>> +XORG_CONF = "/etc/X11/xorg.conf"
>>  
>>  def default_route():
>>      route_file = "/proc/net/route"
>> @@ -276,6 +277,28 @@
>>              return -1
>>      return 0
>>  
>> +def xorg_keymap():

One other thing: could you rename this to _xorg_keymap so it isn't
confused for a public function?

Thanks,
Cole

_______________________________________________
et-mgmt-tools mailing list
et-mgmt-tools@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/et-mgmt-tools

[Index of Archives]     [Fedora Users]     [Fedora Legacy List]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux