On Mon, Mar 3, 2014 at 1:36 AM, Guanglin Xu mzguang...@gmail.com wrote:
to avoid a magic code of 11.
Helped-by: Sun He sunheeh...@gmail.com
Helped-by: Eric Sunshine sunsh...@sunshineco.com
Helped-by: Jacopo Notarstefano jaco...@gmail.com
Signed-off-by: Guanglin Xu mzguang...@gmail.com
---
On Mon, Mar 3, 2014 at 2:18 AM, Dmitry S. Dolzhenko
dmitrys.dolzhe...@yandex.ru wrote:
Dmitry S. Dolzhenko (11):
builtin/pack-objects.c: use ALLOC_GROW() in check_pbase_path()
bundle.c: use ALLOC_GROW() in add_to_ref_list()
cache-tree.c: use ALLOC_GROW() in find_subtree()
commit.c:
On Sat, Mar 1, 2014 at 7:12 AM, Nguyễn Thái Ngọc Duy pclo...@gmail.com wrote:
If $GIT_COMMON_DIR is set, it should be $GIT_COMMON_DIR/hooks/, not
$GIT_DIR/hooks/. Just let rev-parse --git-path handle it.
Signed-off-by: Nguyễn Thái Ngọc Duy pclo...@gmail.com
---
git-am.sh
Thanks for the submission. Some commentary below to expose you to the
review process on this project...
On Mon, Mar 3, 2014 at 2:47 AM, Karthik Nayak karthik@gmail.com wrote:
Replace with skip_prefix(), which uses the inbuilt function
strcmp() to compare.
Explaining the purpose of the
Hi Eric,
Yes, you're right. !! is comfortably concise and also idiomatic in
Git sources.
Thanks,
Guanglin
2014-03-03 16:12 GMT+08:00 Eric Sunshine sunsh...@sunshineco.com:
On Mon, Mar 3, 2014 at 1:36 AM, Guanglin Xu mzguang...@gmail.com wrote:
to avoid a magic code of 11.
Helped-by: Sun He
The format of this email is wrong. The non-commit-message notes should
come between the --- line (- note, there are three minus signs here)
and the patch itself.
On 03/01/2014 08:48 PM, Tanay Abhra wrote:
Signed-off-by: Tanay Abhra tanay...@gmail.com
---
commit.c |2 +-
1 file changed,
to avoid a magic code of 11.
Helped-by: Sun He sunheeh...@gmail.com
Helped-by: Eric Sunshine sunsh...@sunshineco.com
Helped-by: Jacopo Notarstefano jaco...@gmail.com
Signed-off-by: Guanglin Xu mzguang...@gmail.com
---
This is an implementation of the idea#2 of GSoC 2014 microproject.
branch.c
On 03/02/2014 04:55 PM, Matthieu Moy wrote:
Eric Sunshine sunsh...@sunshineco.com writes:
On Sun, Mar 2, 2014 at 3:53 AM, Eric Sunshine sunsh...@sunshineco.com
wrote:
On Sat, Mar 1, 2014 at 9:53 PM, Nguyễn Thái Ngọc Duy pclo...@gmail.com
wrote:
This is rev-list style, where people can do
The old version fixes a maximum length on the buffer, which could be a problem
if one is not certain of the length of get_object_directory().
Using strbuf can avoid the protential bug.
Helped-by: Michael Haggerty mhag...@alum.mit.edu
Helped-by: Eric Sunshine sunsh...@sunshineco.com
Signed-off-by:
2014-03-03 15:41 GMT+08:00 Eric Sunshine sunsh...@sunshineco.com:
On Sat, Mar 1, 2014 at 9:29 PM, Sun He sunheeh...@gmail.com wrote:
Signed-off-by: Sun He sunheeh...@gmail.com
Helped-by: Eric Sunshine sunsh...@sunshineco.com
Helped-by: Michael Haggerty mhag...@alum.mit.edu
---
This patch
Michael Haggerty mhag...@alum.mit.edu writes:
Or perhaps -NUM should fail with an error message if any of the last
NUM commits are merges. In that restricted scenario (which probably
accounts for 99% of rebases), -NUM is equivalent to HEAD~NUM.
Makes sense to me. So, -NUM would actually mean
Replacing memcpy with hashcpy is more directly and elegant.
Leave ppc/sha1.c alone, as it is an isolated component.
Pull cache.h(actually ../cache.h) in just for one memcpy
there is not proper.
Helped-by: Michael Haggerty mhag...@alum.mit.edu
Helped-by: Duy Nguyen pclo...@gmail.com
Am 01.03.2014 07:54, schrieb Torsten Bögershausen:
On 2014-03-01 03.42, Lee Hopkins wrote:
+
+if(ignore_case)
Only looking at ignore_case here closes the door for people
who have a branch foo and Foo at the same time.
(Which means that they are carefully running git pack-refs)
How about
On Mon, Mar 3, 2014 at 4:37 PM, Matthieu Moy
matthieu@grenoble-inp.fr wrote:
Michael Haggerty mhag...@alum.mit.edu writes:
Or perhaps -NUM should fail with an error message if any of the last
NUM commits are merges. In that restricted scenario (which probably
accounts for 99% of
On 03/02/2014 10:09 AM, Eric Sunshine wrote:
On Sun, Mar 2, 2014 at 4:04 AM, Eric Sunshine sunsh...@sunshineco.com wrote:
On Sat, Mar 1, 2014 at 9:53 PM, Nguyễn Thái Ngọc Duy pclo...@gmail.com
wrote:
git rebase -e XYZ is basically the same as
EDITOR=sed -i '1s/pick XYZ/edit XYZ/' $@ \
git
Duy Nguyen pclo...@gmail.com writes:
On Mon, Mar 3, 2014 at 4:37 PM, Matthieu Moy
This would actually be a feature for me: I often want to rebase recent
enough history, and when my @{upstream} isn't well positionned, I
randomly type HEAD~N without remembering what N should be. When N is too
On Mon, Mar 03, 2014 at 10:37:11AM +0100, Matthieu Moy wrote:
Michael Haggerty mhag...@alum.mit.edu writes:
Or perhaps -NUM should fail with an error message if any of the last
NUM commits are merges. In that restricted scenario (which probably
accounts for 99% of rebases), -NUM is
On Mon, Mar 3, 2014 at 5:10 PM, Michael Haggerty mhag...@alum.mit.edu wrote:
On 03/02/2014 10:09 AM, Eric Sunshine wrote:
On Sun, Mar 2, 2014 at 4:04 AM, Eric Sunshine sunsh...@sunshineco.com
wrote:
On Sat, Mar 1, 2014 at 9:53 PM, Nguyễn Thái Ngọc Duy pclo...@gmail.com
wrote:
git rebase -e
We used to show (missing ) next to tests skipped because they are
specified in GIT_SKIP_TESTS. Use (matched by GIT_SKIP_TESTS) instead.
---
t/test-lib.sh | 13 -
1 files changed, 8 insertions(+), 5 deletions(-)
diff --git a/t/test-lib.sh b/t/test-lib.sh
index 1531c24..89a405b
This is a counterpart to GIT_SKIP_TESTS. Mostly useful when debugging.
---
t/README | 15 +++
t/test-lib.sh |8
2 files changed, 23 insertions(+), 0 deletions(-)
diff --git a/t/README b/t/README
index caeeb9d..f939987 100644
--- a/t/README
+++ b/t/README
@@
Built-in commands can specify names for option arguments, that are shown
when usage text is generated for the command. sh based commands should
be able to do the same.
Option argument name hint is any text that comes after [*=?!] after the
argument name up to the first whitespace. Underscores
Duy Nguyen pclo...@gmail.com writes:
On Mon, Mar 3, 2014 at 4:37 PM, Matthieu Moy
matthieu@grenoble-inp.fr wrote:
Michael Haggerty mhag...@alum.mit.edu writes:
Or perhaps -NUM should fail with an error message if any of the last
NUM commits are merges. In that restricted scenario
Duy Nguyen pclo...@gmail.com writes:
Logically, yes. Practically, no. If you have to put multiple -e and
some hashes in one line, wouldn't editing to-do list in your favorite
editor be faster?
An editor is the last resort when the card puncher is broken.
--
David Kastrup
--
To unsubscribe
Hi,
Based on my experience so far as a first-time Google Summer of Code
mentor, I just wrote a blog article containing some hopefully useful
advice for students applying to the program. Please note that this is
my personal opinion only and doesn't necessarily reflect the views of
the Git/libgit2
Hi,
I have a git superproject with 3 submodules. The submodules are cloned from the
same URL but use different branches. Git clones the repo three times and I have
three entries in .git/modules. Is it possible to reuse the first clone for the
next submodule clones?
Stefan
--
To unsubscribe
On Mar 3, 2014, at 7:24 AM, stefan.li...@partner.bmw.de wrote:
I have a git superproject with 3 submodules. The submodules are cloned from
the same URL but use different branches. Git clones the repo three times and
I have three entries in .git/modules. Is it possible to reuse the first
2014-02-28 4:45 GMT+08:00 Dmitry S. Dolzhenko dmitrys.dolzhe...@yandex.ru:
Signed-off-by: Dmitry S. Dolzhenko dmitrys.dolzhe...@gmail.com
---
attr.c | 7 +--
builtin/pack-objects.c | 7 +--
bundle.c | 6 +-
cache-tree.c | 6 +-
I don't think this distinction is necessary, either you have a
case-insensitive file system or you don't. The case
that the .git directory is case-sensitive and the worktree directory isn't
(or the other way around) is
probably so exotic that we can ignore it.
I think Torsten's use case
Hi,
Any chance to have this patch considered?
Thanks in advance,
_g.
Gilles Filippini a écrit , Le 01/03/2014 17:33:
This is to prevent unwanted prefix when such an environment variable
exists. The case occurs for example during the Debian package build
where the git-subtree test suite is
Separate message from command examples to avoid translation issues
such as a dash being omitted in a translation.
Signed-off-by: Sandy Carter sandy.car...@savoirfairelinux.com
---
builtin/branch.c | 10 ++
1 file changed, 6 insertions(+), 4 deletions(-)
diff --git a/builtin/branch.c
This new option prevent git submodule add|update to clone the missing
submodules with the --separate-git-dir option.
Then the submodule will be regular repository and their gitdir will not
be placed in the superproject gitdir/modules directory.
Signed-off-by: Henri GEIST geist.he...@laposte.net
Add missing leading dash to proposed commands in french output when
using the command:
git branch --set-upstream remotename/branchname
and when upstream is gone
Signed-off-by: Sandy Carter sandy.car...@savoirfairelinux.com
---
po/fr.po | 4 ++--
1 file changed, 2 insertions(+), 2
From: Ilya Bobyr ilya.bo...@gmail.com
We used to show (missing ) next to tests skipped because they are
specified in GIT_SKIP_TESTS. Use (matched by GIT_SKIP_TESTS)
instead.
The message below forgets the by. Otherwise looks sensible.
---
t/test-lib.sh | 13 -
1 files changed,
Hello Eric,
Thanks for Pointing out everything, i had a thorough look and fixed a
couple of things.
Here is an Updated Patch.
- Removed unnecessary code and variables.
- Replaced all instances of starts_with() with skip_prefix()
Replace starts_with() with skip_prefix() for better reading
Minor nits.
From: Ilya Bobyr ilya.bo...@gmail.com
This is a counterpart to GIT_SKIP_TESTS. Mostly useful when
debugging.
---
t/README | 15 +++
t/test-lib.sh |8
2 files changed, 23 insertions(+), 0 deletions(-)
diff --git a/t/README b/t/README
index
In record_author_date() parse_gpg_output() ,using skip_prefix() instead of
starts_with() is more elegant and abstracts away the details.
Helped-by: Michael Haggerty mhag...@alum.mit.edu
Signed-off-by: Tanay Abhra tanay...@gmail.com
---
Patch V2 Corrected email formatting ,reapplied the
Signed-off-by: Michael Haggerty mhag...@alum.mit.edu
---
A trivial little cleanup.
cache-tree.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/cache-tree.c b/cache-tree.c
index 0bbec43..7f63c23 100644
--- a/cache-tree.c
+++ b/cache-tree.c
@@ -565,7 +565,7 @@ static struct
Michael Haggerty mhag...@alum.mit.edu writes:
Signed-off-by: Michael Haggerty mhag...@alum.mit.edu
---
A trivial little cleanup.
cache-tree.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/cache-tree.c b/cache-tree.c
index 0bbec43..7f63c23 100644
--- a/cache-tree.c
From: Michael Haggerty mhag...@alum.mit.edu
Cc: Dmitry Dolzhenko dmitrys.dolzhe...@yandex.ru; Sun He
sunheeh...@gmail.com; Brian Gesiak modoca...@gmail.com; Tanay
Abhra tanay...@gmail.com; Kyriakos Georgiou
kyriakos.a.georg...@gmail.com; Siddharth Goel
siddharth98...@gmail.com; Guanglin Xu
On 03/03/2014 05:32 PM, David Kastrup wrote:
[...]
So perhaps all of that should just be
while (*slash == '/')
slash++;
if (!*slash)
return it;
I just fixed a little thing that caught my eye. You OWNED it. You are
On Mon, Mar 3, 2014 at 5:45 AM, Michael Haggerty mhag...@alum.mit.edu wrote:
My secret tip for GSoC success
http://softwareswirl.blogspot.com/2014/03/my-secret-tip-for-gsoc-success.html
I enjoyed reading that BLOG post. I daresay some of the points you
raise are pertinent for any new
Michael Haggerty mhag...@alum.mit.edu writes:
On 03/03/2014 05:32 PM, David Kastrup wrote:
[...]
So perhaps all of that should just be
while (*slash == '/')
slash++;
if (!*slash)
return it;
I just fixed a little
Am 03.03.2014 14:47, schrieb Henri GEIST:
This new option prevent git submodule add|update to clone the missing
submodules with the --separate-git-dir option.
Then the submodule will be regular repository and their gitdir will not
be placed in the superproject gitdir/modules directory.
And
Duy Nguyen pclo...@gmail.com writes:
On Thu, Feb 20, 2014 at 6:26 AM, Junio C Hamano gits...@pobox.com wrote:
Is there a reason not to do just an equivalent of
#define git_pathdup mkpathdup
and be done with it? Am I missing something?
They have a subtle difference: mkpathdup() calls
Lee Hopkins leer...@gmail.com writes:
I went ahead and took a stab at a solution. My solution is more
aggressive than a warning, I actually prevent the creation of
ambiguous refs. My changes are also in refs.c, which may not be
appropriate, but it seemed like the natural place.
I have never
Duy Nguyen pclo...@gmail.com writes:
On Wed, Feb 26, 2014 at 5:44 AM, Junio C Hamano gits...@pobox.com wrote:
Nguyễn Thái Ngọc Duy pclo...@gmail.com writes:
git_path() soon understands the path given to it. Some paths abc may
become def while other ghi may become ijk. We don't want
Jeff King p...@peff.net writes:
On Fri, Feb 28, 2014 at 10:45:39AM -0800, Junio C Hamano wrote:
Exactly. The two features (bitmaps and .keep) are not compatible with
each other, so you have to prioritize one. If you are using static .keep
files, you might want them to continue being
On Mon, Mar 03, 2014 at 10:13:47AM -0800, Junio C Hamano wrote:
Or the flip side: if the user wants to use .keep, we should drop
bitmaps. My point is that we do not know which way the user wants to
go, so we should not tie the options together.
Hmph. I think the short of your later
Michael Haggerty mhag...@alum.mit.edu writes:
On 02/28/2014 10:29 AM, Dmitry S. Dolzhenko wrote:
Thank you for your remarks. In this patch I tried to take them into account.
Dmitry S. Dolzhenko (11):
builtin/pack-objects.c: change check_pbase_path() to use ALLOC_GROW()
bundle.c: change
Jeff King p...@peff.net writes:
I realize that I just bikeshedded on subject lines for half a page, and
part of me wants to go kill myself in shame. But I feel like I see the
technique misapplied often enough that maybe some guidance is merited.
Thanks. What I queued read like these:
$ git
Faiz Kothari faiz.of...@gmail.com writes:
Signed-off-by: Faiz Kothari faiz.of...@gmail.com
---
Implemented write_or_die.c:strbuf_write_or_die() and used in relevant places
to substitute write_or_die(). I spotted other places where strbuf can be used
in place of buf[MAX_PATH] but that would
Eric Sunshine sunsh...@sunshineco.com writes:
On Sat, Mar 1, 2014 at 7:51 AM, He Sun sunheeh...@gmail.com wrote:
2014-03-01 19:21 GMT+08:00 Faiz Kothari faiz.of...@gmail.com:
diff --git a/remote-curl.c b/remote-curl.c
index 10cb011..dee8716 100644
--- a/remote-curl.c
+++ b/remote-curl.c
@@
Jacopo Notarstefano jacopo.notarstef...@gmail.com writes:
Here my proposal differs in that I have no way of knowing which label
is good and which label is bad: I blindly accept two distinct labels
and bisect with those. I gave an example of this behaviour above.
I think you fundamentally
Hi,
I am converting my team's SVN server to GIT but they aren't ready to
transition yet. People are still working out of SVN, so I need to keep
the git-svn clone around to do 'git svn fetch' to continue to keep it
synchronized with SVN.
Once everyone is ready to switch, and after converting all
Michael Haggerty mhag...@alum.mit.edu writes:
-if (!starts_with(buf, author )) {
+if (!skip_prefix(buf, author )) {
If this is the only change, there is not much point, is there? How does
this help? Perhaps there is some way to take advantage of the
difference
Sun He sunheeh...@gmail.com writes:
'pack_tmp_name' is the subject of the utime() check, so report it in the
warning, not the uninitialized 'tmpname'
Signed-off-by: Sun He sunheeh...@gmail.com
---
Changing the subject and adding valid information as tutored by
Eric Sunshine.
Thanks
On Tue, Mar 4, 2014 at 12:01 AM, Junio C Hamano gits...@pobox.com wrote:
Eric Sunshine sunsh...@sunshineco.com writes:
On Sat, Mar 1, 2014 at 7:51 AM, He Sun sunheeh...@gmail.com wrote:
2014-03-01 19:21 GMT+08:00 Faiz Kothari faiz.of...@gmail.com:
diff --git a/remote-curl.c b/remote-curl.c
Eric Sunshine sunsh...@sunshineco.com writes:
diff --git a/branch.c b/branch.c
index 723a36b..ca4e824 100644
--- a/branch.c
+++ b/branch.c
@@ -50,7 +50,7 @@ static int should_setup_rebase(const char *origin)
void install_branch_config(int flag, const char *local, const char *origin,
Simon Ruderich si...@ruderich.org writes:
Most distributions don't require Term::Readkey as dependency,
leaving the user to wonder why the setting doesn't work.
Signed-off-by: Simon Ruderich si...@ruderich.org
Thanks, but is it true that interactive.singlekey requries
Term::ReadKey?
The
Siddharth Goel siddharth98...@gmail.com writes:
Helped-by: Eric Sunshine sunsh...@sunshineco.com
Signed-off-by: Siddharth Goel siddharth98...@gmail.com
---
Added a space after colon in the subject as compared to previous
patch [PATCH v2].
[PATCH v2]:
[CC'ing the submodule area experts.]
Henri GEIST geist.he...@laposte.net writes:
This new option prevent git submodule add|update to clone the missing
submodules with the --separate-git-dir option.
Then the submodule will be regular repository and their gitdir will not
be placed in the
Tanay Abhra tanay...@gmail.com writes:
In record_author_date() parse_gpg_output() ,using skip_prefix() instead of
starts_with() is more elegant and abstracts away the details.
Avoid subjective judgement like more elegant when justifying your
change; you are not your own judge.
The caller of
On Mon, Mar 3, 2014 at 1:31 PM, Junio C Hamano gits...@pobox.com wrote:
Eric Sunshine sunsh...@sunshineco.com writes:
On Sat, Mar 1, 2014 at 7:51 AM, He Sun sunheeh...@gmail.com wrote:
2014-03-01 19:21 GMT+08:00 Faiz Kothari faiz.of...@gmail.com:
diff --git a/remote-curl.c b/remote-curl.c
Jeff King p...@peff.net writes:
On Mon, Mar 03, 2014 at 10:13:47AM -0800, Junio C Hamano wrote:
Or the flip side: if the user wants to use .keep, we should drop
bitmaps. My point is that we do not know which way the user wants to
go, so we should not tie the options together.
Hmph. I
Eric Sunshine sunsh...@sunshineco.com writes:
As a potential GSoC student and newcomer to the project, Faiz would
not have known that this would be considered unwanted churn when he
chose the task from the GSoC microproject page [1]. Perhaps it would
be a good idea to retire this item from
On Mon, Mar 3, 2014 at 1:40 PM, Junio C Hamano gits...@pobox.com wrote:
Michael Haggerty mhag...@alum.mit.edu writes:
-if (!starts_with(buf, author )) {
+if (!skip_prefix(buf, author )) {
If this is the only change, there is not much point, is there? How does
this
Jeff King p...@peff.net writes:
On Fri, Feb 28, 2014 at 12:04:19PM +0900, Brian Gesiak wrote:
From: modocache modoca...@gmail.com
Both your emailed patches have this, which is due to your author name
not matching your sending identity. You probably want to set user.name,
or if you already
On Mon, Mar 03, 2014 at 11:51:06AM -0800, Junio C Hamano wrote:
Yes. Do you need a re-roll from me? I think the last version I sent +
the squash to tie the default to bitmap-writing makes the most sense.
I have 9e20b390 (repack: add `repack.packKeptObjects` config var,
2014-02-26); I do
Jeff King p...@peff.net writes:
Most callbacks would convert to a query system in a pretty
straightforward way, but some that have side effects might be tricky.
Converting them all may be too large for a GSoC project, but I think you
could do it gradually:
1. Convert the parser to read
On Sat, Mar 1, 2014 at 9:53 PM, Nguyễn Thái Ngọc Duy pclo...@gmail.com wrote:
git rebase -e XYZ is basically the same as
EDITOR=sed -i '1s/pick XYZ/edit XYZ/' $@ \
git rebase -i XYZ^
In English, it prepares the todo list for you to edit only commit XYZ
to save your time. The time saving is
Le lundi 03 mars 2014 à 17:45 +, Jens Lehmann a écrit :
Am 03.03.2014 14:47, schrieb Henri GEIST:
This new option prevent git submodule add|update to clone the missing
submodules with the --separate-git-dir option.
Then the submodule will be regular repository and their gitdir will not
Eric Sunshine sunsh...@sunshineco.com writes:
On Mon, Mar 3, 2014 at 1:31 PM, Junio C Hamano gits...@pobox.com wrote:
Eric Sunshine sunsh...@sunshineco.com writes:
On Sat, Mar 1, 2014 at 7:51 AM, He Sun sunheeh...@gmail.com wrote:
2014-03-01 19:21 GMT+08:00 Faiz Kothari faiz.of...@gmail.com:
Most distributions don't require Term::ReadKey as dependency, leaving
the user to wonder why the setting doesn't work.
Signed-off-by: Simon Ruderich si...@ruderich.org
---
On Mon, Mar 03, 2014 at 10:58:58AM -0800, Junio C Hamano wrote:
Thanks, but is it true that interactive.singlekey requries
Suggested-by: Junio C Hamano gits...@pobox.com
Signed-off-by: Simon Ruderich si...@ruderich.org
---
git-add--interactive.perl | 3 +++
1 file changed, 3 insertions(+)
diff --git a/git-add--interactive.perl b/git-add--interactive.perl
index 24bb1ab..d3bca12 100755
--- a/git-add--interactive.perl
Hi ,
I am trying do complete the microproject 4, inorder to apply to GSOC.
I have made the below changes:
https://gist.github.com/anhsirksai/9334565
Post my changes compilation is succes in the source directory.
But when I ran the tests[make in t/ directory] my tests are failing
saying
On Mon, Mar 3, 2014 at 3:35 PM, Junio C Hamano gits...@pobox.com wrote:
Eric Sunshine sunsh...@sunshineco.com writes:
On Mon, Mar 3, 2014 at 1:31 PM, Junio C Hamano gits...@pobox.com wrote:
Eric Sunshine sunsh...@sunshineco.com writes:
It's not obvious from the patch fragment, but 'heads' is
Michael Haggerty mhag...@alum.mit.edu writes:
This might be a reason that -NUM is a bad idea.
Or perhaps -NUM should fail with an error message if any of the last
NUM commits are merges. In that restricted scenario (which probably
accounts for 99% of rebases), -NUM is equivalent to
Matthieu Moy matthieu@grenoble-inp.fr writes:
Michael Haggerty mhag...@alum.mit.edu writes:
Or perhaps -NUM should fail with an error message if any of the last
NUM commits are merges. In that restricted scenario (which probably
accounts for 99% of rebases), -NUM is equivalent to
On Mon, Mar 3, 2014 at 4:11 PM, saikrishna.sripada
saikrishna.srip...@students.iiit.ac.in wrote:
I am trying do complete the microproject 4, inorder to apply to GSOC.
I have made the below changes:
https://gist.github.com/anhsirksai/9334565
Post my changes compilation is succes in the source
This version differs from previous [1] the following changes:
- added three new commits with similar changes in builtin/mktree.c,
cache-tree.c and sha1_file.c.
- updated commit messages: use ALLOC_GROW() in function_name() instead of
change function_name() to use ALLOC_GROW()
-
Signed-off-by: Dmitry S. Dolzhenko dmitrys.dolzhe...@yandex.ru
---
replace_object.c | 8 ++--
1 file changed, 2 insertions(+), 6 deletions(-)
diff --git a/replace_object.c b/replace_object.c
index cdcaf8c..843deef 100644
--- a/replace_object.c
+++ b/replace_object.c
@@ -36,12 +36,8 @@ static
Signed-off-by: Dmitry S. Dolzhenko dmitrys.dolzhe...@yandex.ru
---
dir.c | 5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/dir.c b/dir.c
index 98bb50f..4ae38e4 100644
--- a/dir.c
+++ b/dir.c
@@ -1341,10 +1341,7 @@ static struct path_simplify *create_simplify(const char
Signed-off-by: Dmitry S. Dolzhenko dmitrys.dolzhe...@yandex.ru
---
cache-tree.c | 6 +-
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/cache-tree.c b/cache-tree.c
index 0bbec43..30149d1 100644
--- a/cache-tree.c
+++ b/cache-tree.c
@@ -75,11 +75,7 @@ static struct cache_tree_sub
Signed-off-by: Dmitry S. Dolzhenko dmitrys.dolzhe...@yandex.ru
---
attr.c | 7 +--
1 file changed, 1 insertion(+), 6 deletions(-)
diff --git a/attr.c b/attr.c
index 8d13d70..734222d 100644
--- a/attr.c
+++ b/attr.c
@@ -338,12 +338,7 @@ static void handle_attr_line(struct attr_stack *res,
Use ALLOC_GROW() instead inline code in
locate_rename_dst() and register_rename_src()
Signed-off-by: Dmitry S. Dolzhenko dmitrys.dolzhe...@yandex.ru
---
diffcore-rename.c | 12 ++--
1 file changed, 2 insertions(+), 10 deletions(-)
diff --git a/diffcore-rename.c b/diffcore-rename.c
index
Use ALLOC_GROW() instead inline code in
diffstat_add() and diff_q()
Signed-off-by: Dmitry S. Dolzhenko dmitrys.dolzhe...@yandex.ru
---
diff.c | 12 ++--
1 file changed, 2 insertions(+), 10 deletions(-)
diff --git a/diff.c b/diff.c
index e800666..aebdfda 100644
--- a/diff.c
+++ b/diff.c
Signed-off-by: Dmitry S. Dolzhenko dmitrys.dolzhe...@yandex.ru
---
bundle.c | 6 +-
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/bundle.c b/bundle.c
index e99065c..1388a3e 100644
--- a/bundle.c
+++ b/bundle.c
@@ -14,11 +14,7 @@ static const char bundle_signature[] = # v2 git
Signed-off-by: Dmitry S. Dolzhenko dmitrys.dolzhe...@yandex.ru
---
builtin/pack-objects.c | 9 +++--
1 file changed, 3 insertions(+), 6 deletions(-)
diff --git a/builtin/pack-objects.c b/builtin/pack-objects.c
index c733379..0ffad6f 100644
--- a/builtin/pack-objects.c
+++
Helped-by: He Sun sunheeh...@gmail.com
Signed-off-by: Dmitry S. Dolzhenko dmitrys.dolzhe...@yandex.ru
---
sha1_file.c | 7 +--
1 file changed, 1 insertion(+), 6 deletions(-)
diff --git a/sha1_file.c b/sha1_file.c
index 019628a..3cb17b8 100644
--- a/sha1_file.c
+++ b/sha1_file.c
@@ -2624,12
Signed-off-by: Dmitry S. Dolzhenko dmitrys.dolzhe...@yandex.ru
---
commit.c | 8 ++--
1 file changed, 2 insertions(+), 6 deletions(-)
diff --git a/commit.c b/commit.c
index 6bf4fe0..e004314 100644
--- a/commit.c
+++ b/commit.c
@@ -147,12 +147,8 @@ int register_commit_graft(struct
Signed-off-by: Dmitry S. Dolzhenko dmitrys.dolzhe...@yandex.ru
---
read-cache.c | 6 +-
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/read-cache.c b/read-cache.c
index fb440b4..cbdf954 100644
--- a/read-cache.c
+++ b/read-cache.c
@@ -990,11 +990,7 @@ int add_index_entry(struct
Signed-off-by: Dmitry S. Dolzhenko dmitrys.dolzhe...@yandex.ru
---
patch-ids.c | 5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/patch-ids.c b/patch-ids.c
index bc8a28f..bf81b92 100644
--- a/patch-ids.c
+++ b/patch-ids.c
@@ -83,10 +83,7 @@ static struct patch_id
Junio C Hamano gits...@pobox.com writes:
Matthieu Moy matthieu@grenoble-inp.fr writes:
Michael Haggerty mhag...@alum.mit.edu writes:
Or perhaps -NUM should fail with an error message if any of the last
NUM commits are merges. In that restricted scenario (which probably
accounts for
Michael Haggerty mhag...@alum.mit.edu writes:
Based on my experience so far as a first-time Google Summer of Code
mentor, I just wrote a blog article containing some hopefully useful
advice for students applying to the program. Please note that this is
my personal opinion only and doesn't
On Sun, Mar 02, 2014 at 06:04:39AM +0900, Brian Gesiak wrote:
My name is Brian Gesiak. I'm a research student at the University of
Tokyo, and I'm hoping to participate in this year's Google Summer of
Code by contributing to Git. I'm a longtime user, first-time
contributor--some of you may
Use ALLOC_GROW() instead inline code in
add_commit_info() and read_one_reflog()
Signed-off-by: Dmitry S. Dolzhenko dmitrys.dolzhe...@yandex.ru
---
reflog-walk.c | 12 ++--
1 file changed, 2 insertions(+), 10 deletions(-)
diff --git a/reflog-walk.c b/reflog-walk.c
index b2fbdb2..2899729
Helped-by: He Sun sunheeh...@gmail.com
Signed-off-by: Dmitry S. Dolzhenko dmitrys.dolzhe...@yandex.ru
---
builtin/mktree.c | 5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/builtin/mktree.c b/builtin/mktree.c
index f92ba40..a964d6b 100644
--- a/builtin/mktree.c
+++
Junio C Hamano gits...@pobox.com writes:
Siddharth Goel siddharth98...@gmail.com writes:
Helped-by: Eric Sunshine sunsh...@sunshineco.com
Signed-off-by: Siddharth Goel siddharth98...@gmail.com
---
Added a space after colon in the subject as compared to previous
patch [PATCH v2].
[PATCH
Dmitry S. Dolzhenko dmitrys.dolzhe...@yandex.ru writes:
Use ALLOC_GROW() instead inline code in
diffstat_add() and diff_q()
...instead of open coding it in... may read better.
Signed-off-by: Dmitry S. Dolzhenko dmitrys.dolzhe...@yandex.ru
---
diff.c | 12 ++--
1 file changed, 2
On Sun, Mar 2, 2014 at 6:36 AM, Max Horn m...@quendi.de wrote:
On 01.03.2014, at 00:26, Conley Owens c...@android.com wrote:
$ git --version # This is just the git from MacPorts
git version 1.8.5.5
$ sw_vers
ProductName:Mac OS X
ProductVersion: 10.8.5
BuildVersion: 12F45
I cannot
1 - 100 of 128 matches
Mail list logo