Signed-off-by: Steffen Prohaska <proha...@zib.de>
---
Documentation/gitrepository-layout.txt | 4
1 file changed, 4 insertions(+)
diff --git a/Documentation/gitrepository-layout.txt
b/Documentation/gitrepository-layout.txt
index adf9554ad2..c60bcad44a 100644
--- a/Documen
f7b7a7e05eab7df0ef89dec65
author Steffen Prohaska <proha...@zib.de> 1488122961 +0100
committer Steffen Prohaska <proha...@zib.de> 1488123452 +0100
gpgsig ...
```
GPG signatures would automatically cover the supplementary hashes.
Verification code paths would have to be added to compute the has
List items must be continued with '+' (see [asciidoc]).
[asciidoc] AsciiDoc user guide 17.7. List Item Continuation
http://www.methods.co.nz/asciidoc/userguide.html#X15
Signed-off-by: Steffen Prohaska proha...@zib.de
---
contrib/subtree/git-subtree.txt | 194
, index_stream_convert_blob() is able to handle a large file
if a filter is configured that returns a small result.
Signed-off-by: Steffen Prohaska proha...@zib.de
---
This patch should be applied on top of sp/stream-clean-filter.
index_stream() might internally also be able to handle large files to
some
The single-parameter form is described as the preferred way. Separate
arguments are only supported for backward compatibility. Update the
example to the recommended form.
Signed-off-by: Steffen Prohaska proha...@zib.de
---
Documentation/git-update-index.txt | 2 +-
1 file changed, 1 insertion
Hi,
Is there a way to update the stat information recorded in the index without
reading the file content from disk?
Starting from a clean working copy with a committed `file`, I'd like
touch file
git magic-command file
to bring the index into essentially the same state as
touch
On Aug 27, 2014, at 4:47 PM, Junio C Hamano gits...@pobox.com wrote:
Jeff King p...@peff.net writes:
On Tue, Aug 26, 2014 at 02:54:11PM -0700, Junio C Hamano wrote:
A worse position is to have git_env_bool() that says empty is
false and add a new git_env_ulong() that says empty is unset.
On Aug 26, 2014, at 8:29 PM, Jeff King p...@peff.net wrote:
On Tue, Aug 26, 2014 at 05:23:24PM +0200, Steffen Prohaska wrote:
The caller opened the fd, so it should be responsible for closing it.
Signed-off-by: Steffen Prohaska proha...@zib.de
---
copy.c | 5 +
lockfile.c | 3
The new function will be used to parse GIT_MMAP_LIMIT and
GIT_ALLOC_LIMIT.
Signed-off-by: Steffen Prohaska proha...@zib.de
---
cache.h | 1 +
config.c | 11 +++
2 files changed, 12 insertions(+)
diff --git a/cache.h b/cache.h
index fcb511d..b820b6a 100644
--- a/cache.h
+++ b/cache.h
The caller opened the fd, so it should be responsible for closing it.
Signed-off-by: Steffen Prohaska proha...@zib.de
---
copy.c | 5 +
lockfile.c | 3 +++
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/copy.c b/copy.c
index a7f58fd..d0a1d82 100644
--- a/copy.c
+++ b
It is only the path that matters in the decision whether to filter or
not. Clarify this by making path the single argument of
would_convert_to_git().
Signed-off-by: Steffen Prohaska proha...@zib.de
---
convert.h | 5 ++---
sha1_file.c | 2 +-
2 files changed, 3 insertions(+), 4 deletions
more large
blob test cases
Signed-off-by: Steffen Prohaska proha...@zib.de
---
sha1_file.c | 18 +-
1 file changed, 17 insertions(+), 1 deletion(-)
diff --git a/sha1_file.c b/sha1_file.c
index 00c07f2..d9b5157 100644
--- a/sha1_file.c
+++ b/sha1_file.c
@@ -663,10 +663,26
use GIT_ALLOC_LIMIT to test small sizes.
The cast of size in the call to die() is changed to uintmax_t to match
the format string PRIuMAX.
Signed-off-by: Steffen Prohaska proha...@zib.de
---
t/t1050-large.sh | 2 +-
wrapper.c| 15 ---
2 files changed, 9 insertions(+), 8
actually has been modified on its way from the file system
to the object store.
Signed-off-by: Steffen Prohaska proha...@zib.de
---
convert.c | 55 +--
convert.h | 5 +
sha1_file.c | 27
Changes since v5:
- Introduce and use git_env_ulong().
- Change copy_fd() to not close input fd, which simplified changes to convert.
- More detailed explanation why filter must be marked required in commit
message of 6/6.
- Style fixes.
Steffen Prohaska (6):
convert: drop arguments
On Aug 25, 2014, at 1:38 PM, Jeff King p...@peff.net wrote:
On Sun, Aug 24, 2014 at 06:07:44PM +0200, Steffen Prohaska wrote:
diff --git a/wrapper.c b/wrapper.c
index bc1bfb8..69d1c9b 100644
--- a/wrapper.c
+++ b/wrapper.c
@@ -11,14 +11,18 @@ static void (*try_to_free_routine)(size_t
On Aug 25, 2014, at 2:43 PM, Jeff King p...@peff.net wrote:
On Sun, Aug 24, 2014 at 06:07:46PM +0200, Steffen Prohaska wrote:
The data is streamed to the filter process anyway. Better avoid mapping
the file if possible. This is especially useful if a clean filter
reduces the size
On Aug 22, 2014, at 6:31 PM, Junio C Hamano gits...@pobox.com wrote:
Steffen Prohaska proha...@zib.de writes:
+ if (limit == -1) {
+ const char *env = getenv(GIT_MMAP_LIMIT);
+ limit = env ? atoi(env) * 1024 : 0;
... this should then be changed to atol(env
use GIT_ALLOC_LIMIT to test small sizes.
The cast of size in the call to die() is changed to uintmax_t to match
the format string PRIuMAX.
Signed-off-by: Steffen Prohaska proha...@zib.de
---
t/t1050-large.sh | 2 +-
wrapper.c| 16 ++--
2 files changed, 11 insertions(+), 7
It is only the path that matters in the decision whether to filter or
not. Clarify this by making path the single argument of
would_convert_to_git().
Signed-off-by: Steffen Prohaska proha...@zib.de
---
convert.h | 5 ++---
sha1_file.c | 2 +-
2 files changed, 3 insertions(+), 4 deletions
to
verify that the data actually has been modified on its way from the file
system to the object store.
Signed-off-by: Steffen Prohaska proha...@zib.de
---
convert.c | 60 +--
convert.h | 5 +
sha1_file.c | 27
Changes since v4: use git_parse_ulong() to parse env vars.
Steffen Prohaska (4):
convert: Refactor would_convert_to_git() to single arg 'path'
Change GIT_ALLOC_LIMIT check to use git_parse_ulong()
Introduce GIT_MMAP_LIMIT to allow testing expected mmap size
convert: Stream from fd
more large
blob test cases
Signed-off-by: Steffen Prohaska proha...@zib.de
---
sha1_file.c | 21 -
1 file changed, 20 insertions(+), 1 deletion(-)
diff --git a/sha1_file.c b/sha1_file.c
index 00c07f2..3204f66 100644
--- a/sha1_file.c
+++ b/sha1_file.c
@@ -663,10 +663,29
On Aug 22, 2014, at 12:26 AM, Junio C Hamano gits...@pobox.com wrote:
Steffen Prohaska proha...@zib.de writes:
Similar to testing expectations about malloc with GIT_ALLOC_LIMIT (see
commit d41489), it can be useful to test expectations about mmap.
This introduces a new environment
string PRIuMAX.
Signed-off-by: Steffen Prohaska proha...@zib.de
---
wrapper.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/wrapper.c b/wrapper.c
index bc1bfb8..e91f6e9 100644
--- a/wrapper.c
+++ b/wrapper.c
@@ -11,14 +11,14 @@ static void (*try_to_free_routine)(size_t
It is only the path that matters in the decision whether to filter or
not. Clarify this by making path the single argument of
would_convert_to_git().
Signed-off-by: Steffen Prohaska proha...@zib.de
---
convert.h | 5 ++---
sha1_file.c | 2 +-
2 files changed, 3 insertions(+), 4 deletions
blob test cases
Signed-off-by: Steffen Prohaska proha...@zib.de
---
sha1_file.c | 17 -
1 file changed, 16 insertions(+), 1 deletion(-)
diff --git a/sha1_file.c b/sha1_file.c
index 00c07f2..603673b 100644
--- a/sha1_file.c
+++ b/sha1_file.c
@@ -663,10 +663,25 @@ void
Only changes since PATCH v3: Use ssize_t to store limits.
Steffen Prohaska (4):
convert: Refactor would_convert_to_git() to single arg 'path'
Introduce GIT_MMAP_LIMIT to allow testing expected mmap size
Change GIT_ALLOC_LIMIT check to use ssize_t for consistency
convert: Stream from fd
to
verify that the data actually has been modified on its way from the file
system to the object store.
Signed-off-by: Steffen Prohaska proha...@zib.de
---
convert.c | 60 +--
convert.h | 5 +
sha1_file.c | 27
It is only the path that matters in the decision whether to filter or
not. Clarify this by making path the single argument of
would_convert_to_git().
Signed-off-by: Steffen Prohaska proha...@zib.de
---
convert.h | 5 ++---
sha1_file.c | 2 +-
2 files changed, 3 insertions(+), 4 deletions
blob test cases
Signed-off-by: Steffen Prohaska proha...@zib.de
---
sha1_file.c | 17 -
1 file changed, 16 insertions(+), 1 deletion(-)
diff --git a/sha1_file.c b/sha1_file.c
index 00c07f2..88d64c0 100644
--- a/sha1_file.c
+++ b/sha1_file.c
@@ -663,10 +663,25 @@ void
to
verify that the data actually has been modified on its way from the file
system to the object store.
Signed-off-by: Steffen Prohaska proha...@zib.de
---
convert.c | 60 +--
convert.h | 5 +
sha1_file.c | 27
I revised the testing approach as discussed. Patch 2/3 adds GIT_MMAP_LIMIT,
which allows testing of memory expectations together with GIT_ALLOC_LIMIT.
The rest is unchanged compared to v2.
Steffen Prohaska (3):
convert: Refactor would_convert_to_git() to single arg 'path'
Introduce
On Aug 19, 2014, at 9:53 AM, Jeff King p...@peff.net wrote:
For files 2GB on a 32-bit system (e.g. msysgit), filtering with the
previous code always failed. Now it works. I created the patch to
change git from 'fundamentally doesn't handle this' to 'works as
expected'.
I deal with
On Aug 16, 2014, at 7:00 PM, Andreas Schwab sch...@linux-m68k.org wrote:
Steffen Prohaska proha...@zib.de writes:
The test should confirm that the the file that is added is not mmapped to
memory.
RSS doesn't tell you that. You can mmap a big file without RSS getting
bigger.
All data
On Aug 17, 2014, at 9:27 AM, Jeff King p...@peff.net wrote:
On Sat, Aug 16, 2014 at 06:26:08PM +0200, Steffen Prohaska wrote:
Is the 15MB limit supposed to be imposed somewhere or is it just a guide
of how much memory we expect Git to use in this scenario?
The test should confirm
On Aug 16, 2014, at 12:27 PM, John Keeping j...@keeping.me.uk wrote:
+test_expect_success HAVE_MAX_MEM_USAGE \
+'filtering large input to small output should use little memory' '
+git config filter.devnull.clean cat /dev/null
+git config filter.devnull.required true
+for i in
I'll address the comments about style in a revised patch.
On Aug 4, 2014, at 9:03 PM, Junio C Hamano gits...@pobox.com wrote:
+return apply_filter(path, 0, 0, -1, 0, ca.drv-clean);
What's the significance of -1 here? Does somebody in the
callchain from apply_filter() check if fd 0
It is only the path that matters in the decision whether to filter or
not. Clarify this by making path the single argument of
would_convert_to_git().
Signed-off-by: Steffen Prohaska proha...@zib.de
---
convert.h | 5 ++---
sha1_file.c | 2 +-
2 files changed, 3 insertions(+), 4 deletions
The main difference to the previous version is that I've split off the
refactoring into a separate commit. The rest is polishing the style.
Steffen Prohaska (2):
convert: Refactor would_convert_to_git() to single arg 'path'
convert: Stream from fd to required clean filter instead of mmap
/usr/bin/time. An
alternative would have been tcsh, which could be used to print memory
information as follows:
tcsh -c 'set time=(0 %M); cmd'
Although the logic could perhaps be simplified with tcsh, I chose to use
'time' to avoid a dependency on tcsh.
Signed-off-by: Steffen Prohaska proha
/usr/bin/time. An
alternative would have been tcsh, which could be used to print memory
information as follows:
tcsh -c 'set time=(0 %M); cmd'
Although the logic could perhaps be simplified with tcsh, I chose to use
'time' to avoid a dependency on tcsh.
Signed-off-by: Steffen Prohaska proha
On Jun 10, 2014, at 7:27 AM, Junio C Hamano gits...@pobox.com wrote:
Steffen Prohaska proha...@zib.de writes:
I tend to prefer writing it like so instead:
sh -c '...' -
so that I won't clobber f (or any other name). I wonder if you
can help users of this other pattern as well
the parens, i.e. '!f() ...' and '!f () ...' both work.
[1] https://git.wiki.kernel.org/index.php/Aliases
Signed-off-by: Steffen Prohaska proha...@zib.de
---
contrib/completion/git-completion.bash | 10 ++
t/t9902-completion.sh | 27 +++
2 files changed, 37
as a workaround to declare the desired completion style. For example,
the alias '!f() { : git commit ; if ... ' now triggers commit
completion.
[1] https://git.wiki.kernel.org/index.php/Aliases
Signed-off-by: Steffen Prohaska proha...@zib.de
---
I changed the tests to use test_config, as Eric
as a workaround to declare the desired completion style. For example,
the alias '!f() { : git commit ; if ... ' now triggers commit
completion.
[1] https://git.wiki.kernel.org/index.php/Aliases
Signed-off-by: Steffen Prohaska proha...@zib.de
---
contrib/completion/git-completion.bash | 7
Hello Jonathan,
On Sep 27, 2013, at 8:52 PM, Jonathan Nieder jrnie...@gmail.com wrote:
The latest maintenance release Git v1.8.4.1 is now available.
I can't find the following three minor doc fixes
http://article.gmane.org/gmane.comp.version-control.git/235234
Labeled lists require a double colon.
Signed-off-by: Steffen Prohaska proha...@zib.de
---
Documentation/git.txt | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/Documentation/git.txt b/Documentation/git.txt
index 5d68d33..4c2757e 100644
--- a/Documentation/git.txt
+++ b
git-prune-packed operates on GIT_OBJECT_DIRECTORY. See 'environment.c',
git_object_dir = getenv(DB_ENVIRONMENT);
and cache.h,
#define DB_ENVIRONMENT GIT_OBJECT_DIRECTORY
Signed-off-by: Steffen Prohaska proha...@zib.de
---
Documentation/git-prune-packed.txt | 2 +-
1 file changed, 1
Fixed typo in comment.
Steffen Prohaska (2):
xread, xwrite: Limit size of IO, fixing IO of 2GB and more on Mac OS X
Revert compate/clipped-write.c: large write(2) fails on Mac OS X/XNU
Makefile | 8
compat/clipped-write.c | 13 -
config.mak.uname
The previous commit introduced a size limit on IO chunks on all
platforms. The compat clipped_write() is not needed anymore.
This reverts commit 6c642a878688adf46b226903858b53e2d31ac5c3.
Signed-off-by: Steffen Prohaska proha...@zib.de
---
Makefile | 8
compat/clipped
6c642a878688adf46b226903858b53e2d31ac5c3
compate/clipped-write.c: large write(2) fails on Mac OS X/XNU
Signed-off-by: Steffen Prohaska proha...@zib.de
---
t/t0021-conversion.sh | 14 ++
wrapper.c | 12
2 files changed, 26 insertions(+)
diff --git a/t/t0021
The previous commit introduced a size limit on IO chunks on all
platforms. The compat clipped_write() is not needed anymore.
This reverts commit 6c642a878688adf46b226903858b53e2d31ac5c3.
Signed-off-by: Steffen Prohaska proha...@zib.de
---
Makefile | 8
compat/clipped
This is the revised patch taking the considerations about IO chunk size into
account. The series deletes more than it adds and fixes a bug. Nice.
Steffen Prohaska (2):
xread, xwrite: Limit size of IO, fixing IO of 2GB and more on Mac OS X
Revert compate/clipped-write.c: large write(2) fails
6c642a878688adf46b226903858b53e2d31ac5c3
compate/clipped-write.c: large write(2) fails on Mac OS X/XNU
Signed-off-by: Steffen Prohaska proha...@zib.de
---
t/t0021-conversion.sh | 14 ++
wrapper.c | 12
2 files changed, 26 insertions(+)
diff --git a/t/t0021
] 6c642a878688adf46b226903858b53e2d31ac5c3
compate/clipped-write.c: large write(2) fails on Mac OS X/XNU
Signed-off-by: Steffen Prohaska proha...@zib.de
---
Makefile | 8
builtin/var.c | 1 +
config.mak.uname | 1 +
git-compat-util.h | 5 +
streaming.c
On Aug 19, 2013, at 9:54 AM, John Keeping j...@keeping.me.uk wrote:
You've created compat/clipped-read.c, but...
Makefile | 8
builtin/var.c | 1 +
config.mak.uname | 1 +
git-compat-util.h | 5 +
streaming.c | 1 +
] 6c642a878688adf46b226903858b53e2d31ac5c3
compate/clipped-write.c: large write(2) fails on Mac OS X/XNU
Signed-off-by: Steffen Prohaska proha...@zib.de
---
Makefile | 8
builtin/var.c | 1 +
compat/clipped-read.c | 13 +
config.mak.uname | 1 +
git
On Aug 19, 2013, at 10:20 AM, Johannes Sixt j...@kdbg.org wrote:
Am 19.08.2013 08:38, schrieb Steffen Prohaska:
+test_expect_success EXPENSIVE 'filter large file' '
+git config filter.largefile.smudge cat
+git config filter.largefile.clean cat
+for i in $(test_seq 1 2048); do
] 6c642a878688adf46b226903858b53e2d31ac5c3
compate/clipped-write.c: large write(2) fails on Mac OS X/XNU
Signed-off-by: Steffen Prohaska proha...@zib.de
---
Makefile | 10 +-
builtin/var.c| 10 +-
compat/{clipped-write.c
On Aug 19, 2013, at 6:04 PM, Linus Torvalds torva...@linux-foundation.org
wrote:
I hate your patch for other reasons, though:
The problem for read() is addressed in a similar way by introducing
a wrapper function in compat that always reads less than 2GB.
Why do you do that? We already
Since shortlog isn't using the return value anymore (see previous
commit), the functions can be changed to void.
Signed-off-by: Steffen Prohaska proha...@zib.de
---
utf8.c | 13 ++---
utf8.h | 4 ++--
2 files changed, 8 insertions(+), 9 deletions(-)
diff --git a/utf8.c b/utf8.c
index
strbuf_add_wrapped_text() never returned a value of wraplen.
[1] 14e1a4e1ff70aff36db3f5d2a8b806efd0134d50 utf8: fix off-by-one
wrapping of text
Signed-off-by: Steffen Prohaska proha...@zib.de
---
builtin/shortlog.c | 5 ++---
t/t4201-shortlog.sh | 24
2 files changed, 26
On Dec 9, 2012, at 10:36 AM, Junio C Hamano gits...@pobox.com wrote:
Steffen Prohaska proha...@zib.de writes:
A recent commit [1] fixed a off-by-one wrapping error. As
a side-effect, add_wrapped_shortlog_msg() needs to be changed to always
append a newline.
Could you clarify
A recent commit [1] fixed a off-by-one wrapping error. As
a side-effect, add_wrapped_shortlog_msg() needs to be changed to always
append a newline.
[1] 14e1a4e1ff70aff36db3f5d2a8b806efd0134d50 utf8: fix off-by-one
wrapping of text
Signed-off-by: Steffen Prohaska proha...@zib.de
---
builtin
On Oct 8, 2012, at 6:27 PM, Junio C Hamano wrote:
Once we go into want/have phase, I do not think there is a need
for fundamental change in the protocol (by this, I am not counting a
change to send haves sparsely and possibly backtracking to bisect
history, etc. as fundamental).
I've
On Sep 27, 2012, at 9:11 PM, Junio C Hamano wrote:
Steffen Prohaska proha...@zib.de writes:
iTerm is an alternative to the default terminal emulation program on Mac
OS X. git-web--browse wasn't aware of iTerm and failed to open HTML
help pages when used in a shell session running in iTerm
iTerm is an alternative to the default terminal emulation program on Mac
OS X. git-web--browse wasn't aware of iTerm and failed to open HTML
help pages when used in a shell session running in iTerm, reporting No
known browser available. Now it works as expected.
Signed-off-by: Steffen Prohaska
68 matches
Mail list logo