On Fri, Jun 2, 2017 at 4:24 AM, Prathamesh Chavan wrote:
> This aims to make git-submodule foreach a builtin. This is the very
> first step taken in this direction. Hence, 'foreach' is ported to
> submodule--helper, and submodule--helper is called from git-submodule.sh.
> The code is split up to h
Stefan Beller wrote:
> Today I learned again how public-inbox is awesome! Thanks Eric!
You're welcome :)
> * You can just copy the message ID INCLUDING the surrounding < >
> and public inbox still just shows you the correct message. I had assumed
> you would need to strip off the < > and I d
On 06/03/2017 02:01 AM, Junio C Hamano wrote:
> Your "/images/*" is the "Otherwise" case, isn't it?
Ok, sorry, didn't read your answer thoroughly enough the first time.
The problem is, when I have an entry in .gitignore like this:
/images
Then git will ignore that whole directory (and all of its
On 06/03/2017 02:01 AM, Junio C Hamano wrote:
> Your "/images/*" is the "Otherwise" case, isn't it?
Yes, true. I forgot to mention in my first mail that I also tried
'/images/', which had the same (=none) effect as '/images/*', sorry
about the confusion.
On Fri, Jun 2, 2017 at 5:26 PM, Junio C Hamano wrote:
>
> Issue #06 of May marked it to be merged to 'next':
> https://public-inbox.org/git/
>
> Issue #07 of May marked it for 'master':
> https://public-inbox.org/git/
>
> Issue #08 of May kept it (i.e. no issues discovered in the
> meantime):
> ht
When running 'git submodule foreach' from a subdirectory of your
repository, nested submodules get a bogus value for $sm_path:
For a submodule 'sub' that contains a nested submodule 'nested',
running 'git -C dir submodule foreach echo $path' would report
path='../nested' for the nested submodule. T
Jonathan Nieder writes:
> 2. Hard-coded object ids in tests: As Stefan hinted, many tests beyond
>t00* make assumptions about the exact values of object ids. That's
>bad for maintainability for other reasons beyond the hash function
>transition, too.
>
>It should be possible to s
Johannes Schindelin writes:
> Hi Junio,
>
> On Fri, 2 Jun 2017, Junio C Hamano wrote:
>
>> Samuel Lijin writes:
>>
>> >> What is holding this topic up? Anything Ben or I can do to move this
>> >> closer to `next` or even `master`?
>> >
>> > It's in `next` right now (3196d093d6).
>>
>> Thanks f
Ævar Arnfjörð Bjarmason writes:
> On Fri, Jun 2, 2017 at 4:46 PM, Liam R. Howlett
> wrote:
>
>> 2.13.0 is very much broken for me on SPARC.
>> {maint//git} $ make -j120
>> [...]
>> {maint//git} $ ./git log
>> [1]1004506 bus error (core dumped) ./git log
>>
>> This is with b06d36431 (maint)
Linus Torvalds writes:
> Dereferencing an unaligned pointer may be "undefined" in some
> technical meaning, but it sure as hell isn't undefined in reality, and
> compilers that willfully do stupid things should not be catered to
> overly. Reality is a lot more important.
Thanks for succinctly pu
Rene Pasing writes:
> The problem is, the documentation[1] says: "The rules how the pattern
> matches paths are the same as in .gitignore files; see gitignore[5].",
> so when I have a pattern like '/images/', it should match on all
> files+folders under /images, even the directory itself, right?
Jeff King writes:
> But I think a more compelling case is that there may be an ongoing
> operation in the original repo (e.g., say you are in the middle of
> writing a commit message) when we do a blind copy of the filesystem
> contents. You might racily pick up a lockfile.
>
> Should we find and
Ævar Arnfjörð Bjarmason writes:
> On Fri, Jun 2, 2017 at 9:15 AM, Junio C Hamano wrote:
>> Jeffrey Walton writes:
>>
>>> Is there no switch? Its the most efficient way to accomplish the task.
>>
>> This is a safety to help normal human users from hurting themselves,
>> and it does not make any
Brandon Williams writes:
> On 06/02, Junio C Hamano wrote:
>>
>> > -static int path_to_sha1(const char *path, unsigned char *sha1)
>> > +static int path_to_oid(const char *path, struct object_id *oid)
>> > {
>> > - char hex_sha1[40];
>> > + char hex_oid[GIT_SHA1_HEXSZ];
>> >int i = 0;
>>
Brandon Williams writes:
> On 06/02, Junio C Hamano wrote:
>>
>> I lied. This also conflicts somewhat with Peff's diff-blob topic.
>> I think I resolved them correctly (there needs evil merges applied
>> to two files when merging this topic), and hopefully can push out
>> the result by the end
Jeff King wrote:
> On Fri, Jun 02, 2017 at 12:38:45PM -0700, Jonathan Tan wrote:
>> +--blob-max-bytes=::
>> +This option can only be used with --stdout. If specified, a blob
>> +larger than this will not be packed unless a to-be-packed tree
>> +has that blob with a filename beginning w
Thanks Philip,
I've created a pull request there -
https://github.com/patthoyts/git-gui/pull/12
signature.asc
Description: OpenPGP digital signature
Dear Good day,
I sent this mail praying for it to reach you in good health, since I
myself are in a very critical health condition in which I sleep every
night without knowing if I may be alive to see the next day. I am a
widow suffering from long time illness. I have some funds I inherited
from
On Sat, Jun 3, 2017 at 12:05 AM, Ben Peart wrote:
>
>
> On 6/2/2017 6:28 AM, Ævar Arnfjörð Bjarmason wrote:
>>
>> Add a performance test for the new core.fsmonitor facility using the
>> sample query-fsmonitor hook.
>>
>> This is WIP code for the reasons explained in the setup comments,
>> unfortun
From: "Ævar Arnfjörð Bjarmason"
On Fri, Jun 2, 2017 at 3:41 PM, Philipp Gortan wrote:
Hi git devs,
First off, thanks for your awesome work!
I've been unhappy for quite a while that I had to configure the hooks
manually for each of my repos - until I found out recently that there is
the core.
On Sat, Jun 03, 2017 at 12:47:59AM +0200, Ulrich Mueller wrote:
> > On Fri, 2 Jun 2017, Jeff King wrote:
>
> > The remaining question is whether we want to care about preserving the
> > system %Z for the local-timezone case.
>
> No strong preference here. Maybe go for consistency, and have %
> On Fri, 2 Jun 2017, Jeff King wrote:
> The remaining question is whether we want to care about preserving the
> system %Z for the local-timezone case.
No strong preference here. Maybe go for consistency, and have %Z
always return the same format (either empty, or same as %z). That
would at
On Sat, Jun 03, 2017 at 12:04:32AM +0200, Ulrich Mueller wrote:
> Actually, the POSIX definition for %Z continues: "or by no bytes if no
> timezone information exists." So also returning an empty string would
> be compliant (but maybe not very helpful).
> [...]
> I agree that GMT+0200 could be mis
On Fri, Jun 02, 2017 at 12:38:45PM -0700, Jonathan Tan wrote:
> +--blob-max-bytes=::
> + This option can only be used with --stdout. If specified, a blob
> + larger than this will not be packed unless a to-be-packed tree
> + has that blob with a filename beginning with ".git". The siz
On Thu, Jun 01, 2017 at 02:13:10PM -0700, Stefan Beller wrote:
> >
> >> $ git am /tmp/original_msg.txt
> >> Applying: fsmonitor: add documentation for the fsmonitor extension.
> >> error: patch failed: Documentation/githooks.txt:448
> >> error: Documentation/githooks.txt: patch
On Fri, Jun 02, 2017 at 12:38:43PM -0700, Jonathan Tan wrote:
> > Do we need to future-proof the output format so that we can later
> > use 32-byte hash? The input to pack-objects (i.e. rev-list --objects)
> > is hexadecimal text, and it may not be so bad to make this also
> > text, e.g. " SP LF
On 6/2/2017 6:28 AM, Ævar Arnfjörð Bjarmason wrote:
Add a performance test for the new core.fsmonitor facility using the
sample query-fsmonitor hook.
This is WIP code for the reasons explained in the setup comments,
unfortunately the perf code doesn't easily allow you to run different
setup co
> On Fri, 2 Jun 2017, Jeff King wrote:
> On Fri, Jun 02, 2017 at 07:25:43PM +0200, René Scharfe wrote:
>> On Linux "%z %Z" is expanded to "+0200 CEST" for me, while on Windows I
>> get "Mitteleurop▒ische Sommerzeit Mitteleurop▒ische Sommerzeit". (That
>> "▒" is probably supposed to be an "ä".
On Fri, Jun 02, 2017 at 11:23:30AM +0900, Junio C Hamano wrote:
> René Scharfe writes:
>
> > Am 27.05.2017 um 23:46 schrieb Jeff King:
> >> On Sat, May 27, 2017 at 06:57:08PM +0200, Ævar Arnfjörð Bjarmason wrote:
> >>
> >>> There's another test which breaks if we just s/gmtime/localtime/g. As
>
On Fri, Jun 2, 2017 at 1:17 PM, demerphq wrote:
> Most hash function implementations have code like the following
> (extracted and reduced from hv_macro.h in perl.git [which only
> supports little-endian hash functions]):
Yes.
Please do *not* try to make things overly portable by adding random
m
BTW, a medium-sized (~250k files across 40k dirs) synthetic repo is available
over bittorrent at:
http://bitmover.com/2015-04-03-1M-git-bare.tar.bz2.torrent
I tried Ævar's perf test with that (on a beefy laptop with SSD), and got
significantly slower results with bp/fsmonitor:
Test
On Fri, Jun 02, 2017 at 10:45:38AM +0900, Junio C Hamano wrote:
> > ...I should have read to the end of the sentence. It should also be "in
> > the `$GIT_DIR/remotes/` file". Or just drop "file".
>
> There is another one nearby. Here is what I understand as your
> suggestion (the "just drop" var
Dear Philip,
the previous mail contains a patch against the master of
http://repo.or.cz/git-gui.git
Could you please review it? I am not a TCL developer, so please take
extra care!
Thanks,
Philipp
signature.asc
Description: OpenPGP digital signature
On Fri, Jun 2, 2017 at 10:17 PM, demerphq wrote:
> On 2 June 2017 at 21:32, Ævar Arnfjörð Bjarmason wrote:
>> On Fri, Jun 2, 2017 at 11:49 AM, Martin Ågren wrote:
>>> On 2 June 2017 at 10:51, Ævar Arnfjörð Bjarmason wrote:
On Fri, Jun 2, 2017 at 2:15 AM, Junio C Hamano wrote:
> Martin
On Fri, Jun 2, 2017 at 7:54 PM, Jonathan Nieder wrote:
> Hi Dscho,
>
> Johannes Schindelin wrote:
>> On Thu, 1 Jun 2017, Stefan Beller wrote:
>
>>> We had a discussion off list how much of the test suite is in bad shape,
>>> and "$ git grep ^index" points out a lot of places as well.
>>
>> Maybe w
On 2 June 2017 at 22:14, Ævar Arnfjörð Bjarmason wrote:
> On Fri, Jun 2, 2017 at 10:11 PM, Martin Ågren wrote:
>> On 2 June 2017 at 21:32, Ævar Arnfjörð Bjarmason wrote:
>>> On Fri, Jun 2, 2017 at 11:49 AM, Martin Ågren
>>> wrote:
On 2 June 2017 at 10:51, Ævar Arnfjörð Bjarmason wrote:
>
Signed-off-by: Philipp Gortan
---
The following patch tries to fix git-gui to respect the core.hooksPath config
variable, falling back to the old behavior.
git-gui.sh | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/git-gui.sh b/git-gui.sh
index 5bc21b8..a5335b1 100755
---
On 2 June 2017 at 21:32, Ævar Arnfjörð Bjarmason wrote:
> On Fri, Jun 2, 2017 at 11:49 AM, Martin Ågren wrote:
>> On 2 June 2017 at 10:51, Ævar Arnfjörð Bjarmason wrote:
>>> On Fri, Jun 2, 2017 at 2:15 AM, Junio C Hamano wrote:
Martin Ågren writes:
> I looked into this some more.
On Fri, Jun 02, 2017 at 09:10:07PM +0200, SZEDER Gábor wrote:
> While at it, the first one fixes a minor bug, which allowed e.g. 'git
> log --no-min-parents-foobarbaz' to succeed.
>
> The other two are fairly straightforward starts_with() ->
> skip_prefix() conversions.
These all look fine to me
On Fri, Jun 02, 2017 at 04:11:43PM -0400, Jeff King wrote:
> if (match_opt(arg, "--early-output"), &optarg)) {
> int count = optarg ? atoi(optarg) : 100;
> ...
> }
>
> which is a little nicer and could maybe help other options (I didn't see
> any, though).
I take it back. This wo
On Fri, Jun 2, 2017 at 10:11 PM, Martin Ågren wrote:
> On 2 June 2017 at 21:32, Ævar Arnfjörð Bjarmason wrote:
>> On Fri, Jun 2, 2017 at 11:49 AM, Martin Ågren wrote:
>>> On 2 June 2017 at 10:51, Ævar Arnfjörð Bjarmason wrote:
On Fri, Jun 2, 2017 at 2:15 AM, Junio C Hamano wrote:
> Ma
On Fri, Jun 2, 2017 at 8:45 PM, Jeff King wrote:
> On Fri, Jun 02, 2017 at 10:33:30AM +, Ævar Arnfjörð Bjarmason wrote:
>
>> When the tested repo has an index.lock file it should be removed. This
>> file may be present if e.g. git-status previously crashed in that
>> repo, and it will make a l
On Fri, Jun 02, 2017 at 09:10:09PM +0200, SZEDER Gábor wrote:
> @@ -1785,15 +1785,15 @@ static int handle_revision_opt(struct rev_info *revs,
> int argc, const char **arg
> } else if (!strcmp(arg, "--author-date-order")) {
> revs->sort_order = REV_SORT_BY_AUTHOR_DATE;
>
On 2 June 2017 at 21:32, Ævar Arnfjörð Bjarmason wrote:
> On Fri, Jun 2, 2017 at 11:49 AM, Martin Ågren wrote:
>> On 2 June 2017 at 10:51, Ævar Arnfjörð Bjarmason wrote:
>>> On Fri, Jun 2, 2017 at 2:15 AM, Junio C Hamano wrote:
Martin Ågren writes:
> I looked into this some more.
On Fri, Jun 02, 2017 at 10:33:30AM +, Ævar Arnfjörð Bjarmason wrote:
> When the tested repo has an index.lock file it should be removed. This
> file may be present if e.g. git-status previously crashed in that
> repo, and it will make a lot of git commands fail. Let's try harder
> and remove t
As part of an effort to improve Git support for very large repositories
in which clients typically have only a subset of all version-controlled
blobs, teach pack-objects to support --blob-max-bytes, packing only
blobs not exceeding that size unless the blob corresponds to a file
whose name starts w
On Thu, Jun 01, 2017 at 12:23:25PM -0700, Stefan Beller wrote:
> On Wed, May 31, 2017 at 2:43 PM, Brandon Williams wrote:
> > Under some circumstances (bogus GIT_DIR value or the discovered gitdir
> > is '.git') 'setup_git_directory()' won't initialize key repository
> > state. This leads to inc
Currently, in pack_objects, add_object_entry() distinguishes between 2
types of non-preferred-base objects:
(1) objects that should not be in "to_pack" because an option like
--local or --honor-pack-keep is set
(2) objects that should be in "to_pack"
A subsequent commit will teach pack-obj
Here's a new version addressing Junio's comments.
> Hmph, that statement is a hard to read and agree to. I thought an
> ignored object that is not going to be packed is one that won't hit
> to_pack?
That is true currently, but will not be the full truth once the 2nd
patch is applied. I have exp
On Fri, Jun 02, 2017 at 07:25:43PM +0200, René Scharfe wrote:
> Am 02.06.2017 um 05:08 schrieb Jeff King:
> > In theory the solution is:
> >
> >1. Start using localtime() instead of gmtime() with an adjustment when
> > we are converting to the local timezone (i.e., format-local). We
> >
On Fri, Jun 2, 2017 at 11:49 AM, Martin Ågren wrote:
> On 2 June 2017 at 10:51, Ævar Arnfjörð Bjarmason wrote:
>> On Fri, Jun 2, 2017 at 2:15 AM, Junio C Hamano wrote:
>>> Martin Ågren writes:
>>>
I looked into this some more. It turns out it is possible to trigger
undefined behavior
Since 3ccd681c2a (Merge branch 'sb/submodule-rm-absorb', 2017-01-18)
git-rm tries to absorb any submodules git dir before deleting the
submodule. Correct the documentation to say so.
Signed-off-by: Stefan Beller
---
Documentation/git-rm.txt | 9 +
1 file changed, 5 insertions(+), 4 delet
On 05/31, brian m. carlson wrote:
> On Tue, May 30, 2017 at 10:30:36AM -0700, Brandon Williams wrote:
> > A month or so ago I thought I would lend a hand to Brian and do a round of
> > conversions from sha1 -> struct object_id. Now that Brian's latest series
> > has
> > hit master I can finally s
While at it, the first one fixes a minor bug, which allowed e.g. 'git
log --no-min-parents-foobarbaz' to succeed.
The other two are fairly straightforward starts_with() ->
skip_prefix() conversions.
SZEDER Gábor (3):
revision.c: stricter parsing of '--no-{min,max}-parents'
revision.c: use ski
These two options are parsed using starts_with(), allowing things like
'git log --no-min-parents-foobarbaz' to succeed.
Use strcmp() instead.
Signed-off-by: SZEDER Gábor
---
revision.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/revision.c b/revision.c
index f88c14ba
Instead of starts_with() and a bunch of magic numbers.
Signed-off-by: SZEDER Gábor
---
revision.c | 18 +-
1 file changed, 9 insertions(+), 9 deletions(-)
diff --git a/revision.c b/revision.c
index 2b64b7e0e..ab0279572 100644
--- a/revision.c
+++ b/revision.c
@@ -2142,20 +2142,2
Instead of starts_with() and a bunch of magic numbers.
While at it, there is an indentation fix where processing
'--early-output', and a coding style fix where processing
'--show-notes'.
Signed-off-by: SZEDER Gábor
---
revision.c | 54 --
1 fi
On 06/02, Junio C Hamano wrote:
> Brandon Williams writes:
>
> > Signed-off-by: Brandon Williams
> > ---
> > notes-merge.c | 20 ++--
> > 1 file changed, 10 insertions(+), 10 deletions(-)
> >
> > diff --git a/notes-merge.c b/notes-merge.c
> > index 55dbb3659..962e9b1bc 100644
>
On 05/31, brian m. carlson wrote:
> On Tue, May 30, 2017 at 10:30:58AM -0700, Brandon Williams wrote:
> > @@ -596,47 +596,47 @@ int notes_merge(struct notes_merge_options *o,
> > /* Find merge bases */
> > bases = get_merge_bases(local, remote);
> > if (!bases) {
> > - base_sh
On 06/02, Junio C Hamano wrote:
> Junio C Hamano writes:
>
> > Junio C Hamano writes:
> >
> >> Brandon Williams writes:
> >>
> >>> A month or so ago I thought I would lend a hand to Brian and do a round of
> >>> conversions from sha1 -> struct object_id. Now that Brian's latest
> >>> series h
Jonathan Nieder wrote:
> Here's a rough list of some useful tasks, in no particular order:
>
> 1. bc/object-id: This patch series continues, eliminating assumptions
>about the size of object ids by encapsulating them in a struct.
>One straightforward way to find code that still needs to be
Hi Dscho,
Johannes Schindelin wrote:
> On Thu, 1 Jun 2017, Stefan Beller wrote:
>> We had a discussion off list how much of the test suite is in bad shape,
>> and "$ git grep ^index" points out a lot of places as well.
>
> Maybe we should call out a specific month (or even a longer period) during
Am 02.06.2017 um 05:08 schrieb Jeff King:
In theory the solution is:
1. Start using localtime() instead of gmtime() with an adjustment when
we are converting to the local timezone (i.e., format-local). We
should be able to do this portably.
This is easy to do, and it's bett
Hi all,
I have noticed a strange behaviour when using git-lfs.
If I understood correctly, git-lfs adds patterns to .gitattributes, for
which git then calls lfs for any matches with this pattern.
The problem is, the documentation[1] says: "The rules how the pattern
matches paths are the same
On Fri, Jun 2, 2017 at 4:46 PM, Liam R. Howlett wrote:
> * ?var Arnfj?r? Bjarmason [170602 04:53]:
>> On Fri, Jun 2, 2017 at 2:15 AM, Junio C Hamano wrote:
>> > Martin Ågren writes:
>> >
>> >> I looked into this some more. It turns out it is possible to trigger
>> >> undefined behavior on "next
On Fri, Jun 2, 2017 at 6:10 PM, Johannes Schindelin
wrote:
> Hi,
>
> On Fri, 2 Jun 2017, Junio C Hamano wrote:
>
>> Ævar Arnfjörð Bjarmason writes:
>>
>> > See <20170525200528.22037-1-ava...@gmail.com> for v3
>> > (https://public-inbox.org/git/20170525200528.22037-1-ava...@gmail.com/).
>> >
>> >
On Fri, Jun 2, 2017 at 3:41 PM, Philipp Gortan wrote:
> Hi git devs,
>
> First off, thanks for your awesome work!
>
> I've been unhappy for quite a while that I had to configure the hooks
> manually for each of my repos - until I found out recently that there is
> the core.hooksPath config variabl
Hi,
On Fri, 2 Jun 2017, Junio C Hamano wrote:
> Ævar Arnfjörð Bjarmason writes:
>
> > See <20170525200528.22037-1-ava...@gmail.com> for v3
> > (https://public-inbox.org/git/20170525200528.22037-1-ava...@gmail.com/).
> >
> > This is on top of "[PATCH v4 00/31] Easy to review grep & pre-PCRE
> >
* ?var Arnfj?r? Bjarmason [170602 04:53]:
> On Fri, Jun 2, 2017 at 2:15 AM, Junio C Hamano wrote:
> > Martin Ågren writes:
> >
> >> I looked into this some more. It turns out it is possible to trigger
> >> undefined behavior on "next". Here's what I did:
> >> ...
> >>
> >> This "fixes" the probl
On Fri, Jun 2, 2017 at 9:41 AM, Philipp Gortan wrote:
> Hi git devs,
>
> First off, thanks for your awesome work!
>
> I've been unhappy for quite a while that I had to configure the hooks
> manually for each of my repos - until I found out recently that there is
> the core.hooksPath config variabl
Hi git devs,
First off, thanks for your awesome work!
I've been unhappy for quite a while that I had to configure the hooks
manually for each of my repos - until I found out recently that there is
the core.hooksPath config variable that (when set globally) allows me to
specify a hooks directory t
Hi,
On Thu, 1 Jun 2017, Stefan Beller wrote:
> On Thu, Jun 1, 2017 at 4:40 PM, Junio C Hamano wrote:
> > Johannes Schindelin writes:
> >
> >> Also, about the commit IDs. As long as the tests are consistent (i.e. they
> >> use test_commit rather than plain `git commit`, or at least call
> >> `te
According to the documentation about git-submodule foreach subcommand's
$path variable:
$path is the name of the submodule directory relative to the superproject
But it was observed when the value of the $path value deviates from this
for the nested submodules when the is run from a subdirectory.
This aims to make git-submodule foreach a builtin. This is the very
first step taken in this direction. Hence, 'foreach' is ported to
submodule--helper, and submodule--helper is called from git-submodule.sh.
The code is split up to have one function to obtain all the list of
submodules. This functi
When the tested repo has an index.lock file it should be removed. This
file may be present if e.g. git-status previously crashed in that
repo, and it will make a lot of git commands fail. Let's try harder
and remove the lock.
Signed-off-by: Ævar Arnfjörð Bjarmason
---
t/perf/perf-lib.sh | 9
Add a performance test for the new core.fsmonitor facility using the
sample query-fsmonitor hook.
This is WIP code for the reasons explained in the setup comments,
unfortunately the perf code doesn't easily allow you to run different
setup code for different versions you're testing. This test will
Hi Junio,
On Fri, 2 Jun 2017, Junio C Hamano wrote:
> Samuel Lijin writes:
>
> >> What is holding this topic up? Anything Ben or I can do to move this
> >> closer to `next` or even `master`?
> >
> > It's in `next` right now (3196d093d6).
>
> Thanks for pinging and checking ;-)
>
> I think t
On 2 June 2017 at 10:51, Ævar Arnfjörð Bjarmason wrote:
> On Fri, Jun 2, 2017 at 2:15 AM, Junio C Hamano wrote:
>> Martin Ågren writes:
>>
>>> I looked into this some more. It turns out it is possible to trigger
>>> undefined behavior on "next". Here's what I did:
>>> ...
>>>
>>> This "fixes" th
On Fri, Jun 2, 2017 at 2:15 AM, Junio C Hamano wrote:
> Martin Ågren writes:
>
>> I looked into this some more. It turns out it is possible to trigger
>> undefined behavior on "next". Here's what I did:
>> ...
>>
>> This "fixes" the problem:
>> ...
>> diff --git a/sha1dc/sha1.c b/sha1dc/sha1.c
>>
On Fri, Jun 2, 2017 at 9:15 AM, Junio C Hamano wrote:
> Jeffrey Walton writes:
>
>> Is there no switch? Its the most efficient way to accomplish the task.
>
> This is a safety to help normal human users from hurting themselves,
> and it does not make any sense to have "I have no name, so record
>
Junio C Hamano writes:
> Junio C Hamano writes:
>
>> Brandon Williams writes:
>>
>>> A month or so ago I thought I would lend a hand to Brian and do a round of
>>> conversions from sha1 -> struct object_id. Now that Brian's latest series
>>> has
>>> hit master I can finally send these patches
Jeffrey Walton writes:
> Is there no switch? Its the most efficient way to accomplish the task.
This is a safety to help normal human users from hurting themselves,
and it does not make any sense to have "I have no name, so record
garbage, please" option, switch or setting that is different from
On Fri, Jun 2, 2017 at 3:00 AM, Konstantin Khomoutov
wrote:
> On Fri, Jun 02, 2017 at 02:02:22AM -0400, Jeffrey Walton wrote:
>
>> I'm working on a test machine. It mostly needs to be a clone of
>> upstream. On occasion it needs to test a particular commit.
>>
>> When I attempt to test a commit it
On Fri, Jun 02, 2017 at 02:02:22AM -0400, Jeffrey Walton wrote:
> I'm working on a test machine. It mostly needs to be a clone of
> upstream. On occasion it needs to test a particular commit.
>
> When I attempt to test a commit it produces:
>
> $ git cherry-pick eb3b27a6a543
>
> *** Ple
84 matches
Mail list logo