Am 07.04.2018 um 01:21 schrieb Stefan Beller:
> This applies on top of 464416a2eaadf84d2bfdf795007863d03b222b7c
> (sb/packfiles-in-repository).
> It is also available at
> https://github.com/stefanbeller/git/tree/object-store-3
This series conflicts with 1731a1e239 (replace_object: convert
Jsem Ronald Bernstein, jsem úverový dustojník, dávám pujcky jednotlivcum a
firme pro obchodní a osobní úcely, kontaktujte me, pokud potrebujete jakýkoliv
druh pujcky. Poskytuji pujcky široké verejnosti s úrokovou sazbou 2%.
Hi, Eric,
On Thu, Apr 5, 2018 at 5:46 PM, Eric Sunshine wrote:
> On Thu, Apr 5, 2018 at 3:48 PM, Igor Korot wrote:
>> Is support for 10.8 dropped?
>
> Until about a year ago, I was still periodically building Git from
> source on MacOS 10.5, and
On Fri, Apr 06, 2018 at 10:55:18AM -0400, Jeff King wrote:
> On Fri, Apr 06, 2018 at 02:14:34AM -0400, Eric Sunshine wrote:
>
> > > diff --git a/Documentation/git-config.txt b/Documentation/git-config.txt
> > > @@ -9,13 +9,13 @@ git-config - Get and set repository or global options
> > > SYNOPSIS
On Fri, Apr 06, 2018 at 03:40:56AM -0400, Eric Sunshine wrote:
> On Fri, Apr 6, 2018 at 2:53 AM, Eric Sunshine wrote:
> > On Fri, Apr 6, 2018 at 2:30 AM, Taylor Blau wrote:
> >> +test_expect_success 'uses entry when available' '
> >> + echo bar
On Fri, Apr 06, 2018 at 02:53:45AM -0400, Eric Sunshine wrote:
> On Fri, Apr 6, 2018 at 2:30 AM, Taylor Blau wrote:
> > [...]
> > This commit (and those following it in this series) aim to eventually
> > replace `--get-color` with a consistent alternative. By introducing
> >
On Fri, Apr 06, 2018 at 03:29:48AM -0400, Eric Sunshine wrote:
> On Fri, Apr 6, 2018 at 2:30 AM, Taylor Blau wrote:
> > [...]
> > For consistency, let's introduce `--type=color` and encourage its use
> > with `--default` together over `--get-color` alone.
>
> A couple minor
On Fri, Apr 06, 2018 at 03:04:53AM -0400, Eric Sunshine wrote:
> On Fri, Apr 6, 2018 at 2:39 AM, Taylor Blau wrote:
> > [...]
> > In this patch, we support `--type=` in
> > addition to `--int`, `--bool`, and etc. This allows the aforementioned
> >
Add a repository argument to allow the get_main_ref_store caller
to be more specific about which repository to handle. This is a small
mechanical change; it doesn't change the implementation to handle
repositories other than the_repository yet.
As with the previous commits, use a macro to catch
Remove another global variable on the way.
Signed-off-by: Jonathan Nieder
Signed-off-by: Stefan Beller
---
object-store.h | 2 ++
replace-object.c | 6 ++
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/object-store.h
In ecef23 (inline lookup_replace_object() calls, 2011-05-15) a shortcut
for checking the object replacement was added by setting check_replace_refs
to 0 once the replacements were evaluated to not exist. This works fine in
with the assumption of only one repository in existence.
The
Signed-off-by: Stefan Beller
---
replace-object.c | 9 +
replace-object.h | 3 +--
2 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/replace-object.c b/replace-object.c
index 967d794687..075a48b661 100644
--- a/replace-object.c
+++ b/replace-object.c
@@
Add a repository argument to allow the prepare_replace_object caller
to be more specific about which repository to handle. This is a small
mechanical change; it doesn't change the implementation to handle
repositories other than the_repository yet.
As with the previous commits, use a macro to
Add a repository argument to allow for_each_replace_ref callers to be
more specific about which repository to handle. This is a small
mechanical change; it doesn't change the implementation to handle
repositories other than the_repository yet.
As with the previous commits, use a macro to catch
Add a repository argument to allow callers of lookup_replace_object
to be more specific about which repository to handle. This is a small
mechanical change; it doesn't change the implementation to handle
repositories other than the_repository yet.
As with the previous commits, use a macro to
Add a repository argument to allow the replace_object_pos caller
to be more specific about which repository to handle. This is a small
mechanical change; it doesn't change the implementation to handle
repositories other than the_repository yet.
As with the previous commits, use a macro to catch
Signed-off-by: Stefan Beller
---
replace-object.c | 10 --
1 file changed, 4 insertions(+), 6 deletions(-)
diff --git a/replace-object.c b/replace-object.c
index b3bfd21541..967d794687 100644
--- a/replace-object.c
+++ b/replace-object.c
@@ -72,15 +72,13 @@ static
Signed-off-by: Stefan Beller
---
replace-object.h | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/replace-object.h b/replace-object.h
index e15f0725cb..e7511ea54c 100644
--- a/replace-object.h
+++ b/replace-object.h
@@ -22,14 +22,14 @@ extern
Signed-off-by: Jonathan Nieder
Signed-off-by: Stefan Beller
---
replace-object.c | 31 +++
1 file changed, 15 insertions(+), 16 deletions(-)
diff --git a/replace-object.c b/replace-object.c
index 9272c7acbf..b3bfd21541 100644
Add a repository argument to allow the register_replace_object caller
to be more specific about which repository to handle. This is a small
mechanical change; it doesn't change the implementation to handle
repositories other than the_repository yet.
As with the previous commits, use a macro to
Add a repository argument to allow the do_lookup_replace_object caller
to be more specific about which repository to handle. This is a small
mechanical change; it doesn't change the implementation to handle
repositories other than the_repository yet.
As with the previous commits, use a macro to
lookup_replace_object is a low-level function that most users of the
object store do not need to use directly.
Move it to replace-object.h to avoid a dependency loop in an upcoming
change to its inline definition that will make use of repository.h.
Signed-off-by: Stefan Beller
Refs belong to particular repositories, so the replacements defined by
them should belong to a particular repository as well.
Move the definition of a single object replacement to a new header
"replace-object.h". While at it replace the hardcoded 20 by GIT_MAX_RAWSZ.
Signed-off-by: Jonathan
Signed-off-by: Stefan Beller
---
refs.c | 4 ++--
refs.h | 4 +---
2 files changed, 3 insertions(+), 5 deletions(-)
diff --git a/refs.c b/refs.c
index 793b827e19..ee71733a74 100644
--- a/refs.c
+++ b/refs.c
@@ -1415,9 +1415,9 @@ int refs_for_each_fullref_in(struct ref_store
Signed-off-by: Jonathan Nieder
Signed-off-by: Stefan Beller
---
replace-object.c | 8 +++-
1 file changed, 3 insertions(+), 5 deletions(-)
diff --git a/replace-object.c b/replace-object.c
index 944bcaf242..9272c7acbf 100644
--- a/replace-object.c
+++
Signed-off-by: Stefan Beller
---
refs.c | 13 +
refs.h | 4 +---
repository.h | 5 +
3 files changed, 11 insertions(+), 11 deletions(-)
diff --git a/refs.c b/refs.c
index 0352cbdcc1..793b827e19 100644
--- a/refs.c
+++ b/refs.c
@@ -1608,9 +1608,6
This is more consistent with the project style. The majority of
Git's source files use dashes in preference to underscores in their file
names.
Noticed while adding a header corresponding to this file.
Signed-off-by: Jonathan Nieder
Signed-off-by: Stefan Beller
This applies on top of 464416a2eaadf84d2bfdf795007863d03b222b7c
(sb/packfiles-in-repository).
It is also available at https://github.com/stefanbeller/git/tree/object-store-3
This series will bring the replacement mechanism (git replace)
into the object store.
The first patches are cleaning up a
On 06.04.2018 16:06, Johannes Schindelin wrote:
+ git stash clear &&
+ test_when_finished "git reset --hard HEAD" &&
+ echo foo >file2 &&
+ git stash &&
+ echo bar >file2 &&
+ git stash &&
+ test-chmtime =123456789 file2 &&
+ for type in apply pop
Hi Dscho,
On Fri, Apr 6, 2018 at 3:38 AM, Johannes Schindelin
wrote:
> On Thu, 5 Apr 2018, Elijah Newren wrote:
>> On Thu, Apr 5, 2018 at 12:57 PM, Johannes Schindelin
>> wrote:
>> That all looks great to me. But at this point, it
On Fri, 6 Apr 2018 14:28:40 -0700
Stefan Beller wrote:
> Now that I redid it another way[1], I have the impression that this was the
> right approach, because it allows for a short
> if (o->color_moved)
> condition. If we treat white spaces separately, then we'd have to
>
Compared to 'test-chmtime -v +0 file' which prints the mtime and
and the file name, 'test-chmtime --get file' displays only the mtime.
If it is used in combination with (+|=|=+|=-|-)seconds, it changes
and prints the new value.
test-chmtime -v +0 file | sed 's/[^0-9].*$//'
is now
On Fri, Apr 06, 2018 at 05:57:15PM -0400, Jonathan Primrose wrote:
> > Oops. This is due to my 3adf9fdecf (configure.ac: loosen
> > FREAD_READS_DIRECTORIES test program, 2017-06-14).
> >
> > Neither I nor the original tester noticed, because we're on Linux, which
> > needs that set.
>
> I'm
On 04/06/2018 03:33 PM, Jeff King wrote:
On Fri, Apr 06, 2018 at 02:06:50PM -0400, Jonathan Primrose wrote:
A while ago, the configure test for FREAD_READS_DIRECTORIES was changed
to (attempt to) check for a NULL result from fopen. Unfortunately, the
test will always fail - because it won't
On Fri, Apr 6, 2018 at 2:40 PM, Jeff King wrote:
> On Tue, Apr 03, 2018 at 06:00:05PM +0200, Johannes Schindelin wrote:
>
>> > But if we're at the point of creating custom C builtins for
>> > busybox/dash/etc, you should be able to create a primitive for "read
>> > this using
On Sat, Mar 31, 2018 at 12:02:56PM +0200, Nguyễn Thái Ngọc Duy wrote:
> v8 changes
>
> - prefer BUG() over die()
> - do "1U <<" instead of "1 << " to avoid undefined behavior with
> signed shifting.
> - add more comments based on Jeff's feedback
> - plug a leak in try_delta() when delta_size
On Tue, Apr 03, 2018 at 05:17:00PM +0200, Duy Nguyen wrote:
> It's not that complex. With the EAGER_DEVELOPER patch removed, we can
> have something like this where eager devs just need to put
>
> DEVOPTS = gentle no-suppression
>
> and you put
>
> DEVOPTS = gentle
>
> (bad naming, I
On Tue, Apr 03, 2018 at 06:00:05PM +0200, Johannes Schindelin wrote:
> > But if we're at the point of creating custom C builtins for
> > busybox/dash/etc, you should be able to create a primitive for "read
> > this using buffered stdio, other processes be damned, and return one
> > line at a
On Tue, Apr 03, 2018 at 06:27:55PM +0200, Johannes Schindelin wrote:
> I am very, very grateful for the time Peff spent on reviewing the previous
> iteration, and hope that he realizes just how much the elegance of the
> event-stream-based version is due to his excellent review.
Unfortunately I
On Tue, Apr 03, 2018 at 06:28:42PM +0200, Johannes Schindelin wrote:
> While a neat theoretical construct, state machines are hard to read. In
> this instance, it does not even make a whole lot of sense because we are
> more interested in flags, anyway: has the section been seen? Has the key
>
On Mon, Apr 2, 2018 at 4:51 PM, Jonathan Tan wrote:
> On Mon, 2 Apr 2018 15:48:52 -0700
> Stefan Beller wrote:
>
>> At the time the move coloring was implemented we thought an enum of modes
>> is the best to configure this feature. However as we
On Tue, Apr 03, 2018 at 06:28:34PM +0200, Johannes Schindelin wrote:
> It is much easier to reason about, when the config code to set/unset
> variables or to remove/rename sections does not rely on a global (or
> file-local) variable.
Agreed.
> -static struct {
> +struct config_set_store {
On Tue, Apr 03, 2018 at 06:28:29PM +0200, Johannes Schindelin wrote:
> This extends our config parser so that it can optionally produce an event
> stream via callback function, where it reports e.g. when a comment was
> parsed, or a section header, etc.
>
> This parser will be used subsequently
On Fri, Apr 06 2018, Eric Wong wrote:
> Ævar Arnfjörð Bjarmason wrote:
>> On Fri, Apr 06 2018, Eric Wong wrote:
>> > Ævar Arnfjörð Bjarmason wrote:
>> >
>> >> --- a/perl/Git.pm
>> >> +++ b/perl/Git.pm
>> >> @@ -554,7 +554,7 @@ sub get_record {
>> >> my
Ævar Arnfjörð Bjarmason wrote:
> On Fri, Apr 06 2018, Eric Wong wrote:
> > Ævar Arnfjörð Bjarmason wrote:
> >
> >> --- a/perl/Git.pm
> >> +++ b/perl/Git.pm
> >> @@ -554,7 +554,7 @@ sub get_record {
> >>my ($fh, $rs) = @_;
> >>local $/ = $rs;
> >>my
Hi Jeff,
On Fri, Apr 6, 2018 at 1:04 PM, Jeff King wrote:
> On Mon, Apr 02, 2018 at 03:48:50PM -0700, Stefan Beller wrote:
>
>> The diff options are passed to the compare function as
>> 'hashmap_cmp_fn_data', which are given when the hashmaps
>> are initialized.
>>
>> A later
On Fri, Apr 06 2018, Jeff King wrote:
> On Fri, Apr 06, 2018 at 09:31:22PM +0200, Johannes Schindelin wrote:
>> This patch chooses instead to look for the prefix "error:" at the
>> beginning of the line, so that there can be no ambiguity that any catch
>> was indeed a message generated by Git's
On 06/04/18 07:56 PM, Jeff King wrote:
On Thu, Apr 05, 2018 at 04:18:23PM -0700, Bryan Turner wrote:
The documentation for --work-tree says:
--work-tree=
Set the path to the working tree. It can be an absolute path or a path
relative to the current working directory. This can also be
Signed-off-by: Anders Kaseorg
---
Documentation/git-bisect.txt | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/Documentation/git-bisect.txt b/Documentation/git-bisect.txt
index 4a1417bdcd..4b45d837a7 100644
--- a/Documentation/git-bisect.txt
+++
On Fri, Apr 06, 2018 at 10:13:46PM +0200, Johannes Schindelin wrote:
> On Fri, 6 Apr 2018, Jeff King wrote:
>
> > On Fri, Apr 06, 2018 at 09:31:22PM +0200, Johannes Schindelin wrote:
> >
> > > Suppose, however, that you are a big fan of whales. Or even better: your
> > > IT administrator has a
Hi Peff,
On Fri, 6 Apr 2018, Jeff King wrote:
> On Fri, Apr 06, 2018 at 09:31:22PM +0200, Johannes Schindelin wrote:
>
> > Suppose, however, that you are a big fan of whales. Or even better: your
> > IT administrator has a whale of a time picking cute user names, e.g.
> > referring to you (due
On Mon, Apr 02, 2018 at 03:48:50PM -0700, Stefan Beller wrote:
> The diff options are passed to the compare function as
> 'hashmap_cmp_fn_data', which are given when the hashmaps
> are initialized.
>
> A later patch will make use of the keydata to signal
> different settings for comparision.
I
On Thu, Apr 05, 2018 at 04:18:23PM -0700, Bryan Turner wrote:
> The documentation for --work-tree says:
>
> --work-tree=
>
> Set the path to the working tree. It can be an absolute path or a path
> relative to the current working directory. This can also be controlled
> by setting the
On Fri, Apr 06, 2018 at 09:31:22PM +0200, Johannes Schindelin wrote:
> In 0b294c0abf0 (make deleting a missing ref more quiet, 2008-07-08), we
> added a test to verify that deleting an already-deleted ref does not
> show an error.
Amazing that it took this long to come up.
> Suppose, however,
On Fri, Apr 06, 2018 at 11:55:51PM +0530, Satya Prakash GS wrote:
> We have a distributed filesystem with NFS access. On the NFS mount, I
> was doing a git-clone and if NFS server crashed and came back up while
> the clone is going on, clone fails with the below message:
>
> git clone
On Fri, Apr 6, 2018 at 12:21 PM, Jeff King wrote:
> On Fri, Apr 06, 2018 at 07:09:30PM +, Derrick Stolee wrote:
>
>> Derrick Stolee (4):
>> treewide: rename tree to maybe_tree
>> commit: create get_commit_tree() method
>> treewide: replace maybe_tree with accessor methods
On 4/6/2018 3:21 PM, Jeff King wrote:
On Fri, Apr 06, 2018 at 07:09:30PM +, Derrick Stolee wrote:
Derrick Stolee (4):
treewide: rename tree to maybe_tree
commit: create get_commit_tree() method
treewide: replace maybe_tree with accessor methods
commit-graph: lazy-load trees for
On Fri, Apr 06, 2018 at 02:06:50PM -0400, Jonathan Primrose wrote:
> A while ago, the configure test for FREAD_READS_DIRECTORIES was changed
> to (attempt to) check for a NULL result from fopen. Unfortunately, the
> test will always fail - because it won't compile. The code snippet in
>
In 0b294c0abf0 (make deleting a missing ref more quiet, 2008-07-08), we
added a test to verify that deleting an already-deleted ref does not
show an error.
Our test simply looks for the substring 'error' in the output of the
`git push`, which might look innocuous on the face of it.
Suppose,
Hello, I have the following git send-email command:
git send-email -5 --quiet --thread --no-chain-reply-to --compose
--subject='Recent Base Prototype Changes Summary'
--to=davisda...@google.com --from=davisda...@google.com
It's sending emails but as 5 individual emails and not as subsequent
On 4/6/2018 2:59 PM, Jeff King wrote:
In preparation for callers constructing their own ref_array
structs, let's move our own internal push operation into its
own function.
While we're at it, we can replace REALLOC_ARRAY() with
ALLOC_GROW(), which should give the growth operation
amortized
On Fri, Apr 06, 2018 at 07:09:30PM +, Derrick Stolee wrote:
> Derrick Stolee (4):
> treewide: rename tree to maybe_tree
> commit: create get_commit_tree() method
> treewide: replace maybe_tree with accessor methods
> commit-graph: lazy-load trees for commits
I gave this only a
While walking the commit graph, we load struct commit objects into
the object cache. During this process, we also load struct tree
objects for the root tree of each of these commits. We load these
objects even if we are only computing commit reachability information,
such as a merge base or
The commit-graph file provides quick access to commit data, including
the OID of the root tree for each commit in the graph. When performing
a deep commit-graph walk, we may not need to load most of the trees
for these commits.
Delay loading the tree object for a commit loaded from the graph
Using the commit-graph file to walk commit history removes the large
cost of parsing commits during the walk. This exposes a performance
issue: lookup_tree() takes a large portion of the computation time,
even when Git never uses those trees.
In anticipation of lazy-loading these trees, rename
There are several commit-graph walks that require loading many commits
but never walk the trees reachable from those commits. However, the
current logic in parse_commit() requires the root tree to be loaded.
This only uses lookup_tree(), but when reading commits from the commit-
graph file, the
In preparation for callers constructing their own ref_array
structs, let's move our own internal push operation into its
own function.
While we're at it, we can replace REALLOC_ARRAY() with
ALLOC_GROW(), which should give the growth operation
amortized linear complexity (as opposed to growing by
We have a helper function to allocate ref_array_item
structs, but it only takes a subset of the possible fields
in the struct as initializers. We could have it accept an
argument for _every_ field, but that becomes a pain for the
fields which some callers don't want to set initially.
Instead,
Internally we store a "struct object_id", and all of our
callers have one to pass us. But we insist that they peel it
to its bare-sha1 hash, which we then hashcpy() into place.
Let's pass it around as an object_id, which future-proofs us
for a post-sha1 world.
Signed-off-by: Jeff King
On Thu, Apr 05, 2018 at 01:11:34AM +0200, Harald Nordgren wrote:
> Without digging to much into the `ref-filter` code itself, it seems
> like there is an opportunity to generalize and unfify the logic
> between these two cases. As well as using `ALLOC_GROW`. But maybe that
> is best left as a
On Fri, Feb 16 2018, Johannes Schindelin wrote:
> This is an RFC because it tries to introduce a fundamental new color feature:
> Coloring messages *on stderr*.
I missed this the first time around, and don't have anything to add that
others haven't covered. Just wanted to say I'd love to get
On Fri, Apr 06 2018, Satya Prakash wrote:
> Hi,
>
> We have a distributed filesystem with NFS access. On the NFS mount, I
> was doing a git-clone and if NFS server crashed and came back up while
> the clone is going on, clone fails with the below message:
>
> git clone
Hi,
We have a distributed filesystem with NFS access. On the NFS mount, I
was doing a git-clone and if NFS server crashed and came back up while
the clone is going on, clone fails with the below message:
git clone https://sa...@github.com/fs/private-qa.git
remote: Counting objects: 139419,
On Fri, Apr 06 2018, Eric Wong wrote:
> Ævar Arnfjörð Bjarmason wrote:
>> See https://public-inbox.org/git/86h8oobl36@phe.ftfl.ca/ for the
>> original report.
>
> Thanks for taking a look at this. Also https://bugs.debian.org/894997
>
>> --- a/perl/Git.pm
>> +++
A while ago, the configure test for FREAD_READS_DIRECTORIES was changed
to (attempt to) check for a NULL result from fopen. Unfortunately, the
test will always fail - because it won't compile. The code snippet in
configure.ac translates to:
return f);
Either there's an extra ) or a missing (. A
On Thu, Apr 05, 2018 at 01:06:30PM -0700, Elijah Newren wrote:
> > There are other similar trailing-slash matches in that function, but I'm
> > not sure of all the cases in which they're used. I don't know if any of
> > those would need similar treatment (sorry for being vague; I expect I'd
> >
On Fri, Apr 06, 2018 at 05:57:39PM +0100, Rafael Ascensao wrote:
> On Fri, Apr 6, 2018 at 4:55 PM, Olaf Hering wrote:
> >
> > This does not work. Initially I copied the global config into the
> > repo and set all unwanted values to , like 'smtpuser='.
> > Perhaps the config
On Fri, Apr 6, 2018 at 4:55 PM, Olaf Hering wrote:
>
> This does not work. Initially I copied the global config into the repo and
> set all unwanted values to , like 'smtpuser='. Perhaps the config
> parser recognized that fact, but the consumer does not?
>
Today someone asked
Ævar Arnfjörð Bjarmason wrote:
> See https://public-inbox.org/git/86h8oobl36@phe.ftfl.ca/ for the
> original report.
Thanks for taking a look at this. Also https://bugs.debian.org/894997
> --- a/perl/Git.pm
> +++ b/perl/Git.pm
> @@ -554,7 +554,7 @@ sub get_record {
>
Hi Joel,
On Wed, 4 Apr 2018, Joel Teichroeb wrote:
> I've been working on converting all of git stash to be a
> builtin, however it's hard to get it all working at once with
> limited time, so I've moved around half of it to a new
> stash--helper builtin and called these functions from the
On Fri, Apr 06, 2018 at 05:55:56PM +0200, Olaf Hering wrote:
> > The general strategy in Git's config is that instead of "unsetting",
> > you
> > should overwrite with whatever value you _do_ want. So a config option
> > like sendemail.smtpauth should accept some kind of empty or "none" value
> >
Hi Joel,
On Wed, 4 Apr 2018, Joel Teichroeb wrote:
> diff --git a/builtin/stash--helper.c b/builtin/stash--helper.c
> index 486796bb6a..7c8950bc90 100644
> --- a/builtin/stash--helper.c
> +++ b/builtin/stash--helper.c
> @@ -13,6 +13,7 @@
>
> static const char * const git_stash_helper_usage[]
Am Thu, 5 Apr 2018 12:32:27 -0400
schrieb Jeff King :
> The general strategy in Git's config is that instead of "unsetting", you
> should overwrite with whatever value you _do_ want. So a config option
> like sendemail.smtpauth should accept some kind of empty or "none" value
> to
Hi Joel,
On Wed, 4 Apr 2018, Joel Teichroeb wrote:
> diff --git a/builtin/stash--helper.c b/builtin/stash--helper.c
> index 520cd746c4..486796bb6a 100644
> --- a/builtin/stash--helper.c
> +++ b/builtin/stash--helper.c
> @@ -502,6 +508,49 @@ static int drop_stash(int argc, const char **argv,
Hi Joel,
On Wed, 4 Apr 2018, Joel Teichroeb wrote:
> Add the drop and clear commands to the builtin helper. These two
> are each simple, but are being added together as they are quite
> related.
Makes sense.
> We have to unfortunately keep the drop and clear functions in the
> shell script as
Hi Joel,
completely forgot to reiterate this (my reply to Christian was probably
buried in the thread):
On Wed, 4 Apr 2018, Joel Teichroeb wrote:
> +int cmd_stash__helper(int argc, const char **argv, const char *prefix)
> +{
> + int result = 0;
> + pid_t pid = getpid();
> + const
Hi Joel,
On Wed, 4 Apr 2018, Joel Teichroeb wrote:
> Add a bulitin helper for performing stash commands. Converting
> all at once proved hard to review, so starting with just apply
> let conversion get started without the other command being
s/let/lets the/ s/command/commands/
> finished.
>
>
On Fri, Apr 06, 2018 at 02:14:34AM -0400, Eric Sunshine wrote:
> > diff --git a/Documentation/git-config.txt b/Documentation/git-config.txt
> > @@ -9,13 +9,13 @@ git-config - Get and set repository or global options
> > SYNOPSIS
> >
> > [verse]
> > -'git config' [] [type]
Hi Ævar,
On Fri, 6 Apr 2018, Ævar Arnfjörð Bjarmason wrote:
> Change code in Git.pm that sometimes calls chomp() on undef to only do
> so the value is defined.
>
> This code has been chomping undef values ever since it was added in
> b26098fc2f ("git-svn: reduce scope of input record separator
Change code in Git.pm that sometimes calls chomp() on undef to only do
so the value is defined.
This code has been chomping undef values ever since it was added in
b26098fc2f ("git-svn: reduce scope of input record separator change",
2016-10-14), but started warning due to the introduction of
Hi Joel,
On Wed, 4 Apr 2018, Joel Teichroeb wrote:
> In preparation for converting the stash command incrementally to
> a builtin command, this patch improves test coverage of the option
> parsing. Both for having too many parameters, or too few.
Very good.
> Signed-off-by: Joel Teichroeb
Johannes Schindelin writes:
> Hi Joseph,
> On Fri, 6 Apr 2018, Joseph Mingrone wrote:
>> After upgrading to version 2.17.0, this message appears repeatedly when
>> running `git svn rebase`:
>> Use of uninitialized value $rec in scalar chomp at
>>
Hi Joseph,
On Fri, 6 Apr 2018, Joseph Mingrone wrote:
> After upgrading to version 2.17.0, this message appears repeatedly when
> running `git svn rebase`:
>
> Use of uninitialized value $rec in scalar chomp at
> /usr/local/lib/perl5/site_perl/Git.pm line 557, <$fh> chunk 1.
>
> The value of
Hi Eric,
On Fri, 6 Apr 2018, Eric Sunshine wrote:
> On Thu, Apr 5, 2018 at 6:48 PM, Johannes Schindelin
> wrote:
> > Let's make it easier for users to find out how to customize these colors.
> >
> > Signed-off-by: Johannes Schindelin
> >
Hi Eric,
On Fri, 6 Apr 2018, Eric Sunshine wrote:
> On Thu, Apr 5, 2018 at 6:48 PM, Johannes Schindelin
> wrote:
> > This actually only tests whether the push errors/hints are colored if
> > the respective color.* config settings are `always`, but in the regular
> >
Hello,
After upgrading to version 2.17.0, this message appears repeatedly when
running `git svn rebase`:
Use of uninitialized value $rec in scalar chomp at
/usr/local/lib/perl5/site_perl/Git.pm line 557, <$fh> chunk 1.
The value of chunk varies. For example the message above may end with
I have a business proposal for you which involves a huge amount of
money that run into millions of US dollars that i want to move or
transfer to your country for safe keeping and investment,Please try as
much as possible to get back to me on this, i will explain more to you
when you finally
Hi Junio,
[and welcome back, at least I hope you only read this after a good and
relaxing vacation]
On Fri, 16 Feb 2018, Junio C Hamano wrote:
> Johannes Schindelin writes:
>
> > Now, what would be possible solutions for this?
> >
> > - introduce `int fd` in
Hi Jake,
On Thu, 5 Apr 2018, Jacob Keller wrote:
> On Thu, Apr 5, 2018 at 3:48 PM, Johannes Schindelin
> wrote:
> > From: Ryan Dammrose
> >
> > This is an attempt to resolve an issue I experience with people that are
> > new to Git --
On Thu, Apr 5, 2018 at 6:48 PM, Johannes Schindelin
wrote:
> Let's make it easier for users to find out how to customize these colors.
>
> Signed-off-by: Johannes Schindelin
> ---
> diff --git a/Documentation/config.txt
1 - 100 of 118 matches
Mail list logo