| From | Subject | Date |
|---|---|---|
| Johannes Schindelin | Re: [PATCH] cat-file --batch / --batch-check: do not exi ...
Hi,
maybe s/do not exit/complain/? Because it still exits if the hashes are
Is it really intended that it returns 0 now?
Further, should it not be an error("%s missin... instead? It is an
error, isn't it?
Ciao,
Dscho
--
| Jun 8, 4:34 pm 2008 |
| Lea Wiemann | [PATCH] cat-file --batch / --batch-check: do not exit if ...
Previously, cat-file --batch / --batch-check would silently exit if it
was passed a non-existent SHA1 on stdin. Now it prints "<SHA1>
missing" as in all other cases (and as advertised in the
documentation).
Note that cat-file --batch-check (but not --batch) will still output
"error: unable to find <SHA1>" on stderr if a non-existent SHA1 is
passed, but this does not affect parsing its stdout.
Signed-off-by: Lea Wiemann <LeWiemann@gmail.com>
---
builtin-cat-file.c | 7 +++++--
...
| Jun 8, 4:28 pm 2008 |
| Catalin Marinas | [ANNOUNCE] Stacked GIT 0.14.3
Stacked GIT 0.14.3 release is available from http://www.procode.org/stgit/.
StGIT is a Python application providing similar functionality to Quilt
(i.e. pushing/popping patches to/from a stack) on top of GIT. These
operations are performed using GIT commands and the patches are stored
as GIT commit objects, allowing easy merging of the StGIT patches into
other repositories using standard GIT functionality.
This is a maintenance release including bug-fixes for the "refresh"
command, handling ...
| Jun 8, 4:10 pm 2008 |
| Lea Wiemann | [PATCH] t1006-cat-file.sh: typo
Previously timestamps were removed unconditionally (though this didn't
seem to break this test). Now they are only removed if $no_ts is
non-empty.
Signed-off-by: Lea Wiemann <LeWiemann@gmail.com>
---
The no_ts feature doesn't seem to be particularly important (it still
works if you switch it always on), but for now I'll just fix it so
that it does what the author apparently intended.
t/t1006-cat-file.sh | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git ...
| Jun 8, 4:03 pm 2008 |
| Junio C Hamano | Re: [PATCH] progress.c: avoid use of dynamic-sized array
Heh, thanks for being picky ;-)
Something this small, either way is fine by me. Besides, I've applied the
patch (and the other one) from Boyd already.
--
| Jun 8, 3:02 pm 2008 |
| Boyd Lynn Gerber | Re: [PATCH] progress.c: avoid use of dynamic-sized array
It is his from my orignal modifications. I thought he suggested I make
the changes and resubmit it as a seperate patch. Thanks for the
clarifications.
--
Boyd Gerber <gerberb@zenez.com>
ZENEZ 1042 East Fort Union #135, Midvale Utah 84047
--
| Jun 8, 2:53 pm 2008 |
| Junio C Hamano | Re: [PATCH 1/3] Modified test-lib.sh to output stats to ...
Saving at the beginning to use in test_done, and optionally detecting
Yup. That is what I meant by "not immediate, but something to keep in
mind".
--
| Jun 8, 1:59 pm 2008 |
| Junio C Hamano | Re: [PATCH] provide a new "theirs" strategy, useful for ...
Can this strong special case limitation "only the first one can be the
amend" somehow be loosened? Otherwise I suspect it would not be useful as
a general solution.
--
| Jun 8, 1:59 pm 2008 |
| Paolo Bonzini | Re: [PATCH] provide a new "theirs" strategy, useful for ...
Well, the point of the exercise is to split a *single* commit into a
"base" commit (already available, possibly on another branch) and a
"delta" (the amending, transformed into an independent commit whose
parent is the "base"). Indeed you can do that for any commit.
The script uses the "git-merge-base" to compute the "base", and takes
the following commit (on the path to HEAD) as the "delta". That's what
add the restriction. You can definitely make a two-argument variation
that, ...
| Jun 8, 4:06 pm 2008 |
| Johannes Schindelin | Re: [BUG] git-branch -m cannot rename remote branches
Hi,
That is very much by design; a next fetch would get them with their
current name anyway. A "remote prune" should get rid of any stale
branches.
So yes, "branch -m" with remote branches does not make sense.
Ciao,
Dscho
--
| Jun 8, 3:58 pm 2008 |
| Jakub Narebski | [BUG] git-branch -m cannot rename remote branches
I cannot rename remote branch (possibly packed, and possibly with
reflog) using "git branch"
[gitweb/web@git]$ git branch -m refs/remotes/gitweb-caching/master \
refs/remotes/gitweb-kernel.org/master
error: refname refs/heads/refs/remotes/gitweb-caching/master not found
fatal: Branch rename failed
[gitweb/web@git]$ git branch -m gitweb-caching/master \
gitweb-kernel.org/master
error: refname refs/heads/gitweb-caching/master not ...
| Jun 8, 1:57 pm 2008 |
| Sverre Rabbelier | Re: [PATCH 1/3] Modified test-lib.sh to output stats to ...
Ah, I guess logging to /tmp/git-test-result was more reliable then.
Any suggestions as to what is the best solution here? (We could for
Ah, yes, I was going to have each test log to their own file
(test-results-$$) and then cat the result together, but I figured that
since we are .NOTPARALLEL anyway it would be more efficient not to. I
reckon that whenever we decide to make such a change it will be then
that we modify this script, if it is included?
--
Cheers,
Sverre Rabbelier
--
| Jun 8, 12:02 pm 2008 |
| Junio C Hamano | Re: [PATCH 1/3] Modified test-lib.sh to output stats to ...
[Empty message]
| Jun 8, 11:53 am 2008 |
| Matthieu Moy | Re: [PATCH] Port to 12 other Platforms.
You won't do a push, the maintainer will apply the patch if it seems
The format used to send patches. git format-patch and git send-email
would do that for you, but in particular : what comes after « --- »
and before the patch itself is ignored by the tools, just an
informative message for people reading your email. What comes before
this « --- » is the commit message.
--
Matthieu
--
| Jun 8, 9:21 am 2008 |
| Jakub Narebski | Re: [PATCH] Port to 12 other Platforms.
By the way, if you want to send large series of patches, and you have
your git repository published somewhere (repo.or.cz, gitorious, github,
freedektop, kernel.org), sometimes pull request is used. As you
usually use git-format-patch (perhaps together with git-send-email)
when sending patches, you can use output of git-request-pull there.
Strange. While in git-diff family of commands '-p' means generate
diff in patchset format:
-p Generate patch (see section on generating ...
| Jun 8, 10:19 am 2008 |
| Boyd Lynn Gerber | Re: [PATCH] Port to 12 other Platforms.
I agree, but getting various things changed is near impossible.
Especially since they are either in a retired or unmaintained state.
Although a few are still release updated products, they refuse to change
Agreed.
Thanks,
--
Boyd Gerber <gerberb@zenez.com>
ZENEZ 1042 East Fort Union #135, Midvale Utah 84047
--
| Jun 8, 2:49 pm 2008 |
| Boyd Lynn Gerber | Re: [PATCH] Port to 12 other Platforms.
I moved the message from above my commiter message because I was told to
add comments after the --- with the file being modified. The reasons for
the modifications. I used git format-patch -p -M -B origin to generate
this email. All I did was read the contents into the email I create for
posting. This was the advice I was given on IRC as the best method to
create/send the patch. When on IRC I was omitting the -p and was not
getting the .patch file. After reading the docs over and ...
| Jun 8, 9:33 am 2008 |
| Junio C Hamano | Re: [PATCH] Port to 12 other Platforms.
It might appear that many people somehow hate your patch and ganging up on
it, and if so I apologize for them and I assure you that they do not mean
ill.
There seem to be some confusion either in the SubmittingPatches document
or the way some suggestions have been given in the recent postings by
people, so let's clear it up first.
There are four different kinds of information you would want to convey
when you send patches to the list. This is just a convention around here,
but the tool is ...
| Jun 8, 11:52 am 2008 |
| Jakub Narebski | Re: [PATCH] Port to 12 other Platforms.
Above info is neither in commit message, not in comment in some file.
It would be nice to have it in somewhere, and not only in mailing list
archives.
--
Jakub Narebski
Poland
ShadeHawk on #git
--
| Jun 8, 8:46 am 2008 |
| Boyd Lynn Gerber | Re: [PATCH] Port to 12 other Platforms. (Comments)
I have setup a repo on www.zenez.com.
I just did a git clone give://www.zenez.com/git git.
I am trying to get various communities to use git. So I want all there
changes to go to my master repo. I am trying to get those that still
support/work on SCO OS's to embrase the Open Source Software (OSS)
concepts and give back to the OSS community inspite of the SCO Group and
what they are doing. I am a firm believer of OSS Concepts. I still have
to support many SCO Systems. My clients ...
| Jun 8, 10:58 am 2008 |
| Boyd Lynn Gerber | Re: [PATCH] Port to 12 other Platforms.
This was from my own copy of the master archive. It is my proposal. I
thought you had to get an OK from this list before you do a push to the
main archive. Am I missing something? I am new to this list and the
proper methods for submitting patches. I thought I was following the
guidelines from
http://repo.or.cz/w/git.git?a=blob_plain;f=Documentation/SubmittingPatches;hb=HEAD
What am I missing?
--
Boyd Gerber <gerberb@zenez.com>
ZENEZ 1042 East Fort Union #135, Midvale Utah ...
| Jun 8, 9:04 am 2008 |
| Junio C Hamano | Re: [PATCH] Port to 12 other Platforms.
Thanks.
I'll apply, but I'd reword the part on _XOPEN_SOURCE like this:
Looking at the the various platform headers, I see around line 450:
#if defined(_KERNEL) || !defined(_POSIX_SOURCE) \
&& !defined(_POSIX_C_SOURCE) && !defined(_XOPEN_SOURCE)
which hides u_short and other typedefs that other header files on these
platforms depend on. With _XOPEN_SOURCE defined, sources that include
system header files that depend on the typedefs such as ...
| Jun 8, 2:19 pm 2008 |
| Boyd Lynn Gerber | Re: [PATCH] Port to 12 other Platforms.
So the patch should be
From: Boyd Lynn Gerber <gerberb@zenez.com>
Date: Sun, 8 Jun 2008 11:41:46 -0600
[PATCH] Port to 12 other Platforms.
This patch adds support to compile and run git on 12 additional platforms.
The platforms are based on UNIX Systems Labs (USL)/Novell/SYS V code base.
The most common are Novell UnixWare 2.X.X, SCO UnixWare 7.X.X,
OpenServer 5.0.X, OpenServer 6.0.X, and SCO pre OSR 5 platforms.
This is from
# 1 "/usr/include/netinet/tcp_f.h"
The problem is that ...
| Jun 8, 1:47 pm 2008 |
| Boyd Lynn Gerber | [PATCH] Port to 12 other Platforms.
This patch adds support to compile git on 12 additional platforms.
They are based on UNIX Systems Labs (USL)/Novell and SYS V
based OS's, SCO OpenServer 5.0.X, SCO UnixWare 7.1.4, OpenServer 6.0.X and
SCO pre OSR 5 OS's to build and run git.
Signed-off-by: Boyd Lynn Gerber <gerberb@zenez.com>
--
Boyd Gerber <gerberb@zenez.com>
ZENEZ 1042 East Fort Union #135, Midvale Utah 84047
---
Developer's Certificate of Origin 1.1
By making a contribution to this project, I ...
| Jun 8, 8:28 am 2008 |
| Boyd Lynn Gerber | [PATCH] version 0001 Port to 12 other Platforms.
This patch adds support to compile and run git on 12 additional platforms.
The platforms are based on UNIX Systems Labs (USL)/Novell/SYS V code base.
The most common are Novell UnixWare 2.X.X, SCO UnixWare 7.X.X,
OpenServer 5.0.X, OpenServer 6.0.X, and SCO pre OSR 5 platforms.
This is from
The problem is that git source has blocked some typedefs
by excluding certain <sys/types.h> content.
Looking at the the various platform header, I see around line 450
&& ...
| Jun 8, 2:36 pm 2008 |
| Boyd Lynn Gerber | [PATCH] progress.c: avoid use of dynamic-sized array
Dynamically sized arrays are gcc and C99 construct. Using them hurts
portability to older compilers, although using them is nice in this case
it is not desirable. This patch removes the only use of the construct
in stop_progress_msg(); the function is about writing out a single line
of a message, and the existing callers of this function feed messages
of only bounded size anyway, so use of dynamic array is simply overkill.
This is with suggestions from
Daniel Barkalow ...
| Jun 8, 2:37 pm 2008 |
| Boyd Lynn Gerber | Re: [PATCH] Port to 12 other Platforms. (Comments)
Sorry type-o should have been
git clone git://www.zenez.com/git git
--
Boyd Gerber <gerberb@zenez.com>
ZENEZ 1042 East Fort Union #135, Midvale Utah 84047
--
| Jun 8, 11:01 am 2008 |
| Boyd Lynn Gerber | Re: [PATCH] Port to 12 other Platforms.
This list has been very good. The problem comes from other lists and
Yes, I was a bit confused but the docs/email/IRC. I really apperciate the
message below. I really want to comply with the rules of this list and
make sure my changes make it into the master/core source.
I will have to find all this information. It took me 2 months in my
personal time to find and fix them. I will have to get back on this
Thanks, more later when time permits.
--
Boyd Gerber ...
| Jun 8, 12:31 pm 2008 |
| Alex Riesen | Re: [PATCH] progress.c: avoid use of dynamic-sized array
It is just a temporary buffer, there is no urgent need to
micro-optimize the allocation like that. Maybe just leave
one of the buffers, the one on stack? It is just a progress
message, snprintf will cut it, yes, but it is unlikely to
cause any harm (unless you forget to \n-terminate it).
--
| Jun 8, 2:34 pm 2008 |
| Boyd Lynn Gerber | Re: [PATCH] progress.c: avoid use of dynamic-sized array
I put it in because in the git repo it is not their. Only in the email.
It is fine to remove it.
--
Boyd Gerber <gerberb@zenez.com>
ZENEZ 1042 East Fort Union #135, Midvale Utah 84047
--
| Jun 8, 12:52 pm 2008 |
| Boyd Lynn Gerber | [PATCH] progress.c: avoid use of dynamic-sized array
Dynamically sized arrays are gcc and C99 construct. Using them hurts
portability to older compilers, although using them is nice in this case
it is not desirable. This patch removes the only use of the construct
in stop_progress_msg(); the function is about writing out a single line
of a message, and the existing callers of this function feed messages
of only bounded size anyway, so use of dynamic array is simply overkill.
Signed-off-by: Boyd Lynn Gerber <gerberb@zenez.com>
--
Boyd ...
| Jun 8, 8:26 am 2008 |
| Johannes Schindelin | Re: [PATCH] progress.c: avoid use of dynamic-sized array
Hi,
Do you really want to have your mail signature in the commit message,
You repeat it at the end of the mail anyway?
The patch looks fine to me, though.
Ciao,
Dscho
--
| Jun 8, 12:30 pm 2008 |
| しらいしななこ | Re: [PATCH] progress.c: avoid use of dynamic-sized array
I may be mistaken but isn't this Junio's patch? If so (quoting
from SubmittingPatches document):
If you are forwarding a patch from somebody else, optionally, at
the beginning of the e-mail message just before the commit
message starts, you can put a "From: " line to name that person.
--
Nanako Shiraishi
http://ivory.ap.teacup.com/nanako3/
----------------------------------------------------------------------
Find out how you can get spam free ...
| Jun 8, 2:50 pm 2008 |
| Johannes Sixt | [PATCH] Remove exec bit from builtin-fast-export.c
Signed-off-by: Johannes Sixt <johannes.sixt@telecom.at>
---
Just in case you care...
-- Hannes
0 files changed, 0 insertions(+), 0 deletions(-)
mode change 100755 => 100644 builtin-fast-export.c
diff --git a/builtin-fast-export.c b/builtin-fast-export.c
old mode 100755
new mode 100644
--
1.5.6.rc2.6.g3056b
--
| Jun 8, 7:40 am 2008 |
| Johannes Sixt | [PATCH] make_nonrelative_path: Use is_absolute_path()
This helps porting to Windows.
Signed-off-by: Johannes Sixt <johannes.sixt@telecom.at>
---
path.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/path.c b/path.c
index 7c3b89a..5da41c7 100644
--- a/path.c
+++ b/path.c
@@ -321,7 +321,7 @@ const char *make_nonrelative_path(const char *path)
{
static char buf[PATH_MAX + 1];
- if (path[0] == '/') {
+ if (is_absolute_path(path)) {
if (strlcpy(buf, path, PATH_MAX) >= PATH_MAX)
die ("Too long path: ...
| Jun 8, 7:34 am 2008 |
| Sverre Rabbelier | [PATCH 3/3] Hook up the result aggregation in the test m ...
From: Sverre Rabbelier <srabbelier@gmail.com>
This patch makes 'make' output the aggregated results at the end of each build.
The 'git-test-result' file is removed both before and after each build.
Signed-off-by: Sverre Rabbelier <srabbelier@gmail.com>
---
t/Makefile | 11 +++++++++--
1 files changed, 9 insertions(+), 2 deletions(-)
diff --git a/t/Makefile b/t/Makefile
index c6a60ab..f9ff933 100644
--- a/t/Makefile
+++ b/t/Makefile
@@ -14,18 +14,25 @@ SHELL_PATH_SQ = $(subst ...
| Jun 8, 7:04 am 2008 |
| Jakub Narebski | Re: [PATCH 1/3] Modified test-lib.sh to output stats to ...
Errr... it looks like you forgot to update commit message.
But that aside, I quite like this series.
--
Jakub Narebski
Poland
ShadeHawk on #git
--
| Jun 8, 7:42 am 2008 |
| Sverre Rabbelier | [PATCH 1/3] Modified test-lib.sh to output stats to /tmp ...
From: Sverre Rabbelier <srabbelier@gmail.com>
This change is needed order to aggregate data on the test run later on.
Because writing to the current directory is not possible, we write to /tmp/.
Suggestions for a better location are welcome.
Signed-off-by: Sverre Rabbelier <srabbelier@gmail.com>
---
t/test-lib.sh | 10 ++++++++++
1 files changed, 10 insertions(+), 0 deletions(-)
diff --git a/t/test-lib.sh b/t/test-lib.sh
index 7a8bd27..4585fde 100644
--- a/t/test-lib.sh
+++ ...
| Jun 8, 7:04 am 2008 |
| Sverre Rabbelier | [PATCH 2/3] A simple script to parse the results from th ...
From: Miklos Vajna <vmiklos@frugalware.org>
This is a simple script that aggregates key:value pairs in a file.
Signed-off-by: Miklos Vajna <vmiklos@frugalware.org>
---
t/aggregate-results.sh | 33 +++++++++++++++++++++++++++++++++
1 files changed, 33 insertions(+), 0 deletions(-)
create mode 100755 t/aggregate-results.sh
diff --git a/t/aggregate-results.sh b/t/aggregate-results.sh
new file mode 100755
index 0000000..9e4322c
--- /dev/null
+++ b/t/aggregate-results.sh
@@ -0,0 +1,33 ...
| Jun 8, 7:04 am 2008 |
| Sverre Rabbelier | Re: [PATCH 1/3] Modified test-lib.sh to output stats to ...
Yeah, I intended to fix that when amending the original commit, but I
forgot. I thought I couldn't write to the current dir, but instead I
was writing to /t/trash, which of course gets deleted after every
Thanks!
--
Cheers,
Sverre Rabbelier
--
| Jun 8, 7:45 am 2008 |
| Sebastian Harl | Re: git local repo export
Hi Cyrill,
Imho, that's a fairly elegant way to do that.
You might want to have a look at git-bundle as well. However, that's not
suited for moving the whole repository, but rather to exchange patches
without being able to use any network based protocols.
HTH,
Sebastian
--=20
Sebastian "tokkee" Harl +++ GnuPG-ID: 0x8501C7FC +++ http://tokkee.org/
Those who would give up Essential Liberty to purchase a little Temporary
Safety, deserve neither Liberty nor Safety. -- Benjamin ...
| Jun 8, 6:59 am 2008 |
| Cyrill Gorcunov | git local repo export
Hi,
could someone say me if there is a way to pack git repo in a single file somehow?
I mean - sometime I have to move _whole_ git database by some media without net
access so I can't fetch it. Usually I just create tar.gz for .git directory and
copy it to some media then unpack it in place where I need. But maybe there is
some other more elegant way exist to do the same?
Thanks in advace, please CC me since I'm not subscribed to git list.
- Cyrill -
--
| Jun 8, 6:46 am 2008 |
| Cyrill Gorcunov | Re: git local repo export
[Sebastian Harl - Sun, Jun 08, 2008 at 03:59:49PM +0200]
| Hi Cyrill,
|
| On Sun, Jun 08, 2008 at 05:46:01PM +0400, Cyrill Gorcunov wrote:
| > could someone say me if there is a way to pack git repo in a single
| > file somehow? I mean - sometime I have to move _whole_ git database
| > by some media without net access so I can't fetch it. Usually I just
| > create tar.gz for .git directory and copy it to some media then unpack
| > it in place where I need. But maybe there is some other more ...
| Jun 8, 7:01 am 2008 |
| Junio C Hamano | Re: Unfathomable merge conflict
The commits given by that will contain ones that are based on the file
from a version way older than v2.6.25, which were merged to the history
after v2.6.25.
$ git log -m --pretty=oneline --abbrev-commit v2.6.25..v2.6.26-rc4 -- \
net/mac80211/rc80211_pid_algo.c
2c8dccc... mac80211: rename files
d0709a6... mac80211: RCU-ify STA info structure access
902acc7... mac80211: clean up mesh code
ee38585... mac80211: mesh data structures and first mesh changes
...
| Jun 8, 2:48 am 2008 |
| Vegard Nossum | Unfathomable merge conflict
Hi,
I want to do a kind of manual rebase where I check out an older
version of a branch and simply re-apply a selection of the commits
following this point in history.
So in this specific case, I want to check out v2.6.25 of linux-2.6.git
and re-apply all the changes that were made to the file
net/mac80211/rc80211_pid_algo.c between v2.6.25 and v2.6.26-rc4.
In order to determine which commits I need, I used the following command:
$ git log --follow v2.6.25..v2.6.26-rc4 ...
| Jun 8, 12:57 am 2008 |
| Junio C Hamano | Re: [PATCH] 0003 This patch is to allow 12 different OS' ...
More like this:
From: Boyd Lynn Gerber <gerberb@zenez.com>
Subject: [PATCH] Port to other 12 platforms
Date: Sat, 7 Jun 2008 21:46:48 -0600
To: git@vger.kernel.org
Cc: gitster@pobox.com
This adds support to compile git on 12 platforms (<<whichever one you
ported including UnixWare, SCO,... are listed here.>>).
__USLC__ indicates UNIX System Labs Corperation (USLC), or a
Novell-derived compiler and/or some SysV based OS's. __M_UNIX indicates
...
| Jun 8, 12:45 am 2008 |
| Boyd Lynn Gerber | [PATCH] Port to 12 other Platforms.
This patch adds support to compile git on 12 additional platforms.
The platforms are based on UNIX Systems Labs (USL)/Novell/SYS V code base.
This patch also adds support for Novell UnixWare 2.x.x, UnixWare 7.1.4 and
older, OpenServer 5.0.7 and older OpenServer 6.0.X, and
SCO pre OSR 5 platforms to build and run git.
This is with suggestions and modifications from
Daniel Barkalow <barkalow@iabervon.org>
Junio C Hamano <gitster@pobox.com>
Thomas Harning <harningt@gmail.com>
Jeremy ...
| Jun 8, 10:07 am 2008 |
| Boyd Lynn Gerber | [PATCH] progress.c: avoid use of dynamic-sized array
Dynamically sized arrays are gcc and C99 construct. Using them hurts
portability to older compilers, although using them is nice in this case
it is not desirable. This patch removes the only use of the construct
in stop_progress_msg(); the function is about writing out a single line
of a message, and the existing callers of this function feed messages
of only bounded size anyway, so use of dynamic array is simply overkill.
This is with suggestions and modifications from
Daniel Barkalow ...
| Jun 8, 9:57 am 2008 |
| Junio C Hamano | Re: [PATCH] Fix missing "HEAD" in the RewriteRule
I'll also retitle it to "gitweb setup instruction: rewrite HEAD and root
as well".
Please sign-off your future patches.
Thanks.
--
| Jun 7, 6:17 pm 2008 |
| Junio C Hamano | Re: [PATCH] Documentation/git-remote.txt: remove descrip ...
Is this something we would want to document as a new feature, or just a
regression that makes the existing feature unusable when disconnected from
the network that needs to be fixed in the code?
--
| Jun 8, 1:27 pm 2008 |
| Olivier Marin | remote show/prune: strange -n(--dry-run) option.
Hello,
The git-remote documentation talks about a mysterious -n option for show
and prune that comes from the old git-remote.perl script. This flag was
used to prevent the script from calling ls-remote more than once, FWIU.
Today, the builtin accept an (un)?related -n(--dry-run) flag that does
nothing, actually. It seems broken.
So, is it safe to drop it entirely or is it better to just remove it
from the documentation for compatibility? In the second case, how long
should we wait before ...
| Jun 7, 5:54 pm 2008 |
| Olivier Marin | [PATCH] Documentation/git-remote.txt: remove description ...
From: Olivier Marin <dkr@freesurf.fr>
This option comes from the original git-remote.perl script and is not
used nor needed in the current builtin.
So, remove it from the documentation so that we can reuse it later for
something else.
Signed-off-by: Olivier Marin <dkr@freesurf.fr>
---
Documentation/git-remote.txt | 7 -------
1 files changed, 0 insertions(+), 7 deletions(-)
diff --git a/Documentation/git-remote.txt b/Documentation/git-remote.txt
index e97dc09..e51d232 100644
--- ...
| Jun 8, 4:03 am 2008 |
| dkr+ml.git | [PATCH] Documentation/git-remote.txt: remove description ...
From: Olivier Marin <dkr@freesurf.fr>
This option comes from the original git-remote.perl script and is not
used nor needed in the current builtin.
So, remove it from the documentation so that we can reuse it later for
something else.
Signed-off-by: Olivier Marin <dkr@freesurf.fr>
---
My MUA destroyed the previous patch! Sorry.
Documentation/git-remote.txt | 7 -------
1 files changed, 0 insertions(+), 7 deletions(-)
diff --git a/Documentation/git-remote.txt ...
| Jun 8, 5:22 am 2008 |
| Steven Walter | Re: git-svn: multiple branches directories
git-svn does support it. The way that I interact with repositories
like this is to have the main git-svn remote track only the trunk
(fetch = /trunk:...), and then to add addititonal remotes for each
branch that I want to track. There may be a nicer way to do it, but
git-svn certainly supports the method just described.
--
-Steven Walter <stevenrwalter@gmail.com>
"A human being should be able to change a diaper, plan an invasion,
butcher a hog, conn a ship, design a building, write a ...
| Jun 8, 9:00 am 2008 |
| Tzafrir Cohen | git-svn: multiple branches directories
Hi
Does git-svn support a repository layout where branches may reside under
more than one root?
/trunk
/branches
branch1
branch2
branch3
/somedir
branch4
branch5
branch6
/tags
...
--
Tzafrir Cohen
icq#16849755 jabber:tzafrir.cohen@xorcom.com
+972-50-7952406 mailto:tzafrir.cohen@xorcom.com
http://www.xorcom.com iax:guest@local.xorcom.com/tzafrir
--
| Jun 7, 5:23 pm 2008 |
| Tzafrir Cohen | Re: git-svn: multiple branches directories
Thanks for your answer,
I'm afraid I don't follow you. Where do I have to write that and where?
--
Tzafrir Cohen
icq#16849755 jabber:tzafrir.cohen@xorcom.com
+972-50-7952406 mailto:tzafrir.cohen@xorcom.com
http://www.xorcom.com iax:guest@local.xorcom.com/tzafrir
--
| Jun 8, 11:37 am 2008 |
| Björn | Re: git-svn: multiple branches directories
You can just add more svn-remote to a single git repo with "git svn
init". So you could do:
git svn clone -s svn://host/some/repo
cd repo
git svn init --branches somedir svn://host/some/repo/ svn2
git svn fetch svn2
To fetch all new stuff at once, you would then later use "git svn fetch
--all".
Björn
--
| Jun 8, 11:51 am 2008 |
| Geoffrey Irving | Re: [PATCH / RFC] cherry: cache commit to patch-id pairs ...
Forgot to cc people from previous threads.
Is starting a new thread each time I update a patch the correct way to
do things?
Geoffrey
--
| Jun 7, 10:39 pm 2008 |
| Johannes Schindelin | Re: [PATCH / RFC] cherry: cache commit to patch-id pairs ...
Hi,
I prefer a response in the existing thread, but I guess it is a matter of
personal taste (of the poster).
Ciao,
Dscho
--
| Jun 8, 9:06 am 2008 |
| Johannes Schindelin | Re: [PATCH / RFC] cherry: cache commit to patch-id pairs ...
Hi,
Please keep your commit messages shorter than 77 columns/row, so that
I think get_hash_index() should already return the correct index, IOW it
IIRC the optimal size for a hash set was double the number of non-NULL
Would it not be better to avoid duplicating code here, by having a helper
that returns the index if the entry exists, and the negative index of the
Ah, so here you ignore the error when writing. Thank you. Maybe a
comment that we fail gracefully if the ...
| Jun 8, 10:24 am 2008 |
| Stephan Beyer | [PATCH 5/5] git-name-rev.txt: document --no-undefined an ...
Signed-off-by: Stephan Beyer <s-beyer@gmx.net>
---
See commit 2cf9f445e9074df1a45f5179cef0d4bb8647dd8
Documentation/git-name-rev.txt | 7 +++++++
1 files changed, 7 insertions(+), 0 deletions(-)
diff --git a/Documentation/git-name-rev.txt b/Documentation/git-name-rev.txt
index 12784d7..ffac3f8 100644
--- a/Documentation/git-name-rev.txt
+++ b/Documentation/git-name-rev.txt
@@ -41,6 +41,13 @@ OPTIONS
of linkgit:git-describe[1] more closely. This option
cannot be combined with ...
| Jun 7, 6:36 pm 2008 |
| Stephan Beyer | [PATCH 2/5] Docs: Use "-l::\n--long\n" format in OPTIONS ...
The OPTIONS section of a documentation file contains a list
of the options a git command accepts.
Currently there are several variants to describe the case that
different options (almost) do the same in the OPTIONS section.
Some are:
-f, --foo::
-f|--foo::
-f | --foo::
But AsciiDoc has the special form:
-f::
--foo::
This patch
* applies this form to the documentation of the whole git suite,
* removes useless em-dash prevention, so \--foo becomes --foo.
Signed-off-by: Stephan ...
| Jun 7, 6:36 pm 2008 |
| Stephan Beyer | [PATCH 1/5] git-commit.txt: Add missing long/short options
And split the "-c or -C <commit>" item into two separate items.
Signed-off-by: Stephan Beyer <s-beyer@gmx.net>
---
Hi,
so here's the first one ;-)
Documentation/git-commit.txt | 15 ++++++++-------
1 files changed, 8 insertions(+), 7 deletions(-)
diff --git a/Documentation/git-commit.txt b/Documentation/git-commit.txt
index 40bf63e..a94b4e3 100644
--- a/Documentation/git-commit.txt
+++ b/Documentation/git-commit.txt
@@ -57,14 +57,16 @@ OPTIONS
been modified and deleted, but new ...
| Jun 7, 6:36 pm 2008 |
| Stephan Beyer | [PATCH 3/5] Docs: add some long/short options
Namely:
git-clean.txt: --dry-run --quiet
git-count-objects.txt: --verbose
git-quiltimport.txt: -n
git-remote.txt: -v --verbose
Signed-off-by: Stephan Beyer <s-beyer@gmx.net>
---
Hi,
this was a kind of semi-automated search to find the options that are not
documented in the manual.
Besides deprecated or unofficial ones, my search found those listed here
and those from the following patches.
Regards,
Stephan
PS: Because the patches are so tiny, I didn't want to make several ...
| Jun 7, 6:36 pm 2008 |
| Stephan Beyer | [PATCH 4/5] git-describe.txt: document --always
Signed-off-by: Stephan Beyer <s-beyer@gmx.net>
---
See commit da2478dbb000436b79e813ba7f243d6042f26e66
Documentation/git-describe.txt | 3 +++
1 files changed, 3 insertions(+), 0 deletions(-)
diff --git a/Documentation/git-describe.txt b/Documentation/git-describe.txt
index 3f0b7b2..9f6f483 100644
--- a/Documentation/git-describe.txt
+++ b/Documentation/git-describe.txt
@@ -70,6 +70,9 @@ OPTIONS
Only consider tags matching the given pattern (can be used to avoid
leaking private ...
| Jun 7, 6:36 pm 2008 |
| Shawn O. Pearce | Re: [RFC PATCH] Use SUDO_UID to guess committer identity
The issue is when users run commands though sudo, but forget to set a
value for GIT_COMMITTER_NAME/EMAIL, or to configure ~/.gitconfig in
their personal account. Now git has to guess the values for these
based on the gecos of getuid(), and getuid() is returning the uid
of the service account sudo entered (gitadm), not the real user's
account. So the reflogs show generic "GIT Admin" and not who it was.
Eh, I'm myself not entirely happy with the patch. It honors the
real user's ...
| Jun 7, 5:23 pm 2008 |
| Junio C Hamano | Re: [RFC PATCH] Use SUDO_UID to guess committer identity
In your scenario, is the above "sudo -u gitadm" the exact command line
The thing is, I personally hate pseudo and wish that your solution did not
rely on SUDO_UID which is too specific to that hack.
Sometimes people need to lie to their SCM when doing things in behalf of
somebody else, and I agree we would want to give them a way to do so. And
we do, just like RCS and CVS honor LOGNAME.
If you have /etc/hosts under RCS control but you do not want all the log
entries to say 'root', and ...
| Jun 7, 5:57 pm 2008 |
| Johannes Schindelin | Re: [PATCH] Remove unused code in parse_commit_buffer()
Hi,
Nice catch!
Ciao,
Dscho
--
| Jun 8, 8:46 am 2008 |
| Junio C Hamano | Re: [PATCH] fix attribute handling in bare repositories
Hmm. I do not know if it breaks anything, but if you are indeed in a bare
repository, the files the codepaths affected try to read would not exist
anyway, so I am not sure what this would fix, other than changing the
behaviour of check-attr from noticing that it was asked for nonsense and
bail out to not noticing nor saying anything useful.
--
| Jun 8, 1:46 pm 2008 |
| René Scharfe | Re: git-archive and unwanted .gitattributes
Attributes are taken from three places (in order of increasing
precedence): a table of built-in defaults, .gitattributes files in the
work tree, and the file $GIT_DIR/info/attributes.
René
--
| Jun 8, 8:24 am 2008 |
| René Scharfe | Re: [PATCH] fix attribute handling in bare repositories
The patch does two things: it fixes git-check-attr to work in a bare
repository, and it makes git ignore .gitattributes files in bare
repositories. True, the latter is fixable by simply not creating these
files in the first place.
Duy somehow ended up with them, though, and reported it as strange that
they are not ignored. And I agree: if we don't have a work tree then we
should not look at it. :-)
René
--
| Jun 8, 2:31 pm 2008 |
| René Scharfe | [PATCH] fix attribute handling in bare repositories
Yes, it probably shouldn't do that. What about this patch?
-- snip! --
Attributes can be specified at three different places: the internal table
of default values, the file $GIT_DIR/info/attributes and files named
.gitattributes in the work tree. Since bare repositories don't have a
work tree, git should ignore any .gitattributes files there.
This patch makes git do that, so the only way left for a user to specify
attributes in a bare repository is the file info/attributes (in ...
| Jun 8, 8:16 am 2008 |
| Johannes Schindelin | Re: [PATCH] improve doc heading for git-bisect
Hi,
You still have to move the subject line where it belongs, so I think the
value of --paste is limited. You do have to pay attention anyway (for
example when imitating how other people do it), so I think the current
state should be fine.
Ciao,
Dscho
--
| Jun 8, 8:01 am 2008 |
| Jeff King | Re: [PATCH] improve doc heading for git-bisect
Not necessarily. I often end up posting patches like:
How about this?
-- >8 --
subject
body
---
diffstat
patch
but the --pretty=format recipe I provided works just fine for that.
-Peff
--
| Jun 8, 2:27 pm 2008 |
| Jeff King | Re: [PATCH] improve doc heading for git-bisect
As others have pointed out, the usual way for using format-patch is to
create an mbox that you then pull into your MUA (e.g., I do "git
format-patch --stdout origin >mbox; mutt -f mbox").
However, sometimes it is desirable to cut and paste, either because it
is inconvenient to bring the message into your MUA this way, or if you
are just putting a commit into an email you have already started
writing.
I was going to suggest a "--paste" option for format-patch which would
produce a nicer ...
| Jun 7, 9:35 pm 2008 |
| Jon Loeliger | Re: [PATCH v2] git-commit.txt: Correct option alternatives
Hey Junio,
Say, if we were to document and espouse one form over the other,
which would you recommend or prefer?
jdl
--
| Jun 7, 6:36 pm 2008 |
| Jeff King | Re: [PATCH v2] git-commit.txt: Correct option alternatives
$ sed -ne '/sticked/,/^$/p' Documentation/gitcli.txt
* when a command line option takes an argument, use the 'sticked' form. In
other words, write `"git foo -oArg"` instead of `"git foo -o Arg"` for short
options, and `"git foo --long-opt=Arg"` instead of `"git foo --long-opt Arg"`
for long options. An option that takes optional option-argument must be
written in the 'sticked' form.
'sticked' form must be used:
----------------------------
$ git describe --abbrev HEAD # ...
| Jun 7, 9:23 pm 2008 |
| Paolo Bonzini | Re: [PATCH] provide a new "theirs" strategy, useful for ...
Yes, I had suggested in the original thread to follow up with a "git
rebase -i" to fix the commit message, because "git-rebase--interactive
--help" did not show a -s option. However, I found out that it does
support it, so it is probably better to use "git rebase -i -s theirs
You mean that I should a) drop the example from git-rebase.1, b) reword
it to clarify it, c) drop the patch completely?
Paolo
--
| Jun 7, 7:54 pm 2008 |
| Paolo Bonzini | Re: [PATCH] provide a new "theirs" strategy, useful for ...
Hmm, anyway you do want to test what this strategy would do --- before
merging. The point of "ours"/"theirs" is AFAICS that they *cannot*
produce broken trees (they can cause you to lose committed stuff if used
carelessly, but the resulting tree is already in some branch and
supposedly has already been tested). So breaking the symmetry would not
be good probably.
I guess I see the reason why "ours" is more useful than "theirs". The
reason is that rebase (and "rebase -i" in ...
| Jun 8, 6:38 am 2008 |
| Junio C Hamano | Re: [PATCH] provide a new "theirs" strategy, useful for ...
Yeah, but that is only about the commit log message. The issue of
recording a wrong tree when commits X and Y exist is not alleviated, is
I have to say that the rebase example is too misleading --- unless it is
accompanied by a lot of disclaimers, its risk to give broken result to
people probably is worse than the benefit. I am afraid that we would need
a lot better use case to justify the use of "theirs" than what you wrote.
I have occasionally seen valid situations to use "ours", but I ...
| Jun 8, 1:16 am 2008 |
| Boyd Lynn Gerber | Re: [PATCH] 0004 This patch is to allow 12 different OS' ...
This patch allows some older OS's, UNIX Systems Labs (USL)/Novell and SYS V
based OS's, SCO OpenServer 5.0.X, SCO UnixWare 7.1.4, OpenServer 6.0.X and
SCO pre OSR 5 OS's to build and run git. Applied suggestions from list.
Developer's Certificate of Origin 1.1
By making a contribution to this project, I certify that:
(a) The contribution was created in whole or in part by me and I
have the right to submit it under the open source license
...
| Jun 7, 8:50 pm 2008 |
| Boyd Lynn Gerber | Re: [PATCH] 0004 This patch is to allow 12 different OS' ...
Sorry, I forgot to copy the real 00* file to the machine I work on.
Below, is what should have been sent.
--
Boyd Gerber <gerberb@zenez.com>
ZENEZ 1042 East Fort Union #135, Midvale Utah 84047
--------------------------------Cut-Here-------------------------------
Subject: [PATCH] This patch is to allow 12 different OS's to compile and run git.
This patch allows some older OS's, UNIX Systems Labs (USL)/Novell and SYS V
based OS's, SCO OpenServer 5.0.X, SCO UnixWare 7.1.4, OpenServer ...
| Jun 8, 12:22 am 2008 |
| Boyd Lynn Gerber | Re: [PATCH] 0003 This patch is to allow 12 different OS' ...
So I should have
---
Makefile
Add changes for System V, UnixWare, SCO OS's
---
git-compat-util.h
__USLC__ indicates UNIX System Labs Corperation (USLC), or a
Novell-derived compiler and/or some SysV based OS's.
__M_UNIX indicates XENIX/SCO UNIX/OpenServer before 5.0.7 and prior
release of and SCO OS. It is used just like Apple and BSD, both of these
shouldn't have _XOPEN_SOURCE defined.
---
progress.c
Changes for older OS's that do not support the current methods for ...
| Jun 7, 8:46 pm 2008 |
| Catalin Marinas | Re: [StGIT PATCH 3/5] Add stack creation and initialisat ...
I can't argue much. I propose to commit the patch as is and modify it
afterwards. I haven't touched this code much, it was Yann's
implementation and I forgot all the decisions at that time.
--
Catalin
--
| Jun 8, 3:03 pm 2008 |
| Nico -telmich- Schot ... | Re: How to compare different files in different branches
Thank you all for your fast answers!
I must confess I've overseen -M, as I saw --no-renames and searched
for --renames.
Very nice syntax, somehow guessed that this is available, but didn't
try... my fault.=20
Perhaps we should reference git-rev-list(1) from git-diff(1), too?
Sincerly
Nico
--=20
Think about Free and Open Source Software (FOSS).
http://nico.schottelius.org/documentations/foss/the-term-foss/
PGP: BFE4 C736 ABE5 406F 8F42 F7CF B8BE F92A 9885 188C
| Jun 8, 2:47 am 2008 |
| Junio C Hamano | Re: How to compare different files in different branches
This is just the standard syntax to name blob _anywhere_, so we should not
hide it in git-rev-list(1) but somewhere people would read before reading
any "technical manual" material, perhaps in the tutorial and git(1).
--
| Jun 8, 2:51 am 2008 |
| Johannes Schindelin | Re: [TOY PATCH] git wrapper: show similar command names ...
Hi,
This patch is not meant for inclusion, as indicated by the "TOY PATCH"
prefix.
Ciao,
Dscho
--
| Jun 8, 8:07 am 2008 |
| Junio C Hamano | Re: [TOY PATCH] git wrapper: show similar command names ...
Yes, I understand it.
I was not complaining about the "who cares, we will exit after we are done
with this processing anyway" attitude in this code. I think it is a
reasonable approach to take because it is not likely that this codepath to
change and would start wanting to access the original command table after
it did its munging. I was suggesting a positive improvement to the patch
by making it explicitly documented to help people polish further, that's
Likewise.
--
| Jun 8, 10:53 am 2008 |
| Dirk Süsserott | Re: [TOY PATCH] git wrapper: show similar command names ...
What a pity! :-( I liked it.
--
| Jun 8, 8:14 am 2008 |
| Johannes Schindelin | Re: [TOY PATCH] git wrapper: show similar command names ...
Hi,
On Sun, 8 Jun 2008, Junio C Hamano wrote:
> Dirk S
| Jun 8, 4:26 pm 2008 |
| Catalin Marinas | Re: [StGIT PATCH 3/5] Create a git.Branch class as ances ...
Hidden plan - once the new stgit.lib.git module is complete, we can
mechanism.
--
Catalin
--
| Jun 8, 3:16 pm 2008 |
| Lea Wiemann | Re: [PATCH] Avoid errors from git-rev-parse in gitweb blame
I haven't been following the recent discussion in detail, but here's
another thought: If you want to look up the parents, it's usually faster
(at least when caching is enabled) to get them all in a single call.
IOW, don't look up the parent for each hash as it appears, but collect
all hashes and then get a list of all parents with a single call.
-- Lea
--
| Jun 8, 11:19 am 2008 |
| Jakub Narebski | Re: [PATCH] Avoid errors from git-rev-parse in gitweb blame
If caching is enabled, then parent info can be retrieved from cache.
If caching is disabled, or cache expired (cache miss) you would have
to get whole blame output to get all revisions to get parents for.
This means for a short while twice amount of memory (whole blame in
git-blame, because thats how non-incremental blame works, and whole
blame in gitweb, till reading last byte of blame when git-blame ends);
and that is not good when memory-based cache (be it memcache, mmap,
or other solution) ...
| Jun 8, 1:28 pm 2008 |
| Johannes Schindelin | Re: [PATCH] Adding a cache of commit to patch-id pairs t ...
Hi,
Right, it is insufficient in such a case, but then, it does not really
matter, methinks. The cache is small enough anyway, and I think that many
people will not really use it as much as you do.
However, I realized one very real issue with your patch: you do not
provide a way to _disable_ the caching. I think at least a config
variable is needed, and while at it, a fallback when you cannot write to
the repository.
Ciao,
Dscho
--
| Jun 8, 9:10 am 2008 |
| Florian Köberle | Re: [JGIT PATCH v3 0/23] Implementation of a file tree i ...
Just did some tests and noted that the shell and git does not support
them. I guess I thought that they work because [:alpha:] matched "a".
Hi Robin
The bash shell doesn't support [:alpha:] too:
$ mkdir test
$ cd test
$ touch a
$ touch b
$ touch :
The fnmatch function of the python module fnmatch
example:
$python
>>> from fnmatch import fnmatch
>>> fnmatch("a","[:alpha:]")
False
So I see no reason for displaying a warning.
Best regards,
Florian
--
| Jun 8, 9:37 am 2008 |
| Robin Rosenberg | Re: [JGIT PATCH v3 0/23] Implementation of a file tree i ...
söndagen den 8 juni 2008 18.37.30 skrev Florian Köberle:
Oh, it does. [:alpha:] is a character class, like a-z. So you use [[:alpha:]], which is what git as well as bash
Hmm. Odd:
$ python
Python 2.5.2 (r252:60911, Apr 12 2008, 01:47:55)
[GCC 4.2.3 (4.2.3-6mnb1)] on linux2
True
False
-- robin
--
| Jun 8, 2:28 pm 2008 |
| Florian Köberle | Re: [JGIT PATCH v3 20/23] Added the class NoGitRepositor ...
Robin Rosenberg wrote:
>> + super(String.format("No repository found for file '%s'",
>> + fileWeSearchedRepositoryFor));
>
> This is any odd patterrn. Usually we use + to concatenate strimgs.
Hi Robin,
I know two good reason why you should use formating functions over
string concatenation via the plus sign:
1.) formated strings are better translatable. e.g.
"The string '%s' is more then %s characters long"
is easier to translate then the three strings:
"The string '"
"' is ...
| Jun 8, 9:14 am 2008 |
| Robin Rosenberg | Re: [JGIT PATCH v3 20/23] Added the class NoGitRepositor ...
I'm so ashamed... I used to like printf in C, but I've hardly used it in Java. I
probably will use it more now. Seems Eclipse doesn't check (even GCC
does that).
-- robin
--
| Jun 8, 2:05 pm 2008 |
| Johannes Schindelin | Re: [PATCH v3] git-add--interactive: manual hunk editing mode
Hi,
If you really think this is a bikeshed, I better spend my time elsewhere.
Hth,
Dscho
--
| Jun 8, 4:19 pm 2008 |
| Thomas Rast | [PATCH v3] git-add--interactive: manual hunk editing mode
Subject: [PATCH] git-add--interactive: manual hunk editing mode
Adds a new option 'e' to the 'add -p' command loop that lets you
edit the current hunk in your favourite editor.
Signed-off-by: Thomas Rast <trast@student.ethz.ch>
---
On top of the previous patch, this adds basic testing.
Compared to the competing bikeshed
http://article.gmane.org/gmane.comp.version-control.git/83894
this integrates into the 'add -p' loop. I (still) prefer it because I
use 'add -p' a lot, thus having ...
| Jun 8, 3:32 pm 2008 |
| Johannes Schindelin | Re: [RFC PATCH] git-add--interactive: manual hunk editin ...
Hi,
That is actually where Junio convinced me that my approach is wrong: he
said that you can _only_ reliably split a hunk at common lines.
The thing is: if you split _not_ at a common line, the context of the
second part would _change_ depending if you want to apply the first part
or not.
Ciao,
Dscho
--
| Jun 8, 4:06 pm 2008 |
| Thomas Rast | Re: [RFC PATCH] git-add--interactive: manual hunk editin ...
I think there's no way to split hunks, in the way I currently "help"
with @@ line guessing, using just the --recount feature. Unless the
editor helps you with adding complete correct @@ lines in the middle
of hunks (Emacs does that). I don't think it is at all possible to
remove the middle part of a hunk in Johannes' scheme without somehow
figuring out the corresponding @@ line (or at least its old line
number) or editing away every +/- line.
Then again it's not always easy even with my ...
| Jun 8, 3:33 pm 2008 |
| Johannes Schindelin | Re: [RFC PATCH] git-add--interactive: manual hunk editin ...
Hi,
Oh, sorry, I really meant the "I do not want" literally. It is just me.
That does not mean that your hunk editing from within add -i has no merit.
It's just that this guy is not very interested in that feature.
Ciao,
Dscho
--
| Jun 8, 4:02 pm 2008 |
| Thomas Rast | Re: [RFC PATCH] git-add--interactive: manual hunk editin ...
While there is obviously little point in trying to convince you, let
me briefly explain why I still think it is useful:
I usually run 'add -p' instead of adding specific files. "Nodding
off" each hunk means that I get a last chance to review my changes,
and perhaps skip some of them (possibly for a later commit). With my
proposed editing feature, I can split hunks in the middle too. You
could of course argue that the right way to do this would be staring
at 'git diff', or perhaps scrolling ...
| Jun 8, 3:18 pm 2008 |
| Sverre Rabbelier | Re: [PATCH] A simple script to parse the results from th ...
I used the 'show original message' feature, so that I could edit it
before applying. I changed the name to aggregate-results.sh as
suggested and changed the path to /t/test-results. Now let's see if I
can make a new patch series out of this...
--
Cheers,
Sverre Rabbelier
--
| Jun 8, 4:43 am 2008 |
| Mikael Magnusson | Re: [PATCH] A simple script to parse the results from th ...
git mv oldname newname
git commit --amend ;: not --ammend
--
Mikael Magnusson
--
| Jun 8, 11:06 am 2008 |
| Miklos Vajna | Re: [PATCH] A simple script to parse the results from th ...
Or:
$ git pull git://repo.or.cz/git/vmiklos.git stat
| Jun 7, 7:26 pm 2008 |
| Sverre Rabbelier | Re: [PATCH] A simple script to parse the results from th ...
On Sun, Jun 8, 2008 at 7:27 PM, Johannes Schindelin
Hmmm, yeah, but can you change the filename easily with commit
--ammend / rebase --interactive?
--
Cheers,
Sverre Rabbelier
--
| Jun 8, 10:30 am 2008 |
| Miklos Vajna | Re: [PATCH] A simple script to parse the results from th ...
Sure. I would suggest:
1) Remove that ugly /tmp/git-test-results, place it under t/.
2) Resend a series indicating this is no longer a demonstration but a
real series which you want to be included. ;-)
Ah and it's bikesheding, but probably key_value_parser.sh is not the
best name for such a script. Maybe aggregate-results.sh or something
like that.
| Jun 7, 5:49 pm 2008 |
| Sverre Rabbelier | Re: [PATCH] A simple script to parse the results from th ...
Details, details.
Ah well, the way I did it seemed the most easy at the time, maybe next
time I'll use amend (my favorite feature in git that I usually spell
incorrectly at least twice).
--
Cheers,
Sverre Rabbelier
--
| Jun 8, 11:09 am 2008 |
| Sverre Rabbelier | Re: [PATCH] A simple script to parse the results from th ...
Awesome, what do you want to do with the other patches? I mean, this
patch on it's own doesn't make a lot of sense, but with [1/3] and
[3/3] I think it deserves some proper reviewing by the list.
--
Cheers,
Sverre Rabbelier
--
| Jun 7, 5:34 pm 2008 |
| Miklos Vajna | [PATCH] A simple script to parse the results from the te ...
This is a simple script that aggregates key:value pairs in a file.
Signed-off-by: Miklos Vajna <vmiklos@frugalware.org>
---
Here is a shell version. Just to avoid python.
t/key_value_parser.sh | 33 +++++++++++++++++++++++++++++++++
1 files changed, 33 insertions(+), 0 deletions(-)
create mode 100755 t/key_value_parser.sh
diff --git a/t/key_value_parser.sh b/t/key_value_parser.sh
new file mode 100755
index 0000000..db568fe
--- /dev/null
+++ b/t/key_value_parser.sh
@@ -0,0 +1,33 ...
| Jun 7, 5:18 pm 2008 |
| Johannes Schindelin | Re: [PATCH] A simple script to parse the results from th ...
Hi,
FWIW that's what "commit --amend" and "rebase --interactive" are for.
Ciao,
Dscho
--
| Jun 8, 10:27 am 2008 |
| Sverre Rabbelier | Re: [PATCH] A simple script to parse the results from th ...
Heh, I'm not sure what the protocol is here :P. I could send in the
series with your patch as second... that is, if I can figure out how
I remember trying that but it not working, which was why I put it
Sure, but that's what it was though, a simple key_value_parser, your
version is actually a result aggregator.
--
Cheers,
Sverre Rabbelier
--
| Jun 7, 5:56 pm 2008 |
| René Scharfe | Re: how to git-archive ignore some files?
git-archive uses the routines from attr.c, and this is where the ability
to read attributes from git trees should be added. It's independent from
ignoring files when creating archives. Maybe it would be a good idea to
add this feature; in the meantime users need to resort to adding their
attributes to $GIT_DIR/info/attributes.
Anyway, here's a patch to add export-ignore support to git-archive.
Files and directories with this attribute are not added to archives.
Signed-off-by: Rene Scharfe ...
| Jun 8, 9:42 am 2008 |
| previous day | today | next day |
|---|---|---|
| June 7, 2008 | June 8, 2008 | June 9, 2008 |
