Re: [PATCH 1/4] strbuf_split_buf(): use ALLOC_GROW()

2012-11-05 Thread Michael Haggerty
On 11/04/2012 12:41 PM, Jeff King wrote: > On Sun, Nov 04, 2012 at 07:46:51AM +0100, Michael Haggerty wrote: > >> Use ALLOC_GROW() rather than inline code to manage memory in >> strbuf_split_buf(). Rename "pos" to "nr" because it better describes >> the use of the variable and it better conforms

Re: Support for a series of patches, i.e. patchset or changeset?

2012-11-05 Thread Johannes Sixt
Am 11/6/2012 7:56, schrieb Eric Miao: > On Tue, Nov 6, 2012 at 2:39 PM, Johannes Sixt wrote: >> Am 11/6/2012 1:58, schrieb Eric Miao: >>> E.g. when we merged a series of patches: >>> >>> [PATCH 00/08] >>> [PATCH 01/08] >>> ... >>> [PATCH 08/08] >>> >>> How do we know this whole series afte

Re: [PATCH v4 6/8] longest_ancestor_length(): require prefix list entries to be normalized

2012-11-05 Thread Michael Haggerty
On 10/30/2012 07:23 PM, Ramsay Jones wrote: > [...] > Just to be clear, this is true for the MinGW port to Windows, but *not* > the cygwin port. > > [...] > > ditto Thanks for clarifying these points. It seems like this patch series is not going to be usable (because of fears that it might cause

Re: Support for a series of patches, i.e. patchset or changeset?

2012-11-05 Thread Eric Miao
On Tue, Nov 6, 2012 at 2:39 PM, Johannes Sixt wrote: > Am 11/6/2012 1:58, schrieb Eric Miao: >> On Mon, Nov 5, 2012 at 10:40 PM, Michael J Gruber >> wrote: >>> Eric Miao venit, vidit, dixit 05.11.2012 15:12: The problem is, most cases we have no idea of the base rev1, and commit rev2 >

Re: Support for a series of patches, i.e. patchset or changeset?

2012-11-05 Thread Johannes Sixt
Am 11/6/2012 1:58, schrieb Eric Miao: > On Mon, Nov 5, 2012 at 10:40 PM, Michael J Gruber > wrote: >> Eric Miao venit, vidit, dixit 05.11.2012 15:12: >>> The problem is, most cases we have no idea of the base rev1, and commit rev2 >>> which it's leading up to. E.g. for a single patch which is betw

Re: Support for a series of patches, i.e. patchset or changeset?

2012-11-05 Thread Eric Miao
On Mon, Nov 5, 2012 at 10:40 PM, Michael J Gruber wrote: > Eric Miao venit, vidit, dixit 05.11.2012 15:12: >> The problem is, most cases we have no idea of the base rev1, and commit rev2 >> which it's leading up to. E.g. for a single patch which is between >> commit rev1..rev2, >> how do we find o

Re: [git-users] git compilation problem

2012-11-05 Thread Konstantin Khomoutov
On Mon, Nov 05, 2012 at 09:21:13AM -0800, Mathieu Dutour wrote: First, note that this list is a support channel for Git newbies, and your question appears to be more about Git source code itself so I'd say it better fits the main Git list which is git at vger.kernel.org. I Cc'ed this my response h

Re: git-p4 clone @all error

2012-11-05 Thread Vitor Antunes
On Mon, 5 Nov 2012 17:12:12 -0500 Pete Wyckoff wrote: > Thanks, that is interesting output. > > Adding Vitor who knows his way around the branch detection > code. The first mail by Arthur is here, which focuses more > on the backtrace from "p4 describe" errors, but also suggests > something is

[PATCH] gitweb: make remote_heads config setting work.

2012-11-05 Thread Phil Pennock
Git configuration items can not contain underscores in their name; the 'remote_heads' feature can not be enabled on a per-repository basis with that name. This changes the git-config option to be `gitweb.remoteheads` but does not change the gitweb.conf option, to avoid backwards compatibility issu

Fwd: [PATCH] Remove terminal symbols from non-terminal console

2012-11-05 Thread Michael Naumov
As per discussion on msysgit user group: https://groups.google.com/forum/?fromgroups=#!topic/msysgit/U_a982_a3rc/discussion we found the following patch is required to get rid of weird terminal characters for other tools such as GitExtensions for Windows ---8<--- diff --git a/sideband.c b/sideband

git log --follow --reverse ignores --follow

2012-11-05 Thread Valentin Spreckels
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Hi, I found something looking like a bug: The following lines prepares a small exmaple: # create git repo git init # create file a and commit it touch a && git add a && git commit -m first # rename a to b and commit it git mv a b && git commit -m sec

Re: how the commit ID come from in GIT?

2012-11-05 Thread Konstantin Khomoutov
On Mon, Nov 05, 2012 at 12:04:06PM -0800, hong zhang wrote: > Could anyone explain how the git commit ID will include all the files > that devloper makes changes on? > > How git commit ID works? To expand on the answer provided by Magnus, I would also suggest reading the classic "Git from the bo

Re: "git am" crash (builtin/apply.c:2108) + small repro

2012-11-05 Thread Alexey Spiridonov
Thanks for looking into this, guys! I seem to run into this with some regularity, but my setting is apply.whitespace=strip rather than 'fix'. Is there an obvious workaround? Here are my remaining settings, sanitized for file paths and URLs: svn.rmdir=true push.default=upstream color.ui=auto col

Re: Like commit -a, but...

2012-11-05 Thread Andreas Schwab
Andreas Krey writes: > But is there a direct way to convert the current working tree into a > tree object? You can create a temporary index by setting the GIT_INDEX_FILE env var. Andreas. -- Andreas Schwab, sch...@linux-m68k.org GPG Key fingerprint = 58CA 54C7 6D53 942B 1756 01D3 44D5 214B 8

Re: git-p4 clone @all error

2012-11-05 Thread Pete Wyckoff
a.fou...@amesys.fr wrote on Mon, 05 Nov 2012 02:02 -0800: > Hi, > > Here is my import : > > Importing from //depot@all into X > Initialized empty Git repository in > /home/arthur/projets_git/X/X/.git/ > Importing revision 4258 (43%) > Importing new branch depot/DEV_DATA > > R

Re: Like commit -a, but...

2012-11-05 Thread Tomas Carnecky
On Mon, 05 Nov 2012 21:56:28 +0100, Andreas Krey wrote: > On Mon, 05 Nov 2012 21:29:48 +, Andreas Krey wrote: > ... > > But still I'd like to know if there is a cleaner solution, > > esp. with respect to the index. > > Actually, it seems > > commit -m 'index' > commit -a -m 'worktree' > .

Re: why '--init' in git-submodule update

2012-11-05 Thread Heiko Voigt
Hi, On Mon, Nov 05, 2012 at 05:30:51PM +0100, Francis Moreau wrote: > I'm wondering why the --init option from git-submodule-update is not > at least the defaut. Or even wilder, why this option exists at all and > git-submodule-update always behave like --init was always passed. That was a design

Like commit -a, but...

2012-11-05 Thread Andreas Krey
Hi all, I have a workflow for which I can't quite find the git tooling. Essentially what I want is like 'git commit -a', except that I want the resulting commit on a branch I name instead of the current one, and I want my current index not being modified. At the moment I emulate that via git c

Re: Like commit -a, but...

2012-11-05 Thread Andreas Krey
On Mon, 05 Nov 2012 21:29:48 +, Andreas Krey wrote: ... > But still I'd like to know if there is a cleaner solution, > esp. with respect to the index. Actually, it seems commit -m 'index' commit -a -m 'worktree' ...push git reset HEAD^ git reset --soft HEAD^ might do the index trick. B

Re: how the commit ID come from in GIT?

2012-11-05 Thread Magnus Bäck
On Monday, November 05, 2012 at 15:04 EST, hong zhang wrote: > Could anyone explain how the git commit ID will include all the files > that devloper makes changes on? > > How git commit ID works? In short, a Git commit points to a tree object that describes the full state of the source tree

Re: [PATCHv2 2/2] git p4: add support for 'p4 move' in P4Submit

2012-11-05 Thread Pete Wyckoff
vitor@gmail.com wrote on Mon, 05 Nov 2012 17:37 +: > Pete Wyckoff padd.com> writes: > > > > > From: Gary Gibbons perforce.com> > > > > For -M option (detectRenames) in P4Submit, use 'p4 move' rather > > than 'p4 integrate'. Check Perforce server for exisitence of > > 'p4 move' and use

Re: checkout-index: unable to create file foo (File exists)

2012-11-05 Thread Pete Wyckoff
br...@interlinx.bc.ca wrote on Mon, 05 Nov 2012 10:25 -0500: > On 12-11-04 05:10 PM, Pete Wyckoff wrote: > > Which network filesystem and OS are you using? > > The filesystem is Lustre. So not only is it networked, it is > distributed where the namespace and data store are handled by different >

Re: [PATCHv2 2/2] git p4: add support for 'p4 move' in P4Submit

2012-11-05 Thread Vitor Antunes
Pete Wyckoff padd.com> writes: > > From: Gary Gibbons perforce.com> > > For -M option (detectRenames) in P4Submit, use 'p4 move' rather > than 'p4 integrate'. Check Perforce server for exisitence of > 'p4 move' and use it if present, otherwise revert to 'p4 integrate'. > Hi Pete, I've just

why '--init' in git-submodule update

2012-11-05 Thread Francis Moreau
Hello, I'm wondering why the --init option from git-submodule-update is not at least the defaut. Or even wilder, why this option exists at all and git-submodule-update always behave like --init was always passed. Thanks -- Francis -- To unsubscribe from this list: send the line "unsubscribe git"

Re: Lack of netiquette, was Re: [PATCH v4 00/13] New remote-hg helper

2012-11-05 Thread Felipe Contreras
On Mon, Nov 5, 2012 at 5:00 PM, Michael J Gruber wrote: > [trimmed down heavily by mjg] > Felipe Contreras venit, vidit, dixit 05.11.2012 16:22: >> On Mon, Nov 5, 2012 at 10:25 AM, Michael J Gruber >> wrote: >>> Felipe Contreras venit, vidit, dixit 02.11.2012 17:09: On Fri, Nov 2, 2012 at 12

Re: Lack of netiquette, was Re: [PATCH v4 00/13] New remote-hg helper

2012-11-05 Thread Michael J Gruber
[trimmed down heavily by mjg] Felipe Contreras venit, vidit, dixit 05.11.2012 16:22: > On Mon, Nov 5, 2012 at 10:25 AM, Michael J Gruber > wrote: >> Felipe Contreras venit, vidit, dixit 02.11.2012 17:09: >>> On Fri, Nov 2, 2012 at 12:03 PM, Michael J Gruber >>> wrote: > There is no lack of openn

Re: Lack of netiquette, was Re: [PATCH v4 00/13] New remote-hg helper

2012-11-05 Thread Felipe Contreras
On Mon, Nov 5, 2012 at 4:22 PM, Felipe Contreras wrote: > On Mon, Nov 5, 2012 at 10:25 AM, Michael J Gruber > wrote: >> For one thing, contributors on the kernel list are open to technical >> arguments, and that includes the arguments of others; just like we are >> here. On the other hand, you s

[PATCH v2 5/5] remote-bzr: update working tree

2012-11-05 Thread Felipe Contreras
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 c981fda..1a06a0a 100755 --- a/contrib/remote-helpers/git-remote-bzr +++ b/contrib/re

[PATCH v2 4/5] remote-bzr: add support for remote repositories

2012-11-05 Thread Felipe Contreras
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

[PATCH v2 3/5] remote-bzr: add support for pushing

2012-11-05 Thread Felipe Contreras
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 ed893b0..9e0062f 100755 --- a/contrib/remote-hel

[PATCH v2 0/5] New remote-bzr remote helper

2012-11-05 Thread Felipe Contreras
Hi, I decided to get rid of bzr-fastimport; too much complexity for not really that much of a gain. The only feature I know is missing is support for executable modes and links. I haven't verified that the resulting output is exactly the same as with other tools, so be careful while using this.

[PATCH v2 2/5] remote-bzr: add simple tests

2012-11-05 Thread Felipe Contreras
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

[PATCH v2 1/5] Add new remote-bzr transport helper

2012-11-05 Thread Felipe Contreras
Signed-off-by: Felipe Contreras --- contrib/remote-helpers/git-remote-bzr | 353 ++ 1 file changed, 353 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-

Re: [PATCH v4 00/13] New remote-hg helper

2012-11-05 Thread Felipe Contreras
On Mon, Nov 5, 2012 at 3:13 PM, Michael J Gruber wrote: > Felipe Contreras venit, vidit, dixit 02.11.2012 19:01: >> I talked with some people in #mercurial, and apparently there is a >> concept of a 'changelog' that is supposed to store these changes, but >> since the format has changed, the cont

Re: Lack of netiquette, was Re: [PATCH v4 00/13] New remote-hg helper

2012-11-05 Thread Felipe Contreras
On Mon, Nov 5, 2012 at 10:25 AM, Michael J Gruber wrote: > Felipe Contreras venit, vidit, dixit 02.11.2012 17:09: >> On Fri, Nov 2, 2012 at 12:03 PM, Michael J Gruber >> wrote: >>> Andreas Ericsson venit, vidit, dixit 02.11.2012 10:38: On 11/01/2012 02:46 PM, René Scharfe wrote: > >

Re: checkout-index: unable to create file foo (File exists)

2012-11-05 Thread Brian J. Murrell
On 12-11-04 05:10 PM, Pete Wyckoff wrote: > > Fascinating problem and observations. I thought so as well. > We've been using NFS with git for quite a while and have never > seen such an error. Could be because NFS manages to operate more atomically given that it's just the network exporting of

Re: Support for a series of patches, i.e. patchset or changeset?

2012-11-05 Thread Michael J Gruber
Eric Miao venit, vidit, dixit 05.11.2012 15:12: > The problem is, most cases we have no idea of the base rev1, and commit rev2 > which it's leading up to. E.g. for a single patch which is between > commit rev1..rev2, > how do we find out rev1 and rev2. So, then the question is: What do you know/ha

Re: git log --graph --oneline - false parent-child visualization

2012-11-05 Thread Michael J Gruber
martyone venit, vidit, dixit 05.11.2012 13:57: > Hello, > > the combination of --graph and --oneline sometimes produces output > which is -- at least for me -- not clear about parent-child relation > between commits. I noticed it when using with --simplify-by-decoration > switch but it should not

Re: [PATCH v4 00/13] New remote-hg helper

2012-11-05 Thread Michael J Gruber
Felipe Contreras venit, vidit, dixit 02.11.2012 19:01: > On Fri, Nov 2, 2012 at 5:41 PM, Felipe Contreras > wrote: >> On Fri, Nov 2, 2012 at 3:48 PM, Jeff King wrote: >>> On Thu, Nov 01, 2012 at 05:08:52AM +0100, Felipe Contreras wrote: >>> > Turns out msysgit's remote-hg is not exporting the

Re: Support for a series of patches, i.e. patchset or changeset?

2012-11-05 Thread Eric Miao
The problem is, most cases we have no idea of the base rev1, and commit rev2 which it's leading up to. E.g. for a single patch which is between commit rev1..rev2, how do we find out rev1 and rev2. On Mon, Nov 5, 2012 at 9:39 PM, Michael J Gruber wrote: > Eric Miao venit, vidit, dixit 05.11.2012 0

Re: Support for a series of patches, i.e. patchset or changeset?

2012-11-05 Thread Michael J Gruber
Eric Miao venit, vidit, dixit 05.11.2012 03:26: > Hi All, > > Does anyone know if git has sort of support for a series of patches, i.e. > a patchset or changeset? So whenever we know the SHA1 id of a single > patch/commit, we know the patchset it belongs to. This is normal when > we do big changes

git log --graph --oneline - false parent-child visualization

2012-11-05 Thread martyone
Hello, the combination of --graph and --oneline sometimes produces output which is -- at least for me -- not clear about parent-child relation between commits. I noticed it when using with --simplify-by-decoration switch but it should not be specific to the use of this switch. See this example ou

Re: git-p4 clone @all error

2012-11-05 Thread Arthur
Hi, Here is my import : Importing from //depot@all into X Initialized empty Git repository in /home/arthur/projets_git/X/X/.git/ Importing revision 4258 (43%) Importing new branch depot/DEV_DATA Resuming with change 4258 Importing revision 5828 (63%) Importing new branch

Re: Lack of netiquette, was Re: [PATCH v4 00/13] New remote-hg helper

2012-11-05 Thread Michael J Gruber
Felipe Contreras venit, vidit, dixit 02.11.2012 17:09: > On Fri, Nov 2, 2012 at 12:03 PM, Michael J Gruber > wrote: >> Andreas Ericsson venit, vidit, dixit 02.11.2012 10:38: >>> On 11/01/2012 02:46 PM, René Scharfe wrote: Also, and I'm sure you didn't know that, "Jedem das Seine" (to eac

[PATCH 2/2] link_alt_odb_entries(): take (char *, len) rather than two pointers

2012-11-05 Thread Michael Haggerty
Change link_alt_odb_entries() to take the length of the "alt" parameter rather than a pointer to the end of the "alt" string. This is the more common calling convention and simplifies the code a tiny bit. Signed-off-by: Michael Haggerty --- sha1_file.c | 10 +- 1 file changed, 5 inserti

[PATCH 0/2] Another minor cleanup involving string_lists

2012-11-05 Thread Michael Haggerty
Nothing really earthshattering here. But it's funny how every time I look closely at a site where I think string_lists could be used, I find problems with the old code. In this case is_absolute_path() is called with an argument that is not a null-terminated string, which is incorrect (though harm

[PATCH 1/2] link_alt_odb_entries(): use string_list_split_in_place()

2012-11-05 Thread Michael Haggerty
Change link_alt_odb_entry() to take a NUL-terminated string instead of (char *, len). Use string_list_split_in_place() rather than inline code in link_alt_odb_entries(). This approach saves some code and also avoids the (probably harmless) error of passing a non-NUL-terminated string to is_absolu