Enrico Weigelt writes:
>> > Their webserver seems to be configured quite restrictively
>> > (eg. cannot access files like 'packed-refs').
>>
>> Probably it just doesn't exist.
>
> Aren' these files requied ?
No. If you've never run git pack-refs it's never created.
Andreas.
--
Andreas Schwa
Add a flag for indicating an update to a reference requires force.
Currently the nonfastforward flag of a ref is used for this when
generating status the status message. A separate flag insulates the
status logic from the details of set_ref_status_for_push().
Signed-off-by: Chris Rorvick
---
ca
References are allowed to update from one commit-ish to another if the
former is a ancestor of the latter. This behavior is oriented to
branches which are expected to move with commits. Tag references are
expected to be static in a repository, though, thus an update to a
tag (lightweight and anno
If the reference exists on the remote and the the update is not a
delete, then mark as an update. This is in preparation for handling
tags and branches differently when pushing.
Signed-off-by: Chris Rorvick
---
cache.h | 1 +
remote.c | 18 +++---
2 files changed, 12 insertions(+)
Advising the user to fetch and merge only makes sense if the rejected
reference is a branch. If none of the rejections were for branches,
tell the user they need to force the update(s).
Signed-off-by: Chris Rorvick
---
builtin/push.c | 16 ++--
cache.h| 1 +
remote.c
Pass all rejection reasons back from transport_push(). The logic is
simpler and more flexible with regard to providing useful feedback.
Signed-off-by: Chris Rorvick
---
builtin/push.c | 13 -
builtin/send-pack.c | 4 ++--
transport.c | 17 -
transport.h
Minor changes since from v2 set. Reposting primarily because I mucked
up the Cc: list (again) and hoping to route feedback to the appropriate
audience.
This patch set can be divided into two sets:
1. Provide useful advice for rejected tag references.
push: return reject reasons via a mas
On Sun, Nov 11, 2012 at 07:12:26PM -0800, Junio C Hamano wrote:
> I am physically back but still am far from catching up with the list
> traffic (yet). After skimming my inbox, it appears that you have
> done a magnificent job keeping track of topics in flight and merging
> low-risk fixes early t
I am physically back but still am far from catching up with the list
traffic (yet). After skimming my inbox, it appears that you have
done a magnificent job keeping track of topics in flight and merging
low-risk fixes early to the 'master' branch.
I am hoping that I can ask you to do another issu
> > Their webserver seems to be configured quite restrictively
> > (eg. cannot access files like 'packed-refs').
>
> Probably it just doesn't exist.
Aren' these files requied ?
cu
--
Mit freundlichen Grüßen / Kind regards
Enrico Weigelt
VNC - Virtual Network Consult GmbH
Head Of Developmen
On Sun, 11 Nov 2012, Deniz Türkoglu wrote:
I understand from the feedback that gerrit should get better on making
it possible to review code via e-mail, as pointed out in Nguyen's
mail, a flow like Shawn mentioned[1] can be a good solution.
FWIW, I can fetch the change(s) from gerrit I am inter
I understand from the feedback that gerrit should get better on making
it possible to review code via e-mail, as pointed out in Nguyen's
mail, a flow like Shawn mentioned[1] can be a good solution.
FWIW, I can fetch the change(s) from gerrit I am interested in and
review it any time I want. I curr
The cygwin project recently switched to a new implementation of the
windows api, now using header files from the mingw-64 project. These
new header files are incompatible with the way cygwin.c included the
old headers: cygwin.c can be compiled using the new or the older (mingw)
headers, but differe
Hi,
On Sun, 11 Nov 2012, Jeff King wrote:
> On Sun, Nov 11, 2012 at 05:46:32PM +0100, René Scharfe wrote:
>
> > >However, I couldn't reproduce it on Linux : where the windows
> > >implementations crashes at a ~32000 depth (*not* exactly 32768, mind
> > >you), on linux it happily went through 100
It is currently not possible to use the short-form output of git status
without declaring an alias to do so.
This isn't always desirable therfore, define a git config option which can
be set to display the short-form: status.shortwithbranch
Signed-off-by: Thomas Adam
---
builtin/commit.c | 12
Hi,
It was asked recently whether git status could output the short-form instead
of the long output (via its "-sb" options). To that end, I've created a
rough POC on how this might look. It's deliberately lacking documentation;
I was curious to know whether:
status.shortwithbranch = true
Was g
Support for the "p4 move" command was added in 8e9497c (git p4:
add support for 'p4 move' in P4Submit, 2012-07-12), which checks
to make sure the client and server support the command.
But older versions of p4d may not handle the "-k" argument, and
newer p4d allow disabling "p4 move" with a config
Felipe Contreras writes:
> Implemented now. I'm not handling the 'tip' revision, but most likely
> it's also the '.' revision. In this case a fake 'master' bookmark will
> be created to track that revision.
Hi Felipe. Sorry for the slow response, I've been snowed under with work and
have only ju
Krzysztof Mazur writes:
> That's why I'm proposing in case of SIGQUIT just killing the editor
> (SIGTERM is sufficient for ed).
>
> So git will ignore SIGINT, but die on SIGQUIT (and kill editor
> that ignores SIGQUIT).
system(3) also ignores SIGQUIT.
Andreas.
--
Andreas Schwab, sch...@linux-
On Sun, Nov 11, 2012 at 9:49 PM, Max Horn wrote:
>
> On 11.11.2012, at 14:59, Felipe Contreras wrote:
>
>> People seeking for reference would find it useful.
>
> Hm, I don't understand this commit message. Probably means I am j git
> fast-export --use-done-featureust too dumb, but since I am one
On Sun, 11 Nov 2012, Krzysztof Mazur wrote:
On Sun, Nov 11, 2012 at 11:09:36AM -0200, Thiago Farina wrote:
On Sun, Nov 11, 2012 at 10:14 AM, Felipe Contreras
wrote:
Requiring everyone to use a web browser would limit the amount of ways
people can review patches.
I don't see that as a limitat
On 11.11.2012, at 14:59, Felipe Contreras wrote:
> When an object has already been exported (and thus is in the marks) it's
> flagged as SHOWN, so it will not be exported again, even if in a later
> time it's exported through a different ref.
>
> We don't need the object to be exported again, bu
On 11.11.2012, at 14:59, Felipe Contreras wrote:
> Basically this is what we want:
>
> == pull ==
>
> testgit transport-helper
>
> * export -> import
>
> # testgit.marks git.marks
>
> == push ==
>
> testgit transp
On 11.11.2012, at 14:59, Felipe Contreras wrote:
> It's way simpler. It exerceises the same features of remote helpers.
Typo: exerceises => exercises
Cheers,
Max
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordo
On 11.11.2012, at 14:59, Felipe Contreras wrote:
> People seeking for reference would find it useful.
Hm, I don't understand this commit message. Probably means I am just too dumb,
but since I am one of those people who would likely be seeking for reference, I
would really appreciate if it cou
On Sun, Nov 11, 2012 at 03:24:19PM -0500, Paul Fox wrote:
> krzysztof wrote:
> > Looks ok, but what about SIGQUIT? Some editors like GNU ed (0.4 and 1.6)
> > ignore SIGQUIT, and after SIGQUIT git dies, but editor is still running.
> > After pressing any key ed receives -EIO and prints "stdin: In
krzysztof wrote:
> On Sun, Nov 11, 2012 at 11:31:00AM -0500, Jeff King wrote:
> >
> > Here's a series that I think should resolve the situation for everybody.
> >
> > [1/5]: launch_editor: refactor to use start/finish_command
> >
> > The cleanup I sent out a few minutes ago.
> >
> >
On Sun, Nov 11, 2012 at 1:48 PM, Ramkumar Ramachandra
wrote:
> Felipe Contreras wrote:
>> On Sun, Nov 11, 2012 at 11:32 AM, Ramkumar Ramachandra
>> wrote:
>>> I'm experiencing test failures in contrib/remote-helpers.
>>
>> Which are your versions of hg, and bzr?
>
> Mercurial Distributed SCM (ver
Am 11.11.2012 17:57, schrieb Jeff King:
> @@ -51,6 +51,8 @@ int launch_editor(const char *path, struct strbuf *buffer,
> const char *const *en
> sigchain_push(SIGINT, SIG_IGN);
> ret = finish_command(&p);
> sigchain_pop(SIGINT);
> + if (WIFSIGN
On Sun, Nov 11, 2012 at 11:31:00AM -0500, Jeff King wrote:
>
> Here's a series that I think should resolve the situation for everybody.
>
> [1/5]: launch_editor: refactor to use start/finish_command
>
> The cleanup I sent out a few minutes ago.
>
> [2/5]: launch_editor: ignore SIGINT while
On Sun, Nov 11, 2012 at 7:14 PM, Jeff King wrote:
> On Sun, Nov 11, 2012 at 06:45:32PM +0100, Felipe Contreras wrote:
>
>> > If there is a standard filter, then what is the advantage in doing it as
>> > a pipe? Why not just teach fast-import the same trick (and possibly make
>> > it optional)? Tha
On Sun, Nov 11, 2012 at 01:09:02PM -0500, Jason Pyeron wrote:
> > My google fu has failed me on this issue. I am trying to setup http(s)
> > repositories for git. If I require authenticated users then git asks
> > for a username and password for the first volley of communications, but
> > then doe
jeff wrote:
> On Sun, Nov 11, 2012 at 10:48:46AM -0500, Jeff King wrote:
>
> > Silly me. When I thought through the impact of Paul's patch, I knew that
> > we would notice signal death of the editor. But I totally forgot to
> > consider that the blocked signal is inherited by the child proces
On 11/11/2012 12:15 PM, Jeff King wrote:
On Sun, Nov 11, 2012 at 12:00:44PM -0500, A Large Angry SCM wrote:
a) Leave the name conversion to the export tools, and when they miss
some weird corner case, like 'Author
[...]
b) Do the name conversion in fast-import itself, perhaps optionally,
so if
On Sun, Nov 11, 2012 at 06:45:32PM +0100, Felipe Contreras wrote:
> > If there is a standard filter, then what is the advantage in doing it as
> > a pipe? Why not just teach fast-import the same trick (and possibly make
> > it optional)? That would be simpler, more efficient, and it would make
> >
Hi,
On Sun, Nov 11, 2012 at 6:18 PM, Ramkumar Ramachandra
wrote:
> Felipe Contreras wrote:
>> The code finds the changes in each commit in the list, runs 'git blame'
>> to see which other commits are relevant to those lines, and then adds
>> the author and signer to the list.
>>
>> Finally, it ca
I had to munge the body so the list would post it.
See: http://vger.kernel.org/majordomo-taboos.txt
> -Original Message-
> From: Pyeron, Jason J CTR (US)
> Sent: Thursday, November 08, 2012 2:49 PM
> To: git@vger.kernel.org
> Subject: Failure using webdav basic auth by git client
>
> M
On Sun, Nov 11, 2012 at 6:12 PM, Ramkumar Ramachandra
wrote:
> Felipe Contreras wrote:
>> I got really tired of 'git send-email' always asking me from which address
>> to send mails... that's already configured.
>
> Use sendemail.from. The email sender doesn't necessarily have to be the
> autho
On Sun, Nov 11, 2012 at 5:36 PM, Torsten Bögershausen wrote:
> On 11.11.12 14:59, Felipe Contreras wrote:
>> test_expect_success 'test biridectionality' '
>> + echo -n > marks-cur &&
>> + echo -n > marks-new &&
> Unless I messed up the patch:
>
> Minor issue: still a typo "biridectionality
On Sun, Nov 11, 2012 at 6:39 PM, A Large Angry SCM wrote:
> On 11/11/2012 12:16 PM, Felipe Contreras wrote:
>> And how do you propose to find the commit commands without parsing all
>> the other commands? If you randomly look for lines that begin with
>> 'commit /refs' you might end up in the mid
On Sun, Nov 11, 2012 at 6:15 PM, Jeff King wrote:
> On Sun, Nov 11, 2012 at 12:00:44PM -0500, A Large Angry SCM wrote:
> If there is a standard filter, then what is the advantage in doing it as
> a pipe? Why not just teach fast-import the same trick (and possibly make
> it optional)? That would b
On Sat, Nov 10, 2012 at 06:33:38PM +, Ramsay Jones wrote:
> > We should probably wrap it. I'm planning to queue this on top of Chris's
> > patch:
>
> Unfortunately, I haven't had time yet to test this patch. (Early this week, I
> went into hospital for a "minor" surgical procedure - I have no
On 11/11/2012 12:16 PM, Felipe Contreras wrote:
On Sun, Nov 11, 2012 at 6:00 PM, A Large Angry SCM wrote:
On 11/11/2012 07:41 AM, Felipe Contreras wrote:
Such a filter would probably be quite complicated, and would decrease
performance.
Really?
The fast import stream protocol is pretty si
Hi Felipe,
Felipe Contreras wrote:
> The code finds the changes in each commit in the list, runs 'git blame'
> to see which other commits are relevant to those lines, and then adds
> the author and signer to the list.
>
> Finally, it calculates what percentage of the total relevant commits
> each
On Sun, Nov 11, 2012 at 6:00 PM, A Large Angry SCM wrote:
> On 11/11/2012 07:41 AM, Felipe Contreras wrote:
>> Such a filter would probably be quite complicated, and would decrease
>> performance.
>
> Really?
>
> The fast import stream protocol is pretty simple. All the filter really
> needs to d
On Sun, Nov 11, 2012 at 12:00:44PM -0500, A Large Angry SCM wrote:
> >>>a) Leave the name conversion to the export tools, and when they miss
> >>>some weird corner case, like 'Author >>>consequences, perhaps after an hour of the process.
> [...]
> >>>b) Do the name conversion in fast-import itself
On Sun, Nov 11, 2012 at 11:09:36AM -0200, Thiago Farina wrote:
> On Sun, Nov 11, 2012 at 10:14 AM, Felipe Contreras
> wrote:
> > Requiring everyone to use a web browser would limit the amount of ways
> > people can review patches.
> I don't see that as a limitation as I think everyone has access t
You can run it like format-patch:
% git cc-cmd master..my-branch
And you'll get relevant people to Cc.
The code finds the changes in each commit in the list, runs 'git blame'
to see which other commits are relevant to those lines, and then adds
the author and signer to the list.
Finally, it ca
There's no point in asking this over and over if the user already
properly configured his/her name and email.
Signed-off-by: Felipe Contreras
---
I got really tired of 'git send-email' always asking me from which address to
send mails... that's already configured.
git-send-email.perl | 11 +++
cc-cmd is only per-file, and many times receipients get lost without
seing the full patch series.
So, add an option for series-cc-cmd, which receives as an argument
rev-list options, just like format-patch.
Signed-off-by: Felipe Contreras
---
Documentation/git-send-email.txt | 7 +++
If we pass the full command to run, we should be able to use it in more
extensible ways--see next patch.
Signed-off-by: Felipe Contreras
---
git-send-email.perl | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/git-send-email.perl b/git-send-email.perl
index aea66a0..26d447
cc-cmd is probably not that useful, what we really want is to make a list of
people to Cc on a per-patch-series basis.
This patch series allows just that.
Felipe Contreras (2):
send-email: refactor recipients_cmd()
send-email: add series-cc-cmd option
Documentation/git-send-email.txt
The user's editor likely catches SIGINT (ctrl-C). but if
the user spawns a command from the editor and uses ctrl-C to
kill that command, the SIGINT will likely also kill git
itself (depending on the editor, this can leave the terminal
in an unusable state).
Signed-off-by: Paul Fox
Signed-off-by:
On 11/11/2012 07:41 AM, Felipe Contreras wrote:
On Sat, Nov 10, 2012 at 8:25 PM, A Large Angry SCM wrote:
On 11/10/2012 01:43 PM, Felipe Contreras wrote:
So, the options are:
a) Leave the name conversion to the export tools, and when they miss
some weird corner case, like 'Author
c) Do the
Currently, 'git diff --submodule' displays output with a bold diff
header for non-submodules. So this part is in bold:
diff --git a/file1 b/file1
index 30b2f6c..2638038 100644
--- a/file1
+++ b/file1
For submodules, the header looks like this:
Submodule submodule1 012b072..2
Introduce a diff.submodule configuration variable corresponding to the
'--submodule' command-line option of 'git diff'.
Signed-off-by: Ramkumar Ramachandra
---
Documentation/diff-config.txt|7 ++
Documentation/diff-options.txt |3 +-
cache.h |1 +
d
v1 is here:
http://mid.gmane.org/1349196670-2844-1-git-send-email-artag...@gmail.com
v2 is here:
http://mid.gmane.org/1351766630-4837-1-git-send-email-artag...@gmail.com
This version was prepared in response to Peff's review of v2. As
suggested, I've created a separate function which both '--su
19299a8 (Documentation: Move diff..* from config.txt to
diff-config.txt, 2011-04-07) moved the diff configuration options to
diff-config.txt, but forgot about diff.wordRegex, which was left
behind in config.txt. Fix this.
Signed-off-by: Ramkumar Ramachandra
---
Documentation/config.txt |
From: Paul Fox
The user's editor likely catches SIGINT (ctrl-C). but if
the user spawns a command from the editor and uses ctrl-C to
kill that command, the SIGINT will likely also kill git
itself (depending on the editor, this can leave the terminal
in an unusable state).
Signed-off-by: Paul Fo
We block SIGINT while the editor runs so that git is not
killed accidentally by a stray "^C" meant for the editor or
its subprocesses. This works because most editors ignore
SIGINT.
However, some editor wrappers, like emacsclient, expect to
die due to ^C. We detect the signal death in the editor a
SIGINT is not generally an interesting signal to the user,
since it is typically caused by them hitting "^C" or
otherwise telling their terminal to send the signal.
Signed-off-by: Jeff King
---
I thought about making this an optional parameter for run-command, but
it seems like everybody would wa
git-svn reads usernames and other user queries from an interactive
terminal. This cause GUIs (w/o STDIN connected) to hang waiting forever
for git-svn to complete
(http://code.google.com/p/tortoisegit/issues/detail?id=967).
This change extends the Git::prompt helper, so that it can also be used
f
We do not actually use this parameter; instead we complain
from the child itself (for fork/exec) or from start_command
(if we are using spawn on Windows).
Signed-off-by: Jeff King
---
Just a cleanup I noticed while in the area.
run-command.c | 7 +++
1 file changed, 3 insertions(+), 4 delet
The launch_editor function uses the convenient run_command_*
interface. Let's use the more flexible start_command and
finish_command functions, which will let us manipulate the
parent state while we're waiting for the child to finish.
Signed-off-by: Jeff King
---
editor.c | 10 +-
1 file
On Sun, Nov 11, 2012 at 05:46:32PM +0100, René Scharfe wrote:
> >However, I couldn't reproduce it on Linux : where the windows
> >implementations crashes at a ~32000 depth (*not* exactly 32768, mind
> >you), on linux it happily went through 10 commits. I didn't take
> >time to look much furthe
git-svn reads passwords from an interactive terminal or by using
GIT_ASKPASS helper tool. But if GIT_ASKPASS environment variable is not
set, git-svn does not try to use SSH_ASKPASS as git-core does. This
cause GUIs (w/o STDIN connected) to hang waiting forever for git-svn to
complete (http://code.
Hi,
Am 06.10.2012 20:28 schrieb Junio C Hamano:
> It is either that it was simply forgotten, or after I wrote the part
> you quoted early in January there were discussions later that showed
> the patch was not desirable for some reason. I do not recall which.
I noticed no threads about possible p
Am 10.11.2012 22:13, schrieb Jean-Jacques Lafay:
Le samedi 10 novembre 2012 21:00:10 UTC+1, Philip Oakley a écrit :
From: "Jean-Jacques Lafay" >
Sent: Saturday,
November 10, 2012 5:36 PM
> In large repos, the recursion implementation of contains(commit,
> commit_list)
In this version of the patch the formatting has been corrected.
Warnings for double / in filenames are avoided.
Signed-off-by: Heinrich Schuchardt
---
gitweb/gitweb.perl | 12
1 file changed, 12 insertions(+)
diff --git a/gitweb/gitweb.perl b/gitweb/gitweb.perl
index 10ed9e5..6
On Sun, Nov 11, 2012 at 05:36:53PM +0100, Torsten Bögershausen wrote:
> On 11.11.12 14:59, Felipe Contreras wrote:
> > test_expect_success 'test biridectionality' '
> > + echo -n > marks-cur &&
> > + echo -n > marks-new &&
> Unless I messed up the patch:
>
> Minor issue: still a typo "biridec
On 11.11.12 14:59, Felipe Contreras wrote:
> test_expect_success 'test biridectionality' '
> + echo -n > marks-cur &&
> + echo -n > marks-new &&
Unless I messed up the patch:
Minor issue: still a typo "biridectionality"
Major issue: "echo -n" is still not portable.
Could we simply use
t
On Sun, Nov 11, 2012 at 10:48:46AM -0500, Jeff King wrote:
> Silly me. When I thought through the impact of Paul's patch, I knew that
> we would notice signal death of the editor. But I totally forgot to
> consider that the blocked signal is inherited by the child process. I
> think we just need t
On Sun, Nov 11, 2012 at 09:02:48AM +0200, Kalle Olavi Niemitalo wrote:
> If git did the same thing as cvs here, i.e. ignore the signals in
> the parent process only and check the exit status of the editor,
> I think that would be OK.
Silly me. When I thought through the impact of Paul's patch, I
For repositories it is possible to maintain a README.html which will
be shown on the summary page. This is not possible for the server
root.
German law requires to provide contact data on the web server. This
data could easily be entered in the overview page using a README.html.
Furthermore it is
On Sun, Nov 11, 2012 at 08:20:27PM +0530, Ramkumar Ramachandra wrote:
> >> diff --git a/builtin/diff.c b/builtin/diff.c
> >> index 9650be2..6d00311 100644
> >> --- a/builtin/diff.c
> >> +++ b/builtin/diff.c
> >> @@ -297,6 +297,10 @@ int cmd_diff(int argc, const char **argv, const char
> >> *prefi
Junio C Hamano :
> Perhaps not exactly what you are looking for, but don't we have
> import-tar somewhere in contrib/fast-import hierarchy (sorry, not on
> a machine yet, and I cannot give more details).
If I recall correctly, that can only be used for original import.
I think Andreas Schwab's su
On Sun, Nov 11, 2012 at 02:33:45AM -0800, Junio C Hamano wrote:
> The change seems to think "branch" is the _only_ thing the user
> might want to record per submodule upon "git submodule add".
I felt that earlier floating/tracking submodule patches were biting
off more than they could chew, so I w
Jeff King wrote:
> On Thu, Nov 01, 2012 at 04:13:49PM +0530, Ramkumar Ramachandra wrote:
>
>> diff --git a/builtin/diff.c b/builtin/diff.c
>> index 9650be2..6d00311 100644
>> --- a/builtin/diff.c
>> +++ b/builtin/diff.c
>> @@ -297,6 +297,10 @@ int cmd_diff(int argc, const char **argv, const char
>
By using print_comp as suggested by SZEDER Gábor.
Signed-off-by: Felipe Contreras
---
t/t9902-completion.sh | 15 +--
1 file changed, 5 insertions(+), 10 deletions(-)
diff --git a/t/t9902-completion.sh b/t/t9902-completion.sh
index 66c7af6..9b38b69 100755
--- a/t/t9902-completion.sh
Lots of duplicated code!
No functional changes.
Signed-off-by: Felipe Contreras
---
t/t9902-completion.sh | 76 ++-
1 file changed, 27 insertions(+), 49 deletions(-)
diff --git a/t/t9902-completion.sh b/t/t9902-completion.sh
index 59cdbfd..66c7af
No need to have two versions; if a second argument is specified, use
that, otherwise use stdin.
Signed-off-by: Felipe Contreras
---
t/t9902-completion.sh | 30 +-
1 file changed, 13 insertions(+), 17 deletions(-)
diff --git a/t/t9902-completion.sh b/t/t9902-completio
No need to duplicate that functionality.
Signed-off-by: Felipe Contreras
---
t/t9902-completion.sh | 9 +++--
1 file changed, 3 insertions(+), 6 deletions(-)
diff --git a/t/t9902-completion.sh b/t/t9902-completion.sh
index aff7e44..204c92a 100755
--- a/t/t9902-completion.sh
+++ b/t/t9902-co
The rest of the code uses ' Z$'. Lets use that for
test_completion_long() as well.
Signed-off-by: Felipe Contreras
---
t/t9902-completion.sh | 27 +--
1 file changed, 13 insertions(+), 14 deletions(-)
diff --git a/t/t9902-completion.sh b/t/t9902-completion.sh
index 5c067
So that it's easier to understand what it does.
Also, make sure we pass only the first argument for completion.
Shouldn't cause any functional changes because run_completion only
checks $1.
Signed-off-by: Felipe Contreras
---
t/t9902-completion.sh | 6 +-
1 file changed, 5 insertions(+), 1
These started from a discussion with SZEDER, but then I realized there were
many improvements possible.
Changes since v1:
* A lot more cleanups
Felipe Contreras (6):
completion: add comment for test_completion()
completion: standardize final space marker in tests
completion: simplify test
Signed-off-by: Felipe Contreras
---
contrib/remote-helpers/git-remote-bzr | 60 +--
1 file changed, 43 insertions(+), 17 deletions(-)
diff --git a/contrib/remote-helpers/git-remote-bzr
b/contrib/remote-helpers/git-remote-bzr
index 2bae5d0..f8919f4 100755
--- a/co
Signed-off-by: Felipe Contreras
---
contrib/remote-helpers/git-remote-bzr | 38 +--
contrib/remote-helpers/test-bzr.sh| 32 +
2 files changed, 59 insertions(+), 11 deletions(-)
diff --git a/contrib/remote-helpers/git-remote-bzr
b/c
Signed-off-by: Felipe Contreras
---
contrib/remote-helpers/test-bzr.sh | 111 +
1 file changed, 111 insertions(+)
create mode 100755 contrib/remote-helpers/test-bzr.sh
diff --git a/contrib/remote-helpers/test-bzr.sh
b/contrib/remote-helpers/test-bzr.sh
new f
Signed-off-by: Felipe Contreras
---
contrib/remote-helpers/git-remote-bzr | 2 ++
1 file changed, 2 insertions(+)
diff --git a/contrib/remote-helpers/git-remote-bzr
b/contrib/remote-helpers/git-remote-bzr
index 2c05f35..5b89a05 100755
--- a/contrib/remote-helpers/git-remote-bzr
+++ b/contrib/re
Strictly speaking bzr doesn't need any changes to interact with remote
repositories, but it's dead slow.
Signed-off-by: Felipe Contreras
---
contrib/remote-helpers/git-remote-bzr | 26 --
1 file changed, 24 insertions(+), 2 deletions(-)
diff --git a/contrib/remote-helper
Signed-off-by: Felipe Contreras
---
contrib/remote-helpers/git-remote-bzr | 295 ++
1 file changed, 295 insertions(+)
diff --git a/contrib/remote-helpers/git-remote-bzr
b/contrib/remote-helpers/git-remote-bzr
index b6be9d6..8366234 100755
--- a/contrib/remote-hel
Signed-off-by: Felipe Contreras
---
contrib/remote-helpers/git-remote-bzr | 352 ++
1 file changed, 352 insertions(+)
create mode 100755 contrib/remote-helpers/git-remote-bzr
diff --git a/contrib/remote-helpers/git-remote-bzr
b/contrib/remote-helpers/git-remote-
Hi,
This is a re-roll of the previous series to add support to fetch and push
special modes, and refactor some related code.
Cheers.
Changes since v2:
* Add support for special modes
* Minor refactoring and cleanups
Changes since v1:
* Rewritten to avoid bzr-fastimport
Felipe Contreras (7
Signed-off-by: Felipe Contreras
---
contrib/remote-helpers/git-remote-hg | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/contrib/remote-helpers/git-remote-hg
b/contrib/remote-helpers/git-remote-hg
index 0edb8ac..1d46838 100755
--- a/contrib/remote-helpers/git-remote-hg
+++ b/
Signed-off-by: Felipe Contreras
---
contrib/remote-helpers/git-remote-hg | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/contrib/remote-helpers/git-remote-hg
b/contrib/remote-helpers/git-remote-hg
index 7929eec..0edb8ac 100755
--- a/contrib/remote-helpers/git-remote-hg
++
Hi,
Nothing major.
These are meant to be on top of fc/remote-hg that is on peff/pu.
Felipe Contreras (2):
remote-hg: trivial cleanups
remote-hg: fix for files with spaces
contrib/remote-helpers/git-remote-hg | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
--
1.8.0
--
To unsu
They have been marked as UNINTERESTING for a reason, lets respect that.
Currently the first ref is handled properly, but not the rest, so:
% git fast-export master ^master
Would currently throw a reset for master (2nd ref), which is not what we
want.
% git fast-export master ^foo ^bar ^roo
%
When an object has already been exported (and thus is in the marks) it's
flagged as SHOWN, so it will not be exported again, even if in a later
time it's exported through a different ref.
We don't need the object to be exported again, but we want the ref
updated, which doesn't happen.
Since we ca
First the expected, then the actual, otherwise the diff would be the
opposite of what we want.
Signed-off-by: Felipe Contreras
---
t/t9350-fast-export.sh | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/t/t9350-fast-export.sh b/t/t9350-fast-export.sh
index 0c8d828..b7d30
Setting 'commit' to 'commit' is a no-op. It might have been there to
avoid a compiler warning, but if so, it was the compiler to blame, and
it's certainly not there any more.
Signed-off-by: Felipe Contreras
---
builtin/fast-export.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --g
1 - 100 of 135 matches
Mail list logo