By default, Bugzilla does not search the list of RESOLVED bugs.
You can force it to do so by putting the upper-case word ALL in front of your search query, e.g.: ALL tdelibs
We recommend searching for bugs this way, as you may discover that your bug has already been resolved and fixed in a later release.

Bug 2830

Summary: pinentry-0.9.7-trinity
Product: TDE Reporter: deloptes
Component: tdeutilsAssignee: Timothy Pearson <kb9vqf>
Status: RESOLVED FIXED    
Severity: normal CC: bugwatch, deloptes, gamrat.kristopher, michele.calgaro, slavek.banko
Priority: P5    
Version: R14.1.x [Trinity]   
Hardware: All   
OS: Linux   
See Also: http://bugs.pearsoncomputing.net/show_bug.cgi?id=250
http://bugs.pearsoncomputing.net/show_bug.cgi?id=2595
Compiler Version: TDE Version String:
Application Version: Application Name:
Bug Depends on:    
Bug Blocks: 250, 3161    
Attachments: included ported old KDE3 pinentry to TDE
A debian package build for pinentry-tqt and pinentry-tty
original source code
original debian package archive

Description deloptes 2017-09-06 17:20:31 CDT
Created attachment 2816 [details]
included ported old KDE3 pinentry to TDE

Attached is the code for pinentry-tqt.

Unfortunately I did not have time or desire to fix the entire compile and link process or the debian part.
I used it to produce the pinentry-tqt and moved it into my PATH - works great.

I hope someone more skilled in make/automake or packaging will do the rest. I hacked in a dirty way some of the automake/conf files, to get it done

I asked on the gnupg list if possible to include the tqt part, but either they did not get the mail, or not interested - I did not get answer yet.

The base for the code was 

pinentry_0.9.7.orig.tar.bz2
pinentry_0.9.7-5~bpo8+1.debian.tar.xz

and the old pinentry-qt for KDE3 (QT3) code.
Comment 1 deloptes 2017-09-14 16:39:38 CDT
Created attachment 2817 [details]
A debian package build for pinentry-tqt and pinentry-tty

Included is a debian directory updated so far to build pinentry-tqt and pinentry-tty packages.

I think it needs some polishing, but is also useful in the present state.

regards
Comment 2 Michele Calgaro 2017-09-15 09:14:46 CDT
Hi Emanoil,
good work and thanks. I have added this bug to R14.1.0 bug list.
Do you also have the original tar files you started from?
Comment 3 deloptes 2017-09-15 09:50:48 CDT
Created attachment 2818 [details]
original source code
Comment 4 deloptes 2017-09-15 09:51:50 CDT
Created attachment 2819 [details]
original debian package archive
Comment 5 Michele Calgaro 2017-09-15 10:14:32 CDT
Thanks!!
Comment 6 deloptes 2017-10-21 09:57:17 CDT
Great news regarding this. We may get it in the official gnupg code sooner as we thought. Now it is matter of convincing the gnupg community to accept the patch :)

The original mail from the gnupg-devel list follows

Damien Goutte-Gattat
	
Attachments1:41 PM (3 hours ago)
	
to me, gnupg-devel
On 10/11/2017 06:46 PM, deloptes wrote:

    I wonder if Damien could help with the autotools stuff. It would ease the
    enormously process.


I got a tentative patch which is working-for-me. It is too big (~ 140kb) to send on the mailing-list, so you can pull it from here:

  git://git.incenp.org/pinentry.git pinentry-tqt

As suggested elsewhere in the discussion by Werner, pinentry-tqt is only built if explicitly requested with --enable-pinentry-tqt. Furthermore, if any Qt version is detected, pinentry-qt must be explicitly disabled (with --disable-pinentry-qt) to allow building the TQt pinentry.

I have tested this patch on Slackware and Debian systems, with TQt version R14.0.4. Further testing, especially from TDE users, is welcome.

----------------------------------------------------------------
Damien Goutte-Gattat (1):
      tqt: Add a TQt-based pinentry.

 .gitignore             |    2 +
 Makefile.am            |    9 +-
 configure.ac           |   44 +-
 tqt/Makefile.am        |   60 ++
 tqt/README.SecQ        |   17 +
 tqt/main.cpp           |  241 ++++++
 tqt/pinentrydialog.cpp |  234 ++++++
 tqt/pinentrydialog.h   |   92 +++
 tqt/secqinternal.cpp   |  635 ++++++++++++++++
 tqt/secqinternal_p.h   |  140 ++++
 tqt/secqlineedit.cpp   | 1955 ++++++++++++++++++++++++++++++++++++++++++++++++
 tqt/secqlineedit.h     |  228 ++++++
 tqt/secqstring.cpp     |  939 +++++++++++++++++++++++
 tqt/secqstring.h       |  307 ++++++++
 14 files changed, 4901 insertions(+), 2 deletions(-)
 create mode 100644 tqt/Makefile.am
 create mode 100644 tqt/README.SecQ
 create mode 100644 tqt/main.cpp
 create mode 100644 tqt/pinentrydialog.cpp
 create mode 100644 tqt/pinentrydialog.h
 create mode 100644 tqt/secqinternal.cpp
 create mode 100644 tqt/secqinternal_p.h
 create mode 100644 tqt/secqlineedit.cpp
 create mode 100644 tqt/secqlineedit.h
 create mode 100644 tqt/secqstring.cpp
 create mode 100644 tqt/secqstring.h
Comment 7 deloptes 2017-10-29 14:33:03 CDT
update

Damien Goutte-Gattat

The TQt code is in the "pinentry-tqt" branch. [...] You need to switch to the "pinentry-tqt" branch first.

  $ git clone git://git.incenp.org/pinentry.git pinentry-tqt
  $ cd pinentry-tqt
  $ git checkout pinentry-tqt

He used 14.0.4 and in 14.1 in pinentry-tqt/tqt/secqlineedit.h I had to remove hasMarkedText (as it is flagged obsolete) to compile. To decide if version check is needed at that place in the code. Any advice?

Compile, configure and build with 
  $ ./autogen.sh
  $ ./configure --enable-maintainer-mode
  $ debuild -b -uc -us -j4

produced

pinentry-tty_1.0.1-beta34_amd64.deb
pinentry-tty-dbgsym_1.0.1-beta34_amd64.deb
pinentry-tqt_1.0.1-beta34_amd64.deb
pinentry-tqt-dbgsym_1.0.1-beta34_amd64.deb
pinentry-doc_1.0.1-beta34_all.deb

Tested basic functionality and it works fine.

regards
Comment 8 deloptes 2018-09-10 14:36:54 CDT
*** Bug 2672 has been marked as a duplicate of this bug. ***
Comment 9 deloptes 2018-10-20 15:31:19 CDT
one additional note I came across today. In debian/control dependency needs to be changed to pinentry-tqt.
At the moment it enforces installation of qt5 packages to satisfy pinentry-qt (qt5)
Comment 10 Slávek Banko 2020-11-28 20:12:14 CST
TQt flavor is already part of the pinentry upstream. And pinentry-tqt is added to the extra-dependencies, so packages are already available to users. Therefore, this bug report can be closed.