Bug#702281: Patch

2013-03-09 Thread Nicolas Bourdaud
Thanks Stephan for the patch.

From what I understand, it seems that both of you (Stephan and Vincent)
have previously gconf settings in subpath of /desktop/gnome/keybindings
in addition to the default one set by muffin-common package
(/desktop/gnome/keybindings/custom0/{name|action|bindings}).
Could you confirm that this is the case?

Also I have added your patch to the package in preparation, and
uploaded to mentors a version with it:
http://mentors.debian.net/debian/pool/main/c/cinnamon/cinnamon_1.7.2-1.dsc
or if you prefer using git:
git://anonscm.debian.org/pkg-cinnamon/cinnamon.git

@Vincent: Could you confirm that this version fixes your issue?

I will upload the patch to upstream, as soon as you confirm that it works.

Cheers,

Nicolas



signature.asc
Description: OpenPGP digital signature


Bug#702281: Patch

2013-03-09 Thread Nicolas Bourdaud
On 09/03/2013 11:05, Nicolas Bourdaud wrote:
   Also I have added your patch to the package in preparation, and
 uploaded to mentors a version with it:
 http://mentors.debian.net/debian/pool/main/c/cinnamon/cinnamon_1.7.2-1.dsc

Well it seems that mentors.debian.net has some issues today. It did not
receive my updated package.

So I suggest to use the git repository with git-buildpackage:
git://anonscm.debian.org/pkg-cinnamon/cinnamon.git

Cheers,

Nicolas




signature.asc
Description: OpenPGP digital signature


Bug#702281: Patch

2013-03-07 Thread Stephan Sürken
Hi,

fwiw, I guess this

--- 
root@debian:~# diff -u /usr/lib/cinnamon-settings/modules/cs_keyboard.py.orig 
/usr/lib/cinnamon-settings/modules/cs_keyboard.py
--- /usr/lib/cinnamon-settings/modules/cs_keyboard.py.orig  2013-03-07 
14:48:10.897291292 +0100
+++ /usr/lib/cinnamon-settings/modules/cs_keyboard.py   2013-03-07 
14:44:23.933284714 +0100
@@ -241,7 +241,7 @@
 self.label = label
 self.action = action
 self.entries = []
-self.entries.append(binding)
+self.entries.append(binding if binding else )
 
 def setBinding(self, index, val):
 if val is not None:
---

may qualify as upstream patch (as the setBinding method has the same
guard against None, __init__ should have it too I guess).

At least it fixes this bug for me.

HtH,

Stephan


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org