Re: [Rpm-maint] [rpm-software-management/rpm] GPG_TTY warning causes test-suite failure in mock (#1290)

2020-06-26 Thread Panu Matilainen
Closed #1290 via #1291.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/1290#event-3485664720___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] GPG_TTY warning causes test-suite failure in mock (#1290)

2020-06-25 Thread Panu Matilainen
Here's an simple reproducer that doesn't need messing with mock:

> [pmatilai︎lumikko rpm]$ cp tests/data/RPMS/hello-2.0-1.x86_64.rpm /tmp/
[pmatilai︎lumikko rpm]$ ./rpmsign --addsign /tmp/hello-2.0-1.x86_64.rpm < 
/dev/null
/tmp/hello-2.0-1.x86_64.rpm:
warning: Could not set GPG_TTY to stdin: Inappropriate ioctl for device
[pmatilai︎lumikko rpm]$

Or to reproduce in the test-suite context, `make check < /dev/null`.

So actually the warning is quite appropriate in this situation, that this works 
at all in the test-suite depends on the key not having a passphrase at all so 
nothing will need to ask for it. The right thing to do is probably to set 
GPG_TTY explicitly from the test-suite instead - I'll submit a PR in a minute.

(so typical, I assign this to you but then end up looking at it myself, trying 
to kill time before vacation...)

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/1290#issuecomment-649448251___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] GPG_TTY warning causes test-suite failure in mock (#1290)

2020-06-24 Thread Panu Matilainen
No worries, it's not the first (or last) time mock trips up something because 
it's a rather special environment.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/1290#issuecomment-648809464___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] GPG_TTY warning causes test-suite failure in mock (#1290)

2020-06-24 Thread Michal Domonkos
Oh my. Thanks for sharing, I'll check that.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/1290#issuecomment-648804803___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


[Rpm-maint] [rpm-software-management/rpm] GPG_TTY warning causes test-suite failure in mock (#1290)

2020-06-24 Thread Panu Matilainen
Seems that something in mock environment trips up the new GPG_TTY related 
warning, causing (a bogus) test-suite failure due to the extra output:
https://download.copr.fedorainfracloud.org/results/pmatilai/rpm-snapshot/fedora-rawhide-x86_64/01499743-rpm/build.log.gz

```
 --- /dev/null  2020-06-24 10:47:07.441643129 +
+++ 
/builddir/build/BUILD/rpm-4.16.90-git15102/tests/rpmtests.dir/at-groups/242/stderr
  2020-06-24 10:48:56.462891354 +
@@ -0,0 +1,2 @@
+warning: Could not set GPG_TTY to stdin: Inappropriate ioctl for device
+warning: Could not set GPG_TTY to stdin: Inappropriate ioctl for device
242. rpmsigdig.at:414: 242. rpmsign --addsign (rpmsigdig.at:414): FAILED 
(rpmsigdig.at:420)
```

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/1290___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint