git mailing list

FromSubjectsort iconDate
Alex Riesen
[PATCH] Fix dependencies of parse-options test program
Signed-off-by: Alex Riesen <raa.lkml@gmail.com> --- An old test-parse-options was breaking t0040. Makefile | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/Makefile b/Makefile index cd6d469..af827f6 100644 --- a/Makefile +++ b/Makefile @@ -999,6 +999,8 @@ test-date$X: date.o ctype.o test-delta$X: diff-delta.o patch-delta.o +test-parse-options$X: parse-options.o + .PRECIOUS: $(patsubst test-%$X,test-%.o,$(TEST_PROGRAMS)) test-%$X: test-%.o $(GITL...
Nov 13, 7:16 pm 2007
Johannes Schindelin
Re: [PATCH] Fix dependencies of parse-options test program
Hi, apparently I forgot to send this patch, which I thought was only relevant in the builtin-commit branch of mine: -- snipsnap -- [PATCH] Makefile: add correct dependencies for test-parse-options Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de> --- Makefile | 4 ++++ 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/Makefile b/Makefile index bda8762..2abb8da 100644 --- a/Makefile +++ b/Makefile @@ -996,6 +996,10 @@ test-date$X: date.o ctype.o ...
Nov 13, 7:46 pm 2007
Jakub Narebski
Re: Integrating with hooks
[Cc: Todd A. Jacobs <nospam@codegnome.org>, git@vger.kernel.org] Take a look at gitattributes(5), namely 'filter' attribute. Although instead of implementing it "by hand", perhaps it would be enough to use 'ident' and/or 'export-subs' attribute. P.S. Because of the way git updates files, and git thinks about files it is I think universally regarded to be bad idea to put in a file any Id that depend on commit data. -- Jakub Narebski Warsaw, Poland ShadeHawk on #git -
Nov 13, 7:07 pm 2007
David Reiss
Trivial patch to git-svn
From d9f93dc1c503c5f42b27485b2c35b38e8c9bca44 Mon Sep 17 00:00:00 2001 From: David Reiss <dreiss@dreiss-vmware.(none)> Date: Tue, 13 Nov 2007 13:47:26 -0800 Subject: [PATCH] Fix a typo and add a comma in an error message in git-svn Signed-off-by: David Reiss <dreiss@facebook.com> --- git-svn.perl | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/git-svn.perl b/git-svn.perl index e3e00fd..981cdd6 100755 --- a/git-svn.perl +++ b/git-svn.perl @@ -391,7 +39...
Nov 13, 5:56 pm 2007
shunichi fuji
git-clean won't read global ignore
hi, i setup git to use with eclipse through global excludesfile config. git-status report just ignore files, but git-clean deleted ignore files. ---- $ git-config -l core.excludesfile=/home/pal/.gitignore $ cat /home/pal/.gitignore # ignore for eclipse .project .cproject $ git-status # On branch master nothing to commit (working directory clean) $ git-clean Removing .project -
Nov 13, 5:49 pm 2007
Pierre Habouzit
Re: git-clean won't read global ignore
=2Eproject is not ..project right ? --=20 =C2=B7O=C2=B7 Pierre Habouzit =C2=B7=C2=B7O madcoder@debia= n.org OOO http://www.madism.org
Nov 13, 6:50 pm 2007
Junio C Hamano
[PATCH] revert/cherry-pick: allow starting from dirty work t...
There is no reason to forbid a dirty work tree when reverting or cherry-picking a change, as long as the index is clean. The scripted version used to allow it: case "$no_commit" in t) # We do not intend to commit immediately. We just want to # merge the differences in. head=$(git-write-tree) || die "Your index file is unmerged." ;; *) head=$(git-rev-parse --verify HEAD) || die "You do not have a valid HEAD" files=$(git-diff-index --cach...
Nov 13, 5:16 pm 2007
Johannes Schindelin
Re: [PATCH] revert/cherry-pick: allow starting from dirty wo...
Hi, Probably. Thanks. While we're at cherry-pick: Two days ago I had to rebase in a dirty working directory. Why? Because one of the submodules was not yet ready to be committed to the superproject. And you cannot easily stash away a submodule. Now, my quick and dirty solution was to hack a GIT_IGNORE_SUBMODULES patch: http://repo.or.cz/w/git/mingw/4msysgit.git?a=commitdiff;h=9ef6b6bff4e368934c4262af13b0... But the more fundamental question is: should we eventually have ...
Nov 13, 7:37 pm 2007
Junio C Hamano
[PATCH 1/2] t/t3404: fix test for a bogus todo file.
The test wants to see if there are still remaining tasks, but checked a wrong file. Signed-off-by: Junio C Hamano <gitster@pobox.com> --- t/t3404-rebase-interactive.sh | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/t/t3404-rebase-interactive.sh b/t/t3404-rebase-interactive.sh index 1113904..f1039d1 100755 --- a/t/t3404-rebase-interactive.sh +++ b/t/t3404-rebase-interactive.sh @@ -149,7 +149,7 @@ test_expect_success 'stop on conflicting pick' ' diff -u expect...
Nov 13, 5:14 pm 2007
Johannes Schindelin
Re: [PATCH 1/2] t/t3404: fix test for a bogus todo file.
Hi, Oh well, another fallout of my brilliant name picking. Thanks, Dscho -
Nov 13, 5:52 pm 2007
Linus Torvalds
Strange "beagle" interaction..
Ok, I've made a bugzilla entry for this for the Fedora people, but I thought I'd mention something I noticed yesterday but only tracked down today: it seems like the beagle file indexing code is able to screw up git in subtle ways. I do not know exactly what happens, but the symptoms are random (and quite hard-to-trigger) dirty index contents where git believes that some set of files are not clean in the index. I *suspect* that beagle is playing games with the file access times, causing ...
Nov 13, 4:56 pm 2007
Mike Hommey
Re: Strange "beagle" interaction..
(...) IIRC, beagle stores a bunch of useful information for itself in extended attributes on indexed files. It is likely that it's that that is tampering with the file stats. Mike -
Nov 13, 5:55 pm 2007
J. Bruce Fields
Re: Strange "beagle" interaction..
Last I ran across this, I believe I found it was adding extended attributes to the file. I think it's something like getfattr -d to show all the extended attributes set on the file. Does that show anything? Yeah, I just turned off beagle. It looked to me like it was doing something wrongheaded. -
Nov 13, 5:03 pm 2007
Linus Torvalds
Re: Strange "beagle" interaction..
Yeah, I just straced it and found the same thing. It's saving fingerprints Gaah. The problem is, setting xattrs does actually change ctime. Which means that if we want to make git play nice with beagle, I guess we have to just remove the comparison of ctime. Oh, well. Git doesn't *require* it, but I like the notion of checking the inode really really carefully. But it looks like it may not be an option, because of file indexers hiding stuff behind our backs. Or we could just tell peo...
Nov 13, 5:30 pm 2007
Jon Smirl
Re: Strange "beagle" interaction..
Things like Beagle need a guaranteed log of global inotify events. That would let them efficiently find changes made since the last time they updated their index. Right now every time Beagle starts it hasn't got a clue what has changed in the file system since it was last run. This forces Beagle to rescan the entire filesystem every time it is started. The xattrs are used as cache to reduce this load somewhat. A better solution would be for the kernel to log inotify events to disk in a manner ...
Nov 13, 5:44 pm 2007
J. Bruce Fields
Re: Strange "beagle" interaction..
Wouldn't a simple change-attribute get you most of the way there? All you need is a number that's guaranteed to increase any time a file is updated. Lacking that, git's current approach (snapshot all the stat data, then look closer at any files that appear to have been touched within a second of the stat) seems pretty sensible. -
Nov 13, 5:50 pm 2007
Jon Smirl
Re: Strange "beagle" interaction..
You still need to look at every file in the file system. People can have many millions of files in their file systems (I have two million in mine and that's small). The inotify log is the most efficient solution. I've turned Beagle off simply because it beats on my disk for an hour -- Jon Smirl jonsmirl@gmail.com -
Nov 13, 5:58 pm 2007
David Brown
Re: Strange "beagle" interaction..
Perhaps something similar to FsEvents on OSX which is a daemon that interfaces with the OS to record this very information. It only works across clean reboots, but it does work there. Do a bad shutdown, and your next backup or index take a long time to go scan everything. It would also be wonderful to have this for backups as well. Dave -
Nov 13, 5:49 pm 2007
J. Bruce Fields
Re: Strange "beagle" interaction..
By the way, on Debian or Ubuntu, at least, that requires an "apt-get Just looking at the attribute names and taking a wild guess, it looked to me like beagle was computing a checksum of each file's data and comparing it to a checksum previously stored in an xattr, and using that to decide whether to reindex the file data. With the result that to check whether anything's changed when it starts up again it has to read through the entire filesystem's data. Maybe I'm wrong--I hope so. I'd love t...
Nov 13, 5:21 pm 2007
Todd A. Jacobs
Integrating with hooks
I've created some bash functions which handle tagging some files with revision information, but even after reading the git manual I'm not really sure how to integrate them so that they remove revision expansion before each check-in (to avoid cluttering the repository with keyword substitutions), and add them back (with the current commit info) after each commit. These are the functions: # Show some kind of useful revision string, like the RCS $Id$ string. I # think commit hash, filename,...
Nov 13, 1:37 pm 2007
Johannes Sixt
[PATCH 10/11] Introduce git_etc_gitconfig() that encapsulate...
In a subsequent patch the path to the system-wide config file will be computed. This is a preparation for that change. It turns all accesses of ETC_GITCONFIG into function calls. There is no change in behavior. As a consequence, config.c is the only file that needs the definition of ETC_GITCONFIG. Hence, -DETC_GITCONFIG is removed from the CFLAGS and a special build rule for config.c is introduced. As a side-effect, changing the defintion of ETC_GITCONFIG (e.g. in config.mak) does not trigger a com...
Nov 13, 4:05 pm 2007
Johannes Schindelin
Re: [PATCH 10/11] Introduce git_etc_gitconfig() that encapsu...
Hi, Wouldn't this logically belong into environment.c? Ciao, Dscho -
Nov 13, 5:22 pm 2007
Johannes Sixt
Re: [PATCH 10/11] Introduce git_etc_gitconfig() that encapsu...
I don't think so. environment.c is about things that concern the repository. This is really only about configuration. -- Hannes -
Nov 13, 5:35 pm 2007
Johannes Schindelin
Re: [PATCH 10/11] Introduce git_etc_gitconfig() that encapsu...
Hi, I thought that environment.c is about getting certain settings from the environment. In that sense, git_etc_gitconfig() would live a happier life there. But I do not care all that deeply. I know _I_ would have looked for it in environment.c first, but then, there's always good ole' git-grep. Ciao, Dscho -
Nov 13, 5:43 pm 2007
Johannes Sixt
[PATCH 11/11] Allow ETC_GITCONFIG to be a relative path.
If ETC_GITCONFIG is not an absolute path, interpret it relative to --exec-dir. This makes the installed binaries relocatable because the prefix is not compiled-in. Signed-off-by: Johannes Sixt <johannes.sixt@telecom.at> --- config.c | 13 ++++++++++++- 1 files changed, 12 insertions(+), 1 deletions(-) diff --git a/config.c b/config.c index dd7e9ad..9f014bb 100644 --- a/config.c +++ b/config.c @@ -6,6 +6,7 @@ * */ #include "cache.h" +#include "exec_cmd.h" #define MAXNAME...
Nov 13, 4:05 pm 2007
Johannes Sixt
[PATCH 07/11] builtin run_command: do not exit with -1.
There are shells that do not correctly detect an exit code of -1 as a failure. We simply truncate the status code to the lower 8 bits. Signed-off-by: Johannes Sixt <johannes.sixt@telecom.at> --- git.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/git.c b/git.c index 4a250f7..37d99d6 100644 --- a/git.c +++ b/git.c @@ -256,7 +256,7 @@ static int run_command(struct cmd_struct *p, int argc, const char **argv) status = p->fn(argc, argv, prefix); if (sta...
Nov 13, 4:05 pm 2007
Johannes Sixt
[PATCH 08/11] Close files opened by lock_file() before unlin...
From: Johannes Schindelin <Johannes.Schindelin@gmx.de> This is needed on Windows since open files cannot be unlinked. Signed-off-by: Johannes Sixt <johannes.sixt@telecom.at> --- This was authored by Dscho, but carries only my sign-off. Is this ok? cache.h | 1 + lockfile.c | 17 ++++++++++------- 2 files changed, 11 insertions(+), 7 deletions(-) diff --git a/cache.h b/cache.h index f0a25c7..28870c5 100644 --- a/cache.h +++ b/cache.h @@ -284,6 +284,7 @@ extern in...
Nov 13, 4:05 pm 2007
Johannes Schindelin
Re: [PATCH 08/11] Close files opened by lock_file() before u...
Hi, Signed-off-by: Johannes Schindelin <Johannes.Schindelin@gmx.de> BTW: Hannes, many thanks for your efforts. Much appreciated. Ciao, Dscho -
Nov 13, 5:05 pm 2007
Johannes Sixt
[PATCH 09/11] Allow a relative builtin template directory.
In order to make git relocatable (i.e. not have the prefix compiled-in) the template directory must depend on the location where this git instance is found, which is GIT_EXEC_DIR. The exec path is prepended only if the compiled-in default template directory is to be used and that is relative. Any relative directories that are specified via environment variable or the --exec-dir switch are taken as is. Signed-off-by: Johannes Sixt <johannes.sixt@telecom.at> --- builtin-init-db.c | 16 ++...
Nov 13, 4:05 pm 2007
Johannes Sixt
[PATCH 0/11] Miscellaneous MinGW port fallout
This is a series of smallish, unrelated changes that were necessary for the MinGW port. [PATCH 01/11] t5300-pack-object.sh: Split the big verify-pack test into smaller parts. [PATCH 02/11] t7501-commit.sh: Not all seds understand option -i [PATCH 03/11] t5302-pack-index: Skip tests of 64-bit offsets if necessary. [PATCH 04/11] Skip t3902-quoted.sh if the file system does not support funny names. Some changes to the test suite. [PATCH 05/11] Use is_absolute_path() in sha1_file.c. [PATCH 06/...
Nov 13, 4:04 pm 2007
Johannes Sixt
Re: [PATCH 0/11] Miscellaneous MinGW port fallout
... and so allow that the compiled binaries are installed in any directory that the user chooses. -- Hannes -
Nov 13, 4:10 pm 2007
Junio C Hamano
Re: [PATCH 0/11] Miscellaneous MinGW port fallout
If you can do that without breaking the tests (specifically, the test script should pick up the version of git you just built, not from /usr/bin nor /usr/local/stow/git/bin) that would be great. -
Nov 13, 5:10 pm 2007
Johannes Sixt
Re: [PATCH 0/11] Miscellaneous MinGW port fallout
Sorry, I don't understand your statement. Do you see any tests breaking? These changes are only about where git-init looks up the templates and where the system configuration is looked for. They change git's behavior only if config.mak contains relative paths like this: sysconfdir = ../etc template_dir = ../share/git-core It has nothing to do with where the test suite looks for the executables. -- Hannes -
Nov 13, 5:32 pm 2007
Johannes Schindelin
Re: [PATCH 0/11] Miscellaneous MinGW port fallout
Hi, I guess what Junio is getting at: if your changes could lead to our not needing to hard code defaults, that would be awesome. For example, a very unhappy camper reported recently that installing git with a different prefix triggers a complete rebuild. Ciao, Dscho -
Nov 13, 5:46 pm 2007
Junio C Hamano
Re: [PATCH 0/11] Miscellaneous MinGW port fallout
Yes. Another thing I said was that it would be grave regression for testability if the change leads git to look at somewhere else other than it was told to look via GIT_EXEC_PATH environment, -
Nov 13, 6:22 pm 2007
Johannes Sixt
Re: [PATCH 0/11] Miscellaneous MinGW port fallout
[PATCH 10/11] is one step into this direction. -- Hannes -
Nov 13, 5:54 pm 2007
Johannes Sixt
[PATCH 01/11] t5300-pack-object.sh: Split the big verify-pac...
This makes it easier to spot which of the tests failed. Signed-off-by: Johannes Sixt <johannes.sixt@telecom.at> --- t/t5300-pack-object.sh | 34 ++++++++++++++++++---------------- 1 files changed, 18 insertions(+), 16 deletions(-) diff --git a/t/t5300-pack-object.sh b/t/t5300-pack-object.sh index ba7579c..f1106e6 100755 --- a/t/t5300-pack-object.sh +++ b/t/t5300-pack-object.sh @@ -187,49 +187,51 @@ test_expect_success \ test-3-${packname_3}.idx' test_expect_success \ - 'c...
Nov 13, 4:04 pm 2007
Johannes Sixt
[PATCH 02/11] t7501-commit.sh: Not all seds understand optio...
Use mv instead. Signed-off-by: Johannes Sixt <johannes.sixt@telecom.at> --- t/t7501-commit.sh | 6 ++++-- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/t/t7501-commit.sh b/t/t7501-commit.sh index 4dc35bd..38db2f2 100644 --- a/t/t7501-commit.sh +++ b/t/t7501-commit.sh @@ -69,7 +69,8 @@ test_expect_success \ cat >editor <<\EOF #!/bin/sh -sed -i -e "s/a file/an amend commit/g" $1 +sed -e "s/a file/an amend commit/g" < $1 > $1- +mv $1- $1 EOF c...
Nov 13, 4:04 pm 2007
Johannes Sixt
[PATCH 03/11] t5302-pack-index: Skip tests of 64-bit offsets...
There are platforms where off_t is not 64 bits wide. In this case many tests are doomed to fail. Let's skip them. Signed-off-by: Johannes Sixt <johannes.sixt@telecom.at> --- t/t5302-pack-index.sh | 25 +++++++++++++++++++++++-- 1 files changed, 23 insertions(+), 2 deletions(-) diff --git a/t/t5302-pack-index.sh b/t/t5302-pack-index.sh index 4f58c4c..d93abc4 100755 --- a/t/t5302-pack-index.sh +++ b/t/t5302-pack-index.sh @@ -61,17 +61,33 @@ test_expect_success \ test_expect_success...
Nov 13, 4:04 pm 2007
Johannes Sixt
[PATCH 04/11] Skip t3902-quoted.sh if the file system does n...
Signed-off-by: Johannes Sixt <johannes.sixt@telecom.at> --- t/t3902-quoted.sh | 7 +++++++ 1 files changed, 7 insertions(+), 0 deletions(-) diff --git a/t/t3902-quoted.sh b/t/t3902-quoted.sh index 245fb3b..73da45f 100755 --- a/t/t3902-quoted.sh +++ b/t/t3902-quoted.sh @@ -20,6 +20,13 @@ LF=' ' DQ='"' +echo foo > "Name and an${HT}HT" +test -f "Name and an${HT}HT" || { + # since FAT/NTFS does not allow tabs in filenames, skip this test + say 'Your filesystem does not allow ta...
Nov 13, 4:04 pm 2007
Johannes Sixt
[PATCH 05/11] Use is_absolute_path() in sha1_file.c.
There are some places that test for an absolute path. Use the helper function to ease porting. Signed-off-by: Johannes Sixt <johannes.sixt@telecom.at> --- sha1_file.c | 8 ++++---- 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/sha1_file.c b/sha1_file.c index f007874..560c0e0 100644 --- a/sha1_file.c +++ b/sha1_file.c @@ -86,7 +86,7 @@ int safe_create_leading_directories(char *path) char *pos = path; struct stat st; - if (*pos == '/') + if (is_absolute_path(...
Nov 13, 4:05 pm 2007
Johannes Schindelin
Re: [PATCH 05/11] Use is_absolute_path() in sha1_file.c.
Hi, Is this enough? On Windows, certain "absolute" paths would need "pos += 3", no? Ciao, Dscho -
Nov 13, 5:39 pm 2007
Johannes Sixt
Re: [PATCH 05/11] Use is_absolute_path() in sha1_file.c.
True, but this series is not yet about the MinGW port itself. It will be changed eventually, but not at this time. -- Hannes -
Nov 13, 5:43 pm 2007
Johannes Schindelin
Re: [PATCH 05/11] Use is_absolute_path() in sha1_file.c.
Hi, Okay, fair enough. Ciao, Dscho -
Nov 13, 5:48 pm 2007
Johannes Sixt
[PATCH 06/11] Move #include <sys/select.h> and <sys...
... since all system headers are pulled in via git-compat-util.h Signed-off-by: Johannes Sixt <johannes.sixt@telecom.at> --- git-compat-util.h | 2 ++ help.c | 1 - pager.c | 2 -- 3 files changed, 2 insertions(+), 3 deletions(-) diff --git a/git-compat-util.h b/git-compat-util.h index 7b29d1b..6a8b592 100644 --- a/git-compat-util.h +++ b/git-compat-util.h @@ -52,6 +52,8 @@ #include <fnmatch.h> #include <sys/poll.h> #include <sys/socke...
Nov 13, 4:05 pm 2007
Sergei Organov
[PATCH] git-remote.txt: fix typo
Signed-off-by: Sergei Organov <osv@javad.com> --- Documentation/git-remote.txt | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/Documentation/git-remote.txt b/Documentation/git-remote.txt index 027ba11..0da8704 100644 --- a/Documentation/git-remote.txt +++ b/Documentation/git-remote.txt @@ -79,7 +79,7 @@ caution. Fetch updates for a named set of remotes in the repository as defined by remotes.<group>. If a named group is not specified on the command line...
Nov 13, 2:17 pm 2007
Sergei Organov
[PATCH] user-manual.txt: fix a few mistakes
Signed-off-by: Sergei Organov <osv@javad.com> --- Documentation/user-manual.txt | 8 ++++---- 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/Documentation/user-manual.txt b/Documentation/user-manual.txt index d99adc6..a169ef0 100644 --- a/Documentation/user-manual.txt +++ b/Documentation/user-manual.txt @@ -475,7 +475,7 @@ Bisecting: 3537 revisions left to test after this If you run "git branch" at this point, you'll see that git has temporarily moved you to a new br...
Nov 13, 2:19 pm 2007
Jakub Narebski
Re: Cloning from kernel.org, then switching to another repo
objects/info/git-alternates, similar to objects/info/http-alternates? -- Jakub Narebski Warsaw, Poland ShadeHawk on #git -
Nov 13, 3:33 pm 2007
Kristis Makris
Git and Scmbug integration
Hello, I've been working on adding Git support in Scmbug, a system that integrates software configuration management with bug-tracking. I've run across what seem to be limitations in the Git hooks mechanism (at least in version 1.5.1.3). In particular: (1) http://bugzilla.mkgnu.net/show_bug.cgi?id=3D991 There's no hook that will trigger when a tag is applied. (2) http://bugzilla.mkgnu.net/show_bug.cgi?id=3D992 Git does not offer metadata describing a change on the commit trigger ...
Nov 13, 3:06 pm 2007
Alex Riesen Nov 13, 7:47 pm 2007
previous daytodaynext day
November 12, 2007November 13, 2007November 14, 2007