On Tue, Nov 13, 2012 at 07:42:58AM +0100, Felipe Contreras wrote:
> >> > No, it's not. Those broken names do not come from the environment, but
> >> > from our last-resort guess of the hostname.
> >>
> >> That depends how you define environment, but fine, the point is that
> >> it happens.
> >
> >
On Tue, Nov 13, 2012 at 7:42 AM, Felipe Contreras
wrote:
> 6) GIT_AUTHOR
>
> Who should the emails appear to be from? [Felipe Contreras 4th
> ]
>
> What about after my change?
>
> 6.1) GIT_AUTHOR without anything else
>
> fatal: empty ident name (for ) not allowed
> var GIT_COMMITTER_IDENT: comma
On Tue, Nov 13, 2012 at 6:25 AM, Ramkumar Ramachandra
wrote:
> This patch fixes my original problem. So,
> Reported-by: Ramkumar Ramachandra
> Tested-by: Ramkumar Ramachandra
>
> However, test 4 in test-hg.sh still fails for me:
>
> --- expected2012-11-13 05:22:57.946637384 +
> +++ actua
On Tue, Nov 13, 2012 at 5:01 AM, Jeff King wrote:
> On Tue, Nov 13, 2012 at 04:55:25AM +0100, Felipe Contreras wrote:
>
>> > No, it's not. Those broken names do not come from the environment, but
>> > from our last-resort guess of the hostname.
>>
>> That depends how you define environment, but fi
On Mon, Nov 12, 2012 at 08:51:37PM -0800, Junio C Hamano wrote:
> Jeff King writes:
>
> > Yeah. We tolerate a certain amount of skew (24 hours for --name-rev, and
> > 5 broken commits in a row for --since). But the big ones are usually
> > software bugs (the big kernel ones were from broken "gui
On Tue, Nov 13, 2012 at 04:46:36AM +0100, Felipe Contreras wrote:
> >> +setup () {
> >> + (
> >> + echo "[ui]"
> >> + echo "username = A U Thor "
> >> + ) >> "$HOME"/.hgrc
> >> +}
> >
> > This makes sense, but I wonder if we should use something different from
> > the git author id
On Sun, Nov 11, 2012 at 10:29:06PM +0530, Ramkumar Ramachandra wrote:
> 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/fi
On Sun, Nov 11, 2012 at 10:29:05PM +0530, Ramkumar Ramachandra wrote:
> +static int parse_submodule_params(struct diff_options *options, const char
> *value,
> + struct strbuf *errmsg)
> +{
> + if (!strcmp(value, "log"))
> + DIFF_OPT_SET(options, SUBMOD
Felipe Contreras wrote:
> And here's the one for bzr:
>
> --- a/contrib/remote-helpers/git-remote-bzr
> +++ b/contrib/remote-helpers/git-remote-bzr
> @@ -646,12 +646,12 @@ def get_repo(url, alias):
> global dirname, peer
>
> clone_path = os.path.join(dirname, 'clone')
> -origin = bzrl
Hi Peff,
On Mon, 12 Nov 2012, Jeff King wrote:
> On Tue, Nov 13, 2012 at 04:01:11AM +, Johannes Schindelin wrote:
>
> > > Note that name-rev will produce wrong answers in the face of clock skew.
> > > And I think that you even wrote that code. :)
> >
> > IIRC the cute code to short-circuit
Jeff King writes:
> Yeah. We tolerate a certain amount of skew (24 hours for --name-rev, and
> 5 broken commits in a row for --since). But the big ones are usually
> software bugs (the big kernel ones were from broken "guilt", I think) or
> broken imports (when I published a bunch of skew statist
On Tue, Nov 13, 2012 at 04:01:11AM +, Johannes Schindelin wrote:
> > Note that name-rev will produce wrong answers in the face of clock skew.
> > And I think that you even wrote that code. :)
>
> IIRC the cute code to short-circuit using the date is not from me. If it
> is, I am very ashamed.
Hi Peff,
On Mon, 12 Nov 2012, Jeff King wrote:
> On Tue, Nov 13, 2012 at 01:16:01AM +, Johannes Schindelin wrote:
>
> > > We can do much better than O(number of commits), though, if we stop
> > > traversing down a path when its timestamp shows that it is too old to
> > > contain the commits
On Tue, Nov 13, 2012 at 04:55:25AM +0100, Felipe Contreras wrote:
> > No, it's not. Those broken names do not come from the environment, but
> > from our last-resort guess of the hostname.
>
> That depends how you define environment, but fine, the point is that
> it happens.
If you have a strawm
On Tue, Nov 13, 2012 at 4:27 AM, Jeff King wrote:
> On Tue, Nov 13, 2012 at 01:54:59AM +0100, Felipe Contreras wrote:
>
>> > But we use the environment to default the field, so the distinction
>> > doesn't make much sense to me. Plus, it has always been the case that
>> > you can use git without
On Mon, Nov 12, 2012 at 9:32 PM, Jeff King wrote:
> On Mon, Nov 12, 2012 at 06:41:05PM +0100, Felipe Contreras wrote:
>
>> From: Ramkumar Ramachandra
>>
>> 'hg commit' fails otherwise in some versiosn of mercurial because of
>
> s/versiosn/versions/
>
>> +setup () {
>> + (
>> + echo "[ui]
On Tue, Nov 13, 2012 at 01:16:01AM +, Johannes Schindelin wrote:
> > We can do much better than O(number of commits), though, if we stop
> > traversing down a path when its timestamp shows that it is too old to
> > contain the commits we are searching for. The problem is that the
> > timestamp
On Sun, Nov 11, 2012 at 11:17 PM, Chris Webb wrote:
> 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 sl
On Mon, Nov 12, 2012 at 10:27:27PM -0500, Jeff King wrote:
> On Tue, Nov 13, 2012 at 01:54:59AM +0100, Felipe Contreras wrote:
>
> > > But we use the environment to default the field, so the distinction
> > > doesn't make much sense to me. Plus, it has always been the case that
> > > you can use
On Tue, Nov 13, 2012 at 01:54:59AM +0100, Felipe Contreras wrote:
> > But we use the environment to default the field, so the distinction
> > doesn't make much sense to me. Plus, it has always been the case that
> > you can use git without setting user.*, but instead only using the
> > environmen
Hi Peff,
On Mon, 12 Nov 2012, Jeff King wrote:
> On Mon, Nov 12, 2012 at 11:27:14PM +0100, Jean-Jacques Lafay wrote:
>
> > 2012/11/11 Jeff King :
> > > On Sun, Nov 11, 2012 at 05:46:32PM +0100, René Scharfe wrote:
> > >
> > > Ultimately, I have some ideas for doing this in a breadth-first way,
>
So we don't create a clone unnecessarily.
Signed-off-by: Felipe Contreras
---
contrib/remote-helpers/git-remote-bzr | 48 ---
1 file changed, 28 insertions(+), 20 deletions(-)
diff --git a/contrib/remote-helpers/git-remote-bzr
b/contrib/remote-helpers/git-remot
At least as old as 2.0.
Signed-off-by: Felipe Contreras
---
contrib/remote-helpers/git-remote-bzr | 14 +-
1 file changed, 9 insertions(+), 5 deletions(-)
diff --git a/contrib/remote-helpers/git-remote-bzr
b/contrib/remote-helpers/git-remote-bzr
index f8919f4..6cdfac6 100755
--- a/
Hi,
Here's a fix for older versions of bzr, and a check for local repositories.
Felipe Contreras (2):
remote-bzr: add support for older versions of bzr
remote-bzr: detect local repositories
contrib/remote-helpers/git-remote-bzr | 54 +--
1 file changed, 33 in
On Tue, Nov 13, 2012 at 1:37 AM, Junio C Hamano wrote:
> Joe Perches writes:
>
>> On Tue, 2012-11-13 at 03:21 +0530, Ramkumar Ramachandra wrote:
>>> Felipe Contreras wrote:
>>> > cc-cmd is only per-file, and many times receipients get lost without
>>> > seing the full patch series.
>>>
>>> s/sein
Jeff King writes:
> On Mon, Nov 12, 2012 at 07:02:17PM -0500, Jeff King wrote:
>
>> The one distinction that would make sense to me is pausing to ask when
>> we use "implicit" methods to look up the ident, like concatenating the
>> username with the hostname to get the email.
>
> By the way, I su
On Tue, Nov 13, 2012 at 1:02 AM, Jeff King wrote:
> On Tue, Nov 13, 2012 at 12:42:02AM +0100, Felipe Contreras wrote:
>
>> > Why not use Git::ident_person() here? It saves some code, and would also
>> > respect environment variables. Or better yet...
>>
>> I assume there was a reason why that code
Jeff King writes:
> On Sun, Nov 11, 2012 at 06:20:58AM +0100, Henrich Schuchardt wrote:
>
>> Gitweb pages are structured by divs of class title with grey background.
>> The shortlog, and the log page show the project name as the first title.
>> Page summary only shows an empty grey box above the
Joe Perches writes:
> On Tue, 2012-11-13 at 03:21 +0530, Ramkumar Ramachandra wrote:
>> Felipe Contreras wrote:
>> > cc-cmd is only per-file, and many times receipients get lost without
>> > seing the full patch series.
>>
>> s/seing/seeing
>>
>> > [...]
>>
>> Looks good otherwise.
>
> s/recei
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
On Mon, Nov 12, 2012 at 07:02:17PM -0500, Jeff King wrote:
> The one distinction that would make sense to me is pausing to ask when
> we use "implicit" methods to look up the ident, like concatenating the
> username with the hostname to get the email.
By the way, I suspect this is the answer to "
On Tue, Nov 13, 2012 at 12:42:02AM +0100, Felipe Contreras wrote:
> > Why not use Git::ident_person() here? It saves some code, and would also
> > respect environment variables. Or better yet...
>
> I assume there was a reason why that code was asking for input;
> precisely because it would use t
On Tue, Nov 13, 2012 at 12:35 AM, Jeff King wrote:
> On Sun, Nov 11, 2012 at 06:06:50PM +0100, Felipe Contreras wrote:
>
>> 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 rea
On Tue, 2012-11-13 at 00:37 +0100, Felipe Contreras wrote:
> On Tue, Nov 13, 2012 at 12:13 AM, Joe Perches wrote:
> > On Tue, 2012-11-13 at 00:03 +0100, Felipe Contreras wrote:
[]
> >> For --to-cmd and --cc-cmd? So basically you check the dirname of the
> >> argument passed?
> >
> > yes. basename
On Tue, Nov 13, 2012 at 12:13 AM, Joe Perches wrote:
> On Tue, 2012-11-13 at 00:03 +0100, Felipe Contreras wrote:
>> On Mon, Nov 12, 2012 at 11:52 PM, Joe Perches wrote:
>> > On Tue, 2012-11-13 at 03:21 +0530, Ramkumar Ramachandra wrote:
>> >> Felipe Contreras wrote:
>> >> > cc-cmd is only per-fi
On Sun, Nov 11, 2012 at 06:06:50PM +0100, Felipe Contreras wrote:
> 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 ad
On Sun, Nov 11, 2012 at 06:20:58AM +0100, Henrich Schuchardt wrote:
> Gitweb pages are structured by divs of class title with grey background.
> The shortlog, and the log page show the project name as the first title.
> Page summary only shows an empty grey box above the project details.
> This pr
On Sun, Nov 11, 2012 at 04:13:19AM -0800, Jason Timrod wrote:
> I note that when defining a .mailmap file, that it's honoured by
> git-shotlog(1) by default, but for git-log(1) I have to define an
> entirely new --pretty= formatting option to use it.
>
> Why is this? Why doesn't git-log honour t
On Mon, Nov 12, 2012 at 11:27:14PM +0100, Jean-Jacques Lafay wrote:
> 2012/11/11 Jeff King :
> > On Sun, Nov 11, 2012 at 05:46:32PM +0100, René Scharfe wrote:
> >
> > Ultimately, I have some ideas for doing this in a breadth-first way,
> > which would make it more naturally iterative. It would inv
On Tue, 2012-11-13 at 00:03 +0100, Felipe Contreras wrote:
> On Mon, Nov 12, 2012 at 11:52 PM, Joe Perches wrote:
> > On Tue, 2012-11-13 at 03:21 +0530, Ramkumar Ramachandra wrote:
> >> Felipe Contreras wrote:
> >> > cc-cmd is only per-file, and many times receipients get lost without
> >> > seing
On Mon, Nov 12, 2012 at 6:18 PM, Drew Northup wrote:
> On Mon, Nov 12, 2012 at 11:37 AM, Adam Spiers wrote:
>> As it turned out to be fairly easy, I was wondering if there would be
>> any interest in doing this more formally, i.e. splitting off the
>> framework so that it could be used and improv
Drew Northup writes:
> Something like this may be useful to defuse the "file" parameter, but
> I presume a more definitive fix is in order...
A proper fix will have to add esc_html to the feed generation, something
like this (untested):
diff --git a/gitweb/gitweb.perl b/gitweb/gitweb.perl
index
On Mon, Nov 12, 2012 at 11:52 PM, Joe Perches wrote:
> On Tue, 2012-11-13 at 03:21 +0530, Ramkumar Ramachandra wrote:
>> Felipe Contreras wrote:
>> > cc-cmd is only per-file, and many times receipients get lost without
>> > seing the full patch series.
>>
>> s/seing/seeing
>>
>> > [...]
>>
>> Look
On Tue, 2012-11-13 at 03:21 +0530, Ramkumar Ramachandra wrote:
> Felipe Contreras wrote:
> > cc-cmd is only per-file, and many times receipients get lost without
> > seing the full patch series.
>
> s/seing/seeing
>
> > [...]
>
> Looks good otherwise.
s/receipients/recipients/ too
Practically
On Mon, Nov 12, 2012 at 10:41 PM, Jeff King wrote:
> On Sun, Nov 11, 2012 at 07:48:14PM +0100, Felipe Contreras wrote:
>
>> > 3. Exporters should not use it if they have any broken-down
>> > representation at all. Even knowing that the first half is a human
>> > name and the second hal
2012/11/11 Jeff King :
> On Sun, Nov 11, 2012 at 05:46:32PM +0100, René Scharfe wrote:
>
> Ultimately, I have some ideas for doing this in a breadth-first way,
> which would make it more naturally iterative. It would involve having N
> bits of storage per commit to check N tags, but it would mean t
On Sun, Nov 11, 2012 at 06:35:16PM +0530, Ramkumar Ramachandra wrote:
> Ramkumar Ramachandra wrote:
> > Don't you mean `git diff` in place of `git log -p`
> > though? I don't think `git log --submodule` does anything differently
> > from `git log`.
>
> Sorry for the nonsense. I just realized th
On Tue, Nov 13, 2012 at 03:16:51AM +0530, Ramkumar Ramachandra wrote:
> I'd like to get more commands to call into sequencer.c, so that we can
> (finally) implement a sensible `git continue` and `git abort`.
> Currently, am and rebase come to mind, but they are shell scripts and
> I don't see how
On Mon, Nov 12, 2012 at 04:39:39PM -0500, Mark Levedahl wrote:
> >> diff --git a/Makefile b/Makefile
> >> index f69979e..1cc5d96 100644
> >> --- a/Makefile
> >> +++ b/Makefile
> >> @@ -1082,6 +1082,7 @@ ifeq ($(uname_O),Cygwin)
> >> NO_SYMLINK_HEAD = YesPlease
> >> NO_I
On Sun, Nov 11, 2012 at 07:48:14PM +0100, Felipe Contreras wrote:
> > 3. Exporters should not use it if they have any broken-down
> > representation at all. Even knowing that the first half is a human
> > name and the second half is something else would give it a better
> > shot a
On Mon, Nov 12, 2012 at 3:58 PM, Jeff King wrote:
> On Sun, Nov 11, 2012 at 07:17:20PM -0500, Mark Levedahl wrote:
>
>> 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 w
On Mon, Nov 12, 2012 at 10:13:27PM +0100, Jakub Narębski wrote:
> > Yeah, that looks correct, given the way how the other variables
> > emitted with the same "print" like $descr and $owner are formed.
>
> It looks like good solution to me too.
>
> Nb. the problems with feed are mainly because it
On Sun, Nov 11, 2012 at 10:53:03PM +, Thomas Adam wrote:
> 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.
On Mon, Nov 12, 2012 at 9:36 PM, Junio C Hamano wrote:
> Jeff King writes:
>> On Mon, Nov 12, 2012 at 03:24:13PM -0500, Jeff King wrote:
>>
>>> I think the right answer is going to be a well-placed call to esc_html.
>>
>> I'm guessing the right answer is this:
>>
>> diff --git a/gitweb/gitweb.per
On Sun, Nov 11, 2012 at 07:17:20PM -0500, Mark Levedahl wrote:
> 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
On Mon, Nov 12, 2012 at 6:45 PM, Junio C Hamano wrote:
> A Large Angry SCM writes:
>
>> 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:
>
>>>
On Mon, Nov 12, 2012 at 03:18:02PM +0100, Michael J Gruber wrote:
> 'git replace' parses the revision arguments when it creates replacements
> (so that a sha1 can be abbreviated, e.g.) but not when deleting
> replacements.
>
> Make it parse the argument to 'replace -d' in the same way.
>
> Signe
Jeff King writes:
> On Mon, Nov 12, 2012 at 03:24:13PM -0500, Jeff King wrote:
>
>> I think the right answer is going to be a well-placed call to esc_html.
>
> I'm guessing the right answer is this:
>
> diff --git a/gitweb/gitweb.perl b/gitweb/gitweb.perl
> index 10ed9e5..a51a8ba 100755
> --- a/g
Jeff King writes:
> On Mon, Nov 12, 2012 at 12:08:37PM -0800, Junio C Hamano wrote:
>
>> Jeff King writes:
>>
>> > Changing the interface for get_author_ident_from_commit would be a pain,
>> > but if we just wanted to help filter-branch, we could do something like
>> > this:
>>
>> Yes, that is
On Mon, Nov 12, 2012 at 06:41:05PM +0100, Felipe Contreras wrote:
> From: Ramkumar Ramachandra
>
> 'hg commit' fails otherwise in some versiosn of mercurial because of
s/versiosn/versions/
> +setup () {
> + (
> + echo "[ui]"
> + echo "username = A U Thor "
> + ) >> "$HOME"/.hgr
On Mon, Nov 12, 2012 at 03:24:13PM -0500, Jeff King wrote:
> I think the right answer is going to be a well-placed call to esc_html.
I'm guessing the right answer is this:
diff --git a/gitweb/gitweb.perl b/gitweb/gitweb.perl
index 10ed9e5..a51a8ba 100755
--- a/gitweb/gitweb.perl
+++ b/gitweb/git
On Mon, Nov 12, 2012 at 01:55:46PM -0500, Drew Northup wrote:
> On Sun, Nov 11, 2012 at 6:28 PM, glpk xypron wrote:
> > Gitweb can be used to generate an RSS feed.
> >
> > Arbitrary tags can be inserted into the XML document describing
> > the RSS feed by careful construction of the URL.
> [...]
Jeff King writes:
> On Mon, Nov 12, 2012 at 09:45:31AM -0800, Junio C Hamano wrote:
>
>> Jeff King writes:
>>
>> > On Mon, Oct 29, 2012 at 07:33:47PM +0100, Thomas Ackermann wrote:
>> >
>> >> This patch addresses Junios comment in WC:
>> >> "Misapplication of a patch fixed; the ones near the ti
On Mon, Nov 12, 2012 at 12:08:37PM -0800, Junio C Hamano wrote:
> Jeff King writes:
>
> > Changing the interface for get_author_ident_from_commit would be a pain,
> > but if we just wanted to help filter-branch, we could do something like
> > this:
>
> Yes, that is the direction I was alluding
Jeff King writes:
> Changing the interface for get_author_ident_from_commit would be a pain,
> but if we just wanted to help filter-branch, we could do something like
> this:
Yes, that is the direction I was alluding to.
Callers of get_author_ident_from_commit can also do the same and
avoid reb
Hi,
this patch allows tcsh-users to get the benefits of the awesome
git-completion.bash script. It could also help other shells do the same.
==
The current tcsh-completion support for Git, as can be found on the
internet, takes the approach of defining the possible completions
explicitly. This
On Mon, Nov 12, 2012 at 09:44:49AM -0800, Junio C Hamano wrote:
> How did this message happen?
>
> Subject: [PATCH 2/5] launch_editor: ignore SIGINT while the editor has
> control
> To: Kalle Olavi Niemitalo
> Cc: Paul Fox , git@vger.kernel.org
> Date: Sun, 11 Nov 2012 11:55:11
On Mon, Nov 12, 2012 at 09:44:01AM -0800, Junio C Hamano wrote:
> Jeff King writes:
>
> > The only ident-parsing function we currently provide is
> > get_author_ident_from_commit. This is not very
> > flexible for two reasons:
> >
> > 1. It takes a commit as an argument, and can't read from
>
> On Mon, Nov 12, 2012 at 12:12 PM, Kevin wrote:
> > Maybe handy to say that you're on a Powerpc platform.
Oh, and yes, I'm on 2 x 2-core ("4-core") machine.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info
On Mon, Nov 12, 2012 at 09:45:31AM -0800, Junio C Hamano wrote:
> Jeff King writes:
>
> > On Mon, Oct 29, 2012 at 07:33:47PM +0100, Thomas Ackermann wrote:
> >
> >> This patch addresses Junios comment in WC:
> >> "Misapplication of a patch fixed; the ones near the tip needs to
> >> update the l
On Mon, Nov 12, 2012 at 11:37 AM, Adam Spiers wrote:
> Hi all,
>
> I've been pretty impressed with git's test framework, and I'm not
> aware of many other (decent) shell-based test frameworks out there.
> (One that springs to mind is the one used by rvm, but last time I
> looked - admittedly a whi
Junio C Hamano wrote:
> Phil Hord writes:
>
>> State token strings which may be emitted and their meanings:
>> merge a merge is in progress
>> am an am is in progress
>> am-is-emptythe am patch is empty
>> rebase a rebase is in progr
Michael Haggerty writes:
> The log message of the original commit (0454dd93bf) described the
> following scenario: a /home partition under which user home directories
> are automounted, and setting GIT_CEILING_DIRECTORIES=/home to avoid
> hitting /home/.git, /home/.git/objects, and /home/objects
Nguyen Thai Ngoc Duy writes:
> On Sat, Oct 27, 2012 at 7:33 PM, Junio C Hamano wrote:
>>
>> Nguyen Thai Ngoc Duy wrote:
>>>
>>>Notice the --cc in the first line, which is combined diff. Usually
>>>combined-diff is between two points and one parent. Though somehow git
>>>passes 4 parents down co
Phil Hord writes:
> State token strings which may be emitted and their meanings:
> merge a merge is in progress
> am an am is in progress
> am-is-emptythe am patch is empty
> rebase a rebase is in progress
> rebase-interactive a
Jeff King writes:
> On Mon, Oct 29, 2012 at 07:33:47PM +0100, Thomas Ackermann wrote:
>
>> This patch addresses Junios comment in WC:
>> "Misapplication of a patch fixed; the ones near the tip needs to
>> update the links to point at the html files, though."
>>
>> See older mail in this thread:
A Large Angry SCM writes:
> 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
How did this message happen?
Subject: [PATCH 2/5] launch_editor: ignore SIGINT while the editor has
control
To: Kalle Olavi Niemitalo
Cc: Paul Fox , git@vger.kernel.org
Date: Sun, 11 Nov 2012 11:55:11 -0500
Message-ID: <2012165510.gb19...@sigill.intra.peff.net>
Refere
Jeff King writes:
>> Major issue: "echo -n" is still not portable.
>>
>> Could we simply use
>>
>> touch marks-cur &&
>> touch marks-new
>
> Yes, "echo -n" is definitely not portable. Our preferred way of
> creating an empty file is just ">file".
Yes.
And it is misleading to use "touch" i
Heinrich Schuchardt writes:
> In this version of the patch the formatting has been corrected.
>
> Warnings for double / in filenames are avoided.
>
> Signed-off-by: Heinrich Schuchardt
>
> ---
The above is *not* a proper log message.
Those of us who are reading the messages on the list would k
Jeff King writes:
> The only ident-parsing function we currently provide is
> get_author_ident_from_commit. This is not very
> flexible for two reasons:
>
> 1. It takes a commit as an argument, and can't read from
> commit headers saved on disk.
>
> 2. It will only parse authors, not com
Jonathan Nieder writes:
> Ramkumar Ramachandra wrote:
>
>> Oops, I read about `--pretty` in pretty-formats.txt and didn't realize
>> that `--format` existed. However, your patch is still wrong because
>> there seems to be a subtle (and confusing) difference between
>> `--pretty` and `--format`.
Turns out fast-export throws bad 'reset' commands because of a behavior
in transport-helper that is not even needed.
Either way, better to ignore them, otherwise the user will get warnings
when we OK them.
Signed-off-by: Felipe Contreras
---
contrib/remote-helpers/git-remote-hg | 3 +++
1 file
There's no concept of HEAD in mercurial, but let's try our best to do
something sensible.
Signed-off-by: Felipe Contreras
---
contrib/remote-helpers/git-remote-hg | 2 ++
1 file changed, 2 insertions(+)
diff --git a/contrib/remote-helpers/git-remote-hg
b/contrib/remote-helpers/git-remote-hg
in
Turns out repo.revs was introduced quite late, and it doesn't do
anything fancy for our refspec; only list all the numbers in that range.
Signed-off-by: Felipe Contreras
---
contrib/remote-helpers/git-remote-hg | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/contrib/remote-he
From: Ramkumar Ramachandra
'hg commit' fails otherwise in some versiosn of mercurial because of
the missing user information. Other versions simply throw a warning and
guess though.
Signed-off-by: Ramkumar Ramachandra
Signed-off-by: Felipe Contreras
---
contrib/remote-helpers/test-hg.sh | 9 +
Hi,
Just a few fixes, nothing major.
Felipe Contreras (3):
remote-hg: fix compatibility with older versions of hg
remote-hg: try the 'tip' if no checkout present
remote-hg: avoid bad refs
Ramkumar Ramachandra (1):
remote-hg: add missing config for basic tests
contrib/remote-helpers/git
Maybe handy to say that you're on a Powerpc platform.
On Mon, Nov 12, 2012 at 5:32 PM, Douglas Mencken wrote:
> *Any* git clone fails with:
>
> fatal: premature end of pack file, 106 bytes missing
> fatal: index-pack failed
>
> At first, I tried 1.8.0, and it failed. Then I tried to build 1.7.10.
> -Original Message-
> From: Felipe Contreras
> Sent: Monday, November 12, 2012 11:44 AM
>
> On Mon, Nov 12, 2012 at 5:37 PM, Adam Spiers
> wrote:
>
> > I've been pretty impressed with git's test framework, and I'm not
> > aware of many other (decent) shell-based test frameworks out ther
On Mon, Nov 12, 2012 at 5:37 PM, Adam Spiers wrote:
> I've been pretty impressed with git's test framework, and I'm not
> aware of many other (decent) shell-based test frameworks out there.
> (One that springs to mind is the one used by rvm, but last time I
> looked - admittedly a while ago now -
On Mon, Nov 12, 2012 at 4:45 PM, Jonathan Nieder wrote:
> Max Horn wrote:
>
>> Aha, now I understand what this patch is about. So I would suggest
>> this alternate commit message:
>>
>> remote-testgit: make it explicit clear that we use the 'done' feature
>>
>> Previously we relied on passing
Hi all,
I've been pretty impressed with git's test framework, and I'm not
aware of many other (decent) shell-based test frameworks out there.
(One that springs to mind is the one used by rvm, but last time I
looked - admittedly a while ago now - it had limitations).
Recently a situation arose whe
Max Horn wrote:
> Aha, now I understand what this patch is about. So I would suggest
> this alternate commit message:
>
> remote-testgit: make it explicit clear that we use the 'done' feature
>
> Previously we relied on passing '--use-done-feature ' to git
> fast-export, which is easy to mis
Ramkumar Ramachandra wrote:
> Oops, I read about `--pretty` in pretty-formats.txt and didn't realize
> that `--format` existed. However, your patch is still wrong because
> there seems to be a subtle (and confusing) difference between
> `--pretty` and `--format`. In the latter, you can't omit th
Enrico asked:
> Could it be that certain files spent parts of their historical lifetime
> inside the ignored paths ?
I left out one possibly important piece of information: My initial 'git
svn fetch' used '-r' to "cauterize" the history, both because there is a lot of
it (almost 12 yea
'git replace' parses the revision arguments when it creates replacements
(so that a sha1 can be abbreviated, e.g.) but not when deleting
replacements.
Make it parse the argument to 'replace -d' in the same way.
Signed-off-by: Michael J Gruber
---
Notes:
v3 safeguards the hex buffer against
esr:
>Junio C Hamano pobox.com>:
>> 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.
You m
> -Original Message-
> From: Jeff King
> Sent: Sunday, November 11, 2012 1:29 PM
>
> 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
Ulrich Spörlein venit, vidit, dixit 09.11.2012 19:27:
> On Fri, 2012-11-09 at 11:16:47 -0500, Jeff King wrote:
>> On Fri, Nov 09, 2012 at 04:52:48PM +0100, Matthieu Moy wrote:
>>
>>> Ulrich Spörlein writes:
>>>
>> 2. Why the hell is the commit hash dependent on the ordering of the
>> paren
On 11.11.2012, at 22:22, Felipe Contreras wrote:
> 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
>>
1 - 100 of 104 matches
Mail list logo
