PAYRE NATHAN p1508475 writes:
> Without the "print" used for testing.
But still smoe broken indentation:
> git-send-email.perl | 90
> +
> 1 file changed, 63 insertions(+), 27 deletions(-)
>
> diff --git a/git-send-email.perl b/git-send-ema
On Wed, Dec 6, 2017 at 5:55 PM, Nathan PAYRE wrote:
> Junio C Hamano : writes:
>> Also "reusable in other place" is by itself not an unconditional
>> plus, until readers can be convinced that that 'other place' really
>> wants to be able to call this function. Is there some untold
>> motivation b
On Thu, Dec 7, 2017 at 1:18 AM, Jacob Keller wrote:
> On Wed, Dec 6, 2017 at 4:16 PM, Jacob Keller wrote:
>>
>> This causes a regression in the --relative option which prevents it
>> from working properly.
>>
>> If I have a repository with a modified file in a subdirectory, such as:
>>
>> a/file
On 06/12/17 21:07, Jeff Hostetler wrote:
>
>
> On 12/6/2017 12:39 PM, Ramsay Jones wrote:
>> Hi Jeff,
>>
>> commit f1862e8153 ("partial-clone: define partial clone settings
>> in config", 2017-12-05), which is part of your 'jh/partial-clone'
>> branch, introduces the partial_clone_get_default_f
On Wed, Dec 6, 2017 at 5:04 PM, Jeff King wrote:
> On Wed, Dec 06, 2017 at 04:35:17PM -0800, Jacob Keller wrote:
>
>> Subject: [PATCH] diff: add test showing regression to --relative
>
> Since we'd hopefully not ever merge that regression, I think this patch
> ought to stand on its own. In which c
On Wed, Dec 06, 2017 at 05:04:02PM -0800, Jacob Keller wrote:
> What you outlined above is probably the best we can do. We could even
> add some extra helper which does that for us if we want.
>
> I sent a patch that merely reverts the change to --relative and adds a
> test for now though.
Thank
On Wed, Dec 6, 2017 at 4:56 PM, Jeff King wrote:
> On Wed, Dec 06, 2017 at 04:38:29PM -0800, Jacob Keller wrote:
>
>> >> But nope, it looks like the culprit is f7923a5ece (diff: use
>> >> skip_to_optional_val(), 2017-12-04), which switched over parsing of
>> >> "--relative".
>> >
>> > Oh, actually
On Wed, Dec 06, 2017 at 04:35:17PM -0800, Jacob Keller wrote:
> Subject: [PATCH] diff: add test showing regression to --relative
Since we'd hopefully not ever merge that regression, I think this patch
ought to stand on its own. In which case it probably wants to say
something like:
diff: test
From: Jacob Keller
Recently, commit f7923a5ece00 ("diff: use skip_to_optional_val()",
2017-12-04) changed how we parsed some diff options, preferring
skip_to_optional_val instead of a more complex skip_prefix() setup.
Unfortunately, this breaks --relative, because of the semantics of
skip_to_opt
On Wed, Dec 06, 2017 at 04:38:29PM -0800, Jacob Keller wrote:
> >> But nope, it looks like the culprit is f7923a5ece (diff: use
> >> skip_to_optional_val(), 2017-12-04), which switched over parsing of
> >> "--relative".
> >
> > Oh, actually, I guess I was half-right. It feeds &options->prefix as t
On Wed, Dec 6, 2017 at 4:24 PM, Jeff King wrote:
> On Wed, Dec 06, 2017 at 07:22:35PM -0500, Jeff King wrote:
>
>> On Wed, Dec 06, 2017 at 04:01:51PM -0800, Jacob Keller wrote:
>>
>> > I think I narrowed this down to "git diff-index --name-only --relative
>> > HEAD" producing a list of files *not*
From: Jacob Keller
Signed-off-by: Jacob Keller
---
t/t4045-diff-relative.sh | 5 +
1 file changed, 5 insertions(+)
diff --git a/t/t4045-diff-relative.sh b/t/t4045-diff-relative.sh
index 3950f5034d31..41e4f59b2ffb 100755
--- a/t/t4045-diff-relative.sh
+++ b/t/t4045-diff-relative.sh
@@ -70,4
On Wed, Dec 06, 2017 at 07:22:35PM -0500, Jeff King wrote:
> On Wed, Dec 06, 2017 at 04:01:51PM -0800, Jacob Keller wrote:
>
> > I think I narrowed this down to "git diff-index --name-only --relative
> > HEAD" producing a list of files *not* relative to the current
> > directory.
>
> Hmm, my gue
On Wed, Dec 06, 2017 at 04:01:51PM -0800, Jacob Keller wrote:
> I think I narrowed this down to "git diff-index --name-only --relative
> HEAD" producing a list of files *not* relative to the current
> directory.
Hmm, my guess would have been something funny in the setup code
forgetting our origin
On Wed, Dec 6, 2017 at 4:16 PM, Jacob Keller wrote:
> On Sun, Dec 3, 2017 at 9:04 AM, Christian Couder
> wrote:
>> From: Christian Couder
>>
>> Let's simplify diff option parsing using skip_to_opt_val().
>>
>> Signed-off-by: Christian Couder
>> ---
>> diff.c | 22 --
>> 1 f
On Sun, Dec 3, 2017 at 9:04 AM, Christian Couder
wrote:
> From: Christian Couder
>
> Let's simplify diff option parsing using skip_to_opt_val().
>
> Signed-off-by: Christian Couder
> ---
> diff.c | 22 --
> 1 file changed, 8 insertions(+), 14 deletions(-)
>
> diff --git a/di
On Wed, Dec 6, 2017 at 4:01 PM, Jacob Keller wrote:
> On Wed, Dec 6, 2017 at 3:53 PM, Jacob Keller wrote:
>> Hi,
>>
>> I'm still investigating, but thought I'd send an email. I recently
>> updated to jch branch, and found that completion for git commit does
>> not work as expected.
>>
>> If I hav
Hi Łukasz,
On Tue, 5 Dec 2017, Łukasz Stelmach wrote:
> Convert author's name and e-mail address from the UTF-8 (or any other)
> encoding in load_last_commit function the same way commit message is
> converted.
>
> Amending commits in git-gui without such conversion breaks UTF-8
> strings. For e
On Wed, Dec 6, 2017 at 3:53 PM, Jacob Keller wrote:
> Hi,
>
> I'm still investigating, but thought I'd send an email. I recently
> updated to jch branch, and found that completion for git commit does
> not work as expected.
>
> If I have a git repository with a modified file in a subdirectiory,
>
Hi,
I'm still investigating, but thought I'd send an email. I recently
updated to jch branch, and found that completion for git commit does
not work as expected.
If I have a git repository with a modified file in a subdirectiory,
then git commit produces the name of the subdirectory instead of
t
On Thu, Dec 7, 2017 at 12:02 AM, Nathan Payre
wrote:
> +sub parse_header_line {
> + my $lines = shift;
> + my $parsed_line = shift;
> +
> + foreach (split(/\n/, $lines)) {
> + if (/^(To|Cc|Bcc):\s*(.+)$/i) {
> + $parsed_line->{lc $1} = [ parse
Nathan PAYRE writes:
> Junio C Hamano : writes:
>
>> ... throughout this patch, not limited to this section, indentation
>> is strange and there seem to be many "print" that show messages that
>> do not seem to be meant for end-user consumption. I can see that
>> this aspires to improve the read
The existing code mixes parsing of email header with regular
expression and actual code. Extract the parsing code into a new
subroutine 'parse_header_line()'. This improves the code readability
and make parse_header_line reusable in other place.
Signed-off-by: Nathan Payre
Signed-off-by: Matthieu
Junio C Hamano : writes:
> ... throughout this patch, not limited to this section, indentation
> is strange and there seem to be many "print" that show messages that
> do not seem to be meant for end-user consumption. I can see that
> this aspires to improve the readability, but not quite yet ;-)
Brandon Williams writes:
> @@ -56,6 +62,41 @@ static void die_initial_contact(int unexpected)
> "and the repository exists."));
> }
>
> +static enum protocol_version discover_version(struct packet_reader *reader)
> +{
> + enum protocol_version version = protocol_unknown
Brandon Williams writes:
> Remove code duplication and use the existing 'get_refs_via_connect()'
> function to retrieve a remote's heads in 'fetch_refs_via_pack()' and
> 'git_transport_push()'.
>
> Signed-off-by: Brandon Williams
> ---
> transport.c | 18 --
> 1 file changed, 4
On Wed, Dec 6, 2017 at 5:00 PM, Brandon Williams wrote:
> On 12/05, Eric Sunshine wrote:
>> 557a5998d (submodule: remove gitmodules_config, 2017-08-03) touched a
>> fair number of built-in commands. It's not clear from the current
>> patch's commit message if diff-tree is the only command which
>>
Junio C Hamano writes:
> I've however queued the following on top of the entire series. I do
> not mind squashing it into this step, though.
> Subject: [PATCH] t2020: test variations that matter
That sounds and looks fine to me. I'm going to do this on my
side (getting it from 'pu' once the s
A regression was introduced in 557a5998d (submodule: remove
gitmodules_config, 2017-08-03) to how attribute processing was handled
in bare repositories when running the diff-tree command.
By default the attribute system will first try to read ".gitattribute"
files from the working tree and then fa
On 12/05, Eric Sunshine wrote:
> On Tue, Dec 5, 2017 at 5:13 PM, Brandon Williams wrote:
> > A regression was introduced in 557a5998d (submodule: remove
> > gitmodules_config, 2017-08-03) to how attribute processing was handled
> > in bare repositories when running the diff-tree command.
> >
> > B
Brandon Williams writes:
> In order to allow for code sharing with the server-side of fetch in
> protocol-v2 convert upload-pack to be a builtin.
>
> Signed-off-by: Brandon Williams
> ---
This looks obvious and straight-forward to a cursory look.
I vaguely recalled and feared that we on purpos
On 12/05, Stefan Beller wrote:
> On Tue, Dec 5, 2017 at 2:13 PM, Brandon Williams wrote:
> > A regression was introduced in 557a5998d (submodule: remove
> > gitmodules_config, 2017-08-03) to how attribute processing was handled
> > in bare repositories when running the diff-tree command.
> >
> > B
On Wed, Dec 6, 2017 at 4:00 PM, Gumbel, Matthew K
wrote:
> I've noticed that when I run 'git worktree add /path/to/new/tree',
> the post-checkout hook does not fire, even though the worktree
> manpage explicitly states that "worktree add" will, "Create
> and checkout into it."
>
> Is this the in
On Wed, Dec 06, 2017 at 10:08:23AM -0500, Derrick Stolee wrote:
> There are several places in Git where we refer to the size of an object by
> an 'unsigned long' instead of a 'size_t'. In 64-bit Linux, 'unsigned long'
> is 8 bytes, but in 64-bit Windows it is 4 bytes.
>
> The main issue with this
Nathan Payre writes:
> The existing code mixes parsing of email header with regular
> expression and actual code. Extract the parsing code into a new
> subroutine 'parse_header_line()'. This improves the code readability
> and make parse_header_line reusable in other place.
>
> Signed-off-by: Na
Hi Brandon,
On Mon, 4 Dec 2017, Brandon Williams wrote:
> Commit 74ed43711fd (grep: enable recurse-submodules to work on
> objects, 2016-12-16) taught 'tree_entry_interesting()' to be able to
> match across submodule boundaries in the presence of wildcards. This is
> done by performing literal
On 12/6/2017 12:39 PM, Ramsay Jones wrote:
Hi Jeff,
commit f1862e8153 ("partial-clone: define partial clone settings
in config", 2017-12-05), which is part of your 'jh/partial-clone'
branch, introduces the partial_clone_get_default_filter_spec()
function without any callers. Could you please c
Hello all,
I've noticed that when I run 'git worktree add /path/to/new/tree', the
post-checkout hook does not fire, even though the worktree manpage explicitly
states that "worktree add" will, "Create and checkout into it."
Is this the intended behavior? Seems like maybe a bug, but I'm not su
On Wed, Dec 6, 2017 at 7:56 PM, Daniel Jacques wrote:
> On Wed, Dec 6, 2017 at 1:47 PM, Junio C Hamano wrote:
>>
>> Johannes Sixt writes:
>>
>> > The updated series works for me now. Nevertheless, I suggest to squash
>> > in the following change to protect against IFS and globbing characters in
On Wed, Dec 6, 2017 at 1:47 PM, Junio C Hamano wrote:
>
> Johannes Sixt writes:
>
> > The updated series works for me now. Nevertheless, I suggest to squash
> > in the following change to protect against IFS and globbing characters in
> > $INSTLIBDIR.
>
> Yeah, that is very sensible.
>
> > diff -
Johannes Sixt writes:
> The updated series works for me now. Nevertheless, I suggest to squash
> in the following change to protect against IFS and globbing characters in
> $INSTLIBDIR.
Yeah, that is very sensible.
> diff --git a/Makefile b/Makefile
> index 7ac4458f11..08c78a1a63 100644
> --- a
On Wed, Dec 6, 2017 at 10:34 AM, Johannes Sixt wrote:
> I am sorry for not responding in detail. I think we've reached a mutual
> understanding of our workflows.
>
> Though, from the ideas you tossed around most recently, you seem to want to
> make git-commit into a kitchen-sink for everything. I
I am sorry for not responding in detail. I think we've reached a mutual
understanding of our workflows.
Though, from the ideas you tossed around most recently, you seem to want
to make git-commit into a kitchen-sink for everything. I have my doubts
that this will be a welcome change. Just beca
Am 05.12.2017 um 22:35 schrieb Junio C Hamano:
> Dan Jacques writes:
>
>> Thanks for checking! The patch that you quoted above looks like it's from
>> this "v4" thread; however, the patch that you are diffing against in your
>> latest reply seems like it is from an earlier version.
>>
>> I believ
On 12/06, Derrick Stolee wrote:
> There are several places in Git where we refer to the size of an
> object by an 'unsigned long' instead of a 'size_t'. In 64-bit Linux,
> 'unsigned long' is 8 bytes, but in 64-bit Windows it is 4 bytes.
>
> The main issue with this conversion is that large objects
> Can you bisect to see when the feature stopped working as you expect?
I will see if I can do that but might take some time.
> It finds up to which commit each line survived without getting touched since
> the oldest commit in the range.
Right, this is where it is failing in my case.
With a h
On Sunday 03 December 2017 07:38 AM, Junio C Hamano wrote:
Kaartic Sivaraam writes:
NOTE: Though a commit-hash is a "syntactically" valid branch name,
it is generally not considered as one for the use cases of
"git check-ref-format --branch". That's because a user who does
"git check-ref-form
Hi Jeff,
commit f1862e8153 ("partial-clone: define partial clone settings
in config", 2017-12-05), which is part of your 'jh/partial-clone'
branch, introduces the partial_clone_get_default_filter_spec()
function without any callers. Could you please confirm that this
is intentional and that, presu
Nick Snyder writes:
> This can be reproduced on Linux and Mac. This behavior seems to be a bug.
Can you bisect to see when the feature stopped working as you expect?
Unlike a forward blame, where the command tries to find a commit
that is responsible for a line being in the final result (i.e.
t
Am 06.12.2017 um 16:08 schrieb Derrick Stolee:
Hi Derrick,
> If there are small, isolated chunks that can be done safely, then this
> may be a good target for a first patch.
Here are some pointers to past discussions:
-
https://public-inbox.org/git/trinity-9f703269-6f73-4f6d-b90b-45e09e1c094c-14
On December 6, 2017 11:40 AM, Junio C Hamano wrote:
>"Randall S. Becker" writes:
>> Having the git-manpages repo available is fantastic for platforms that
>> cannot easily build documentation on demand, for example, when too
>> many dependencies that do not build properly.
>> It would be really
Torsten Bögershausen writes:
>> Looks like t0027-auto-crlf.sh is failing on Windows:
>> https://travis-ci.org/git/git/jobs/312135514
>>
>> Could that patch be related?
>>
>> - Lars
>>
> Chances are high, I will have a look.
> Thanks for noticing.
Thanks.
Ann T Ropea writes:
> v7: fix style issues (redirection, here-dox, long lines,
> setting/exporting/unsetting of env var): cf.
>
Thanks. I'd say this is polished enough already. Let's stop
rerolling, wait for a few days for others to give final comments and
merge it to 'next'.
I've however
"Randall S. Becker" writes:
> Having the git-manpages repo available is fantastic for platforms
> that cannot easily build documentation on demand, for example,
> when too many dependencies that do not build properly.
>
> It would be really nice to have a version of install-doc-quick.sh to either
The existing code mixes parsing of email header with regular
expression and actual code. Extract the parsing code into a new
subroutine 'parse_header_line()'. This improves the code readability
and make parse_header_line reusable in other place.
Signed-off-by: Nathan Payre
Signed-off-by: Matthieu
On 2017-12-06 16:14, Lars Schneider wrote:
>
>> On 04 Dec 2017, at 22:46, Junio C Hamano wrote:
>>
>>
>> * tb/check-crlf-for-safe-crlf (2017-11-27) 1 commit
>> - convert: tighten the safe autocrlf handling
>>
>> The "safe crlf" check incorrectly triggered for contents that does
>> not use CRLF as
Other thread attached as context.
Having the git-manpages repo available is fantastic for platforms that cannot
easily build documentation on demand, for example, when too many dependencies
that do not build properly.
It would be really nice to have a version of install-doc-quick.sh to either:
-Original Message-
On December 6, 2017 3:49 AM, Jeff King wrote:
>On Wed, Dec 06, 2017 at 09:14:57AM +0100, Ævar Arnfjörð Bjarmason wrote:
>> > I'm trying to upgrade the NonStop port from 2.3.7 upward eventually
>> > to
>> > 2.15.1 and hit a snag on documentation. The xmlto component is a
This can be reproduced on Linux and Mac. This behavior seems to be a bug.
On Wed, Nov 29, 2017 at 12:06 AM, Nick Snyder wrote:
> I have a repo that reproduces a behavior with `git blame --reverse`
> that surprises me. https://github.com/nicksnyder/git-blame-bug
>
> The readme explains the observe
The existing code mixes parsing of email header with regular
expression and actual code. Extract the parsing code into a new
subroutine 'parse_header_line()'. This improves the code readability
and make parse_header_line reusable in other place.
Signed-off-by: Nathan Payre
Signed-off-by: Matthieu
> On 04 Dec 2017, at 22:46, Junio C Hamano wrote:
>
>
> * tb/check-crlf-for-safe-crlf (2017-11-27) 1 commit
> - convert: tighten the safe autocrlf handling
>
> The "safe crlf" check incorrectly triggered for contents that does
> not use CRLF as line endings, which has been corrected.
>
> Will
There are several places in Git where we refer to the size of an object
by an 'unsigned long' instead of a 'size_t'. In 64-bit Linux, 'unsigned
long' is 8 bytes, but in 64-bit Windows it is 4 bytes.
The main issue with this conversion is that large objects fail to load
(they seem to hash and s
On Wed, Dec 6, 2017 at 3:02 PM, Saurabh Dixit wrote:
> Hi,
>
> I am new here. I just wondered if the Merge Requests (aka., Pull
> Requests on GitHub) are also imported or cloned while
> cloning/importing a Git repository, say from GitHub to BitBucket.
> While I consider that, it may not be possibl
Hi,
I am new here. I just wondered if the Merge Requests (aka., Pull
Requests on GitHub) are also imported or cloned while
cloning/importing a Git repository, say from GitHub to BitBucket.
While I consider that, it may not be possible because of the URL to a
remote is already set and cannot be alt
Kindly Assist Me
In good faith from Mr.Sonami, actually could you please consider to
help me to relocate this sum of five million,three hundred thousand
dollars(US$5.3 m) to your country for establishing a medium industry
in your country. The said 5.3 millions dollars was deposited in our
bank by
Greetings from Dr. mohammad ouattara.
Assalamu`Alaikum.
My Name is Dr. mohammad ouattara, I am a banker by profession. I'm
from Ouagadougou, Burkina Faso, West Africa. My reason for contacting
you is to transfer an abandoned $14.6M to your account.
The owner of this fund died since 2004 with his
On Wed, Dec 06, 2017 at 09:14:57AM +0100, Ævar Arnfjörð Bjarmason wrote:
> > I'm trying to upgrade the NonStop port from 2.3.7 upward eventually to
> > 2.15.1 and hit a snag on documentation. The xmlto component is a bit new to
> > me and I hit the following error:
Did it work before in v2.3.7? I
On Wed, Dec 06 2017, Randall S. Becker jotted:
> Hi All,
>
> I'm trying to upgrade the NonStop port from 2.3.7 upward eventually to
> 2.15.1 and hit a snag on documentation. The xmlto component is a bit new to
> me and I hit the following error:
>
> XMLTO git-remote-testgit.1
> xmlto: /home/g
68 matches
Mail list logo