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

2015-03-27 Thread Sundararajan R
s are true. Helped-by: Junio C Hamano Helped-by: Eric Sunshine Helped-by: Matthieu Moy Signed-off-by: Sundararajan R --- Have made the edits suggested by Matthew and Kevin. t/t7102-reset.sh | 74 1 file changed, 74 insertions(+) diff --

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

2015-03-27 Thread Sundararajan R
dies with a message about the ambiguous argument. When none of the above cases hold, - behaves like @{-1}. Helped-by: Junio C Hamano Helped-by: Eric Sunshine Signed-off-by: Sundararajan R --- Corrected a minor style error. builtin/reset.c | 21 +++-- 1 file changed, 19 ins

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 wh

[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 2/2] reset: add tests for git reset -

2015-03-18 Thread Sundararajan R
e. Helped-by: Junio C Hamano Helped-by: Eric Sunshine Helped-by: Matthieu Moy Signed-off-by: Sundararajan R --- 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. Please let me know if t

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

2015-03-18 Thread Sundararajan R
dies with a message about the ambiguous argument. Helped-by: Junio C Hamano Helped-by: Eric Sunshine Signed-off-by: Sundararajan R --- Thank you Eric and Junio for your patient feedback. As verify_filename() and verify_non_filename() die and return,respectively when passed the argument '-

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

2015-03-12 Thread Sundararajan R
Helped-by: Matthieu Moy Signed-off-by: Sundararajan R --- 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 let me know if there is something el

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

2015-03-10 Thread Sundararajan R
A similar ambiguity 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 Helped-by: Eric Sunshine Signed-off-by: Sundararajan R --- Have made the modific

[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 Thanks-to: Junio C Hamano --- I have added 6 tests to check for the following cases: git reset - with no @{-1} git reset - with no @{-1} and file

[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 Thanks-to: Junio C Hamano --- I have attempted to resolve the ambiguity when there exists a file named - by communicating to the user that he/she can

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 wrote: Sundararajan R 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 pat

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

2015-03-06 Thread Sundararajan R
t;@{-1}". For this I have a rough idea 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 --- builtin/reset

[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