Ubuntu
pinentry package

pinentry-qt4 doesn't raise on top, need to select that window manually

Bug #657219 reported by Azamat S. Kalimoulline on 2010-10-09

This bug affects 4 people

	Status	Importance	Assigned to
pinentry (Debian)	Fix Released	Unknown	debbugs #581617
pinentry (Ubuntu)	Fix Released	Undecided	Unassigned
Maverick	Fix Released	Medium	Unassigned

Bug Description

Binary package hint: pinentry-qt4

KUbuntu Maverick. When sending signed or encrypting mail from contact Alt-Enter that should sent mail raises pinentry windows, but doesn't changed focus to pinentry. And user need to select it manually using mouse. And only then type password.

TEST CASE: using the -release package, sign a source package and notice that the pinentry window stays in back. Install the updated package and then do it again. Notice that the window appears on top where you want it.

See original description

Tags:

Related branches

lp:ubuntu/natty/pinentry

lp:ubuntu/maverick-proposed/pinentry

Revision history for this message

Bruno (bruno666-666) wrote on 2010-10-10:

There's a patch here : https://bugs.g10code.com/gnupg/issue1162

Changed in pinentry-qt4 (Ubuntu):
status:	New → Confirmed

Felix Geyer (debfx) on 2010-10-16

affects:

pinentry-qt4 (Ubuntu) → pinentry (Ubuntu)

Revision history for this message

Felix Geyer (debfx) wrote on 2010-10-16:

The patch from https://bugs.g10code.com/gnupg/issue1162 just sets the pinentry window to always on top.
Imho that's a very bad workaround.

This patch from Fedora works fine:
http://pkgs.fedoraproject.org/gitweb/?p=pinentry.git;a=blob;f=0004-Fix-qt4-pinentry-window-created-in-the-background.patch;h=d4bfa428490466931afc6c0264f5c0e5ccf12d1d;hb=HEAD

Revision history for this message

Rohan Garg (rohangarg) wrote on 2010-10-16:

Dedbiff for maverick SRU
The current pinentry has a major bug, the password entry window does not come at the top, causing alot of inconvenience to users.This debdiff adds kubuntu_01_fix_stayontop.diff ( stolen from fedora ;) ) to fix this issue.Another debdiff for natty as well.

Test Case :
1) Install pinentry-qt4
2) Use a app that calls pinentry-qt4 ( such as debuild -S -sa or similar )
3) Make sure pinentry window is at the top

Revision history for this message

Rohan Garg (rohangarg) wrote on 2010-10-16:

Proper maverick debdiff

tags:

added: kubuntu patch

Felix Geyer (debfx) on 2010-10-16

Changed in pinentry (Ubuntu):
status:	Confirmed → Triaged

Revision history for this message

Rohan Garg (rohangarg) wrote on 2010-10-16:

pinentry.debdiff Edit (2.3 KiB, text/plain)

Natty debdiff

Revision history for this message

Rohan Garg (rohangarg) wrote on 2010-10-16:

maverick.debdiff Edit (2.5 KiB, text/plain)

Bug Watch Updater (bug-watch-updater) on 2010-10-16

Changed in pinentry (Debian):
status:	Unknown → New

Scott Kitterman (kitterman) on 2010-10-17

Changed in pinentry (Ubuntu Maverick):
status:	New → In Progress
assignee:	nobody → Scott Kitterman (kitterman)
importance:	Undecided → Medium
tags:	added: regression-release

Revision history for this message

Launchpad Janitor (janitor) wrote on 2010-10-17:

This bug was fixed in the package pinentry - 0.8.0-1ubuntu1

---------------
pinentry (0.8.0-1ubuntu1) natty; urgency=low

[ Rohan Garg ]
* Add kubuntu_01_fix_stayontop.diff to make pinetry-qt4 stay on top (LP: #657219)

  [ Scott Kitterman ]
  * Drop obsolete pinentry-qt package
    - Drop package from debian/control
    - pinentry-doc suggests pinentry-qt4 instead
    - Switch configure flag to --disable-pinentry-qt in debian/rules
-- Scott Kitterman <email address hidden> Sun, 17 Oct 2010 12:26:29 -0400

Changed in pinentry (Ubuntu):
status:	Triaged → Fix Released

Revision history for this message

Scott Kitterman (kitterman) wrote on 2010-10-17:

Uploaded for Maverick after confirming locally that it fixes the problem.

description:	updated
Changed in pinentry (Ubuntu Maverick):
assignee:	Scott Kitterman (kitterman) → nobody

Revision history for this message

Martin Pitt (pitti) wrote on 2010-10-20: Please test proposed package

Accepted pinentry into maverick-proposed, the package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

Changed in pinentry (Ubuntu Maverick):
status:	In Progress → Fix Committed
tags:	added: verification-needed

Revision history for this message

Rohan Garg (rohangarg) wrote on 2010-10-20:

#10

Seems to work for me from -proposed, adding last 150 lines of term.log as attachment

Revision history for this message

Rohan Garg (rohangarg) wrote on 2010-10-20:

#11

install log Edit (12.5 KiB, text/plain)

Revision history for this message

Felix Geyer (debfx) wrote on 2010-10-20:

#12

I've tested the package from maverick-proposed. The dialog correctly gets the focus when it pops up.

tags:

added: verification-done
removed: verification-needed

Revision history for this message

Azamat S. Kalimoulline (turtle-bazon) wrote on 2010-10-21:

#13

Tested today pinentry-qt4 from maverick-proposed. All works fine as expected.

Revision history for this message

Bruno (bruno666-666) wrote on 2010-10-21:

#14

You rock guys! Thanks a lot, it works fine now.

Revision history for this message

Launchpad Janitor (janitor) wrote on 2010-10-27:

#15

This bug was fixed in the package pinentry - 0.8.0-1ubuntu0.1

---------------
pinentry (0.8.0-1ubuntu0.1) maverick-proposed; urgency=low

* Add kubuntu_01_fix_stayontop.diff to make pinetry-qt4 stay on top (LP: #657219)
-- Rohan Garg <email address hidden> Sat, 16 Oct 2010 22:27:17 +0530