git-log(1) Doesn't Describe -L start,end Fully.

2017-05-11 Thread Ralph Corderoy
Hi,

Arch Linux's git 2.12.2-4's git-log(1) says

-L ,:, -L ::
Trace the evolution of the line range given by ","
(or the function name regex ) within the .  You
may not give any pathspec limiters.  This is currently limited
to a walk starting from a single revision, i.e. you may only
give zero or one positive revision arguments.  You can specify
this option more than once.

 and  can take one of these forms:

number
If  or  is a number, it specifies an absolute
line number (lines count from 1).

It doesn't say one or both of start and end may be omitted.  Thus `-L
,:foobar.c' is valid with start defaulting to 1 and end to the number of
lines in foobar.c.  

Nor does it explain if 1,42 covering all of foobar.c as it is now swells
to be 1,314 as the commits are walked to larger versions of foobar.c.

This was noticed because I've started with foo.c and bar.c, and each was
`git mv'd to foobar.c in separate commits that were then each merged.
I'm trying to `git log' HEAD's foobar.c and have it walk past the merges
to show be foo.c and bar.c's history.

`git log HEAD -- foobar.c foo.c bar.c' works, but one has to specify all
the old names.  `git log -L ,:foobar.c' works for a simple test case,
but I don't know if changes outside of foobar.c's current 1,42 lines
would always show up.  And also -L triggers patches and --no-patch
doesn't stop that;  bug?  The -L logic seems particular to `git log' so
I can't have git-rev-list(1) do that work to provide the commits and
then log them separately.

-- 
Cheers, Ralph.
https://plus.google.com/+RalphCorderoy


Re: [darcs-devel] Darcs and git: plan of action

2005-04-20 Thread Ralph Corderoy

Hi Ray,

 Give me a case where assuming it's a replace will do the wrong thing,
 for C code, where it's a variable or function name.

How about two patches.

1.  s/foo/bar/ throughout file because foo() has been decided upon
as the name of a new globally visible forthcoming function but was
already in use as a static function.

2.  Add definition of new foo().

Patch 1 mustn't be a `darcs replace' despite it changing every occurence
of the C token foo into bar.

Cheers,


Ralph.

-
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html