cvs commit: ports/security/gnupg Makefile

[Joe Marcus Clarke]

On Sun, 2007-09-02 at 13:10 -0700, Doug Barton wrote:
> On Sun, 2 Sep 2007, Roman Bogorodskiy wrote:
> 
> > novel       2007-09-02 11:08:10 UTC
> >
> >  FreeBSD ports repository
> >
> >  Modified files:
> >    security/gnupg       Makefile
> >  Log:
> >  Add RUN_DEPEND on security/pinentry because gpg is almost useless
> >  without it.
> >
> >  PR:             115760  http://www.FreeBSD.org/cgi/query-pr.cgi?pr=115760
> >  Submitted by:   novel
> >  Approved by:    maintainer timeout (1 week, linimon ok)
> >
> >  Revision  Changes    Path
> >  1.106     +2 -1      ports/security/gnupg/Makefile
> >
> > http://www.FreeBSD.org/cgi/cvsweb.cgi/ports/security/gnupg/Makefile.diff?&r1=1.105&r2=1.106&f=h
> 
> I don't think this is a good idea for a few reasons. First off, the gnupg 
> port already has a pkg-message that is pretty clear about the fact that 
> you need to pick a pinentry dialog. Second, I don't think that the 
> pinentry port itself is a good choice in its current state. I just did a 
> quick test and as far as I can tell it seems to want to build all of them, 
> which means depending on QT3, and GTK 1 and 2.
> 
> I sort of think that this might be reasonable if the pinentry port grew 
> OPTIONS, which I would even be willing to work on if lofi thought it was a 
> good idea. But I don't think the overhead of drawing all of the dialogs in 
> is worth it, and I don't see an easy way of guessing which one the user 
> would want by default.
> 
> Can this change be backed out till there has been a little discussion?

Not to mention, it breaks the package build:

http://www.marcuscom.com/tb/errors/6.2-MarcusCom/gnupg-2.0.4.log

Joe

-- 
PGP Key : http://www.marcuscom.com/pgp.asc
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 187 bytes
Desc: This is a digitally signed message part
Url : http://lists.freebsd.org/pipermail/cvs-all/attachments/20070902/a8660fa6/attachment.pgp