@pmatilai commented on this pull request.


> +[0],
+[],
+[Autosign already configured
+])
+
+RPMTEST_CHECK([
+runroot rm -rf /root/.config/rpm
+runroot ${RPM_CONFIGDIR_PATH}/rpm-setup-autosign -p gpg
+],
+[0],
+[],
+[stderr])
+
+RPMTEST_CHECK([
+cmd=$(grep ^sudo stderr | cut -c6-)
+test -n "${cmd}" && runroot ${cmd}

Righty, totally forgot this construct is there. It comes up in the GH action 
log as the culprit but still didn't connect...

I guess the idea was to ensure the command we output really works. So I think 
I'll just drop the "is a tty" test from the script for now, that's cosmetic at 
best anyway.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3665#discussion_r2022162167
You are receiving this because you are subscribed to this thread.

Message ID: <rpm-software-management/rpm/pull/3665/review/[email protected]>
_______________________________________________
Rpm-maint mailing list
[email protected]
https://lists.rpm.org/mailman/listinfo/rpm-maint

Reply via email to