On Fri, Jan 11, 2013 at 06:21:11PM +0700, Nguyễn Thái Ngọc Duy wrote:
- for (i = 0; i specs; i++) {
+ for (i = 0; i pathspec-nr; i++) {
if (!seen[i])
- die(_(pathspec '%s' did not match any files),
pathspec[i]);
+
On Sun, Dec 30, 2012 at 7:05 PM, Jeff King p...@peff.net wrote:
So I was thinking about this, which led to some coding, which led to
some benchmarking.
I like your way of thinking! May I suggest you take a new year break
first, then think about reachability bitmaps ;-) 2013 will be an
exciting
On Sat, Dec 29, 2012 at 7:41 AM, Jeff King p...@peff.net wrote:
I wonder if we could do even better, though. For a traversal, we only
need to look at the commit header. We could potentially do a progressive
inflate and stop before getting to the commit message (which is the bulk
of the data,
On Sat, Dec 29, 2012 at 12:07 PM, Jeff King p...@peff.net wrote:
On Sat, Dec 29, 2012 at 11:34:09AM +0700, Nguyen Thai Ngoc Duy wrote:
On Sat, Dec 29, 2012 at 7:41 AM, Jeff King p...@peff.net wrote:
I wonder if we could do even better, though. For a traversal, we only
need to look
On Fri, Dec 28, 2012 at 1:24 PM, Junio C Hamano gits...@pobox.com wrote:
+ while ((t_ch = *text) != '\0'
+(!(flags WM_PATHNAME) || t_ch
!= '/')) {
Why do we look at (flags WM_PATHMAME) and not special here?
On Fri, Dec 28, 2012 at 1:24 PM, Junio C Hamano gits...@pobox.com wrote:
if (*++p == '*') {
const uchar *prev_p = p - 2;
while (*++p == '*') {}
- if ((prev_p == text || *prev_p == '/')
On Mon, Dec 24, 2012 at 1:19 PM, Jeff King p...@peff.net wrote:
On Mon, Dec 24, 2012 at 12:28:45PM +0700, Nguyen Thai Ngoc Duy wrote:
You want to know what commit was I at when I typed `git branch
mybranch`? The problem is git doesn't record this information and
doesn't have the slightest
On Mon, Dec 24, 2012 at 1:27 PM, Junio C Hamano gits...@pobox.com wrote:
Nguyen Thai Ngoc Duy pclo...@gmail.com writes:
On Mon, Dec 24, 2012 at 12:34 PM, Tomas Carnecky
tomas.carne...@gmail.com wrote:
Maybe we should store this information. reflog is a perfect place for
this, I think
On Tue, Dec 25, 2012 at 2:10 AM, Junio C Hamano gits...@pobox.com wrote:
I looked briefly at reflog before writing my previous mail and noticed
that when I create a new branch (usually using git checkout -b branch
ref) it does not record the base commit.
Hmph. Perhaps you are referring to
On Mon, Dec 24, 2012 at 11:09 AM, Seth Robertson in-gitv...@baka.org wrote:
In message 20121224035825.GA17203@zuhnb712, Woody Wu writes:
How can I find out what's the staring reference point (a commit number
or tag name) of a locally created branch? I can use gitk to find out it
On Mon, Dec 24, 2012 at 12:34 PM, Tomas Carnecky
tomas.carne...@gmail.com wrote:
Maybe we should store this information. reflog is a perfect place for
this, I think. If this information is reliably available, git rebase
can be told to rebase my whole branch instead of my choosing the
base
On Thu, Dec 20, 2012 at 1:36 AM, Junio C Hamano gits...@pobox.com wrote:
@@ -627,7 +628,7 @@ enum interesting tree_entry_interesting(const struct
name_entry *entry,
return entry_interesting;
if (item-use_wildcard) {
-
On Thu, Dec 20, 2012 at 8:55 AM, Nguyen Thai Ngoc Duy pclo...@gmail.com wrote:
As long as simpler patterns fnmatch() groks (namely, '?', '*', and
'[class]' wildcards only) are not slowed down by replacing it with
wildmatch(), that is, of course.
I'm concerned about performance vs fnmatch too
On Wed, Dec 19, 2012 at 8:08 PM, Nguyễn Thái Ngọc Duy pclo...@gmail.com wrote:
For those who have not followed, nd/wildmatch brings another
fnmatch-like implementation which can nearly replace fnmatch.
System fnmatch() seems to behave differently in some cases. It's
better to stay away and use
On Thu, Dec 20, 2012 at 3:34 AM, Jeff King p...@peff.net wrote:
Part of me thinks this is just gross, because :(noglob) is the right
solution. But after spending a few hours trying it this morning, there
is a ton of refactoring required to make it work correctly everywhere
(although we could
On Thu, Dec 20, 2012 at 12:24 AM, Junio C Hamano gits...@pobox.com wrote:
When that happens, we should want to retain the same do not bother
to descend into subdirectories that will never match optimization
for a pattern like Doc*tion/**/*.txt. Because of FNM_PATHNAME, we
can tell if a
On Mon, Dec 17, 2012 at 3:40 PM, Junio C Hamano gits...@pobox.com wrote:
Traditionally, %C(color attr) always emitted the ANSI color
sequence; it was up to the scripts that wanted to conditionally
color their output to omit %C(...) specifier when they do not want
colors.
Optionally allow
On Mon, Dec 17, 2012 at 6:33 PM, Dinesh Subramani
dinesh.subram...@gmail.com wrote:
I am using the below command :
git log --stat --decorate=full --since=date
Can you please let me know if the above command will list all the
commits and would not skip any of the commits due to History
On Sun, Dec 16, 2012 at 2:20 PM, Junio C Hamano gits...@pobox.com wrote:
Nguyễn Thái Ngọc Duy pclo...@gmail.com writes:
entry_count is used in update_one() for two purposes:
1. to skip through the number of processed entries in in-memory index
2. to record the number of entries this
On Sat, Dec 15, 2012 at 2:45 PM, Felipe Contreras
felipe.contre...@gmail.com wrote:
Couple of facts first:
a) This code was already merged
b) This code is for a test
c) I'm the only developer so far
Cruft in the codebase is a problem for git *developers* because it makes
the code harder to
On Sun, Dec 16, 2012 at 2:19 AM, Ramsay Jones
ram...@ramsay1.demon.co.uk wrote:
The failing tests make use of a POSIX character class, '[:xdigit:]'
in this case, which some versions of the fnmatch() library function
do not support. In the spirit of commit f1cf7b79 (t3070: disable
unreliable
On Thu, Dec 13, 2012 at 9:04 AM, Junio C Hamano gits...@pobox.com wrote:
@@ -324,7 +325,14 @@ static int update_one(struct cache_tree *it,
if (!sub)
die(cache-tree.c: '%.*s' in '%s' not found,
entlen, path +
On Sat, Dec 15, 2012 at 5:09 AM, Jeff King p...@peff.net wrote:
I always compile git with gcc -Wall -Werror, because it catches a lot
of dubious constructs, and we usually keep the code warning-free.
However, I also typically compile with -O0 because I end up debugging
a fair bit.
Sometimes,
On Thu, Dec 13, 2012 at 8:13 PM, Jeff King p...@peff.net wrote:
If you are using --format=%C(red) or similar placeholders,
they are the odd duck by not respecting the auto-color mode.
But they should, shouldn't they? Just asking. I may do it to when I
revive
On Wed, Dec 12, 2012 at 7:53 AM, Tomas Carnecky
tomas.carne...@gmail.com wrote:
If it doesn't, it would be trivial to add. It's a one-liner. It's been a while
since I used Lua, but it would be something like this:
void L_putenv(lua_State *L) {
putenv(lua_tostring(L, 1));
}
On Mon, Dec 10, 2012 at 1:50 PM, Junio C Hamano gits...@pobox.com wrote:
Nguyễn Thái Ngọc Duy pclo...@gmail.com writes:
diff --git a/cache-tree.c b/cache-tree.c
index 28ed657..989a7ff 100644
--- a/cache-tree.c
+++ b/cache-tree.c
@@ -248,6 +248,7 @@ static int update_one(struct cache_tree
On Mon, Dec 10, 2012 at 1:31 PM, Junio C Hamano gits...@pobox.com wrote:
Nguyen Thai Ngoc Duy pclo...@gmail.com writes:
On Sat, Dec 8, 2012 at 12:59 AM, Junio C Hamano gits...@pobox.com wrote:
* If somebody has a working replacement URL, we could use that
instead, of course. Takers
On Sat, Dec 8, 2012 at 12:59 AM, Junio C Hamano gits...@pobox.com wrote:
* If somebody has a working replacement URL, we could use that
instead, of course. Takers?
A possible alternative could be
https://www.ohloh.net/p/git/contributors/summary
Nice charts!
--
Duy
--
To unsubscribe
On Wed, Dec 5, 2012 at 2:39 AM, Junio C Hamano gits...@pobox.com wrote:
Or a project commit hook?
Surely. It is OK to have cd t make test-lint in your
pre-commit hook.
No, what I meant is a shared pre-commit script that all git devs are
encouraged (or forced) to install so bugs are found
On Fri, Nov 30, 2012 at 7:06 AM, Junio C Hamano gits...@pobox.com wrote:
Nguyen Thai Ngoc Duy pclo...@gmail.com writes:
An alternative might be to add a phoney bit next to used in the
cache_tree structure, mark the cache tree as phoney when we skip an
entry marked as CE_REMOVE or CE_ITA
On Wed, Nov 28, 2012 at 9:27 AM, Jeff King p...@peff.net wrote:
A tree with meta-paths like '.' or '..' does not work well
with git; the index will refuse to load it or check it out
to the filesystem (and even if we did not have that safety,
it would look like we were overwriting an untracked
On Sun, Nov 25, 2012 at 5:44 PM, Michael Haggerty mhag...@alum.mit.edu wrote:
On the contrary, there is *constant* traffic on the mailing list about
incompatibilities between different shell implementations (sh, dash,
bash, etc), not to mention those in other utilities (sed, grep, etc)
that
On Mon, Nov 26, 2012 at 8:45 AM, Brandon Casey draf...@gmail.com wrote:
From: Brandon Casey bca...@nvidia.com
I hate to have the battle of the patches, but I kinda prefer the
append_signoff code in sequencer.c over the code in log-tree.c as a
foundation to build on.
So, this series is
On Wed, Nov 21, 2012 at 7:05 AM, Junio C Hamano gits...@pobox.com wrote:
* nd/unify-appending-of-s-o-b (2012-11-15) 1 commit
- Unify appending signoff in format-patch, commit and sequencer
I am not sure if the logic to refrain from adding a sign-off based
on the existing run of sign-offs
On Tue, Nov 20, 2012 at 4:20 AM, Junio C Hamano gits...@pobox.com wrote:
$ time git rev-list --quiet HEAD -- '*.c'
real0m40.770s
user0m40.290s
sys 0m0.256s
With the patch
$ time ~/w/git/git rev-list --quiet HEAD -- '*.c'
real0m34.288s
user0m33.997s
sys 0m0.205s
On Fri, Nov 16, 2012 at 1:54 PM, Thomas Gay t...@tokyois.com wrote:
If you set receive.unpacklimit to 1 on the receiving end, does it still
crash?
Yes. The crash log looks the same too.
If it still says unpack-objects died of signal 11 then it was not
done the right way. The receiving end
On Wed, Nov 14, 2012 at 2:55 AM, Douglas Mencken dougmenc...@gmail.com wrote:
Could you try re-building git with the
NO_THREAD_SAFE_PREAD build variable set?
Yeah! It works!!!
--- evil/Makefile
+++ good/Makefile
@@ -957,6 +957,7 @@
HAVE_PATHS_H = YesPlease
On Fri, Nov 16, 2012 at 3:42 AM, Brandon Casey draf...@gmail.com wrote:
Interestingly this patch triggers the fault that it fixes.
I was surprised that there was no blank line before my S-o-b
and thought I broke something. It turns out I used unmodified
format-patch and it mistook the
On Fri, Nov 16, 2012 at 12:13 AM, Jan H. Schönherr
schn...@cs.tu-berlin.de wrote:
#define isprint(x) (sane_istest(x, GIT_ALPHA | GIT_DIGIT | GIT_SPACE | \
GIT_PUNCT | GIT_REGEX_SPECIAL | GIT_GLOB_SPECIAL | \
- GIT_PATHSPEC_MAGIC))
+ GIT_PATHSPEC_MAGIC) \
On Fri, Nov 16, 2012 at 12:44 PM, Thomas Gay t...@tokyois.com wrote:
Using Git 1.8 on Mac OS X 10.7.5. I just added a large binary file to
How large exactly?
my repo, and each time I try to push it, Git crashes. I've attached
the crash log to this email and pasted the console output below.
On Mon, Nov 12, 2012 at 4:15 AM, David Lang da...@lang.hm wrote:
Using a web browser requires connectivity at the time you are doing the
review.
Mailing list based reviews can be done at times when you don't have
connectivity.
I am not against email-based reviews but I'd like to point out
On Fri, Nov 9, 2012 at 5:25 PM, John Szakmeister j...@szakmeister.net wrote:
On Fri, Nov 9, 2012 at 4:27 AM, Tomas Carnecky tomas.carne...@gmail.com
wrote:
[snip]
When merging two branches, git only looks at the tips. It doesn't inspect
their histories to see how the files were moved around.
On Thu, Nov 8, 2012 at 3:57 PM, Jonathon Mah j...@me.com wrote:
Nathan,
I find myself performing similar actions to you: using git add -p to stage
hunks, sometimes editing the staged patch; and keeping mental notes of things
I wanted to revert, sometimes changing them in the editor in
On Tue, Nov 06, 2012 at 12:53:27AM -0800, Jonathon Mah wrote:
It appears the index forgot that those files were new. So not only
has the intent-to-add status been lost, but git status shows a file
existing in neither HEAD nor the index as not-new-but-modified.
Tracing through it, I narrowed
On Mon, Nov 5, 2012 at 4:00 AM, Johannes Sixt j...@kdbg.org wrote:
Patterns beginning with a slash are converted to Windows paths before
test-wildmatch gets to see them. Use a different first character.
Or we could prepend the paths with something, which is then cut out by
test-wildmatch. Not
On Tue, Oct 30, 2012 at 4:28 PM, Orgad Shaneh
Any news? This still doesn't work with 1.8.0.
Nope, sorry. It's still in my todo list.
--
Duy
--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at
On Sat, Oct 27, 2012 at 7:33 PM, Junio C Hamano gits...@pobox.com wrote:
Nguyen Thai Ngoc Duy pclo...@gmail.com 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 combined
On Sat, Oct 27, 2012 at 4:26 AM, Aaron Schrab aa...@schrab.com wrote:
I came across this odd question on stackoverflow:
http://stackoverflow.com/q/13092854/1507392
If git diff is run with ... as a separate argument between two commit-ish
arguments causes it to produce strange output.
On Thu, Oct 25, 2012 at 4:45 PM, Jeff King p...@peff.net wrote:
On Tue, Oct 23, 2012 at 09:24:51AM +0700, Nguyen Thai Ngoc Duy wrote:
These functions are called in sequencer.c, which is part of
libgit.a. This makes libgit.a potentially require builtin/merge.c for
external git commands
On Wed, Oct 24, 2012 at 12:21 PM, Uri Moszkowicz u...@4refs.com wrote:
Continuing to work on improving clone times, using git gc
--aggressive has resulted in a large number of tags combining into a
single file but now I have a large number of files in the objects
directory - 131k for a ~2.7GB
On Wed, Oct 24, 2012 at 1:30 AM, Uri Moszkowicz u...@4refs.com wrote:
I have a large repository which I ran git gc --aggressive on that
I'm trying to clone on a local file system. I would expect it to
complete very quickly with hard links but it's taking about 6min to
complete with no checkout
On Sun, Oct 21, 2012 at 7:20 PM, Drew Northup n1xim.em...@gmail.com wrote:
On Tue, Oct 9, 2012 at 1:58 AM, Matthieu Moy
matthieu@grenoble-inp.fr wrote:
David Aguilar dav...@gmail.com writes:
I would advise against the file locking, though. You ain't gonna need
it ;-)
What do you
On Sat, Oct 20, 2012 at 2:50 AM, Junio C Hamano gits...@pobox.com wrote:
Not particularly interested in the cause, but not so strongly
against it to veto it.
I wonder how many people keep old branches like I do, which are
usually far from remotes.
Doesn't it make more sense to use a
On Sat, Oct 20, 2012 at 4:03 AM, Junio C Hamano gits...@pobox.com wrote:
* nd/wildmatch (2012-10-15) 13 commits
Allows pathname patterns in .gitignore and .gitattributes files
with double-asterisks foo/**/bar to match any number of directory
hierarchies.
I suspect that this needs to be
On Thu, Oct 18, 2012 at 12:45 PM, Junio C Hamano gits...@pobox.com wrote:
This replaces the earlier wip with a real thing.
We never advertised the --notes option to format-patch (or
anything related to the pretty format options for that matter)
because the behaviour of these options was
On Fri, Oct 19, 2012 at 6:10 AM, Uri Moszkowicz u...@4refs.com wrote:
I'm testing out the sparse checkout feature of Git on my large (14GB)
repository and am running into a problem. When I add dir1/ to
sparse-checkout and then run git read-tree -mu HEAD I see dir1 as
expected. But when I add
On Wed, Oct 17, 2012 at 5:53 PM, Peter Krefting pe...@softwolves.pp.se wrote:
Hi!
The output of git format-patch and git diff --stat no longer becomes
localized when using 1.8.0.rc2, compared to 1.7.12
It's the result of this discussion [1]. I don't remember exactly the
open issues. But I
On Wed, Oct 17, 2012 at 7:09 PM, Jan H. Schönherr
schn...@cs.tu-berlin.de wrote:
const unsigned char sane_ctype[256] = {
- 0, 0, 0, 0, 0, 0, 0, 0, 0, S, S, 0, 0, S, 0, 0, /* 0.. 15 */
- 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, /* 16.. 31 */
+ X, X, X, X,
Adam, do you have time to continue this series? I can help polish it
for inclusion, but I don't want to step in your way if you are quietly
updating it.
On Tue, Oct 16, 2012 at 5:31 AM, Junio C Hamano gits...@pobox.com wrote:
+
+If `-z` is specified, the output is a series of lines of the form:
On Tue, Oct 16, 2012 at 9:09 PM, Adam Spiers g...@adamspiers.org wrote:
I was *intending* to finish it off soon, but I have been really busy
with work and other commitments recently, which has prevented this. I
don't currently have any unpublished changes which would conflict with
your recent
On Mon, Oct 15, 2012 at 6:05 PM, Matthieu Moy
matthieu@grenoble-inp.fr wrote:
Angelo Borsotti angelo.borso...@gmail.com writes:
the push command checks first if the tips of the branches match those
of the remote references, and if it does uploads the snapshot.
The update does two things:
On Sun, Oct 14, 2012 at 12:04 PM, Junio C Hamano gits...@pobox.com wrote:
Nguyễn Thái Ngọc Duy pclo...@gmail.com writes:
Signed-off-by: Nguyễn Thái Ngọc Duy pclo...@gmail.com
---
The comment-fix seems to be new but otherwise this is unchanged,
right?
Right.--
Duy
--
To unsubscribe from
On Sun, Oct 14, 2012 at 7:59 PM, René Scharfe
rene.scha...@lsrfire.ath.cx wrote:
+const unsigned char sane_ctype2[256] = {
+ CN, CN, CN, CN, CN, CN, CN, CN, CN, CN, CN, CN, CN, CN, CN, CN, /*
0..15 */
+ CN, CN, CN, CN, CN, CN, CN, CN, CN, CN, CN, CN, CN, CN, CN, CN, /*
16..31 */
On Sun, Oct 14, 2012 at 03:59:31PM +0200, René Scharfe wrote:
Am 14.10.2012 15:25, schrieb Nguyen Thai Ngoc Duy:
On Sun, Oct 14, 2012 at 7:59 PM, René Scharfe
rene.scha...@lsrfire.ath.cx wrote:
With that, couldn't you squeeze the other two classes into the existing
sane_type
On Mon, Oct 15, 2012 at 12:36 AM, Junio C Hamano gits...@pobox.com wrote:
With your teach attr.c match the same optimization as dir.c
series, you would need something like this
diff --git i/attr.c w/attr.c
index 6d39406..528e935 100644
--- i/attr.c
+++ w/attr.c
@@ -710,7 +710,7 @@ static
On Sun, Oct 14, 2012 at 12:44 AM, Ramsay Jones
ram...@ramsay1.demon.co.uk wrote:
Signed-off-by: Ramsay Jones ram...@ramsay1.demon.co.uk
---
Hi Junio,
This test in the current pu branch (commit bb0e6bf Merge branch
'aw/rebase-am-failure-detection' into pu, 11-10-2012) overwrites
the
On Sun, Oct 14, 2012 at 12:02 PM, Junio C Hamano gits...@pobox.com wrote:
Nguyễn Thái Ngọc Duy pclo...@gmail.com writes:
Signed-off-by: Nguyễn Thái Ngọc Duy pclo...@gmail.com
---
The description to justify why it is ctype2[] seems to have been
lost. Intended?
Nope. I added the
On Fri, Oct 12, 2012 at 2:22 PM, Johannes Sixt j.s...@viscovery.net wrote:
Am 10/10/2012 12:40, schrieb Nguyễn Thái Ngọc Duy:
fnmatch on glibc-2.12.1 returns no match. glibc-2.15 returns ok.
There are many more cases that fail with the fnmatch() that we ship in
compat/fnmatch. To test this on
On Thu, Oct 11, 2012 at 3:03 AM, Junio C Hamano gits...@pobox.com wrote:
It would save time from both of us if you can check what is queued
on 'pu'. I do not think I touched the code for off-by-one bugs
there, though.
'pu' looks good.
--
Duy
--
To unsubscribe from this list: send the line
On Fri, Oct 12, 2012 at 10:05:06AM -0700, Junio C Hamano wrote:
Nguyen, how about updating the match () shell function in 3070 so
that it not just says not-ok, but indicates what failed (wildmatch
failed, or wildmatch passed but fnmatch failed), at least when the
test is run as ./t3070-*.sh -v
On Fri, Oct 12, 2012 at 12:09:57PM -0700, Junio C Hamano wrote:
This is not entirely your fault, but please don't do that cd ...
The original test had cd bare, made an assumption that step will
never fail (which is mostly correct), and ran everything afterward
in that subdirectory.
Adding
On Thu, Oct 11, 2012 at 2:04 PM, Michael Haggerty mhag...@alum.mit.edu wrote:
Maybe I'm being too much of a purist, but I don't think that git should
retroactively reinterpret history on its own initiative in a way that
might not be correct (e.g., maybe your encoding changed from ASCII to
On Fri, Oct 05, 2012 at 01:03:28PM -0700, Junio C Hamano wrote:
OK. I think I was surprised that some messages were controlled by
advice.* but gave no hints about that and I found that out by other
means. I'll check all the advice messages.
It's about the commands, not the documents. For
On Thu, Oct 11, 2012 at 4:13 AM, Junio C Hamano gits...@pobox.com wrote:
This sounds a lot stronger than let to me. All existing callers
that assumed that buf to be emptied by the function now has to clear
it. quote: stop resetting output buffer of quote_path_relative()
may better describe
On Thu, Oct 11, 2012 at 9:18 PM, Matthieu Moy
matthieu@grenoble-inp.fr wrote:
For example, it makes the output of git status look like:
# On branch master
# Changes to be committed:
# (use git reset HEAD file... to unstage)
# Set advice.statusHints to false to turn off this message
On Wed, Oct 10, 2012 at 6:51 PM, Johannes Sixt j.s...@viscovery.net wrote:
Thanks for working on this issue!
Am 10/10/2012 13:34, schrieb Nguyễn Thái Ngọc Duy:
+ linkgit:gitattributes[5]). Note that .gitattributes are only
+ support for searching files in working directory.
Does
On Wed, Oct 10, 2012 at 7:12 PM, Johannes Sixt j.s...@viscovery.net wrote:
Is there already an established definition which the correct
.gitattributes are?
If I ask to grep the index then to me it should read only the index.
Although other people can counter that they may want different
On Wed, Oct 10, 2012 at 7:43 PM, Johannes Sixt j.s...@viscovery.net wrote:
Am 10/10/2012 14:32, schrieb Nguyen Thai Ngoc Duy:
git-archive has --worktree-attributes to specify where attributes come
from. Sparse checkout can choose to read index version first then
worktree's or the other way
This is the documentation part of
1a9d7e9 (attr.c: read .gitattributes from index as well. - 2007-08-14)
06f33c1 (Read attributes from the index that is being checked out - 2009-03-13)
Signed-off-by: Nguyễn Thái Ngọc Duy pclo...@gmail.com
---
I looked around but did not see anywhere mentioning
On Thu, Oct 11, 2012 at 4:41 AM, Junio C Hamano gits...@pobox.com wrote:
Nguyễn Thái Ngọc Duy pclo...@gmail.com writes:
@@ -690,16 +689,18 @@ static int path_matches(const char *pathname, int
pathlen,
* contain the trailing slash
*/
- if (pathlen baselen ||
+
On Wed, Oct 10, 2012 at 9:21 PM, Johannes Sixt j.s...@viscovery.net wrote:
+ git commit -m new
+ echo Binary file HEAD:t matches expect
+ git grep text HEAD -- t actual
+ test_cmp expect actual
+ git reset HEAD^
+'
And in yet another test, should
git grep
On Tue, Oct 09, 2012 at 04:38:32PM +0700, Nguyen Thai Ngoc Duy wrote:
#5 0x0815e736 in userdiff_find_by_path (path=0x820e7f0
HEAD:Documentation/.gitattributes) at userdiff.c:278
#6 0x081058ca in grep_source_load_driver (gs=0xbfffd978) at grep.c:1504
A bandage patch may look like
On Wed, Oct 10, 2012 at 3:47 AM, Junio C Hamano gits...@pobox.com wrote:
Nguyễn Thái Ngọc Duy pclo...@gmail.com writes:
diff --git a/wildmatch.c b/wildmatch.c
index 7b64a6b..2382873 100644
--- a/wildmatch.c
+++ b/wildmatch.c
@@ -11,8 +11,8 @@
#include stddef.h
#include ctype.h
On Wed, Oct 10, 2012 at 1:59 AM, Junio C Hamano gits...@pobox.com wrote:
Jeff King p...@peff.net writes:
I think we just need to have callers of grep_source_init provide us with
the actual pathname (or NULL, in the case of GREP_SOURCE_BUF). That is
where the information is lost.
Yes. I
On Tue, Oct 9, 2012 at 2:57 PM, Michael Haggerty mhag...@alum.mit.edu wrote:
+ - A trailing /** matches everything inside. For example,
+ abc/** is equivalent to `/abc/`.
It seems odd that you add a leading slash in this example. I assume
that is because of the rule that a pattern
On Wed, Oct 10, 2012 at 12:33 PM, Junio C Hamano gits...@pobox.com wrote:
Nguyen Thai Ngoc Duy pclo...@gmail.com writes:
On Wed, Oct 10, 2012 at 1:59 AM, Junio C Hamano gits...@pobox.com wrote:
Jeff King p...@peff.net writes:
I think we just need to have callers of grep_source_init provide
On Wed, Oct 10, 2012 at 12:31 PM, Junio C Hamano gits...@pobox.com wrote:
Nguyen Thai Ngoc Duy pclo...@gmail.com writes:
Git's ctype does not seem to be complete for wildmatch's use so
ctype.h is required. But that can be easily fixed later on.
Until later on, I cannot even compile
On Tue, Oct 9, 2012 at 6:33 AM, Junio C Hamano gits...@pobox.com wrote:
* Unify pathspec semantics
This has happened and commands that used to take only path prefix
style pathspecs now take globs as well [ND]
I've been thinking about it lately and will probably restart soon with
a
confusing in multiple attribute manipulation:
*.c attr1
*.h -attr2
!foo.[ch] attr1 -attr2
So not supported and forbidden are equally OK. I just want to
raise a point that it has some use before we go for forbidden.
Nguyen Thai Ngoc Duy pclo...@gmail.com writes:
On Sat, Oct 6, 2012 at 12:36 PM, Junio C
On Sat, Oct 6, 2012 at 12:36 PM, Junio C Hamano gits...@pobox.com wrote:
Or the user might think path/ attr1 sets attr1 for all files under
path/ because it does not make sense to attach attributes to a
directory in git.
...
We may not have a need to assign a real attribute to a
On Sat, Oct 6, 2012 at 2:54 PM, Junio C Hamano gits...@pobox.com wrote:
Shouldn't we do the same for quoting fnmatch(3) metacharacters? To
match a path component 'a' followed by an asterisk followed by 'b',
you could then write 'a\*b'. Same for quoting the backslash itself.
I think my patch
On Sat, Oct 6, 2012 at 6:33 PM, Philip Oakley philipoak...@iee.org wrote:
Asciidoc 8.2.6 does not like me writing
Put \# if you need a literal #.. so I go with backslash and
hash instead. `\!` displays fine both in man page and html format.
'!' changed to `!` because it looks clearer in
On Wed, Oct 3, 2012 at 10:44 PM, Alexey Spiridonov
snarkmas...@gmail.com wrote:
Thanks a lot for trying this.
My hashes match. I just re-reproduced it on two flavors of Linux (64
and 32-bit), with two different Git versions (see below). What
platform are you using?
x86, 32 bit. Perhaps it
On Thu, Oct 4, 2012 at 12:45 PM, Junio C Hamano gits...@pobox.com wrote:
Nguyen Thai Ngoc Duy pclo...@gmail.com writes:
On Thu, Oct 4, 2012 at 1:49 AM, Junio C Hamano gits...@pobox.com wrote:
I would recommend against listing any advice.* in the command manual
pages. They are meant to give
On Fri, Oct 5, 2012 at 5:30 PM, Peter Krefting pe...@softwolves.pp.se wrote:
These files are from rsync.git commit
f92f5b166e3019db42bc7fe1aa2f1a9178cd215d, which was the last commit
before rsync turned GPL-3.
However:
diff --git a/test-wildmatch.c b/test-wildmatch.c
[...]
+ * This
On Fri, Oct 5, 2012 at 2:01 PM, Johannes Sixt j.s...@viscovery.net wrote:
Am 10/4/2012 9:39, schrieb Nguyễn Thái Ngọc Duy:
- - If the pattern does not contain a slash '/', git treats it as
- a shell glob pattern and checks for a match against the
- pathname relative to the location of the
On Thu, Oct 04, 2012 at 09:39:02AM -0700, Junio C Hamano wrote:
Assuming that we do want to match x/y with x/**/y, I suspect
that '**' matches anything including a slash would not give us
that semantics. Is it something we can easily fix in the wildmatch
code?
Something like this may suffice.
On Sat, Oct 6, 2012 at 1:48 AM, Junio C Hamano gits...@pobox.com wrote:
Nguyễn Thái Ngọc Duy pclo...@gmail.com writes:
+Unlike `.gitignore`, negative patterns are not supported.
+Patterns that match directories are also not supported.
Is are not supported the right phrasing?
I think it
On Thu, Oct 4, 2012 at 4:34 PM, Michael Haggerty mhag...@alum.mit.edu wrote:
On Thu, Oct 4, 2012 at 4:34 PM, Michael Haggerty mhag...@alum.mit.edu wrote:
Given that there is no obvious interpretation for what a construct like
x**y would mean, and many plausible guesses (most of which sound
On Tue, Oct 2, 2012 at 10:09 PM, Ramkumar Ramachandra
artag...@gmail.com wrote:
David Glasser wrote:
Is the newish push.default documented in the git push manpage
anywhere? I don't see it mentioned (and there are several references
to the default behavior), but maybe I'm missing something. Is
1 - 100 of 209 matches
Mail list logo