Hi Aleksey,
On 2015-08-19 09:33, Цапков Алексей wrote:
When installing the Git is not a dialog box appears with a choice ssh.
What could be the reason?
I assume that you are referring to the Git for Windows installer (please state
such details in the future). And I assume you were wondering
On Sun, Aug 16, 2015 at 7:16 PM, Duy Nguyen pclo...@gmail.com wrote:
On Sun, Aug 16, 2015 at 12:17 PM, David Turner dtur...@twopensource.com
wrote:
Previously, some calls lookup_untracked would pass a full path. But
lookup_untracked assumes that the portion of the path up to and
including
When in the middle of t7063, we are sure untracked cache is supported,
so we can use --force-untracked-cache to skip the support detection
phase and save a few seconds. It's also good that --force-untracked-cache
is exercised in the test suite.
Signed-off-by: Nguyễn Thái Ngọc Duy
The first patch is a split from David's v3 patch because it should
belong to dt/untracked-sparse. The second is basically David's v3.
The third patch fixes untracked_cache_invalidate_path(). David fixed
it in v2, but there's another bug in this code.
David Turner (1):
untracked-cache: fix
From: David Turner dtur...@twopensource.com
Previously, some calls lookup_untracked would pass a full path. But
lookup_untracked assumes that the portion of the path up to and
including to the untracked_cache_dir has been removed. So
lookup_untracked would be looking in the untracked_cache for
First, the current code in untracked_cache_invalidate_path() is wrong
because it can only handle paths a or a/b, not a/b/c because
lookup_untracked() only looks for entries directly under the given
directory. In the last case, it will look for the entry b/c in
directory a instead. This means if
Karthik Nayak karthik@gmail.com writes:
+ /*
+ * First we obtain all regular branch refs and if the HEAD is
+ * detached then we insert that ref to the end of the ref_fist
+ * so that it can be printed first.
+ */
Nit: the end of the sentence reads funny (why put
The old option parsing code in this plumbing command predates this
API, so option parsing was done more manually. Using the new API
brings send-pack more in line with push, and accepts new variants
like --no-* for negating options.
Signed-off-by: Dave Borowitz dborow...@google.com
---
This field was set in transport_set_option, but never read in the push
code. The push code basically ignores the smart_options field
entirely, and derives its options from the flags arguments to the
push* callbacks. Note that in git_transport_push there are already
several args set from flags that
Signed-off-by: Dave Borowitz dborow...@google.com
---
cache.h | 1 +
config.c | 6 +++---
2 files changed, 4 insertions(+), 3 deletions(-)
diff --git a/cache.h b/cache.h
index 6bb7119..95d9594 100644
--- a/cache.h
+++ b/cache.h
@@ -1392,6 +1392,7 @@ extern int
Signed-off-by: Dave Borowitz dborow...@google.com
---
Documentation/git-send-pack.txt | 11 ++-
1 file changed, 10 insertions(+), 1 deletion(-)
diff --git a/Documentation/git-send-pack.txt b/Documentation/git-send-pack.txt
index 6a6..0a0a3fb 100644
---
Signed-off-by: Dave Borowitz dborow...@google.com
---
Documentation/config.txt | 8
builtin/push.c | 50 ++--
builtin/send-pack.c | 27 +-
3 files changed, 70 insertions(+), 15 deletions(-)
diff --git
Karthik Nayak karthik@gmail.com writes:
--- a/ref-filter.c
+++ b/ref-filter.c
@@ -155,6 +155,8 @@ static void push_new_state(struct ref_formatting_state
**stack)
strbuf_init(s-output, 0);
s-prev = *stack;
+ if (*stack)
+ s-quote_style =
On Wed, Aug 19, 2015 at 9:19 PM, Matthieu Moy
matthieu@grenoble-inp.fr wrote:
Karthik Nayak karthik@gmail.com writes:
On Thu, Jul 30, 2015 at 12:59 PM, Matthieu Moy
matthieu@grenoble-inp.fr wrote:
IIRC, historicaly Git allowed some weirdly named refs which made some
commands
On Fri, Aug 14, 2015 at 7:22 PM, Junio C Hamano gits...@pobox.com wrote:
Dave Borowitz dborow...@google.com writes:
diff --git a/send-pack.c b/send-pack.c
index 2a64fec..6ae9f45 100644
--- a/send-pack.c
+++ b/send-pack.c
@@ -370,7 +370,7 @@ int send_pack(struct send_pack_args *args,
On Wed, Aug 19, 2015 at 8:26 PM, Matthieu Moy
matthieu@grenoble-inp.fr wrote:
Karthik Nayak karthik@gmail.com writes:
+static void end_atom_handler(struct atom_value *atomv, struct
ref_formatting_state **state)
+{
+ struct ref_formatting_state *current = *state;
+ if
On Wed, Aug 19, 2015 at 8:26 PM, Matthieu Moy
matthieu@grenoble-inp.fr wrote:
Karthik Nayak karthik@gmail.com writes:
This is a temporary step before porting 'tag.c' to use 'ref-filter'
completely. As this is a temporary step, most of the code
introduced here will be removed when
Karthik Nayak karthik@gmail.com writes:
+static void pop_state(struct ref_formatting_state **stack)
+{
+ struct ref_formatting_state *current = *stack;
+ struct ref_formatting_state *prev = current-prev;
+
+ if (prev)
+ strbuf_addbuf(prev-output,
Karthik Nayak karthik@gmail.com writes:
On Thu, Jul 30, 2015 at 12:59 PM, Matthieu Moy
matthieu@grenoble-inp.fr wrote:
IIRC, historicaly Git allowed some weirdly named refs which made some
commands ambiguous (e.g. a branch named after an option like '-d').
We're forbidding their
On Wed, Aug 19, 2015 at 8:26 PM, Matthieu Moy
matthieu@grenoble-inp.fr wrote:
Karthik Nayak karthik@gmail.com writes:
+static void pop_state(struct ref_formatting_state **stack)
+{
+ struct ref_formatting_state *current = *stack;
+ struct ref_formatting_state *prev =
On Wed, Aug 19, 2015 at 8:53 PM, Matthieu Moy
matthieu@grenoble-inp.fr wrote:
Karthik Nayak karthik@gmail.com writes:
Remove the error reporting variable to make the code easier to port
over to using ref-filter APIs. This variable is not require as in
s/is not require/d/
Will
Karthik Nayak karthik@gmail.com writes:
Remove the error reporting variable to make the code easier to port
over to using ref-filter APIs. This variable is not require as in
s/is not require/d/
--
Matthieu Moy
http://www-verimag.imag.fr/~moy/
--
To unsubscribe from this list: send the
On Wed, Aug 19, 2015 at 8:48 PM, Matthieu Moy
matthieu@grenoble-inp.fr wrote:
Karthik Nayak karthik@gmail.com writes:
+ /*
+ * First we obtain all regular branch refs and if the HEAD is
+ * detached then we insert that ref to the end of the ref_fist
+ * so that it
Karthik Nayak karthik@gmail.com writes:
+static void end_atom_handler(struct atom_value *atomv, struct
ref_formatting_state **state)
+{
+ struct ref_formatting_state *current = *state;
+ if (!current-at_end)
+ die(_(format: `end` atom used without a supporting
Karthik Nayak karthik@gmail.com writes:
This is a temporary step before porting 'tag.c' to use 'ref-filter'
completely. As this is a temporary step, most of the code
introduced here will be removed when 'tag.c' is ported over to use
'ref-filter' APIs
If you resend: missing '.' at the end
Add a new flag --signed-if-possible to push and send-pack that sends a
push certificate if and only if the server advertised a push cert
nonce. If not, at least warn the user that their push may not be as
secure as they thought.
Signed-off-by: Dave Borowitz dborow...@google.com
---
Signed-off-by: Dave Borowitz dborow...@google.com
---
Documentation/gitremote-helpers.txt | 3 +++
1 file changed, 3 insertions(+)
diff --git a/Documentation/gitremote-helpers.txt
b/Documentation/gitremote-helpers.txt
index 82e2d15..78e0b27 100644
--- a/Documentation/gitremote-helpers.txt
+++
Like --atomic, --signed will fail if the server does not advertise the
necessary capability. In addition, it requires gpg on the client side.
Signed-off-by: Dave Borowitz dborow...@google.com
---
Documentation/git-push.txt | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git
Signed-off-by: Dave Borowitz dborow...@google.com
---
Documentation/git-send-pack.txt | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/Documentation/git-send-pack.txt b/Documentation/git-send-pack.txt
index b5d09f7..6a6 100644
--- a/Documentation/git-send-pack.txt
+++
Changes since v1:
- Rebased on 44e02239
- Use options --[no-]signed|--signed=(yes|no|if-asked)
- Support general yes/true/1/etc. option parsing.
- Convert builtin/send-pack.c to use option parsing API for better code
reuse.
- Various cleanups as suggested by Junio.
v1 can be found at:
On Wed, Aug 19, 2015 at 9:14 PM, Matthieu Moy
matthieu@grenoble-inp.fr wrote:
Karthik Nayak karthik@gmail.com writes:
--- a/ref-filter.c
+++ b/ref-filter.c
@@ -155,6 +155,8 @@ static void push_new_state(struct ref_formatting_state
**stack)
strbuf_init(s-output, 0);
On Wed, Aug 19, 2015 at 9:24 PM, Karthik Nayak karthik@gmail.com wrote:
On Wed, Aug 19, 2015 at 9:14 PM, Matthieu Moy
matthieu@grenoble-inp.fr wrote:
Karthik Nayak karthik@gmail.com writes:
--- a/ref-filter.c
+++ b/ref-filter.c
@@ -155,6 +155,8 @@ static void
On Wed, Aug 19, 2015 at 8:26 AM, Dave Borowitz dborow...@google.com wrote:
The old option parsing code in this plumbing command predates this
API, so option parsing was done more manually. Using the new API
brings send-pack more in line with push, and accepts new variants
like --no-* for
Stefan Beller sbel...@google.com writes:
+static int module_list_compute(int argc, const char **argv,
+ const char *prefix,
+ struct pathspec *pathspec)
+{
+ int i;
+ char *max_prefix, *ps_matched = NULL;
+ int
Nguyễn Thái Ngọc Duy pclo...@gmail.com writes:
The first patch is a split from David's v3 patch because it should
belong to dt/untracked-sparse. The second is basically David's v3.
The third patch fixes untracked_cache_invalidate_path(). David fixed
it in v2, but there's another bug in this
Junio C Hamano gits...@pobox.com writes:
Micronit. Even though multiplication is commutative, the order of
arguments to xcalloc() looks odd. It lets you say I want an array
with nmemb elements, and each of its is size-bytes long by giving
it nmemb and then size.
Unrelated tangent, but
Paul Tan pyoka...@gmail.com writes:
(In addition, I fixed a small mistake with the struct tree_desc array
size.)
Yeah, one thing I forgot to mention was that I suspect this kind of
oneway merge already in other places in the code, and a future
mini-project might be to inspect them all and try
Stefan Beller sbel...@google.com writes:
From: Heiko Voigt hvo...@hvoigt.net
We should not die when reading the submodule config cache since the user
might not be able to get out of that situation when the configuration is
part of the history.
We should handle this condition later when the
On Wed, Aug 19, 2015 at 11:17 AM, Junio C Hamano gits...@pobox.com wrote:
Stefan Beller sbel...@google.com writes:
+static int module_list_compute(int argc, const char **argv,
+ const char *prefix,
+ struct pathspec *pathspec)
+{
+
On Wed, Aug 19, 2015 at 11:22 AM, Junio C Hamano gits...@pobox.com wrote:
Junio C Hamano gits...@pobox.com writes:
Micronit. Even though multiplication is commutative, the order of
arguments to xcalloc() looks odd. It lets you say I want an array
with nmemb elements, and each of its is
On Wed, Aug 19, 2015 at 12:08 PM, Junio C Hamano gits...@pobox.com wrote:
Stefan Beller sbel...@google.com writes:
`module_clone` is part of the update command, which I want to convert
to C next.
Signed-off-by: Stefan Beller sbel...@google.com
---
This is the only patch that broke by a
Dave Borowitz dborow...@google.com writes:
Signed-off-by: Dave Borowitz dborow...@google.com
---
Documentation/git-send-pack.txt | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/Documentation/git-send-pack.txt b/Documentation/git-send-pack.txt
index b5d09f7..6a6
From: Lars Schneider larsxschnei...@gmail.com
Hi,
as discussed with Luke in [PATCH] git-p4: fix faulty paths for case insensitive
systems I added a test case for my path fix. I also changed the trigger for the
fix to the command line parameter --fix-paths.
Cheers,
Lars
Lars Schneider (1):
On Wed, Aug 19, 2015 at 2:00 PM, Stefan Beller sbel...@google.com wrote:
On Wed, Aug 19, 2015 at 8:26 AM, Dave Borowitz dborow...@google.com wrote:
The old option parsing code in this plumbing command predates this
API, so option parsing was done more manually. Using the new API
brings
From: Lars Schneider larsxschnei...@gmail.com
PROBLEM:
We run P4 servers on Linux and P4 clients on Windows. For an unknown
reason the file path for a number of files in P4 does not match the
directory path with respect to case sensitivity.
E.g. `p4 files` might return
//depot/path/to/file1
Dave Borowitz dborow...@google.com writes:
Add a new flag --signed-if-possible to push and send-pack that sends a
push certificate if and only if the server advertised a push cert
nonce. If not, at least warn the user that their push may not be as
secure as they thought.
Signed-off-by: Dave
527ec39 (generate-cmdlist: parse common group commands, 2015-05-21)
replaced generate-cmdlist.sh with a more functional Perl version,
generate-cmdlist.perl. The Perl version gleans named tags from a new
common groups section in command-list.txt and recognizes those tags in
command list section
On Wed, Aug 19, 2015 at 3:56 PM, Junio C Hamano gits...@pobox.com wrote:
Dave Borowitz dborow...@google.com writes:
Signed-off-by: Dave Borowitz dborow...@google.com
---
Documentation/git-send-pack.txt | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git
On Wed, Aug 19, 2015 at 3:58 PM, Junio C Hamano gits...@pobox.com wrote:
Dave Borowitz dborow...@google.com writes:
Add a new flag --signed-if-possible to push and send-pack that sends a
push certificate if and only if the server advertised a push cert
nonce. If not, at least warn the user
Summary: Certain diffs containing angle brackets ('' or '' characters) are
displayed incorrectly in gitk's markup words and color words modes. The bug
is caused by a feature that isn't aware of the formatting difference for the
word diff modes.
How to reproduce the bug:
Create a file with
Matthieu Moy matthieu@grenoble-inp.fr writes:
You replaced the quote_style argument with ref_formatting_state, and I
think you should have kept this argument and added ref_formatting_state.
The other option is to add an extra indirection like
struct ref_formatting_state {
int
Stefan Beller sbel...@google.com writes:
`module_clone` is part of the update command, which I want to convert
to C next.
Signed-off-by: Stefan Beller sbel...@google.com
---
This is the only patch that broke by a last minute fix,
so this replaces the previous PATCH 7/7.
... which still
git fast-export | git fast-import fails to preserve a commit that replaces
a symlink with a directory. Add a failing test case demonstrating this
bug.
The fast-export output for the commit in question looks like
commit refs/heads/master
mark :4
author …
committer …
data 4
two
M
Dave Borowitz dborow...@google.com writes:
I produced the patch with git format-patch --subject-prefix='PATCH
v2' --cover-letter @{u}.. and mailed with git send-email
--to=git@vger.kernel.org,gits...@pobox.com 0*.patch; is there a way
that would have preserved whitespace better?
No need to
Eric Sunshine sunsh...@sunshineco.com writes:
In addition to reviving 527ec39^:generate-cmdlist.sh and extending it, I
also re-indented it with tabs instead of spaces, so it's helpful to
ignore whitespace changes when comparing the old and new versions of the
shell script.
Hmph. Perhaps we
If you mis-type a git command starting with a non-letter, git
internals will spit out some errors at you.
$ git 5fetch
error: invalid key: pager.5fetch
error: invalid key: alias.5fetch
git: '5fetch' is not a git command. See 'git --help'.
Did you mean this?
fetch
$ git \#fetch
error:
Maybe something along the lines of a git-subtree merge. I am not sure
how to do that exactly, and I am not sure that it would be worth the
trouble to setup for a small case...
On Tue, Aug 18, 2015 at 1:11 PM, Anish Tondwalkar
tondwal...@virginia.edu wrote:
I stashed some changes, then refactored
Paul Tan pyoka...@gmail.com writes:
On Tue, Aug 18, 2015 at 5:36 PM, Matthieu Moy
matthieu@grenoble-inp.fr wrote:
I don't remember the details of the regression we had with the shell
version, but that would probably deserve an additional test to enforce
the Hopefully there will be no
Hello.
When installing the Git is not a dialog box appears with a choice ssh. What
could be the reason?
Здравствуйте.
При установке гита у меня не появляется диалоговое окно с выбором ssh.
Пожалуйста, подскажите в чем может быть причина?
--
Shincho itte kudasai. Anime miteru hito!
--
To
On Mon, Aug 17, 2015 at 12:33:40PM -0700, Junio C Hamano wrote:
Have you checked how this change affects that codepath? To put it
differently, does am skip have the same issue without this fix?
Hmm, I adopted Dscho's test to run git am --skip and it did not fail.
I think it's because am_skip()
On Tue, Aug 18, 2015 at 5:36 PM, Matthieu Moy
matthieu@grenoble-inp.fr wrote:
I don't remember the details of the regression we had with the shell
version, but that would probably deserve an additional test to enforce
the Hopefully there will be no regressions part of your message.
61 matches
Mail list logo