Le 12/04/2016 11:52, Duy Nguyen a écrit :
>
> Ex-translator speaking. Context is important. If it's me, I would even
> go as far as marking the whole line translatable just to give more
> context (it's mostly just copy and translate a few words then). But
> then again, I was half developer half tra
On 04/27/2016 11:15 PM, Junio C Hamano wrote:
> Junio C Hamano writes:
>
>> If a casual reader sees this code:
>>
>> ref_transaction_delete(transaction, r->name, r->sha1,
>> REF_ISPRUNING | REF_NODEREF, NULL, &err)
>>
>> it gives an incorrect impression that there may
For people who still find this thread in Google, there is a way to do that
now:
*git config --add pull.ff only*
>From the documentation:
pull.ff
By default, Git does not create an extra merge commit when merging a
commit that is a descendant of the current commit. Instead, the tip of the
cur
This should handle .gitconfig files that specify things like:
[http]
cookieFile = "~/.gitcookies"
Signed-off-by: Brian Norris
---
Documentation/config.txt | 3 +++
http.c | 2 +-
2 files changed, 4 insertions(+), 1 deletion(-)
diff --git a/Documentation/config.txt b/D
Signed-off-by: Brian Norris
---
Documentation/config.txt | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Documentation/config.txt b/Documentation/config.txt
index 42d2b50477b2..a775ad885a76 100644
--- a/Documentation/config.txt
+++ b/Documentation/config.txt
@@ -1660,7 +1660,7
Junio C Hamano writes:
> A cursory bisection suggests that this used to work up to at least
> v2.7.4; this must be a recent regression.
No, false report. This seems to be broken from ages ago.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord..
Junio C Hamano writes:
> My undertanding of what _should_ happen in the world order as
> currently defined (not necessarily implemented) is:
>
> * "git add -- A B" must work the same way as "git add -- B A" and
>"git add -- A; git add -- B"
>
> * "git add -- path/to/subdir/file", when any o
On Thu, Apr 28, 2016 at 9:48 AM, Junio C Hamano wrote:
> Junio C Hamano writes:
> When given multiple paths, e.g. "git add A/B/C A/D", it tries to
> "optimize" things by first finding common leading directory (in this
> case "A/") and doing something slightly different, and I think the
> bug Andr
On Wed, 2016-04-27 at 18:57 +0200, Michael Haggerty wrote:
> This started as a modest attempt to move the last couple of patches
> mentioned here [1] to before the vtable patches. I wanted to do that
> because having real changes mixed with the vtable refactoring was
> making rebasing and stuff awk
Signed-off-by: Mike Hommey
---
connect.c | 11 ++-
connect.h | 9 +
2 files changed, 11 insertions(+), 9 deletions(-)
diff --git a/connect.c b/connect.c
index c53f3f1..29569b3 100644
--- a/connect.c
+++ b/connect.c
@@ -231,13 +231,6 @@ int server_supports(const char *feature)
Signed-off-by: Mike Hommey
---
connect.c | 108 +-
connect.h | 2 ++
2 files changed, 60 insertions(+), 50 deletions(-)
diff --git a/connect.c b/connect.c
index ce216cb..c71563f 100644
--- a/connect.c
+++ b/connect.c
@@ -649,6 +649,62
Signed-off-by: Mike Hommey
---
connect.c | 27 ---
1 file changed, 12 insertions(+), 15 deletions(-)
Note this makes http://marc.info/?l=git&m=146183714532394 irrelevant.
diff --git a/connect.c b/connect.c
index 29569b3..ce216cb 100644
--- a/connect.c
+++ b/connect.c
@@
On Wed, 2016-04-27 at 18:57 +0200, Michael Haggerty wrote:
+retry:
...
> + if (--attempts_remaining > 0)
> + goto retry;
could this be a loop instead of using gotos?
> + /*
> + * There is a directory in the way,
> +
On Thu, Apr 28, 2016 at 10:41:12AM -0700, Junio C Hamano wrote:
> Mike Hommey writes:
>
> > As you may be aware, I'm working on a git remote helper to access
> > mercurial repositories (https://github.com/glandium/git-cinnabar/).
> >
> > At the moment, a small part is written in C, relying on the
Stefan Beller writes:
> It doesn't even have to be a submodule related thing at all.
>
> I can imagine that `git gc` could learn to walk nested repos
> (not submodules, just repos on disk inside the work tree).
> And for that use case we'd maybe want to have a setting
> to pack the nested repos m
Jeff King writes:
> Keep in mind that submodule interactions may be triggered from other
> non-submodule commands. So "git fetch", for instance, may end up caring
> about whether you pass "http.*" or "credential.*" down to the
> submodules.
> I do not think "fetch" should grow submodule-specific
On Thu, Apr 28, 2016 at 02:08:43PM -0700, Stefan Beller wrote:
> > 1. Ones where we _know_ that the config is nonsense to pass along,
> > _and_ where a user might conceivably make use of the
> > just-the-top-level version of it (core.worktree
> > comes to mind, though of course th
On Thu, Apr 28, 2016 at 4:00 PM, Rafal Klys wrote:
> trailer: load config to handle core.commentChar
This subject is describing low-level details of the patch rather than
giving a high-level overview. A possible rewrite might be:
trailer: respect core.commentChar
> Fall throught git_default
On Thu, Apr 28, 2016 at 2:03 PM, Jeff King wrote:
> On Thu, Apr 28, 2016 at 12:52:03PM -0700, Junio C Hamano wrote:
>
>> "git" is not always about submodules, so "-c-but-not-for-submodules"
>> option does not belong to "git" wrapper.
>>
>> Users use "git -c" and hope to affect what happens in subm
On Thu, Apr 28, 2016 at 2:00 PM, Jeff King wrote:
> On Thu, Apr 28, 2016 at 12:28:21PM -0700, Junio C Hamano wrote:
>
>> Jeff King writes:
>>
>> > It's definitely sufficient, it's just annoying if a user shows up every
>> > week and says "I want X.Y", and then somebody else shows up a week later
On Thu, Apr 28, 2016 at 10:00 PM, Rafal Klys wrote:
> Fall throught git_default_config when reading config to update the
> comment_line_char from default '#' to possible different value set in
> core.commentChar.
>
> Signed-off-by: Rafal Klys
> ---
>
> Added fallthru instead of reading config thi
On Thu, Apr 28, 2016 at 12:52:03PM -0700, Junio C Hamano wrote:
> "git" is not always about submodules, so "-c-but-not-for-submodules"
> option does not belong to "git" wrapper.
>
> Users use "git -c" and hope to affect what happens in submodules,
> only because "git submodule" support is still i
On Thu, Apr 28, 2016 at 12:28:21PM -0700, Junio C Hamano wrote:
> Jeff King writes:
>
> > It's definitely sufficient, it's just annoying if a user shows up every
> > week and says "I want X.Y", and then somebody else shows up a week later
> > and says "I want X.Z".
> >
> > Are we serving any pur
Christian Couder writes:
>> I do not think you need to think about "free"ing.
>
> Yeah, lockfile.h says:
> ...
> and:
> ...
Yup, we are now on the same page.
>> Even if the libified version of the apply internal can be called
>> multiple times to process multiple patch inputs, there is no need
David Turner writes:
> Since there is a manual check for that case, the code will fail at test
> time.
I see a difference between "We make it hard to write incorrect code"
and "We keep it easy to write incorrect code but a runtime check
will catch mistakes anyway", and I tend to prefer the forme
Jean-Noël AVILA writes:
> Cherry-picked, fixed and PR to Jiang Xin
>
Thanks
> JN
> --
> 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://vger.kernel.org/majordomo-info.html
--
To unsubscribe f
Fall throught git_default_config when reading config to update the
comment_line_char from default '#' to possible different value set in
core.commentChar.
Signed-off-by: Rafal Klys
---
Added fallthru instead of reading config third time.
Added test, updated commit message.
I even tried to chan
When there is no .gitmodules file availabe to initialize a submodule
from, `submodule_from_path` just returns NULL. We need to check for
that and abort gracefully.
When `git submodule update` was implemented in shell, this error out
with the warning
Submodule path '%s' not initialized
May
When there is no .gitmodules file availabe to initialize a submodule
from, `submodule_from_path` just returns NULL. We need to check for
that and abort gracefully. When `submodule init` was implemented in shell,
a missing .gitmodules file would result in an error message
No url found for submo
On Thu, Apr 28, 2016 at 12:52 PM, Junio C Hamano wrote:
> Stefan Beller writes:
>
>> So when going with that philosophy, the user might be missing
>> switches like
>>
>> -c-for-this-repo-only core.worktree=... -c
>> submodule.worktree=align-relative-to-parent
>>
>> i.e. when shifting the resp
Junio C Hamano writes:
> Users use "git -c" and hope to affect what happens in submodules,
> only because "git submodule" support is still immature and does not
> have options to do that. You certainly smell a linkage between
> "pass options to a selected subset of submodules" and your recent
>
Stefan Beller writes:
> So when going with that philosophy, the user might be missing
> switches like
>
> -c-for-this-repo-only core.worktree=... -c
> submodule.worktree=align-relative-to-parent
>
> i.e. when shifting the responsibility to the user, we need to have
> switches to pass options
Johannes Schindelin writes:
> - if (starts_with(var, "credential."))
> + if (starts_with(var, "credential.") ||
> + (starts_with(var, "http.") &&
> + ends_with(var, ".extraheader")))
I know you are fond of indenting with HT without aligning things
On Thu, Apr 28, 2016 at 12:28 PM, Junio C Hamano wrote:
> Jeff King writes:
>
>> It's definitely sufficient, it's just annoying if a user shows up every
>> week and says "I want X.Y", and then somebody else shows up a week later
>> and says "I want X.Z".
>>
>> Are we serving any purpose in vettin
Jeff King writes:
> It's definitely sufficient, it's just annoying if a user shows up every
> week and says "I want X.Y", and then somebody else shows up a week later
> and says "I want X.Z".
>
> Are we serving any purpose in vetting each one (and if so, what)?
Personally I do not think we would
Cherry-picked, fixed and PR to Jiang Xin
JN
--
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://vger.kernel.org/majordomo-info.html
On Thu, Apr 28, 2016 at 11:28 AM, Stefan Beller wrote:
> When there is no .gitmodules file availabe to initialize a submodule
> from, `submodule_from_path` just returns NULL. We need to check for
> that and abort gracefully. When `submodule init` was implemented in shell,
> a missing .gitmodules f
On Thu, Apr 28, 2016 at 12:06:56PM -0700, Junio C Hamano wrote:
> Jeff King writes:
>
> > On Thu, Apr 28, 2016 at 09:09:44AM -0700, Stefan Beller wrote:
> >
> >> > I think the key thing with a blacklist is somebody has to go to the work
> >> > to audit the existing keys.
> >>
> >> Would it be s
Jeff King writes:
> On Thu, Apr 28, 2016 at 09:09:44AM -0700, Stefan Beller wrote:
>
>> > I think the key thing with a blacklist is somebody has to go to the work
>> > to audit the existing keys.
>>
>> Would it be sufficient to wait until someone screams at the mailing list
>> for some key to be
David Turner writes:
> From: Nguyễn Thái Ngọc Duy
> ...
> The biggest gain is not having to verify the trailing SHA-1, which
> takes lots of time especially on large index files. But this also
> opens doors for further optimiztions:
optimizAtion
> Git can poke the daemon via unix domain socket
When there is no .gitmodules file availabe to initialize a submodule
from, `submodule_from_path` just returns NULL. We need to check for
that and abort gracefully. When `submodule init` was implemented in shell,
a missing .gitmodules file would result in an error message
No url found for submo
On Thu, Apr 28, 2016 at 10:48:38AM -0700, Stefan Beller wrote:
> >> How does it overwrite the GIT_CURL_VERBOSE variable?
> >
> > You can't use both, as they are both triggered using the CURLOPT_VERBOSE
> > option of curl. The main difference is that with GIT_CURL_VERBOSE, we
> > rely on curl to pr
On Thu, Apr 28, 2016 at 10:44 AM, Jeff King wrote:
> On Thu, Apr 28, 2016 at 10:26:06AM -0700, Stefan Beller wrote:
>
>> > +'GIT_TRACE_CURL'::
>> > + Enables a curl full trace dump of all incoming and outgoing data,
>> > + including descriptive information, of the git transport protoco
On Wed, 2016-04-27 at 14:15 -0700, Junio C Hamano wrote:
> Junio C Hamano writes:
>
> > If a casual reader sees this code:
> >
> > ref_transaction_delete(transaction, r->name, r->sha1,
> >REF_ISPRUNING | REF_NODEREF, NULL, &err)
> >
> > it gives an incorrect impressi
On Thu, Apr 28, 2016 at 10:26:06AM -0700, Stefan Beller wrote:
> > +'GIT_TRACE_CURL'::
> > + Enables a curl full trace dump of all incoming and outgoing data,
> > + including descriptive information, of the git transport protocol.
> > + This is similar to doing curl --trace-ascii
On Wed, 2016-04-27 at 16:37 -0400, Jeff King wrote:
> On Wed, Apr 27, 2016 at 04:34:53PM -0400, David Turner wrote:
>
> > > I thought the point is that one is a lesser check than the other,
> > > and
> > > we
> > > need different rules for different situations. So we might allow
> > > deletion on
Mike Hommey writes:
> As you may be aware, I'm working on a git remote helper to access
> mercurial repositories (https://github.com/glandium/git-cinnabar/).
>
> At the moment, a small part is written in C, relying on the git code
> base, but eventually, there would be more C.
>
> As I want to ge
Jeff King writes:
>> + for (w = 0; (w < width) && (i + w < size); w++) {
>> + if (nohex && (i + w + 1 < size) && ptr[i + w] == '\r'
>> +&& ptr[i + w + 1] == '\n') {
>> +i += (w + 2 - width);
>> +
On Thu, Apr 28, 2016 at 4:57 AM, Elia Pinto wrote:
> Implement the GIT_TRACE_CURL environment variable to allow a
> greater degree of detail of GIT_CURL_VERBOSE, in particular
> the complete transport header and all the data payload exchanged.
> It might be useful if a particular situation could r
Alexander 'z33ky' Hirsch <1ze...@gmail.com> writes:
> Previously git-pull would silently ignore the --verify-signatures
> option.
I do not see a point of making it error out. Adding a warning about
the option being ignored might be a worthwhile thing to do (e.g. it
may solicit responses from tho
On Thu, Apr 28, 2016 at 2:52 AM, Alexander 'z33ky' Hirsch
<1ze...@gmail.com> wrote:
> Previously git-pull would silently ignore the --verify-signatures
> option.
>
> Signed-off-by: Alexander 'z33ky' Hirsch <1ze...@gmail.com>
> ---
>
> We had some discussion back in December about a patch that added
CC'ed Jean-Noël who maintains French translation.
Antonin writes:
> "tous le dépôts distants" -> "tous les dépôts distants"
> ---
> po/fr.po | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/po/fr.po b/po/fr.po
> index 88b0b8a7..36c7c99 100644
> --- a/po/fr.po
> +++ b/po/
On Thu, Apr 28, 2016 at 9:39 AM, Junio C Hamano wrote:
> Stefan Beller writes:
>
>> I think (pure speculation), that it the error is in the context
>> (repository) switching logic.
>> What happens if you alter the order, i.e. give testfile first and then
>> the files in the nested
>> repos?
>>
>>
Hi,
On Thu, 28 Apr 2016, Jeff King wrote:
> On Thu, Apr 28, 2016 at 08:25:29AM -0700, Stefan Beller wrote:
>
> > > +test_expect_success 'cmdline credential config passes submodule update' '
> > > + # advance the submodule HEAD so that a fetch is required
> > > + git commit --allow-em
On Thu, Apr 28, 2016 at 09:09:44AM -0700, Stefan Beller wrote:
> > I think the key thing with a blacklist is somebody has to go to the work
> > to audit the existing keys.
>
> Would it be sufficient to wait until someone screams at the mailing list
> for some key to be blacklisted? (I mean in the
Junio C Hamano writes:
> * "git add -- path/to/subdir/file", when any of path/, path/to/,
>path/to/subdir/ is a Git repository that is different from the
>current Git repository, must fail.
If any of the leading directories in that long path is actually a
git repository that is differen
Stefan Beller writes:
> I think (pure speculation), that it the error is in the context
> (repository) switching logic.
> What happens if you alter the order, i.e. give testfile first and then
> the files in the nested
> repos?
>
> git add -- file path/to/subdir/file
>
> should do internally
On Wed, Apr 27, 2016 at 11:10 PM, Andrew J wrote:
> Hi Stefan,
>
> On Wed, Apr 27, 2016 at 9:08 AM, Stefan Beller wrote:
>> I think (pure speculation), that it the error is in the context
>> (repository) switching logic.
>> What happens if you alter the order, i.e. give testfile first and then
>>
On Mon, Apr 25, 2016 at 7:55 PM, Junio C Hamano wrote:
> Eric Sunshine writes:
>
>>> + /*
>>> +* Since lockfile.c keeps a linked list of all created
>>> +* lock_file structures, it isn't safe to free(lock_file).
>>> +*/
>>> + struct lock_file *lock_file;
>>
>>
> On 28 Apr 2016, at 15:38, Jeff King wrote:
>
> These functions should be used by any code which spawns a
> submodule process, which may happen in submodule.c (e.g.,
> for spawning fetch). Let's move them there and make them
> public so that submodule--helper can continue to use them.
>
> Sin
On Thu, Apr 28, 2016 at 7:12 AM, Mike Hommey wrote:
> Some remote systems can employ restricted shells that aren't very smart
> with quotes, so avoid quoting when it's not strictly necessary.
>
> The list of "safe" characters comes from Mercurial's shell quoting
> function used for its ssh client
On Thu, Apr 28, 2016 at 8:39 AM, Jeff King wrote:
> On Thu, Apr 28, 2016 at 08:37:10AM -0700, Jacob Keller wrote:
>
>> I think I prefer a blacklist approach, since it reduces the need for
>> future changes, since most cases will either not put config on the
>> environment or (based on feedback on
On Thu, Apr 28, 2016 at 8:56 AM, Stefan Beller wrote:
> On Thu, Apr 28, 2016 at 7:02 AM, Johannes Schindelin
> wrote:
>> Hi Peff,
>>
>> On Thu, 28 Apr 2016, Jeff King wrote:
>>
>>> On Thu, Apr 28, 2016 at 08:17:53AM -0400, Jeff King wrote:
>>>
>>> > So that case _is_ correct right now. It's just
On Thu, Apr 28, 2016 at 7:02 AM, Johannes Schindelin
wrote:
> Hi Peff,
>
> On Thu, 28 Apr 2016, Jeff King wrote:
>
>> On Thu, Apr 28, 2016 at 08:17:53AM -0400, Jeff King wrote:
>>
>> > So that case _is_ correct right now. It's just that t5550 isn't testing
>> > the shell script part, which is brok
On Thu, Apr 28, 2016 at 08:37:10AM -0700, Jacob Keller wrote:
> I think I prefer a blacklist approach, since it reduces the need for
> future changes, since most cases will either not put config on the
> environment or (based on feedback on the mailing list and bug reports)
> the user will believe
On Thu, Apr 28, 2016 at 8:36 AM, Jeff King wrote:
> On Thu, Apr 28, 2016 at 08:28:29AM -0700, Jacob Keller wrote:
>
>> > diff --git a/git-submodule.sh b/git-submodule.sh
>> > index 2a84d7e..3a40d4b 100755
>> > --- a/git-submodule.sh
>> > +++ b/git-submodule.sh
>> > @@ -200,6 +200,7 @@ sanitize_sub
On Thu, Apr 28, 2016 at 8:28 AM, Jeff King wrote:
> On Thu, Apr 28, 2016 at 08:25:29AM -0700, Stefan Beller wrote:
>
>> > +test_expect_success 'cmdline credential config passes submodule update' '
>> > + # advance the submodule HEAD so that a fetch is required
>> > + git commit --allow
On Thu, Apr 28, 2016 at 08:30:45AM -0700, Stefan Beller wrote:
> > +/*
> > + * This function is intended as a callback for use with
> > + * git_config_from_parameters(). It ignores any config options which
> > + * are not suitable for passing along to a submodule, and accumulates the
> > rest
> >
On Thu, Apr 28, 2016 at 6:49 AM, Jeff King wrote:
> On Thu, Apr 28, 2016 at 02:19:37PM +0200, Johannes Schindelin wrote:
>
>> > Should we consider just white-listing all of "http.*"?
>> >
>> > That would help other cases which have come up, like:
>> >
>> > http://thread.gmane.org/gmane.comp.vers
On Thu, Apr 28, 2016 at 08:28:29AM -0700, Jacob Keller wrote:
> > diff --git a/git-submodule.sh b/git-submodule.sh
> > index 2a84d7e..3a40d4b 100755
> > --- a/git-submodule.sh
> > +++ b/git-submodule.sh
> > @@ -200,6 +200,7 @@ sanitize_submodule_env()
> > sanitized_config=$(git submodule--
On Thu, Apr 28, 2016 at 6:38 AM, Jeff King wrote:
> These functions should be used by any code which spawns a
> submodule process, which may happen in submodule.c (e.g.,
> for spawning fetch). Let's move them there and make them
> public so that submodule--helper can continue to use them.
>
> Sine
On Thu, Apr 28, 2016 at 08:25:29AM -0700, Stefan Beller wrote:
> > +test_expect_success 'cmdline credential config passes submodule update' '
> > + # advance the submodule HEAD so that a fetch is required
> > + git commit --allow-empty -m foo &&
> > + git push "$HTTPD_DOCUMENT_RO
On Thu, Apr 28, 2016 at 6:37 AM, Jeff King wrote:
> Commit 14111fc (git: submodule honor -c credential.* from
> command line, 2016-02-29) taught git-submodule.sh to save
> the sanitized value of $GIT_CONFIG_PARAMETERS when clearing
> the environment for a submodule. However, it failed to
> export
On Thu, Apr 28, 2016 at 8:25 AM, Jeff King wrote:
> On Thu, Apr 28, 2016 at 08:24:05AM -0700, Jacob Keller wrote:
>
>> On Thu, Apr 28, 2016 at 6:36 AM, Jeff King wrote:
>> > Commit 14111fc (git: submodule honor -c credential.* from
>> > command line, 2016-02-29) accidentally wrote $HTTP_URL. It
>
On Thu, Apr 28, 2016 at 8:21 AM, Stefan Beller wrote:
> On Thu, Apr 28, 2016 at 6:37 AM, Jeff King wrote:
>> Right now we test only the cloning case, but there are other
>> interesting cases (e.g., fetching). Let's pull the setup
>> bits into their own test, which will make things flow more
>> lo
On Thu, Apr 28, 2016 at 08:21:21AM -0700, Stefan Beller wrote:
> > -test_expect_success 'cmdline credential config passes into submodules' '
> > +test_expect_success 'set up repo with http submodules' '
>
> set up or setup?
>
> $ grep -r "set up" |wc -l
> 69
> $ grep -r "setup" |wc -l
> 1162
>
On Thu, Apr 28, 2016 at 6:37 AM, Jeff King wrote:
> Commit 14111fc (git: submodule honor -c credential.* from
> command line, 2016-02-29) taught git-submodule.sh to save
> the sanitized value of $GIT_CONFIG_PARAMETERS when clearing
> the environment for a submodule. However, it failed to
> export
On Thu, Apr 28, 2016 at 08:24:05AM -0700, Jacob Keller wrote:
> On Thu, Apr 28, 2016 at 6:36 AM, Jeff King wrote:
> > Commit 14111fc (git: submodule honor -c credential.* from
> > command line, 2016-02-29) accidentally wrote $HTTP_URL. It
> > happened to work because we ended up with "credential.
Hi Jeff, thanks for your comments.
On Mon, Apr 25, 2016 at 11:37 PM, Jeff King wrote:
> On Mon, Apr 25, 2016 at 11:15:05PM +0200, Guido Martínez wrote:
>
>> I run a server with several git mirrors, that are updated every hour. On
>> that same server, users clone those projects and work on them. W
On Thu, Apr 28, 2016 at 6:36 AM, Jeff King wrote:
> Commit 14111fc (git: submodule honor -c credential.* from
> command line, 2016-02-29) accidentally wrote $HTTP_URL. It
> happened to work because we ended up with "credential..helper",
> which we treat the same as "credential.helper", applying it
On Thu, Apr 28, 2016 at 6:37 AM, Jeff King wrote:
> Right now we test only the cloning case, but there are other
> interesting cases (e.g., fetching). Let's pull the setup
> bits into their own test, which will make things flow more
> logically once we start adding more tests which use the
> setup
On Thu, Apr 28, 2016 at 11:57:48AM +, Elia Pinto wrote:
> diff --git a/imap-send.c b/imap-send.c
> index 938c691..61c6787 100644
> --- a/imap-send.c
> +++ b/imap-send.c
> @@ -1444,6 +1444,12 @@ static CURL *setup_curl(struct imap_server_conf *srvc)
> if (0 < verbosity || getenv("GIT_CURL
On Thu, Apr 28, 2016 at 11:57:47AM +, Elia Pinto wrote:
> +static void curl_dump(const char *text, unsigned char *ptr, size_t size,
> char nohex, char nopriv)
We usually use "int" for our boolean flags. Space savings don't matter
outside of a struct (and if they did, you should be using a si
Currently, the path extracted from the url is passed as last argument to
the program/command passed to git_connect(). In every case the function
is used in the git code base, it's enough, but in order to allow the
reuse of e.g. the GIT_SSH/GIT_SSH_COMMAND logic, additional flexibility
is welcome.
In Mercurial ssh urls, the path part of the url is relative to the home
directory of the account being logged to instead of being absolute.
Add a flag allowing git_connect() to handle this kind of usecase.
Signed-off-by: Mike Hommey
---
connect.c | 10 +++---
connect.h | 1 +
2 files chang
As you may be aware, I'm working on a git remote helper to access
mercurial repositories (https://github.com/glandium/git-cinnabar/).
At the moment, a small part is written in C, relying on the git code
base, but eventually, there would be more C.
As I want to get rid of the dependency on Mercuri
It can be useful to the caller of git_connect() to get access to stderr,
so add a flag that makes start_command allocate a file descriptor for
it.
Signed-off-by: Mike Hommey
---
connect.c | 2 ++
connect.h | 1 +
2 files changed, 3 insertions(+)
diff --git a/connect.c b/connect.c
index 919bf9e.
Some remote systems can employ restricted shells that aren't very smart
with quotes, so avoid quoting when it's not strictly necessary.
The list of "safe" characters comes from Mercurial's shell quoting
function used for its ssh client side. There likely are more that could
be added to the list.
Hi Peff,
On Thu, 28 Apr 2016, Jeff King wrote:
> On Thu, Apr 28, 2016 at 08:17:53AM -0400, Jeff King wrote:
>
> > So that case _is_ correct right now. It's just that t5550 isn't testing
> > the shell script part, which is broken. Probably running "git submodule
> > update" in the resulting clone
On Thu, Apr 28, 2016 at 02:19:37PM +0200, Johannes Schindelin wrote:
> > Should we consider just white-listing all of "http.*"?
> >
> > That would help other cases which have come up, like:
> >
> > http://thread.gmane.org/gmane.comp.version-control.git/264840
> >
> > which wants to turn off h
On Thu, Apr 28, 2016 at 02:19:37PM +0200, Johannes Schindelin wrote:
> > Should we consider just white-listing all of "http.*"?
> >
> > That would help other cases which have come up, like:
> >
> > http://thread.gmane.org/gmane.comp.version-control.git/264840
> >
> > which wants to turn off h
Hi,
currently I am using --cc-cmd for larger patch series which are of
interest of different parties (e.g. kernel tree wide stuff) and I do not
want to spam all of them by patches which are not of their interest but
I still want them to receive the cover letter because that is useful to
get a conte
On Thu, Apr 28, 2016 at 02:05:20PM +0200, Lars Schneider wrote:
> I am no expert in the Submodule code but I think the cloning of
> the submodules is not yet guarded with sanitize_submodule_env [3].
> That means the submodule is cloned with the GIT_CONFIG_PARAMETERS
> of the super project. That mi
Before 14111fc (git: submodule honor -c credential.* from
command line, 2016-02-29), it was sufficient for code which
spawned a process in a submodule to just set the child
process's "env" field to "local_repo_env" to clear the
environment of any repo-specific variables.
That commit introduced a m
These functions should be used by any code which spawns a
submodule process, which may happen in submodule.c (e.g.,
for spawning fetch). Let's move them there and make them
public so that submodule--helper can continue to use them.
Sine they're now public, let's also provide a basic overview
of th
Commit 14111fc (git: submodule honor -c credential.* from
command line, 2016-02-29) taught git-submodule.sh to save
the sanitized value of $GIT_CONFIG_PARAMETERS when clearing
the environment for a submodule. However, it failed to
export the result, meaning that it had no effect for any
sub-program
Commit 14111fc (git: submodule honor -c credential.* from
command line, 2016-02-29) accidentally wrote $HTTP_URL. It
happened to work because we ended up with "credential..helper",
which we treat the same as "credential.helper", applying it
to all URLs.
Signed-off-by: Jeff King
---
t/t5550-http-
Right now we test only the cloning case, but there are other
interesting cases (e.g., fetching). Let's pull the setup
bits into their own test, which will make things flow more
logically once we start adding more tests which use the
setup.
Let's also introduce some whitespace to the clone-test to
On Thu, Apr 28, 2016 at 08:17:53AM -0400, Jeff King wrote:
> So that case _is_ correct right now. It's just that t5550 isn't testing
> the shell script part, which is broken. Probably running "git submodule
> update" in the resulting clone would cover that.
>
> And for the fetch case, we probably
Previously git-pull would silently ignore the --verify-signatures
option.
Signed-off-by: Alexander 'z33ky' Hirsch <1ze...@gmail.com>
---
I made the error-message conform to the CodingGuidelines (removed
capitalization and full stop).
Also, in the previous mail I said that I proposed a patch for
1 - 100 of 121 matches
Mail list logo