On Sun, Sep 10, 2017 at 08:27:40AM +0200, René Scharfe wrote:
> >>if (junk_work_tree) {
> >>strbuf_addstr(&sb, junk_work_tree);
> >>remove_dir_recursively(&sb, 0);
> >> - strbuf_reset(&sb);
> >>}
> >> + strbuf_release(&sb);
> >> }
> >
> > The code defin
Jeff King writes:
> On Sat, Sep 09, 2017 at 02:37:20AM +0900, Junio C Hamano wrote:
>
>> Using log-tree traversal machinery instead of just get_revision()
>> would probably mean we would slow it down quite a bit unless we are
>> careful, but at the same time, things like "git shortlog -G"
>> woul
On Sun, Sep 10, 2017 at 06:45:08AM +0200, Michael Haggerty wrote:
> > So nothing to see here, but since I spent 20 minutes scratching my head
> > (and I know others look at Coverity output and may scratch their heads
> > too), I thought it was worth writing up. And also if I'm wrong, it would
> >
René Scharfe writes:
> Am 06.09.2017 um 21:51 schrieb Junio C Hamano:
>> Rene Scharfe writes:
>>
>>> If format_tracking_info() returns 0 only if it didn't touch its strbuf
>>> parameter, so it's OK to exit early in that case. Clean up sb in the
>>> other case.
>>
>> These two "if"s confuse me
Am 08.09.2017 um 11:21 schrieb Jeff King:
> Note that the non-stdin path no longer looks at the "mailmap" entry of
> "struct shortlog" (instead we use the one cached inside pretty.c). But
> we still waste time loading it. I'm not sure if it's worth addressing
> that. It's only once per program invo
On Sun, Sep 10, 2017 at 10:44:46AM +0200, René Scharfe wrote:
> Am 08.09.2017 um 11:21 schrieb Jeff King:
> > Note that the non-stdin path no longer looks at the "mailmap" entry of
> > "struct shortlog" (instead we use the one cached inside pretty.c). But
> > we still waste time loading it. I'm no
Am 10.09.2017 um 09:30 schrieb Jeff King:
> On Sun, Sep 10, 2017 at 08:27:40AM +0200, René Scharfe wrote:
>
if (junk_work_tree) {
strbuf_addstr(&sb, junk_work_tree);
remove_dir_recursively(&sb, 0);
- strbuf_reset(&sb);
>
> On 03 Aug 2017, at 10:18, Christian Couder wrote:
>
> Let's add an odb_helper_init() function to send an 'init'
> instruction to the helpers. This 'init' instruction is
> especially useful to get the capabilities that are supported
> by the helpers.
>
> So while at it, let's also add a parse_
> On 03 Aug 2017, at 10:18, Christian Couder wrote:
>
> To properly test passing objects from Git to an external odb
> we need an odb-helper script that supports a 'put'
> capability/instruction.
>
> For now we will support only sending raw blobs, so the
> supported capability/instruction will
Ramsay Jones venit, vidit, dixit 09.09.2017 15:13:
> Hi Adam,
>
> I ran the test-suite on the 'pu' branch last night (simply because
> that was what I had built at the time!), which resulted in a PASS,
> but t6120 was showing a 'TODO passed' for #52.
>
> This is a test introduced by Michael's 'mg
> On 03 Aug 2017, at 10:18, Christian Couder wrote:
>
> ...
>
> * The "helpers" (registered commands)
>
> Each helper manages access to one external ODB.
>
> There are 2 different modes for helper:
>
> - Helpers configured using "odb..scriptCommand" are
>launched each time Git wants to
From: Lars Schneider
Most of the Travis CI commands are in the '.travis.yml'. The yml format
does not support functions and therefore code duplication is necessary
to run commands across all builds.
To fix this, add a library for common CI functions. Move all Travis CI
code into dedicated script
From: Lars Schneider
Hi,
the patches were previously discussed here:
https://public-inbox.org/git/xmqqinifrrzh@gitster.mtv.corp.google.com/#t
Over there Junio posted the original patch to skip a build on Travis
CI. The patch became 2/2 in this series.
Cheers,
Lars
Base Ref: master
Web-Dif
From: Lars Schneider
If we push a branch and a tag pointing to the HEAD of this branch,
then Travis CI would run the build twice. This wastes resources and
slows the testing.
Add a function to detect this situation and skip the build the branch
if appropriate. Invoke this function on every build
On 10/09/17 13:27, Michael J Gruber wrote:
> Ramsay Jones venit, vidit, dixit 09.09.2017 15:13:
[snip]
>> So, it looks like all ULIMIT_STACK_SIZE tests need to be disabled
>> on cygwin. I also wonder about the ULIMIT_FILE_DESCRIPTORS tests,
>> but haven't looked into it.
>>
>> Given that 'ulimit'
On Sun, Sep 10, 2017 at 12:37:06PM +0200, René Scharfe wrote:
> > And in fact, being called from a signal handler means we should
> > generally avoid touching malloc or free (which could be holding locks).
> > That would mean preferring a leak to strbuf_release(). Of course that is
> > the tip of
larsxschnei...@gmail.com writes:
> From: Lars Schneider
>
> Most of the Travis CI commands are in the '.travis.yml'. The yml format
> does not support functions and therefore code duplication is necessary
> to run commands across all builds.
>
> To fix this, add a library for common CI functions.
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.
We are at week #6 of this cycle.
Junio C Hamano writes:
> I still think we would want to turn warning() to die(), but it
> probably is better to do so in a separate follow-up patch. That
> will give us a good place to record the reason why the current "just
> call a warning() and pretend as if nothing bad happend" is wrong.
An
Kevin Willford writes:
> I agree with this when you are not dealing with a sparse-checkout.
> When using a sparse-checkout I expect git not to touch things
> outside of what I have specified in my sparse-checkout file. If it
> does, it should let me know or put my working directory in a
> state
On Wed, Sep 06, 2017 at 12:34:48PM +0900, Junio C Hamano wrote:
> Sam Bobroff writes:
>
> > If "--notes=..." is used with "git format-patch", the notes are
> > prefixed with the ref's local name and indented, which looks odd and
> > exposes the path of the ref.
> >
> > Extend the test that suppre
21 matches
Mail list logo