[PATCH v4 2/2] reset: add tests for git reset -

2015-03-27 Thread Sundararajan R
-by: Eric Sunshine sunsh...@sunshineco.com Helped-by: Matthieu Moy matthieu@grenoble-inp.fr Signed-off-by: Sundararajan R dyou...@gmail.com --- Have made the edits suggested by Matthew and Kevin. t/t7102-reset.sh | 74 1 file changed, 74

[PATCH v4 1/2] reset: add '-' shorthand for '@{-1}'

2015-03-27 Thread Sundararajan R
argument. When none of the above cases hold, - behaves like @{-1}. Helped-by: Junio C Hamano gits...@pobox.com Helped-by: Eric Sunshine sunsh...@sunshineco.com Signed-off-by: Sundararajan R dyou...@gmail.com --- Corrected a minor style error. builtin/reset.c | 21 +++-- 1 file

Fwd: [RFC] [GSoC Proposal Draft] Unifying git branch -l,git tag -l and git for-each-ref

2015-03-23 Thread Sundararajan R
Hi all, I have attempted a microproject [1][2] and this is my first draft of the proposal.I have included only the matter regarding my approach to solving the problem and shall add my personal details later. Please be kind enough to go through my proposal and suggest modifications or detailing

[GSoC Project Help] Unifying git branch -l, git tag -l and git for-each-ref

2015-03-19 Thread Sundararajan R
Hi all, I am a Computer Science sophomore at IIT Kanpur. I am interested in contributing to git in GSoC 2015. I have been using git for the past one year and am pretty comfortable with its commands which is what made me think about contributing to git. I have attempted the microproject “adding

[v3 PATCH 1/2] reset: add '-' shorthand for '@{-1}'

2015-03-18 Thread Sundararajan R
argument. Helped-by: Junio C Hamano gits...@pobox.com Helped-by: Eric Sunshine sunsh...@sunshineco.com Signed-off-by: Sundararajan R dyou...@gmail.com --- Thank you Eric and Junio for your patient feedback. As verify_filename() and verify_non_filename() die and return,respectively when passed

[v3 PATCH 2/2] reset: add tests for git reset -

2015-03-18 Thread Sundararajan R
-by: Eric Sunshine sunsh...@sunshineco.com Helped-by: Matthieu Moy matthieu@grenoble-inp.fr Signed-off-by: Sundararajan R dyou...@gmail.com --- Thank you for your feedback Torsten and Eric. I have made modifications suggested by you. I have also acted on Matthieu's suggestions on the archive

[v2 PATCH 2/2] reset: add tests for git reset -

2015-03-12 Thread Sundararajan R
...@sunshineco.com Helped-by: Matthieu Moy matthieu@grenoble-inp.fr Signed-off-by: Sundararajan R dyou...@gmail.com --- Thank you for your feedback Torsten and Eric. I have now made the modifications suggested by you. I have also incorporated the suggestions given by Matthieu on the archive. Please

[v2 PATCH 1/2] reset: add '-' shorthand for '@{-1}'

2015-03-10 Thread Sundararajan R
occurs when the file @{-1} exits. Therefore, when the files '-' or '@{-1}' exist then the program dies with a message about the ambiguous argument. Helped-by: Junio C Hamano gits...@pobox.com Helped-by: Eric Sunshine sunsh...@sunshineco.com Signed-off-by: Sundararajan R dyou...@gmail.com --- Have

[PATCH 1/2] Adding - shorthand for @{-1} in RESET command

2015-03-09 Thread Sundararajan R
Please give feedback and suggest things I may have missed out on. I hope I have incorporated all the suggestions. Signed-off-by: Sundararajan R dyou...@gmail.com Thanks-to: Junio C Hamano --- I have attempted to resolve the ambiguity when there exists a file named - by communicating to the user

[PATCH 2/2] Added tests for git reset -

2015-03-09 Thread Sundararajan R
As you had suggested @Junio, I have added the required tests. Please let me know if there is something is I should add. Signed-off-by: Sundararajan R dyou...@gmail.com Thanks-to: Junio C Hamano --- I have added 6 tests to check for the following cases: git reset - with no @{-1} git reset

Re: [PATCH] [GSoC Microproject]Adding - shorthand for @{-1} in RESET command

2015-03-08 Thread Sundararajan R
On Sun, Mar 8, 2015 at 1:04 PM Junio C Hamano gits...@pobox.com wrote: Sundararajan R dyou...@gmail.com writes: diff --git a/builtin/reset.c b/builtin/reset.c index 4c08ddc..62764d4 100644 --- a/builtin/reset.c +++ b/builtin/reset.c @@ -203,8 +203,16 @@ static void parse_args(struct pathspec

[PATCH] [GSoC Microproject]Adding - shorthand for @{-1} in RESET command

2015-03-06 Thread Sundararajan R
that I would have to call get-sha1() on @{-1} to check if there is an object matching with it. But I am not able to think of the details. Please guide me with that and give feedback for this patch. Signed-off-by: Sundararajan R dyou...@gmail.com --- builtin/reset.c | 12 +++- 1 file changed

[GSoC microproject help] Allow - as a short-hand for @{-1} in more places

2015-03-06 Thread Sundararajan R
Hi All, I am a sophomore at Indian Institute of Technology Kanpur and am a GSoC aspirant for git. Although I have been using git from a long time, this is the first occasion when I have picked up reading its source code. Can somebody please help me by telling me how to start off with the above