cvs commit: ports/security/gnupg Makefile
dougb at FreeBSD.org
Tue Sep 4 15:18:55 PDT 2007
On Tue, 4 Sep 2007, Michael Nottebrock wrote:
> Doug Barton schrieb:
>>> OPTIONS would be reasonable in this case. We can enable ncurses backend
>>> by default and user will be able to configure the port to make it use
>>> other backends he/she wants.
>> That is basically what I had in mind. I'd like to hear from lofi, but
>> my offer to help with that is still good.
> Security/pinentry is an "old school" master-port for the
> pinentry-[toolkit] slave-ports. I stopped doing master-slave ports of
> that sort after that one precisely because you end up in situations like
> this where people manage to miss the ports they are supposed to use
> despite the fact they are being pointed to them in pkg-messages and they
> can be very easily found in a search.
So it sounds to me like you're saying that the pinentry port is not
designed to be used directly?
> Apparently even committers sometimes cannot see the wood for the trees
> because Roman could have just added options for each of the pinentry
> slave ports to the already existing gnupg options menu in his PR
That's an interesting idea that I hadn't considered. I think doing that,
with a default of the curses version would probably be ok ... the only
concern I have with that is what to do if the user chooses more than one
and they conflict. Generate a fatal error?
> I would like that better than a runtime dependency on an
> option-ifyed pinentry port, but not by much, because the main reason why
> I never added a runtime dependency on any pinentry to the gnupg port
> (back when it was still gnupg-devel) still remains: Whatever pinentry
> you depend on by default through whatever indirection, it will be always
> be the wrong one for the package users out there. That is why the
> pkg-message in gnupg exists.
Well as I said at the start, I agree with this approach, but if others are
determined to add a dependency, I think doing it in a less painful way is
a good idea.
This .signature sanitized for your protection
More information about the cvs-all