You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@yetus.apache.org by "Allen Wittenauer (JIRA)" <ji...@apache.org> on 2017/05/17 22:42:05 UTC
[jira] [Commented] (YETUS-315) smart-apply-patch committer mode
should support signed commits
[ https://issues.apache.org/jira/browse/YETUS-315?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16014877#comment-16014877 ]
Allen Wittenauer commented on YETUS-315:
----------------------------------------
Hi. Playing catch-up.
Maybe if I talk about my personal workflow it will help?
Typically, I'll get a patch, do a quick once over, and then use smart-apply-patch in non-committer mode to test it out. After I'm finished vetting it, I'll then hop over to my ASF directory, run s-a-p in --committer mode, and then do a git push. My hope was that --committer would be "one stop shopping".
I'm starting to think this may have been naïve and/or too optimistic.
So what if we make "--committer" a bit like "--jenkins" in test-patch? It effectively becomes a macro for what we "recommend" as the default state, but individuals can selectively pick their feature set via specific command line flags. If someone picks --committer and one of those states fails (not git-am compatible, no GPG, whatever), we fail passing along git's message.
Thoughts?
> smart-apply-patch committer mode should support signed commits
> --------------------------------------------------------------
>
> Key: YETUS-315
> URL: https://issues.apache.org/jira/browse/YETUS-315
> Project: Yetus
> Issue Type: New Feature
> Components: Test Patch
> Reporter: Allen Wittenauer
> Assignee: Suraj Acharya
> Fix For: 0.5.0
>
>
> A new {{--gpg-sign}} flag is added to the smart-apply-patch.sh.
> This will allow the committer to sign the commit.
> GPG Agent will handle the passphrase request.
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)