Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 767181 - app-crypt/pinentry-1.1.1 - please add USE=efl
Summary: app-crypt/pinentry-1.1.1 - please add USE=efl
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Mikle Kolyada (RETIRED)
URL: https://github.com/gentoo/gentoo/pull...
Whiteboard:
Keywords: PullRequest
: 771003 (view as bug list)
Depends on:
Blocks:
 
Reported: 2021-01-25 19:54 UTC by Bertrand Jacquin
Modified: 2021-02-23 15:33 UTC (History)
4 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Bertrand Jacquin 2021-01-25 19:54:24 UTC
Hi,

As per https://git.gnupg.org/cgi-bin/gitweb.cgi?p=pinentry.git;a=blob;f=NEWS, the latest version of pinentry adds an EFL based pinentry.

Could you please add a USE flag to allow this new frontend to be build ?

Thanks,
Bertrand

Reproducible: Always
Comment 1 Michelangelo Scopelliti 2021-02-09 11:07:31 UTC
efl frontend is built if efl is found on the system; it would also nice to ave an option for pinentry-efl in eselect-pinentry (I currently use it on one of my systems without any issues)
Comment 2 Bertrand Jacquin 2021-02-09 19:03:07 UTC
(In reply to Michelangelo Scopelliti from comment #1)
> efl frontend is built if efl is found on the system

This is correct, however using undeclared dependencies is breaking reverse dependency model, slot bump etc (unsure about SONAME tracking as part of portage)
Comment 3 Michelangelo Scopelliti 2021-02-09 20:06:35 UTC
(In reply to Bertrand Jacquin from comment #2)
> (In reply to Michelangelo Scopelliti from comment #1)
> > efl frontend is built if efl is found on the system
> 
> This is correct, however using undeclared dependencies is breaking reverse
> dependency model, slot bump etc (unsure about SONAME tracking as part of
> portage)

Well, pinentry-1.1.1 has the configure switch --enable-pinentry-efl that should account for a conditional (if I understood what you mean)
Comment 4 Bertrand Jacquin 2021-02-09 22:36:25 UTC
(In reply to Michelangelo Scopelliti from comment #3)
> (In reply to Bertrand Jacquin from comment #2)
> > (In reply to Michelangelo Scopelliti from comment #1)
> > > efl frontend is built if efl is found on the system
> > 
> > This is correct, however using undeclared dependencies is breaking reverse
> > dependency model, slot bump etc (unsure about SONAME tracking as part of
> > portage)
> 
> Well, pinentry-1.1.1 has the configure switch --enable-pinentry-efl that
> should account for a conditional (if I understood what you mean)

Yes, this is the proposal here, to add a USE flag to control the --{enable,disable}-pinentry-efl
Comment 5 Bertrand Jacquin 2021-02-09 23:24:32 UTC
https://github.com/gentoo/gentoo/pull/19396 has been created on this purpose
Comment 6 Mikle Kolyada (RETIRED) archtester Gentoo Infrastructure gentoo-dev Security 2021-02-23 15:29:36 UTC
*** Bug 771003 has been marked as a duplicate of this bug. ***
Comment 7 Larry the Git Cow gentoo-dev 2021-02-23 15:33:12 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=aa876f1a70835eceb5005a1175f48f448c5d959c

commit aa876f1a70835eceb5005a1175f48f448c5d959c
Author:     Bertrand Jacquin <bertrand@jacquin.bzh>
AuthorDate: 2021-02-09 23:14:56 +0000
Commit:     Mikle Kolyada <zlogene@gentoo.org>
CommitDate: 2021-02-23 15:33:09 +0000

    app-eselect/eselect-pinentry: add pinentry-efl
    
    As per https://git.gnupg.org/cgi-bin/gitweb.cgi?p=pinentry.git;a=blob;f=NEWS,
    pinentry 1.1.1 adds an EFL based pinentry.
    
    Closes: https://bugs.gentoo.org/767181
    Signed-off-by: Bertrand Jacquin <bertrand@jacquin.bzh>
    Package-Manager: Portage-3.0.13, Repoman-3.0.2
    Signed-off-by: Mikle Kolyada <zlogene@gentoo.org>

 .../eselect-pinentry/eselect-pinentry-0.7.2.ebuild | 23 ++++++++++++++++++++++
 .../eselect-pinentry/files/pinentry.eselect-0.7.2  | 12 +++++++++++
 2 files changed, 35 insertions(+)

Additionally, it has been referenced in the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=0c904c5599cc8bbeb5ff15658137a748953ffcec

commit 0c904c5599cc8bbeb5ff15658137a748953ffcec
Author:     Bertrand Jacquin <bertrand@jacquin.bzh>
AuthorDate: 2021-02-09 23:09:38 +0000
Commit:     Mikle Kolyada <zlogene@gentoo.org>
CommitDate: 2021-02-23 15:33:09 +0000

    app-crypt/pinentry: add pinentry-fltk
    
    As per https://git.gnupg.org/cgi-bin/gitweb.cgi?p=pinentry.git;a=blob;f=NEWS,
    pinentry 1.1.1 adds an EFL based pinentry.
    
    Bug: https://bugs.gentoo.org/767181
    Bug: https://bugs.gentoo.org/771003
    Signed-off-by: Bertrand Jacquin <bertrand@jacquin.bzh>
    Package-Manager: Portage-3.0.13, Repoman-3.0.2
    Closes: https://github.com/gentoo/gentoo/pull/19396
    Signed-off-by: Mikle Kolyada <zlogene@gentoo.org>

 app-crypt/pinentry/metadata.xml             |  3 +
 app-crypt/pinentry/pinentry-1.1.1-r1.ebuild | 90 +++++++++++++++++++++++++++++
 2 files changed, 93 insertions(+)