I've a submodule located in a subdirectory
({git_rep}/home/{directory}/{submodule}), and I wanted to move the whole
directory up a level ({git_rep}/{directory}/{submodule}). But when I
used 'git mv {directory} ../' the '.gitmodule' file didn't get modified.
Best regards,
Albin Otterhäll
--
To uns
On Fri, Apr 15, 2016 at 12:04:49AM +0200, Christian Couder wrote:
> On Tue, Apr 12, 2016 at 4:59 PM, Stefan Beller wrote:
> > On Tue, Apr 12, 2016 at 2:42 AM, Duy Nguyen wrote:
> >>> On Mon, Apr 11, 2016 at 7:51 AM, Stefan Beller wrote:
> Hi Greg,
>
> Thanks for your talk at the G
On Fri, Apr 15, 2016 at 11:51 AM, Duy Nguyen wrote:
> On Fri, Apr 15, 2016 at 12:04:49AM +0200, Christian Couder wrote:
>>
>> There is a draft of an article about the first part of the Contributor
>> Summit in the draft of the next Git Rev News edition:
>>
>> https://github.com/git/git.github.io/b
On Thu, Apr 14, 2016 at 1:47 AM, David Turner wrote:
>> > + fd = unix_stream_connect(socket_path);
>> > + if (refresh_cache) {
>> > + ret = write_in_full(fd, "refresh", 8) != 8;
>>
>> Since we've moved to unix socket and had bidirectional communication,
>> it's probably a
On Fri, Apr 15, 2016 at 6:18 PM, Christian Couder
wrote:
> On Fri, Apr 15, 2016 at 11:51 AM, Duy Nguyen wrote:
>> On Fri, Apr 15, 2016 at 12:04:49AM +0200, Christian Couder wrote:
>>>
>>> There is a draft of an article about the first part of the Contributor
>>> Summit in the draft of the next Gi
Hi Junio,
On Thu, 14 Apr 2016, Junio C Hamano wrote:
> The pattern taken by all the validations in this function is:
>
> if (notice a violation exists) {
> err = report(... VIOLATION_KIND ...);
> if (err)
> return err;
> }
>
> where
Hi Junio,
On Thu, 14 Apr 2016, Junio C Hamano wrote:
> Even though a Git commit object is designed to be capable of storing
> any binary data as its payload, in practice people use it to describe
> the changes in textual form, and tools like "git log" are designed to
> treat the payload as text.
Hi,
the checkout command prevents me from checking out a branch in the
current worktree if it is already checked out in another worktree.
However, if I rebase the branch in the current worktree onto the
branch in the other worktree, I end up in a situation where the same
branch is checked out twi
Make git fully relocatable at runtime extending the runtime prefix
calculation. Handle absolute and relative paths in argv0. Handle no path
at all in argv0 in a system-specific manner. Replace assertions with
initialised variables and checks that lead to fallback to the static
prefix.
---
Notes:
Johannes Schindelin writes:
>> But that is a selfish code that declares it is the ultimate and
>> final form of the function, never to be enhanced later. To allow
>> and invite future enhancements, make the last test follow the same
>> pattern.
>
> FWIW I agree with this reasoning. Sorry for lea
On Fri, Apr 15, 2016 at 2:51 AM, Duy Nguyen wrote:
> On Fri, Apr 15, 2016 at 12:04:49AM +0200, Christian Couder wrote:
>> On Tue, Apr 12, 2016 at 4:59 PM, Stefan Beller wrote:
>> > On Tue, Apr 12, 2016 at 2:42 AM, Duy Nguyen wrote:
>> >>> On Mon, Apr 11, 2016 at 7:51 AM, Stefan Beller
>> >>> w
Nguyễn Thái Ngọc Duy writes:
> This keeps top dir a bit less crowded. And because these programs are
> for testing purposes, it makes sense that they stay somewhere in t/
But leaves many *.o files after "make clean". Even "distclean" does
not clean them.
>
> Signed-off-by: Nguyễn Thái Ngọc Du
Reto Hablützel writes:
> the checkout command prevents me from checking out a branch in the
> current worktree if it is already checked out in another worktree.
>
> However, if I rebase the branch in the current worktree onto the
> branch in the other worktree, I end up in a situation where the s
There has been talk of a git mascot a while back in 2005. Some people
mentioned a fish or a turtle. Since all the great open source projects
like Linux or RethinkDB have a cute mascot, git definitely needs one
as well. A mascot gives people a recognizable persona towards which
they can direct their
On Fri, Apr 15, 2016 at 01:18:46PM +0200, Christian Couder wrote:
> On Fri, Apr 15, 2016 at 11:51 AM, Duy Nguyen wrote:
> > On Fri, Apr 15, 2016 at 12:04:49AM +0200, Christian Couder wrote:
> >>
> >> There is a draft of an article about the first part of the Contributor
> >> Summit in the draft o
> On April 15, 2016 12:42 PM> Behalf Of Christian Howe wrote
> There has been talk of a git mascot a while back in 2005. Some people
> mentioned a fish or a turtle. Since all the great open source projects like
> Linux or RethinkDB have a cute mascot, git definitely needs one as well. A
> mascot gi
Michael Weiser writes:
> Make git fully relocatable at runtime extending the runtime prefix
> calculation. Handle absolute and relative paths in argv0. Handle no path
> at all in argv0 in a system-specific manner. Replace assertions with
> initialised variables and checks that lead to fallback t
From: Stefan Beller
In order to produce the smallest possible diff and combine several diff
hunks together, we implement a heuristic from GNU Diff which moves diff
hunks forward as far as possible when we find common context above and
below a diff hunk. This sometimes produces less readable diffs
From: Jacob Keller
I took up Stefan's patch, and modified it to resolve a couple issues. I
also tried to implement the suggestions from Junio's review, as well as
the suggestions I had. It appears to produce equivalent output as Jeff's
script. This version is still missing a few things, and it is
From: Jacob Keller
It is a common pattern in xdl_change_compact to check that hashes and
strings match. The resulting code to perform this change causes very
long lines and makes it hard to follow the intention. Introduce a helper
function xdl_hash_and_recmatch which performs both checks to incre
On Fri, Apr 15, 2016 at 9:51 AM, Jacob Keller wrote:
> From: Jacob Keller
>
> I took up Stefan's patch, and modified it to resolve a couple issues. I
> also tried to implement the suggestions from Junio's review, as well as
> the suggestions I had. It appears to produce equivalent output as Jeff'
Junio C Hamano writes:
> Nguyễn Thái Ngọc Duy writes:
>
>> This keeps top dir a bit less crowded. And because these programs are
>> for testing purposes, it makes sense that they stay somewhere in t/
>
> But leaves many *.o files after "make clean". Even "distclean" does
> not clean them.
Per
Le vendredi 15 avril 2016, 11:41:52 Christian Howe a écrit :
> There has been talk of a git mascot a while back in 2005. Some people
> mentioned a fish or a turtle. Since all the great open source projects
> like Linux or RethinkDB have a cute mascot, git definitely needs one
> as well. A mascot gi
On Fri, Apr 15, 2016 at 1:14 AM, Albin Otterhäll wrote:
> I've a submodule located in a subdirectory
> ({git_rep}/home/{directory}/{submodule}), and I wanted to move the whole
> directory up a level ({git_rep}/{directory}/{submodule}). But when I
> used 'git mv {directory} ../' the '.gitmodule' fi
On Fri, Apr 15, 2016 at 10:02 AM, Stefan Beller wrote:
>> @@ -458,11 +458,11 @@ int xdl_change_compact(xdfile_t *xdf, xdfile_t *xdfo,
>> long flags) {
>> * the group.
>> */
>> while (ixs > 0 && xdl_hash_and_recmatch(recs, i
Jacob Keller writes:
> From: Jacob Keller
>
> It is a common pattern in xdl_change_compact to check that hashes and
> strings match. The resulting code to perform this change causes very
> long lines and makes it hard to follow the intention. Introduce a helper
> function xdl_hash_and_recmatch w
Stefan Beller writes:
> Actually we would only need to have the empty line count in the
> second loop as the first loop shifted it as much up(backwards) as
> possible, such that the hunk sits on one end of the movable
> range. The second loop would iterate over the whole range, so that
> would be
Jeff King writes:
> On Fri, Apr 15, 2016 at 01:18:46PM +0200, Christian Couder wrote:
>
>> On Fri, Apr 15, 2016 at 11:51 AM, Duy Nguyen wrote:
>> > On Fri, Apr 15, 2016 at 12:04:49AM +0200, Christian Couder wrote:
>> >>
>> >> There is a draft of an article about the first part of the Contributor
On Fri, Apr 15, 2016 at 10:27 AM, Junio C Hamano wrote:
> Stefan Beller writes:
>
>> Actually we would only need to have the empty line count in the
>> second loop as the first loop shifted it as much up(backwards) as
>> possible, such that the hunk sits on one end of the movable
>> range. The se
On Fri, Apr 15, 2016 at 10:31:39AM -0700, Junio C Hamano wrote:
> Last time I checked, I think the accesses to attributes from the
> convert.c thing was one of the things that are cumbersome to make
> safe in multi-threaded world.
Multi-threaded grep has the same problem. I think we started with
Stefan Beller writes:
> I think we need to be aggressive to find adjacent groups and only after
> that is done we should think about optimizing look&feel?
OK. I was just gauging to see if those involved in the codepath
thought things through, and apparently you did, so I'm happy ;-)
Thanks.
--
On Fri, Apr 15, 2016 at 10:18 AM, Stefan Beller wrote:
> On Fri, Apr 15, 2016 at 1:14 AM, Albin Otterhäll wrote:
>> I've a submodule located in a subdirectory
>> ({git_rep}/home/{directory}/{submodule}), and I wanted to move the whole
>> directory up a level ({git_rep}/{directory}/{submodule}). B
Stefan Beller writes:
> I think I can reproduce the problem. A regression test (which currently fails)
> could look like
Thanks. I however do not think this is a regression.
Changes around 0656781f (mv: update the path entry in .gitmodules
for moved submodules, 2013-08-06) did introduce "git m
On Fri, Apr 15, 2016 at 11:21 AM, Junio C Hamano wrote:
> Stefan Beller writes:
>
>> I think I can reproduce the problem. A regression test (which currently
>> fails)
>> could look like
>
> Thanks. I however do not think this is a regression.
>
> Changes around 0656781f (mv: update the path ent
On Fri, Apr 15, 2016 at 2:13 AM, Jeff King wrote:
> On Sun, Apr 10, 2016 at 12:15:10AM +0530, Karthik Nayak wrote:
>
>> +static void refname_atom_parser(struct used_atom *atom, const char *arg)
>> +{
>> + if (!arg)
>> + atom->u.refname.option = R_NORMAL;
>> + else if (!strcmp(a
On Fri, Apr 15, 2016 at 10:02 AM, Stefan Beller wrote:
> On Fri, Apr 15, 2016 at 9:51 AM, Jacob Keller
> wrote:
>> From: Jacob Keller
>>
>> I took up Stefan's patch, and modified it to resolve a couple issues. I
>> also tried to implement the suggestions from Junio's review, as well as
>> the s
On Fri, Apr 15, 2016 at 10:10 AM, Stefan Beller wrote:
> On Fri, Apr 15, 2016 at 10:02 AM, Stefan Beller wrote:
>>> @@ -458,11 +458,11 @@ int xdl_change_compact(xdfile_t *xdf, xdfile_t *xdfo,
>>> long flags) {
>>> * the group.
>>> */
>>>
On Fri, Apr 15, 2016 at 10:25 AM, Junio C Hamano wrote:
> Jacob Keller writes:
>
>> From: Jacob Keller
>>
>> It is a common pattern in xdl_change_compact to check that hashes and
>> strings match. The resulting code to perform this change causes very
>> long lines and makes it hard to follow the
On Fri, Apr 15, 2016 at 10:33 AM, Stefan Beller wrote:
> On Fri, Apr 15, 2016 at 10:27 AM, Junio C Hamano wrote:
>> Stefan Beller writes:
>>
>>> Actually we would only need to have the empty line count in the
>>> second loop as the first loop shifted it as much up(backwards) as
>>> possible, suc
When directories are moved using `git mv` all files in the directory
have been just moved, but no further action was taken on them. This
was done by assigning the mode = WORKING_DIRECTORY to the files
inside a moved directory.
submodules however need to update their link to the git directory as
we
We've got a lot of refs, but pretty frequently we only want to fetch
one. It's silly for the server to send a bunch of refs that the client
is just going to ignore. Here are some patches that fix that.
Let me know if this seems reasonable.
(and I'll start in on another round of index-helper as
The local variable 'options' was shadowing a global of the same name.
Signed-off-by: David Turner
---
remote-curl.c | 18 +-
1 file changed, 9 insertions(+), 9 deletions(-)
diff --git a/remote-curl.c b/remote-curl.c
index 15e48e2..b9b6a90 100644
--- a/remote-curl.c
+++ b/remote-
Signed-off-by: David Turner
---
http-backend.c | 12 +++-
1 file changed, 7 insertions(+), 5 deletions(-)
diff --git a/http-backend.c b/http-backend.c
index 8870a26..a4f0066 100644
--- a/http-backend.c
+++ b/http-backend.c
@@ -450,9 +450,7 @@ static void get_info_refs(char *arg)
When fetching over http, send the requested refspec to the server.
The server will then only send refs matching that refspec. It is
permitted for old servers to ignore that parameter, and the client
will automatically handle this.
When the server has many refs, and the client only wants a few, th
For single-branch clones (when we know in advance what the remote
branch name will be), send a refspec so that the server doesn't
tell us about any other refs.
Signed-off-by: David Turner
---
builtin/clone.c | 16 +++-
t/t5552-http-fetch-branch.sh | 5 +
transport-h
Allow clients to pass a "refspec" parameter through to upload-pack;
upload-pack will only advertise refs which match that refspec.
Signed-off-by: David Turner
---
http-backend.c | 9 +++
upload-pack.c | 81 --
2 files changed, 88 inse
Add parameters for a list of refspecs to transport_get_remote_refs and
get_refs_list. These parameters are presently unused -- soon, we will
use them to implement fetches which only learn about a subset of refs.
Signed-off-by: David Turner
---
builtin/clone.c | 2 +-
builtin/fetch.c |
On Fri, Apr 15, 2016 at 12:19 PM, David Turner wrote:
> We've got a lot of refs, but pretty frequently we only want to fetch
> one. It's silly for the server to send a bunch of refs that the client
> is just going to ignore. Here are some patches that fix that.
>
> Let me know if this seems reas
On 2016-04-15 19:18, Stefan Beller wrote:
> On Fri, Apr 15, 2016 at 1:14 AM, Albin Otterhäll wrote:
>> I've a submodule located in a subdirectory
>> ({git_rep}/home/{directory}/{submodule}), and I wanted to move the whole
>> directory up a level ({git_rep}/{directory}/{submodule}). But when I
>> u
---
git-p4.py | 12
1 file changed, 12 insertions(+)
diff --git a/git-p4.py b/git-p4.py
index 527d44b..a81795f 100755
--- a/git-p4.py
+++ b/git-p4.py
@@ -2320,6 +2320,15 @@ def extractFilesFromCommit(self, commit):
fnum = fnum + 1
return files
+def extract
On Fri, Apr 15, 2016 at 9:51 AM, Jacob Keller wrote:
> From: Stefan Beller
>
> In order to produce the smallest possible diff and combine several diff
> hunks together, we implement a heuristic from GNU Diff which moves diff
> hunks forward as far as possible when we find common context above and
Stephan Beyer writes:
> test_cmp_rev() took exactly two parameters, the expected revision
> and the revision to test. This commit generalizes this function
> such that it takes any number of at least two revisions: the
> expected one and a list of actual ones. The function returns true
> if and o
Stefan Beller writes:
>> +diff.emptyLineHeuristic::
>
> I was looking at the TODO here and thought about the name:
> It should not encode the `emptyLine` into the config option as
> it is only one of many heuristics.
>
> It should be something like `diff.heuristic=lastEmptyLine`
> The we could ad
On Fri, Apr 15, 2016 at 12:57 PM, Stefan Beller wrote:
> I was looking at the TODO here and thought about the name:
> It should not encode the `emptyLine` into the config option as
> it is only one of many heuristics.
>
> It should be something like `diff.heuristic=lastEmptyLine`
> The we could ad
On Fri, Apr 15, 2016 at 1:06 PM, Junio C Hamano wrote:
> Stefan Beller writes:
>
>>> +diff.emptyLineHeuristic::
>>
>> I was looking at the TODO here and thought about the name:
>> It should not encode the `emptyLine` into the config option as
>> it is only one of many heuristics.
>>
>> It should
On Thu, 2016-04-14 at 17:04 +0700, Duy Nguyen wrote:
> On Thu, Apr 14, 2016 at 1:47 AM, David Turner <
> dtur...@twopensource.com> wrote:
> > On Wed, 2016-04-13 at 20:43 +0700, Duy Nguyen wrote:
> > > On Wed, Apr 13, 2016 at 7:32 AM, David Turner <
> > > dtur...@twopensource.com> wrote:
> > > > +NO
On Fri, 2016-04-15 at 18:25 +0700, Duy Nguyen wrote:
> On Thu, Apr 14, 2016 at 1:47 AM, David Turner <
> dtur...@twopensource.com> wrote:
> > > > + fd = unix_stream_connect(socket_path);
> > > > + if (refresh_cache) {
> > > > + ret = write_in_full(fd, "refresh", 8) != 8;
>
Jan Durovec writes:
> ---
A few issues. Please:
(1) Sign-off your work.
(2) Try to find those who are familiar with the area and Cc them.
"git shortlog -s -n --since=18.months --no-merges git-p4.py"
may help.
(3) Follow the style of existing commits when giving a title to
On Fri, Apr 15, 2016 at 2:06 AM, Jeff King wrote:
> On Thu, Apr 14, 2016 at 04:05:30PM -0400, Jeff King wrote:
>
>> It looks like that's a little tricky for %(upstream) and %(push), which
>> have extra tracking options, but it's pretty trivial for %(symref):
>> [...]
>> I suspect it could work for
Jacob Keller writes:
> On Fri, Apr 15, 2016 at 1:06 PM, Junio C Hamano wrote:
>
>> I actually do not think these knobs should exist when the code is
>> mature enough to be shipped to the end users.
>>
>> Use "diff.compactionHeuristics = " as an opaque set of bits to
>> help the developers while
Stephan Beyer writes:
> test_cmp_rev() from t/test-lib-functions.sh is used to make many
> tests clearer.
>
> Signed-off-by: Stephan Beyer
> ---
>
> Notes:
> This change is in some way independent of the bisect topic but
> the next patch is based on this (for t6030).
It seems that with
Stephan Beyer writes:
> The bisect algorithm allows different outcomes if, for example,
> the number of commits between a good and a bad commit is even.
> The current test relies on a specific behavior (for example,
> the behavior of the halfway() implementation). By disabling
> halfway(), some s
On Sat, Apr 16, 2016 at 01:57:23AM +0530, Karthik Nayak wrote:
> I had a look at your patch and even tested it, seems solid, I like how you
> integrated all these atoms together under refname_atom_parser_internal().
> I'm squashing this in, for my re-roll. Thanks.
Great, thanks for picking it up.
Hi,
On Tue, Apr 12, 2016 at 6:23 PM, Junio C Hamano wrote:
> Elijah Newren writes:
>
>> Elijah Newren (6):
>> Remove duplicate code
>> Avoid checking working copy when creating a virtual merge base
>> Add merge testcases for when index doesn't match HEAD
>> merge-octopus: Abort if index
Stephan Beyer writes:
> +test_expect_success 'bisect algorithm works in linear history with an odd
> number of commits' '
> + git bisect start A7 &&
> + git bisect next &&
> + test_cmp_rev HEAD A3 A4
> +'
> +
> +test_expect_success 'bisect algorithm works in linear history with an ev
On Fri, Apr 15, 2016 at 1:30 PM, Junio C Hamano wrote:
> Jacob Keller writes:
>
>> On Fri, Apr 15, 2016 at 1:06 PM, Junio C Hamano wrote:
>>
>>> I actually do not think these knobs should exist when the code is
>>> mature enough to be shipped to the end users.
>>>
>>> Use "diff.compactionHeurist
Stephan Beyer writes:
> Signed-off-by: Stephan Beyer
> ---
> bisect.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/bisect.c b/bisect.c
> index 7996c29..901e4d3 100644
> --- a/bisect.c
> +++ b/bisect.c
> @@ -984,6 +984,8 @@ int bisect_next_all(const char *prefix, int no_checkout)
>
Stephan Beyer writes:
> Setting the macro DEBUG_BISECT to 1 enables debugging information
> for the bisect algorithm. The code did not compile due to struct
> changes.
>
> Signed-off-by: Stephan Beyer
> ---
Thanks.
This is something that we should do as a preparatory clean-up patch
before the
On Fri, Apr 15, 2016 at 2:15 PM, Jacob Keller wrote:
> On Fri, Apr 15, 2016 at 1:30 PM, Junio C Hamano wrote:
>> Jacob Keller writes:
>>
>>> On Fri, Apr 15, 2016 at 1:06 PM, Junio C Hamano wrote:
>>>
I actually do not think these knobs should exist when the code is
mature enough to be
Stephan Beyer writes:
> If DEBUG_BISECT is set to 1, bisect does not only show debug
> information but also changes the algorithm behavior: halfway()
> is always false.
>
> This commit makes the algorithm independent of DEBUG_BISECT.
>
> Signed-off-by: Stephan Beyer
> ---
Another good candidate
Stephan Beyer writes:
> Large repositories with a huge amount of merge commits in the
> bisection process could lead to stack overflows in git bisect.
> In order to prevent this, this commit uses an *iterative* version
> for counting the number of ancestors of a commit.
Yay!
> -/*
> - * This is
Jacob Keller writes:
>>> What you have is a pure developer support; aim to come up with "good
>>> enough" way, giving developers an easier way to experiment with, and
>>> remove it before the feature is shipped to the end user.
>
> What are your thoughts on adding this do the gitattributes diff
>
Stephan Beyer writes:
> This is a preparation for subsequent changes.
> During a bisection process, we want to augment commits with
> further information to improve speed.
>
> Signed-off-by: Stephan Beyer
> ---
Makes sense.
--
To unsubscribe from this list: send the line "unsubscribe git" in
t
From: Jacob Keller
It is a common pattern in xdl_change_compact to check that hashes and
strings match. The resulting code to perform this change causes very
long lines and makes it hard to follow the intention. Introduce a helper
function recs_match which performs both checks to increase
code re
From: Jacob Keller
Second version of my series with a few more minor fixups. I left the
diff command line and configuration option alone for now, suspecting
that long term we either (a) remove it or (b) use a gitattribute, so
there is no reason to bikeshed the name or its contents right now.
TOD
From: Stefan Beller
In order to produce the smallest possible diff and combine several diff
hunks together, we implement a heuristic from GNU Diff which moves diff
hunks forward as far as possible when we find common context above and
below a diff hunk. This sometimes produces less readable diffs
On Fri, Apr 15, 2016 at 2:44 PM, Junio C Hamano wrote:
> Jacob Keller writes:
>
What you have is a pure developer support; aim to come up with "good
enough" way, giving developers an easier way to experiment with, and
remove it before the feature is shipped to the end user.
>>
>> W
Stephan Beyer writes:
> @@ -43,15 +43,17 @@ static int count_distance(struct commit_list *entry)
> int nr = 0;
> struct commit_list *todo = NULL;
> commit_list_append(entry->item, &todo);
> + marker++;
>
> while (todo) {
> struct commit *commit = pop_co
Stephan Beyer writes:
> Signed-off-by: Stephan Beyer
> ---
> bisect.c | 16 ++--
> 1 file changed, 10 insertions(+), 6 deletions(-)
Nice.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at
Stephan Beyer writes:
> It makes no sense that the argument for count_distance() and
> halfway() is a commit list when only its first commit is relevant.
>
> Signed-off-by: Stephan Beyer
> ---
Makes sense (modulo perhaps s/elem/commit/).
> bisect.c | 16
> 1 file changed, 8 i
Stephan Beyer writes:
> We introduce the concept of rising and falling distances
> (in addition to a halfway distance).
> This will be useful in subsequent commits.
>
> Signed-off-by: Stephan Beyer
> ---
> bisect.c | 33 +++--
> 1 file changed, 23 insertions(+), 10 d
On 15 April 2016 at 21:27, Junio C Hamano wrote:
> Jan Durovec writes:
>
>> ---
>
> A few issues. Please:
>
> (1) Sign-off your work.
>
> (2) Try to find those who are familiar with the area and Cc them.
>
> "git shortlog -s -n --since=18.months --no-merges git-p4.py"
> may help.
>
>
Stephan Beyer writes:
> The total number of commits in a bisect process is a property of
> the bisect process. Making this property global helps to make the code
> clearer.
OK.
>
> Signed-off-by: Stephan Beyer
> ---
> bisect.c | 74
> ++
Stephan Beyer writes:
> Let us use the term "weight" for the number of ancestors
> of each commit, and "distance" for the number
> min{weight, #commits - weight}. (Bisect finds the commit
> with maximum distance.)
>
> In these terms, "count_distance()" is the wrong name of
> the function. So it i
On Thu, Apr 14, 2016 at 05:32:16PM -0400, Isaac Levy wrote:
> After the authenticated request, curl says it's keeping the connection
> open, but the next fetch seems to do two handshakes again. The
> unauthenticated request closes the connection, so the 2nd handshake is
> forced, but I'm not sure
Stephan Beyer writes:
> This is a preparation commit with copy-and-paste involved.
> The function do_find_bisection() is changed and copied to
> two almost similar functions compute_all_weights() and
> compute_relevant_weights().
>
> The function compute_relevant_weights() stops when a
> "halfway
On Fri, Apr 15, 2016 at 06:21:20PM -0400, Jeff King wrote:
> I think we can take that down to _two_ requests pretty easily. We know
> in the very first request that the server told us something like:
>
> < WWW-Authenticate: Basic realm="GitHub"
>
> but curl doesn't remember that. However, we s
On Fri, Apr 15, 2016 at 02:56:21PM -0700, Jacob Keller wrote:
> @@ -470,8 +477,9 @@ int xdl_change_compact(xdfile_t *xdf, xdfile_t *xdfo,
> long flags) {
>* the line next of the current change group, shift
> forward
>* the group.
>
Stephan Beyer writes:
> The idea is to reverse the DAG and perform a traversal
> starting on all sources of the reversed DAG.
Please clarify what you mean by "sources" here. Those who read log
message in Git context would know that you mean the commit graph by
"DAG", and "reversed DAG" means "h
On Fri, Apr 15, 2016 at 3:46 PM, Jeff King wrote:
> On Fri, Apr 15, 2016 at 02:56:21PM -0700, Jacob Keller wrote:
>
>> @@ -470,8 +477,9 @@ int xdl_change_compact(xdfile_t *xdf, xdfile_t *xdfo,
>> long flags) {
>>* the line next of the current change group, shift
>> forwar
Stephan Beyer writes:
> +static struct commit *extract_merge_to_queue(struct commit_list **merges)
> +{
> + assert(merges);
> +
> + struct commit_list *p, *q;
> + struct commit *found;
> +
"gcc -Werror -Wdecl-after-statement" will barf at this.
--
To unsubscribe from this list: send
Stephan Beyer writes:
> The documentation says that when the maximum possible distance
> is found, the algorithm stops immediately. That feature is
> reestablished by this commit.
>
> Signed-off-by: Stephan Beyer
> ---
>
> Notes:
> I plugged a memory leak here.
... relative to patch series
From: Jacob Keller
Third version of my series with a few more minor fixups. I left the
diff command line and configuration option alone for now, suspecting
that long term we either (a) remove it or (b) use a gitattribute, so
there is no reason to bikeshed the name or its contents right now.
TODO
From: Jacob Keller
It is a common pattern in xdl_change_compact to check that hashes and
strings match. The resulting code to perform this change causes very
long lines and makes it hard to follow the intention. Introduce a helper
function recs_match which performs both checks to increase
code re
From: Stefan Beller
In order to produce the smallest possible diff and combine several diff
hunks together, we implement a heuristic from GNU Diff which moves diff
hunks forward as far as possible when we find common context above and
below a diff hunk. This sometimes produces less readable diffs
This is a version based on Jacobs v2, with the same fixes as in his v3
(hopefully),
changing the heuristic, such that CRLF confusion might be gone.
TODO:
* add some tests
* think about whether we need a git attribute or not (I did some
thinking, and if we do need to configure this at all, this
From: Jacob Keller
It is a common pattern in xdl_change_compact to check that hashes and
strings match. The resulting code to perform this change causes very
long lines and makes it hard to follow the intention. Introduce a helper
function recs_match which performs both checks to increase
code re
In order to produce the smallest possible diff and combine several diff
hunks together, we implement a heuristic from GNU Diff which moves diff
hunks forward as far as possible when we find common context above and
below a diff hunk. This sometimes produces less readable diffs when
writing C, Shell
On Fri, Apr 15, 2016 at 4:01 PM, Stefan Beller wrote:
> In order to produce the smallest possible diff and combine several diff
> hunks together, we implement a heuristic from GNU Diff which moves diff
> hunks forward as far as possible when we find common context above and
> below a diff hunk. Th
On Fri, Apr 15, 2016 at 4:05 PM, Jacob Keller wrote:
> There's a few places that will need cleaning up (comments and such)
> that mention empty line still, but that's not surprising. I am going
> to test this for a bit on my local repos, and see if it makes any
> difference to the old heuristic as
1 - 100 of 117 matches
Mail list logo
