Since v4, I have taken Junio's suggestion to change '@' to 'HEAD' as early as reasonably possible. And also added new test to check the no-op case of 'checkout HEAD/@'. And the second patch removes all the perl prerequisites from the tests which use patch mode functionality, as pointed out by Phillip. And have also taken into account Eric's suggestion to not remove perl prerequisites while amending them in patch (1/2) and do it all together in patch (2/2). Ghanshyam Thakkar (2): add-patch: classify '@' as a synonym for 'HEAD' add -p tests: remove PERL prerequisites add-patch.c | 8 ------ builtin/checkout.c | 4 ++- builtin/reset.c | 4 ++- t/t2016-checkout-patch.sh | 46 +++++++++++++++++++--------------- t/t2020-checkout-detach.sh | 12 +++++++++ t/t2024-checkout-dwim.sh | 2 +- t/t2071-restore-patch.sh | 46 ++++++++++++++++++---------------- t/t3904-stash-patch.sh | 6 ----- t/t7105-reset-patch.sh | 37 ++++++++++++++------------- t/t7106-reset-unborn-branch.sh | 2 +- t/t7514-commit-patch.sh | 6 ----- 11 files changed, 91 insertions(+), 82 deletions(-) -- 2.43.0
