On 09 May 2016, at 08:19, Johannes Schindelin
wrote:
> To test that extra HTTP headers are passed correctly, t5551 verifies that
> a fetch succeeds when two required headers are passed, and that the fetch
> does not succeed when those headers are not passed.
>
> However, this test would also s
On Mon, May 09, 2016 at 08:18:52AM +0200, Johannes Schindelin wrote:
> +# Apache 2.2 does not understand , so we use RewriteCond.
> +# And as RewriteCond unfortunately lacks "not equal" matching, we use this
> +# ugly trick to fail *unless* the two headers are present.
> +RewriteCond %{HTTP:x-magi
On Mon, May 09, 2016 at 08:19:00AM +0200, Johannes Schindelin wrote:
> To test that extra HTTP headers are passed correctly, t5551 verifies that
> a fetch succeeds when two required headers are passed, and that the fetch
> does not succeed when those headers are not passed.
>
> However, this test
Hi Peff,
do you sleep at all?
On Mon, 9 May 2016, Jeff King wrote:
> On Mon, May 09, 2016 at 08:19:00AM +0200, Johannes Schindelin wrote:
>
> > test_expect_success 'custom http headers' '
> > - test_must_fail git fetch "$HTTPD_URL/smart_headers/repo.git" &&
> > + test_must_fail git -c http
On Mon, May 09, 2016 at 10:13:51AM +0200, Johannes Schindelin wrote:
> do you sleep at all?
Actually, I just woke up. Nothing like some Git ML to get the blood
pumping in the morning.
> Yeah, a faulty Apache config will unfortunately *skip* the entire test, as
> httpd refuses to start.
If you c
On Mon, May 09, 2016 at 08:30:51AM +0200, demerphq wrote:
> > - perl -le 'print for $ARGV[0]..$ARGV[1]' -- "$@"
> > + test_seq_counter__=$1
> > + while test "$test_seq_counter__" -le $2
> > + do
> > + echo "$test_seq_counter__"
> > + test_seq_cou
Hi Peff,
On Mon, 9 May 2016, Jeff King wrote:
> On Mon, May 09, 2016 at 08:18:52AM +0200, Johannes Schindelin wrote:
>
> > +# Apache 2.2 does not understand , so we use RewriteCond.
> > +# And as RewriteCond unfortunately lacks "not equal" matching, we use this
> > +# ugly trick to fail *unless*
On Mon, May 09, 2016 at 04:03:48PM +0200, Johannes Schindelin wrote:
> How about this:
>
> As RewriteCond does not allow testing for *non*-matches, we simply
> match the desired case first and let it pass by marking the
> RewriteRule as '[L]' ("last rule, do not process any othe
On Sun, May 08, 2016 at 11:48:31AM -0700, Junio C Hamano wrote:
> Ævar Arnfjörð Bjarmason writes:
>
> > I.e. I really expect --verbose to be a more verbose version of the
> > primary thing a command is doing, which in the case of "commit
> > --amend" is giving me info I need to modify the commit
Hi Pranit,
On Sun, 8 May 2016, Pranit Bauva wrote:
> On Sun, May 8, 2016 at 12:34 PM, Johannes Schindelin
> wrote:
> >
> > On Fri, 6 May 2016, Pranit Bauva wrote:
> >
> >> diff --git a/builtin/bisect--helper.c b/builtin/bisect--helper.c
> >> index 3324229..d8de651 100644
> >> --- a/builtin/bisec
Hi Duy,
On Sun, 8 May 2016, Duy Nguyen wrote:
> On Sun, May 8, 2016 at 1:33 PM, Johannes Schindelin
> wrote:
> > The claim is that this libifies the procedure. But it makes the code
> > really nasty for use as a library: if this is run in a thread (and you
> > know that we are going to have to d
Hi Peff,
On Mon, 9 May 2016, Jeff King wrote:
> On Mon, May 09, 2016 at 04:03:48PM +0200, Johannes Schindelin wrote:
>
> > How about this:
> >
> > As RewriteCond does not allow testing for *non*-matches, we simply
> > match the desired case first and let it pass by marking the
> > R
Hey Johannes,
On Mon, May 9, 2016 at 8:29 PM, Johannes Schindelin
wrote:
> Hi Pranit,
>
> On Sun, 8 May 2016, Pranit Bauva wrote:
>
>> On Sun, May 8, 2016 at 12:34 PM, Johannes Schindelin
>> wrote:
>> >
>> > On Fri, 6 May 2016, Pranit Bauva wrote:
>> >
>> >> diff --git a/builtin/bisect--helper.c
Jeff King writes:
> I guess I do not really consider the template content to be the primary
> thing the command is doing. It is subjective, though. I don't feel
> strongly enough to keep discussing it if other people don't agree.
I just see the primary thing of what "commit -e" does is to help
u
On Mon, May 9, 2016 at 4:33 AM, Jeff King wrote:
> On Mon, May 09, 2016 at 08:30:51AM +0200, demerphq wrote:
>> > - perl -le 'print for $ARGV[0]..$ARGV[1]' -- "$@"
>> > + test_seq_counter__=$1
>> > + while test "$test_seq_counter__" -le $2
>> > + do
>> > + ech
Duy Nguyen writes:
> So among the options we have so far, which way should we go, or leave it as
> is?
Thanks for reminding me.
I like that version you sent with "I may have rushed to judgment"
comment the most. Perhaps I can just queue it with s/PATH/PROG/
fixup?
>
> On Tue, May 3, 2016 at
On Sat, May 07, 2016 at 06:15:19PM +0200, Torsten Bögershausen wrote:
> These tests fail here under Mac OS,
> they pass under Linux:
> commit ff3d9b660a4b6e9d3eeb664ce1febe717adff737
> I haven't had a chance to dig further.
I assume you mean t6302. It looks like the difference is not Mac OS, but
Hi there,
skipping through some failed tests I found more (smaller) problems
inside the test... when test arguments are empty they need to be
quoted (quite a lot test in this sentence).
Error is like
t4151-am-abort.sh[5]: test: argument expected
My patch:
*** t4151-am-abort.sh Mon May 9 17:5
On Mon, May 09, 2016 at 12:02:45PM -0400, Eric Sunshine wrote:
> > I had that thought, too, but I think it would be an error to do so.
> > test_seq is supposed to be a replacement for "seq", which does not
> > understand non-numeric sequences.
>
> Although, the comment block just above test_seq()
Heiko Voigt writes:
>> > - When upstream adds a new submodule, I have to do the same manual
>> > work to change the options for that new submodule.
>>
>> Because a new module is not automatically "init"ed by default?
>>
>> Isn't "config only" vs "config with gitmodules fallback" orthogonal
>>
On Mon, May 9, 2016 at 12:09 PM, Armin Kunaschik
wrote:
> skipping through some failed tests I found more (smaller) problems
> inside the test... when test arguments are empty they need to be
> quoted (quite a lot test in this sentence).
>
> Error is like
> t4151-am-abort.sh[5]: test: argument exp
Johannes Schindelin writes:
> +# Apache 2.2 does not understand , so we use RewriteCond.
> +# And as RewriteCond unfortunately lacks "not equal" matching, we use this
> +# ugly trick to fail *unless* the two headers are present.
> +RewriteCond %{HTTP:x-magic-one} =abra
> +RewriteCond %{HTTP:x-mag
On Mon, May 9, 2016 at 12:22 PM, Eric Sunshine wrote:
> On Mon, May 9, 2016 at 12:09 PM, Armin Kunaschik
> wrote:
>> *** t4151-am-abort.sh Mon May 9 17:51:44 2016
>> --- t4151-am-abort.sh.orig Fri Apr 29 23:37:00 2016
>> ! test 3 -eq "$(git ls-files -u | wc -l)" &&
>> ! test 3
On Mon, May 9, 2016 at 12:07 PM, Jeff King wrote:
> On Sat, May 07, 2016 at 06:15:19PM +0200, Torsten Bögershausen wrote:
>> These tests fail here under Mac OS,
>> they pass under Linux:
>> commit ff3d9b660a4b6e9d3eeb664ce1febe717adff737
>> I haven't had a chance to dig further.
>
> I assume you m
Sorry, this was my first patch to the list. I'll do better :-)
You are right about the "wc -l" parts. Maybe I was a bit over
pessimistic. Throw away my last mail.
In my case test 9 ran unsuccessful because of an empty "git ls-files -u"
This reduces the diff to this one (hopefully the right way now
Johannes Schindelin writes:
> Okay, I already force-pushed my extra-http-header branch and the next
> iteration will sport this paragraph.
The new explanation is well written and can and should also replace
the comment before the implementation in the configuration file to
help readers.
To be h
On Mon, May 09, 2016 at 12:30:43PM -0400, Eric Sunshine wrote:
> The latter seems very preferable, though perhaps it could be made more
> concise like this?
>
> sign=
> test_have_prereq GPG && sign=-s
>
> (But that's a minor issue.)
I agree that is nicer, but I wanted to keep the defini
Hi,
similar to t3513, in t6041 tar is used with the -z flag which is not portable
and should be removed the same way as in t3513.
Regards,
Armin
--
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 http://v
On Mon, May 09, 2016 at 09:42:32AM -0700, Junio C Hamano wrote:
> > Hopefully your patch to remove the -c ... sanitizing makes it to `master`
> > soon, then I can submit my next iteration.
>
> Or we can just merge that "do not sanitize" branch in, and then
> queue the "next iteration" which I'd a
Johannes Schindelin writes:
> This is a heavily version of patches we carried in Git for Windows for
> way too long without submitting them upstream.
>
> In this iteration, I also claim authorship for the patch because by now
> Kusma's changes were so contorted and mutilated beyond recognition by
The test uses the 'z' option, i.e. "compress the output while at
it", which is GNUism and not portable.
Reported-by: Armin Kunaschik
Signed-off-by: Stefan Beller
---
Thanks Armin for reporting these GNUism!
Are there any more? (So we can do these patches as a
series instead of one by one:)
Hi list,
eight t5601 tests don't run with my version of sed.
The reason is a trailing space in the sed expression. See below:
#IPv6
for tuah in ::1 [::1] [::1]: user@::1 user@[::1] user@[::1]:
[user@::1] [user@::1]:
do
ehost=$(echo $tuah | sed -e "s/1]:/1]/ "| tr -d "[]")
test_exp
Hi Stefan,
I'm currently in the process of skipping through the failed tests on my AIX box.
There are more tests which require GNU tools like mktemp
(t7610-mergetool.sh) or readlink (t7800-difftool.sh).
But I don't have a solution or workaround for these two.
But at least there are not more failin
Johannes Schindelin writes:
> +core.hideDotFiles::
> + (Windows-only) If true, mark newly-created directories and files whose
> + name starts with a dot as hidden. If 'dotGitOnly', only the `.git/`
> + directory is hidden, but no other files starting with a dot. The
> + default
>> Here is what I imagine
>> When B mirrors from A, B sets up this special ref for its repository,
>> e.g. refs/meta/submodule-B and have a symbolic ref pointing at that.
>> (e.g. SUBMODULE_CONFIG pointing at refs/meta/submodule-B,
>> which has a worktree which contains a .gitmodules files which
>>
On 7 May 2016 at 08:07, Christian Couder wrote:
>
> On Fri, May 6, 2016 at 5:31 PM, Thierry Suzanne
> wrote:
> > Sorry, can't help :(
>
> I just sent a patch. It would be nice if you could test it to confirm
> that you get a nicer error message.
>
> Also please reply below other people's comment
Duy Nguyen writes:
> I vote one step at a time, leave multi-thread support for future.
> There's a lot more shared state than file descriptors anyway, at least
> there are object db and index access and probably a couple of hidden
> static variables somewhere. And I'm not sure if multi-thread rea
On Mon, May 9, 2016 at 12:49 PM, Jeff King wrote:
> On Mon, May 09, 2016 at 12:30:43PM -0400, Eric Sunshine wrote:
> Since you as the author of 618310a seem to agree with this direction,
> here it is as a real patch.
Thanks for working on this.
> Subject: [PATCH] t6302: simplify non-gpg cases
>
Jeff King writes:
> On Mon, May 09, 2016 at 09:42:32AM -0700, Junio C Hamano wrote:
>
>> > Hopefully your patch to remove the -c ... sanitizing makes it to `master`
>> > soon, then I can submit my next iteration.
>>
>> Or we can just merge that "do not sanitize" branch in, and then
>> queue the
From: Torsten Bögershausen
The sed expression for IPv6, "Tested User And Host" or "tuah" used a wrong
sed expression, which doesn't work under all versions of sed.
Reported-By: Armin Kunaschik
Signed-off-by: Torsten Bögershausen
---
t/t5601-clone.sh | 2 +-
1 file changed, 1 insertion(+), 1 d
Hello,
I stuck around the deadlock inside git when running git upload-pack .
command. A debugging shown that the bottom process (it starts several
processes to implement the task) hangs writing to stderr. I managed to
reproduce the issue with a tiny bash script. The repository and the
script is fo
On Mon, May 9, 2016 at 10:49 AM, Eugene Petrenko
wrote:
> Hello,
>
> I stuck around the deadlock inside git when running git upload-pack .
> command. A debugging shown that the bottom process (it starts several
> processes to implement the task) hangs writing to stderr. I managed to
> reproduce th
On Mon, May 9, 2016 at 10:53 AM, Stefan Beller wrote:
> On Mon, May 9, 2016 at 10:49 AM, Eugene Petrenko
> wrote:
>> Hello,
>>
>> I stuck around the deadlock inside git when running git upload-pack .
>> command. A debugging shown that the bottom process (it starts several
>> processes to implemen
[please don't top-post on this list]
On Mon, May 9, 2016 at 12:35 PM, Armin Kunaschik
wrote:
> Sorry, this was my first patch to the list. I'll do better :-)
> You are right about the "wc -l" parts. Maybe I was a bit over
> pessimistic. Throw away my last mail.
Done :-)
> In my case test 9 ran
I'm currently concentrating on finding problems with my setup... this
is already a tough job :-)
I'm a git beginner, and Documentation/SubmittingPatches would keep me
busy for a week.
So anybody feel free to submit this thingy.
Armin
--
To unsubscribe from this list: send the line "unsubscribe git
Torsten Bögershausen writes:
> On 08.05.16 20:20, Junio C Hamano wrote:
>> Torsten Bögershausen writes:
>>
>>> May a simple
>>> printf "1\n2\n3\n4\n5\n6\n7\n8\n9\n10\n"
>>>
>>> be an option ?
>> If you were to do that, at least have the decency to make it more
>> readable by doing something li
Jeff King writes:
> On Mon, May 09, 2016 at 12:02:45PM -0400, Eric Sunshine wrote:
>
>> > I had that thought, too, but I think it would be an error to do so.
>> > test_seq is supposed to be a replacement for "seq", which does not
>> > understand non-numeric sequences.
>>
>> Although, the comment
Junio C Hamano writes:
> Yes, I think the comment should just go. Nobody used that alphabet
> form since it was written in d17cf5f3 (tests: Introduce test_seq,
> 2012-08-04).
>
>> I don't really care either way whether it is replaced or not (at one
>> point there were some people really interest
tbo...@web.de writes:
> From: Torsten Bögershausen
>
> The sed expression for IPv6, "Tested User And Host" or "tuah" used a wrong
> sed expression, which doesn't work under all versions of sed.
>
> Reported-By: Armin Kunaschik
> Signed-off-by: Torsten Bögershausen
> ---
Good. Thanks.
> t/t5
Stefan Beller writes:
> diff --git a/t/t6041-bisect-submodule.sh b/t/t6041-bisect-submodule.sh
> index c6b7aa6..62b8a2e 100755
> --- a/t/t6041-bisect-submodule.sh
> +++ b/t/t6041-bisect-submodule.sh
> @@ -8,7 +8,7 @@ test_description='bisect can handle submodules'
> git_bisect () {
> git s
Something like this follows Documentation/SubmittingPatches, except
that it further needs your Sign-off before mine, which I can forge
if you say it is OK.
Thanks for a report and an analysis of the issue.
-- >8 --
From: Armin Kunaschik
Subject: t4151: make sure argument to 'test -z' is given
8
+ cc Jens as he authored both t6041 as well as t3513 in
the series leading to ad25da009e2a3730 (2014-07-21,
Merge branch 'jl/submodule-tests')
On Mon, May 9, 2016 at 11:46 AM, Junio C Hamano wrote:
> Stefan Beller writes:
>
>> diff --git a/t/t6041-bisect-submodule.sh b/t/t6041-bisect-submodule.s
On Mon, May 9, 2016 at 11:56 AM, Junio C Hamano wrote:
> Something like this follows Documentation/SubmittingPatches, except
> that it further needs your Sign-off before mine, which I can forge
> if you say it is OK.
The sign-off is a simple line at the end of the explanation for
the patch, which
From: "Stefan Beller"
On Fri, May 6, 2016 at 12:57 PM, Junio C Hamano wrote:
Marc Branchaud writes:
On 2016-05-06 02:54 PM, Junio C Hamano wrote:
I wonder if can we come up with a short and sweet notation to remind
futhre readers that this "initialization" is not initializing but
merely s
On May 9, 2016 3:40 PM Philip Oakley wrote:
> From: "Stefan Beller"
> > On Fri, May 6, 2016 at 12:57 PM, Junio C Hamano
> wrote:
> >> Marc Branchaud writes:
> >>
> >>> On 2016-05-06 02:54 PM, Junio C Hamano wrote:
>
> I wonder if can we come up with a short and sweet notation to
>
tbo...@web.de writes:
> +#define get_sha1_from_cache(path) get_sha1_from_index (&the_index, (path))
> #endif
Micronit: lose the extra SP; i.e. "get_sha1_from_index(&the_index, (path))".
> diff --git a/read-cache.c b/read-cache.c
> index d9fb78b..a3ef967 100644
> --- a/read-cache.c
> +++ b/read
On Mon, May 9, 2016 at 2:36 PM, Junio C Hamano wrote:
> Subject: test-lib-functions.sh: remove misleading comment on test_seq
>
> We never used the "letters" form since we came up with "test_seq" to
> replace use of non-portable "seq" in our test script, which we
> introduced it at d17cf5f3 (tests
On Fri, 2016-05-06 at 18:13 +0200, Michael Haggerty wrote:
> Thanks to David, Junio, and Peff for their comments on v1 of this
> patch series [1]. I think I have addressed all of the points that
> were
> brought up. Plus I fixed a pre-existing bug that I noticed myself
> while adding some more test
On Mon, May 9, 2016 at 2:56 PM, Junio C Hamano wrote:
> Something like this follows Documentation/SubmittingPatches [...]
>
> -- >8 --
> From: Armin Kunaschik
> Subject: t4151: make sure argument to 'test -z' is given
>
> 88d50724 (am --skip: revert changes introduced by failed 3way merge,
> 2015
tbo...@web.de writes:
> + if (stats->stat_bits & earlyout)
> + break; /* We found what we have been searching for */
Are we sure if our callers are only interested in just one bit at a
time? Otherwise, if we want to ensure all of the given bits are
set,
i
Eric Sunshine writes:
> On Mon, May 9, 2016 at 2:56 PM, Junio C Hamano wrote:
>> Something like this follows Documentation/SubmittingPatches [...]
>>
>> -- >8 --
>> From: Armin Kunaschik
>> Subject: t4151: make sure argument to 'test -z' is given
>>
>> 88d50724 (am --skip: revert changes introd
From: Nguyễn Thái Ngọc Duy
This allows signal handlers and atexit functions to realize this
situation and not clean up.
Signed-off-by: Nguyễn Thái Ngọc Duy
Signed-off-by: David Turner
---
builtin/gc.c | 2 +-
cache.h | 2 +-
daemon.c | 2 +-
setup.c | 4 +++-
4 files changed, 6
From: Nguyễn Thái Ngọc Duy
We detach after creating and opening the socket, because otherwise
we might return control to the shell before index-helper is ready to
accept commands. This might lead to flaky tests.
Signed-off-by: Nguyễn Thái Ngọc Duy
Signed-off-by: David Turner
---
Documentatio
Make git checkout (and other unpack_tree operations) preserve the
untracked cache and watchman status. This is valuable for two reasons:
1. Often, an unpack_tree operation will not touch large parts of the
working tree, and thus most of the untracked cache will continue to be
valid.
2. Even if th
Add a new command (and command-line arg) to allow index-helpers to
exit cleanly.
This is mainly useful for tests.
Signed-off-by: David Turner
---
Documentation/git-index-helper.txt | 3 +++
index-helper.c | 31 ++-
t/t7900-index-helper.sh
From: Nguyễn Thái Ngọc Duy
Signed-off-by: Nguyễn Thái Ngọc Duy
Signed-off-by: David Turner
---
read-cache.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/read-cache.c b/read-cache.c
index d9fb78b..16cc487 100644
--- a/read-cache.c
+++ b/read-cache.c
@@ -1345,7 +1345,7 @@
Formatting and patch naming fixes from Junio. No substantive changes.
David Turner (8):
index-helper: log warnings
unpack-trees: preserve index extensions
watchman: add a config option to enable the extension
index-helper: kill mode
index-helper: don't run if already running
index-hel
From: Nguyễn Thái Ngọc Duy
Later, we will introduce git index-helper to share this memory with
other git processes.
We only unmap it when we discard the index (although the kernel may of
course choose to page it out).
Signed-off-by: Nguyễn Thái Ngọc Duy
Signed-off-by: David Turner
---
cache.
From: Nguyễn Thái Ngọc Duy
The extension contains a bitmap, one bit for each entry in the
index. If the n-th bit is zero, the n-th entry is considered
unchanged, we can ce_mark_uptodate() it without refreshing. If the bit
is non-zero and we found out the corresponding file is clean after
refresh,
From: Nguyễn Thái Ngọc Duy
The previous patch has the logic to clear bits in 'WAMA' bitmap. This
patch has logic to set bits as told by watchman. The missing bit,
_using_ these bits, are not here yet.
A lot of this code is written by David Turner originally, mostly from
[1]. I'm just copying and
From: Nguyễn Thái Ngọc Duy
Signed-off-by: Nguyễn Thái Ngọc Duy
Signed-off-by: David Turner
---
Documentation/git-index-helper.txt | 3 +++
Documentation/git-update-index.txt | 6 ++
builtin/update-index.c | 16
3 files changed, 25 insertions(+)
diff --git a/
For installations that have centrally-managed configuration, it's
easier to set a config once than to run update-index on every
repository.
Signed-off-by: David Turner
---
.gitignore| 1 +
Documentation/config.txt | 4
Makefile | 1 +
read-ca
From: Nguyễn Thái Ngọc Duy
Watchman is hidden behind index-helper. Before git tries to read the
index from shm, it notifies index-helper through the socket and waits
for index-helper to prepare a file for sharing memory (with
MAP_SHARED). index-helper then contacts watchman, updates 'WAMA'
extens
Soon, we'll want to automatically start index-helper, so we need
a mode that silently exits if it can't start up (either because
it's not in a git dir, or because another one is already running).
Signed-off-by: David Turner
---
Documentation/git-index-helper.txt | 4
index-helper.c
From: Nguyễn Thái Ngọc Duy
Instead of reading the index from disk and worrying about disk
corruption, the index is cached in memory (memory bit-flips happen
too, but hopefully less often). The result is faster read. Read time
is reduced by 70%.
The biggest gain is not having to verify the traili
Instead of writing warnings to stderr, write them to a log. Later, we'll
probably be daemonized, so writing to stderr will be pointless.
Signed-off-by: David Turner
---
Documentation/git-index-helper.txt | 3 +++
index-helper.c | 32
2 files
Add a config option to populate the untracked cache.
For installations that have centrally-managed configuration, it's
easier to set a config once than to run update-index on every
repository.
Signed-off-by: David Turner
---
Documentation/config.txt | 4
read-cache.c | 7 ++
Introduce a new config option, indexhelper.autorun, to automatically
run git index-helper before starting up a builtin git command. This
enables users to keep index-helper running without manual
intervention.
Signed-off-by: David Turner
---
Documentation/config.txt | 4
read-cache.c
Signed-off-by: David Turner
---
index-helper.c | 7 +++
t/t7900-index-helper.sh | 9 +
2 files changed, 16 insertions(+)
diff --git a/index-helper.c b/index-helper.c
index 4ed1610..b275f6e 100644
--- a/index-helper.c
+++ b/index-helper.c
@@ -458,6 +458,13 @@ int main(int arg
From: Nguyễn Thái Ngọc Duy
All the known heavy code blocks are measured (except object database
access). This should help identify if an optimization is effective or
not. An unoptimized git-status would give something like below (92% of
time is accounted). To sum up the effort of making git scale
From: Nguyễn Thái Ngọc Duy
There are "holes" in the index-helper approach because the shared
memory is not verified again by git. If $USER is compromised, shared
memory could be modified. But anyone who could do this could already
modify $GIT_DIR/index. A more realistic risk is some bugs in
index
David Turner writes:
> On Fri, 2016-05-06 at 18:13 +0200, Michael Haggerty wrote:
>> Thanks to David, Junio, and Peff for their comments on v1 of this
>> patch series [1]. I think I have addressed all of the points that
>> were
>> brought up. Plus I fixed a pre-existing bug that I noticed myself
Junio C Hamano writes:
> Stop promising that we may allow using "letters"; this would open an
> easier reimplementation that does not rely on $PERL, if somebody
> later wants to.
>
> Signed-off-by: Junio C Hamano
And I am not the one who particularly wants to, but here is the
previous patch sen
On Mon, May 9, 2016 at 5:08 PM, Junio C Hamano wrote:
> Subject: [PATCH] test-lib-functions.sh: rewrite test_seq without Perl
>
> Rewrite the 'seq' imitation only with commands and features
> that are typically found as built-in in modern POSIX shells,
> instead of relying on Perl to run a single-
On Mon, May 9, 2016 at 4:45 PM, Junio C Hamano wrote:
> Eric Sunshine writes:
>> On Mon, May 9, 2016 at 2:56 PM, Junio C Hamano wrote:
>>> Something like this follows Documentation/SubmittingPatches [...]
>>>
>>> -- >8 --
>>> From: Armin Kunaschik
>>> Subject: t4151: make sure argument to 'test
On Mon, May 9, 2016 at 4:24 PM, Karthik Nayak wrote:
> On Mon, May 9, 2016 at 11:17 PM, Eric Sunshine
> wrote:
>> Should we cite bc9acea (ref-filter: implement %(if), %(then), and
>> %(else) atoms, 2016-04-25) here as an example of a commit for which
>> this was problematic (and which indeed brok
Hmmm, I seem to be getting
$ cat t/trash*7900*/err
fatal: Already running
after running t7900 and it fails at #5, after applying
"index-helper: optionally automatically run".
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.ker
On 05/09/2016 11:05 PM, Junio C Hamano wrote:
> David Turner writes:
>> [...]
>> I generally like to put the bug fixes before the tests for those fixes
>> (so that bisect on the complete suite works). But maybe the git policy
>> is different.
>
> The Git policy only asks not to break bisection.
Michael Haggerty writes:
>> The "demonstrate an existing breakage first" order makes it slightly
>> easier to review and follow a long series, as it forces the reviewer
>> to see the issue first and think about possible avenues to solve it
>> for themselves, before seeing a paticular solution. F
On Mon, 2016-05-09 at 14:40 -0700, Junio C Hamano wrote:
> Hmmm, I seem to be getting
>
> $ cat t/trash*7900*/err
> fatal: Already running
>
> after running t7900 and it fails at #5, after applying
> "index-helper: optionally automatically run"
It still passes for me (with or without USE
David Turner writes:
> On Mon, 2016-05-09 at 14:40 -0700, Junio C Hamano wrote:
>> Hmmm, I seem to be getting
>>
>> $ cat t/trash*7900*/err
>> fatal: Already running
>>
>> after running t7900 and it fails at #5, after applying
>> "index-helper: optionally automatically run"
>
> It still
Junio C Hamano writes:
> David Turner writes:
>
>> On Mon, 2016-05-09 at 14:40 -0700, Junio C Hamano wrote:
>>> Hmmm, I seem to be getting
>>>
>>> $ cat t/trash*7900*/err
>>> fatal: Already running
>>>
>>> after running t7900 and it fails at #5, after applying
>>> "index-helper: option
Here are the topics that have been cooking. Commits prefixed with
'-' are only in 'pu' (proposed updates) while commits prefixed with
'+' are in 'next'. The ones marked with '.' do not appear in any of
the integration branches, but I am still holding onto them.
The 'master' branch now has the te
On Mon, 2016-05-09 at 15:32 -0700, Junio C Hamano wrote:
> Junio C Hamano writes:
>
> > David Turner writes:
> >
> > > On Mon, 2016-05-09 at 14:40 -0700, Junio C Hamano wrote:
> > > > Hmmm, I seem to be getting
> > > >
> > > > $ cat t/trash*7900*/err
> > > > fatal: Already running
> >
On 10/05/16 00:12, David Turner wrote:
> On Mon, 2016-05-09 at 15:32 -0700, Junio C Hamano wrote:
>> Junio C Hamano writes:
>>
>>> David Turner writes:
>>>
On Mon, 2016-05-09 at 14:40 -0700, Junio C Hamano wrote:
> Hmmm, I seem to be getting
>
> $ cat t/trash*7900*/err
[+cc Junio as this should be the final version]
On Mon, May 09, 2016 at 01:47:35PM -0400, Eric Sunshine wrote:
> > Subject: [PATCH] t6302: simplify non-gpg cases
> >
> > When commit 618310a taught t6302 to run without the GPG
>
> 618310a (t6302: skip only signed tags rather than all tests when G
On Mon, May 09, 2016 at 11:36:09AM -0700, Junio C Hamano wrote:
> Junio C Hamano writes:
>
> > Yes, I think the comment should just go. Nobody used that alphabet
> > form since it was written in d17cf5f3 (tests: Introduce test_seq,
> > 2012-08-04).
> >
> >> I don't really care either way whethe
On Mon, May 9, 2016 at 10:40 PM, Jeff King wrote:
> On Mon, May 09, 2016 at 01:47:35PM -0400, Eric Sunshine wrote:
>> > - test_prepare_expect <<-\EOF | sed -e "s/Z$//" >expect &&
>> > + cat <<-\EOF | sed -e "s/Z$//" >expect &&
>>
>> To make this as close to a reversion as possible, thi
The final batch of git-rev-parse tests work against a non-local object
database named ../repo.git rather than the typically-named ../.git. It
prepares by renaming .git/ to repo.git/ and pointing GIT_DIR at
../repo.git, but never restores the name to .git/, which can be
problematic for tests added i
Tests run by test_rev_parse() are nearly identical; each invokes
git-rev-parse with a single option and compares the result against an
expected value. Such duplication makes it onerous to extend the tests
since any change needs to be repeated in each test. Reduce the
duplication by parameterizing t
1 - 100 of 115 matches
Mail list logo
