Skip to content

Conversation

dscho
Copy link
Member

@dscho dscho commented Dec 13, 2018

The oneline notwithstanding, 1337498 (completion: use __gitcomp_builtin for format-patch, 2018-11-03) changed also the way send-email options are completed, by asking the git send-email command itself what options it offers.

Necessarily, this must fail when built with NO_PERL because send-email itself is a Perl script. Which means that we need the PERL prerequisite for the send-email test case in t9902.

Changes since v1:

  • replaced the commit message by the cover letter, as it was deemed to be more informative.

@dscho
Copy link
Member Author

dscho commented Dec 13, 2018

/submit

@gitgitgadget
Copy link

gitgitgadget bot commented Dec 13, 2018

Submitted as pull.99.git.gitgitgadget@gmail.com

@dscho dscho force-pushed the t9902-no-perl-fix branch from 32e6bf2 to 2a055a3 Compare December 13, 2018 13:52
The oneline notwithstanding, 1337498 (completion: use _gitcompbuiltin
for format-patch, 2018-11-03) changed also the way send-email options
are completed, by asking the git send-email command itself what options
it offers.

Necessarily, this must fail when built with NO_PERL because send-email
itself is a Perl script. Which means that we need the PERL prerequisite
for the send-email test case in t9902.

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
@dscho dscho force-pushed the t9902-no-perl-fix branch from 2a055a3 to 8e3ddff Compare December 13, 2018 13:54
@dscho
Copy link
Member Author

dscho commented Dec 13, 2018

/submit

@gitgitgadget
Copy link

gitgitgadget bot commented Dec 13, 2018

Submitted as pull.99.v2.git.gitgitgadget@gmail.com

@gitgitgadget
Copy link

gitgitgadget bot commented Dec 15, 2018

This branch is now known as js/t9902-send-email-completion-fix.

@gitgitgadget
Copy link

gitgitgadget bot commented Dec 15, 2018

This patch series was integrated into pu via git@6be6e66.

@gitgitgadget
Copy link

gitgitgadget bot commented Dec 15, 2018

This patch series was integrated into next via git@6be6e66.

@gitgitgadget
Copy link

gitgitgadget bot commented Dec 15, 2018

This patch series was integrated into master via git@6be6e66.

@gitgitgadget
Copy link

gitgitgadget bot commented Dec 15, 2018

This patch series was integrated into maint via git@6be6e66.

@gitgitgadget gitgitgadget bot closed this Dec 15, 2018
@dscho dscho deleted the t9902-no-perl-fix branch December 15, 2018 21:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant