seahorse problems with gnupg 1.4.2 (dapper version)

[Allison Karlitskaya]

when using seahorse with the new gpg in dapper assigning ownertrust doesn't always work.

if you are using gpg 1.4.1 (breezy version) it always works
if you invoke 'seahorse' from a terminal it always works

the case in which it does not work, then, is when using the new version with seahorse not invoked from a terminal (ie: opened from the menu or from alt+f2).

this might be a bug in seahorse or in libgpgme. it's probably not a bug in gpg, but who knows?

[Allison Karlitskaya (desrt)]

i straced seahorse. the gpg instance that it forks off to exit the trust level on the key exits like this:

open("/dev/tty", O_RDWR|O_LARGEFILE) = -1 ENXIO (No such device or address)
[...locale error message lookup stuff...]
write(2, "gpg: ", 5) = 5
write(2, "cannot open `/dev/tty\': No such "..., 50) = 50
unlink("/home/desrt/.gnupg/.#lk0x8111d48.moonpix.5189") = 0
write(1, "pub:f:1024:17:435102983B03D8EE:1"..., 267) = 267
munmap(0xb7fa0000, 4096) = 0
exit_group(2) = ?

so this explains why it works when seahorse is started from inside a terminal.

[Allison Karlitskaya (desrt)]

this is a gpg bug. the new command completion stuff opens the tty even if --no-tty is requested. i've made up a patch that fixes it and i'll send it off to the gpg guys.

[Allison Karlitskaya (desrt)]

here's a dpatch to fix the gnupg dapper package. i've tested it and it fixes the problem for me.

it's exactly the same as the patch i've sent upstream except for the dpatchification.

[Allison Karlitskaya (desrt)]

patch has been accepted/committed upstream and will be included for 1.4.3

[Daniel Holbach (dholbach)]

What about seahorse? Do we have to fix it as well?

[Allison Karlitskaya (desrt)]

seahorse is fine. i just don't know how to remove it from the bug :)

[Martin Pitt (pitti)]

 gnupg (1.4.2-2ubuntu2) dapper; urgency=low
 .
   * Add 20_no_tty_fix.dpatch:
     - Do not open /dev/tty if --no-tty is specified, since this breaks
       programs like seahorse.
     - Patch also accepted upstream.
     - Thanks to Ryan Lortie <email address hidden> for the patch.
     - Malone #5570