From: David Dombrowsky <6thstreetradio@gmail.com>
To: cygwin@cygwin.com
Subject: Re: pinentry-curses not available?
Date: Tue, 22 Jan 2019 14:36:00 -0000 [thread overview]
Message-ID: <2283765c-1f5a-1a09-8901-2ef9239c14bd@gmail.com> (raw)
In-Reply-To: <CAHdnY6O0_XTFYnwhQ-Eg5AVAhhTB5yuocGERBV_4TWPkdV+AoA@mail.gmail.com>
[-- Attachment #1: Type: text/plain, Size: 1362 bytes --]
On 11/5/18 11:06 AM, David Dombrowsky wrote:> On Sat, Nov 3, 2018 at
2:23 PM Marco Atzeri <marco.atzeri@gmail.com> wrote:
>>> In any case, I got pinentry-1.1.0 to compile against a cygwin-supplied
>>> version of ncurses, and it mostly works (you still need to tell it to
>>> not detach from the tty, not sure what's up with that).
>>
>> can you clarify ?
>> May be it was the reason why I and Yaakov found it not functional
>
> That's very possible.
>
> I downloaded the stock source for pinentry 1.1.0 from gnupg.org and
> tried to build it. After "fixing" one of the build errors (ttytype is
> a macro and needs to be renamed in the pinentry source, at least on my
> cygwin install), I got it to build and install.
For the record, I've had to do this on a couple of machines, so I
figured I'd post the patch. Still doesn't work reliably. I need to log
in to the desktop, start a screen session, and then run
gpg-agent --daemon --disable-scdaemon --no-detach -vv --pinentry-program
/usr/local/bin/pinentry-curses > agent.log 2>&1 &
to get it to work. After that I can sign git commits all day without
having to RDP back into the box.
Not the best, but it works. If anyone has a better solution, please let
me know.
--
David Dombrowsky, Software Engineer
davek@6thstreetradio.org | 518-374-3204
https://www.linkedin.com/in/david-dombrowsky-94334415
[-- Attachment #2: pinentry-1.1.0-cygwin.patch --]
[-- Type: text/x-patch, Size: 3301 bytes --]
diff -ur orig/pinentry-1.1.0/pinentry/pinentry.c pinentry-1.1.0/pinentry/pinentry.c
--- orig/pinentry-1.1.0/pinentry/pinentry.c 2017-12-03 11:13:15.000000000 -0500
+++ pinentry-1.1.0/pinentry/pinentry.c 2018-11-02 15:29:16.945203400 -0400
@@ -97,7 +97,7 @@
Don't reset them. */
int grab = pinentry.grab;
char *ttyname = pinentry.ttyname;
- char *ttytype = pinentry.ttytype;
+ char *ttytype_ = pinentry.ttytype_;
char *ttyalert = pinentry.ttyalert;
char *lc_ctype = pinentry.lc_ctype;
char *lc_messages = pinentry.lc_messages;
@@ -135,7 +135,7 @@
if (use_defaults)
{
free (pinentry.ttyname);
- free (pinentry.ttytype);
+ free (pinentry.ttytype_);
free (pinentry.ttyalert);
free (pinentry.lc_ctype);
free (pinentry.lc_messages);
@@ -194,7 +194,7 @@
{
pinentry.grab = grab;
pinentry.ttyname = ttyname;
- pinentry.ttytype = ttytype;
+ pinentry.ttytype_ = ttytype_;
pinentry.ttyalert = ttyalert;
pinentry.lc_ctype = lc_ctype;
pinentry.lc_messages = lc_messages;
@@ -950,8 +950,8 @@
}
break;
case 'N':
- pinentry.ttytype = strdup (pargs.r.ret_str);
- if (!pinentry.ttytype)
+ pinentry.ttytype_ = strdup (pargs.r.ret_str);
+ if (!pinentry.ttytype_)
{
#ifndef HAVE_W32CE_SYSTEM
fprintf (stderr, "%s: %s\n", this_pgmname, strerror (errno));
@@ -1070,10 +1070,10 @@
}
else if (!strcmp (key, "ttytype"))
{
- if (pinentry.ttytype)
- free (pinentry.ttytype);
- pinentry.ttytype = strdup (value);
- if (!pinentry.ttytype)
+ if (pinentry.ttytype_)
+ free (pinentry.ttytype_);
+ pinentry.ttytype_ = strdup (value);
+ if (!pinentry.ttytype_)
return gpg_error_from_syserror ();
}
else if (!strcmp (key, "ttyalert"))
@@ -1755,7 +1755,7 @@
{
snprintf (buffer, sizeof buffer, "%s %s %s",
pinentry.ttyname? pinentry.ttyname : "-",
- pinentry.ttytype? pinentry.ttytype : "-",
+ pinentry.ttytype_? pinentry.ttytype_ : "-",
pinentry.display? pinentry.display : "-" );
buffer[sizeof buffer -1] = 0;
rc = assuan_send_data (ctx, buffer, strlen (buffer));
diff -ur orig/pinentry-1.1.0/pinentry/pinentry.h pinentry-1.1.0/pinentry/pinentry.h
--- orig/pinentry-1.1.0/pinentry/pinentry.h 2017-12-03 11:13:05.000000000 -0500
+++ pinentry-1.1.0/pinentry/pinentry.h 2018-11-02 15:25:42.578732900 -0400
@@ -75,7 +75,7 @@
supported. (Assuan: "OPTION ttyname TTYNAME".) */
char *ttyname;
/* The type of the terminal. (Assuan: "OPTION ttytype TTYTYPE".) */
- char *ttytype;
+ char *ttytype_;
/* Set the alert mode (none, beep or flash). */
char *ttyalert;
/* The LC_CTYPE value for the terminal. (Assuan: "OPTION lc-ctype
diff -ur orig/pinentry-1.1.0/pinentry/pinentry-curses.c pinentry-1.1.0/pinentry/pinentry-curses.c
--- orig/pinentry-1.1.0/pinentry/pinentry-curses.c 2017-12-03 11:13:05.000000000 -0500
+++ pinentry-1.1.0/pinentry/pinentry-curses.c 2018-11-02 15:27:00.269222900 -0400
@@ -1188,7 +1188,7 @@
}
#endif
- rc = dialog_run (pinentry, pinentry->ttyname, pinentry->ttytype);
+ rc = dialog_run (pinentry, pinentry->ttyname, pinentry->ttytype_);
do_touch_file (pinentry);
return rc;
}
[-- Attachment #3: Type: text/plain, Size: 219 bytes --]
--
Problem reports: http://cygwin.com/problems.html
FAQ: http://cygwin.com/faq/
Documentation: http://cygwin.com/docs.html
Unsubscribe info: http://cygwin.com/ml/#unsubscribe-simple
prev parent reply other threads:[~2019-01-22 14:36 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-11-02 19:47 David Dombrowsky
2018-11-02 20:21 ` cyg Simple
2018-11-02 20:47 ` David Dombrowsky
2018-11-03 18:23 ` Marco Atzeri
2018-11-05 16:06 ` David Dombrowsky
2019-01-22 14:36 ` David Dombrowsky [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=2283765c-1f5a-1a09-8901-2ef9239c14bd@gmail.com \
--to=6thstreetradio@gmail.com \
--cc=cygwin@cygwin.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).