Jeff King p...@peff.net writes:
I was thinking that you would be self-contained inside the
contrib/mw-to-git directory, and therefore you would have to teach your
code how to install the Git module, and you could not longer just cp
git-remote-mediawiki into the right place to install it.
Junio C Hamano gits...@pobox.com writes:
I'll queue this patch after tweaking the test part like this.
I agree your version is better, thanks.
Jorge: this means if you have to edit the patch further, you'll have to
start with the version in Junio's pu. But hopefully you won't have to.
--
On Wed, Jun 12, 2013 at 2:32 AM, Felipe Contreras
felipe.contre...@gmail.com wrote:
On Tue, Jun 11, 2013 at 7:13 PM, Johan Herland jo...@herland.net wrote:
This is a pure code movement of the machinery for copying notes to
rewritten objects. This code was located in builtin/notes.c for
On Tue, Jun 11, 2013 at 04:26:20PM +0200, Michael Haggerty wrote:
Please note that if there is some bizarre filesystem somewhere for
which, for a single, static file
lstat() reports S_ISLNK and readlink() fails with ENOENT or EINVAL
[...]
then the inner loop would never terminate.
Introduce advice.rmHints to choose whether to display advice or not
when git rm fails. Defaults to true, in order to preserve current behavior.
As an example, the message:
error: 'foo.txt' has changes staged in the index
(use --cached to keep the file, or -f to force removal)
When 'git rm' fails, it now displays a single message
with the list of files involved, instead of displaying
a list of messages with one file each.
As an example, the old message:
error: 'foo.txt' has changes staged in the index
(use --cached to keep the file, or -f to force
Hi Yann,
Am 2013-06-11 19:06, schrieb Yann Droneaud:
I'm trying to setup a workflow to track vendor releases (upstream).
Each new release are provided as an archive of source code, data,
documentation, etc.
For each vendor releases, fixes need to be applied before making them
available to
On Wed, Jun 12, 2013 at 08:55:12AM +0200, Matthieu Moy wrote:
But I think we have already crossed that bridge somewhat with Git.pm.
And if you add your module as perl/Git/MediaWiki.pm and use the existing
perl build system, then it is not any extra effort from the build
system.
I'm not
After a few hours, here's a quick summary of your opinions:
-'git status' should be the command to display the information instead
of a --status flag
-the SHA1 of the patch being applied currently is a very important
information
-displaying how we got to this state would be nice
I had in
Le 12/06/2013 12:17, Mathieu Liénard--Mayor a écrit :
Now, I'm not sure if we should always display the list of commits
already applied and those left to apply. What I mean is that maybe it
would be better to make status require a flag to display the two lists.
Something like (not sure about
Le 2013-06-12 13:12, Célestin Matte a écrit :
Le 12/06/2013 12:17, Mathieu Liénard--Mayor a écrit :
Now, I'm not sure if we should always display the list of commits
already applied and those left to apply. What I mean is that maybe
it
would be better to make status require a flag to display
John Keeping wrote:
On Wed, Jun 12, 2013 at 12:16:28AM +0530, Ramkumar Ramachandra wrote:
John Keeping wrote:
Ugh, why this roundabout-passive-past tone? Use imperative tone
like this:
...
vs.
We normally use the imperative in commit messages, perhaps like
On Tue, Jun 11, 2013 at 11:48:21PM +0200, Michael Haggerty wrote:
Split repack_without_ref() into multiple passes:
* collect the list of refnames that should be deleted from packed_refs
* delete those refnames from the cache
* write the remainder to the packed-refs file
The purpose
On 06/12/2013 01:38 PM, Jeff King wrote:
On Tue, Jun 11, 2013 at 11:48:21PM +0200, Michael Haggerty wrote:
Split repack_without_ref() into multiple passes:
* collect the list of refnames that should be deleted from packed_refs
* delete those refnames from the cache
* write the remainder
On Tue, Jun 11, 2013 at 07:10:11PM +0530, Ramkumar Ramachandra wrote:
Presumably, Felipe is the fire hazard that we are talking about, and
nobody else is to blame. He must be removed to prevent future
fires. This is the perception of the regulars, correct?
Then why haven't you removed
On Tue, Jun 11, 2013 at 11:48:24PM +0200, Michael Haggerty wrote:
The API docs are not clear about whether it is kosher to read
lock_file::fd directly. It is only done in one file outside of
lockfile.c. So this patch stores the fd of the lockfile separately in
struct packed_ref_cache, even
Jeff King wrote:
And I think that is where the benevolent dictator role comes in. They
weigh not just the points made in the discussion (or a summary of it),
but also use their judgement on who is making comments (how many people,
the utility of their past comments) and other factors (other
On Tue, Jun 11, 2013 at 06:19:23PM -0500, Felipe Contreras wrote:
Fair? Fairness requires to judge each action without biases, nor
double standards. In the case of an open source community it requires
you to listen to the arguments before dismissing them, and consider
the patches before
On Wed, Jun 12, 2013 at 1:23 PM, Mathieu Liénard--Mayor
mathieu.lienard--ma...@ensimag.fr wrote:
Le 2013-06-12 13:12, Célestin Matte a écrit :
Le 12/06/2013 12:17, Mathieu Liénard--Mayor a écrit :
Now, I'm not sure if we should always display the list of commits
already applied and those
Theodore Ts'o wrote:
But if people who *are* senior developers in the git community decide,
on their own, that someone isn't worth listening to, there's the
punishment has been inflicted, and this happens without banning
someone from posting or removing them from the mailing list.
Yes, I have
On Tue, Jun 11, 2013 at 11:48:32PM +0200, Michael Haggerty wrote:
Now that we keep track of the packed-refs file metadata, we can detect
when the packed-refs file has been modified since we last read it, and
we do so automatically every time that get_packed_ref_cache() is
called. So there is
Mathieu Liénard--Mayor mathieu.lienard--ma...@ensimag.fr writes:
$ git status
# HEAD detached from ecb9f3e
# You are currently editing a832578... my_commit_message [3/5] while
rebasing.
Showing the commit message here is too much IMHO. With a typical
50-characters message, it already gives
Le 2013-06-12 14:44, shawn wilson a écrit :
Either leave it or just show the next, last, and current commit. Not
a whole --continue, --amend, etc stuff. The first time I had to
rebase
(about a month ago) it took me a minute to Google and figure the rest
out.
Well, the current output looks
On Tue, Jun 11, 2013 at 11:48:20PM +0200, Michael Haggerty wrote:
*This patch series must be built on top of mh/reflife.*
Applying on top of what Junio has in mh/reflife seems to create
conflicts at the first patch. I didn't look into it, though, but just
read the patches and looked at the
On Mon, Jun 10, 2013 at 11:43 PM, Junio C Hamano gits...@pobox.com wrote:
Antoine Pelisse apeli...@gmail.com writes:
On Sun, Jun 9, 2013 at 10:07 PM, Junio C Hamano gits...@pobox.com wrote:
When any ignore blank option is used, there will be lines that
actually has changes (hence should be
Junio C Hamano gitster at pobox.com writes:
* rr/remove-contrib-some (2013-06-02) 1 commit
(merged to 'next' on 2013-06-05 at fc15705)
+ contrib: remove continuous/ and patches/
Remove stale contrib/ material.
Will merge to 'master'.
What about contrib/blameview by Aneesh Kumar K.V
On Wed, Jun 12, 2013 at 6:56 AM, Theodore Ts'o ty...@mit.edu wrote:
On Tue, Jun 11, 2013 at 07:10:11PM +0530, Ramkumar Ramachandra wrote:
Presumably, Felipe is the fire hazard that we are talking about, and
nobody else is to blame. He must be removed to prevent future
fires. This is the
Charles McGarvey chazmcgarvey at brokenzipper.com writes:
It is convenient for the user to be able to customize the path to perl if they
do not want to use the system perl. This may be the case, for example, if the
user wants to use the plackup httpd but its extra dependencies are not
On Wed, Jun 12, 2013 at 7:27 AM, Theodore Ts'o ty...@mit.edu wrote:
On Tue, Jun 11, 2013 at 06:19:23PM -0500, Felipe Contreras wrote:
Fair? Fairness requires to judge each action without biases, nor
double standards. In the case of an open source community it requires
you to listen to the
Junio C Hamano gitster at pobox.com writes:
Chico Sokol chico.sokol at gmail.com writes:
Is there any official documentation of tree objets format? Are tree
objects encoded specially in some way? How can I parse the inflated
contents of a tree object?
We're suspecting that there is
Philip Oakley philipoakley at iee.org writes:
From: Michael Haggerty mhagger at alum.mit.edu
Sent: Tuesday, June 11, 2013 7:52 PM
As my mother would say, politeness costs nothing
Does your mother program C? We could use her around here
I think she programmed in Smalltalk and
Not many changes since v3. This version is only for fixing last typos and
applying Junio's recommandations for [21/31], which I forgot in v4.
At this point, the only patch worth reviewing is [21/31].
Changes with v4 :
- fixed typos
- Change in [21/31]: all the processing is done within the
Signed-off-by: Célestin Matte celestin.ma...@ensimag.fr
Signed-off-by: Matthieu Moy matthieu@grenoble-inp.fr
---
contrib/mw-to-git/git-remote-mediawiki.perl | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/contrib/mw-to-git/git-remote-mediawiki.perl
Explicitly assign local variable $/ as undef and make a proper
one-instruction-by-line indentation
Signed-off-by: Célestin Matte celestin.ma...@ensimag.fr
Signed-off-by: Matthieu Moy matthieu@grenoble-inp.fr
---
contrib/mw-to-git/git-remote-mediawiki.perl | 5 -
1 file changed, 4
Perl's split function takes a regex pattern argument. You can also
feed it an expression, which is then compiled into a regex at runtime.
It therefore works to pass your pattern via single quotes, but it is
much less obvious to a reader that the argument is meant to be a
regex, not a static
%basetimestamps declaration was lost in the middle of subroutines
Signed-off-by: Célestin Matte celestin.ma...@ensimag.fr
Signed-off-by: Matthieu Moy matthieu@grenoble-inp.fr
---
contrib/mw-to-git/git-remote-mediawiki.perl | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff
Signed-off-by: Célestin Matte celestin.ma...@ensimag.fr
Signed-off-by: Matthieu Moy matthieu@grenoble-inp.fr
---
contrib/mw-to-git/git-remote-mediawiki.perl | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/contrib/mw-to-git/git-remote-mediawiki.perl
- strings which don't need interpolation are single-quoted for more clarity and
slight gain of performance
- interpolation is preferred over concatenation in many cases, for more clarity
- variables are always used with the ${} operator inside strings
- strings including double-quotes are written
Follow Subroutines::RequireFinalReturn
Signed-off-by: Célestin Matte celestin.ma...@ensimag.fr
Signed-off-by: Matthieu Moy matthieu@grenoble-inp.fr
---
contrib/mw-to-git/git-remote-mediawiki.perl | 18 ++
1 file changed, 18 insertions(+)
diff --git
Follow perlcritic's InputOutput::RequireEncodingWithUTF8Layer policy
Signed-off-by: Célestin Matte celestin.ma...@ensimag.fr
Signed-off-by: Matthieu Moy matthieu@grenoble-inp.fr
---
contrib/mw-to-git/git-remote-mediawiki.perl | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff
A split ' ' is turned into a split / /, which changes its behaviour: the
old method matched a run of whitespaces (/\s*/), while the new one will match a
single space, which is what we want here. Indeed, in other contexts,
changing split(' ') to split(/ /) could potentially be a regression,
Signed-off-by: Célestin Matte celestin.ma...@ensimag.fr
Signed-off-by: Matthieu Moy matthieu@grenoble-inp.fr
---
contrib/mw-to-git/git-remote-mediawiki.perl | 56 -
1 file changed, 32 insertions(+), 24 deletions(-)
diff --git
Signed-off-by: Célestin Matte celestin.ma...@ensimag.fr
Signed-off-by: Matthieu Moy matthieu@grenoble-inp.fr
---
contrib/mw-to-git/git-remote-mediawiki.perl | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/contrib/mw-to-git/git-remote-mediawiki.perl
Subroutines' parameters should be assigned to variable before doing anything
else
Besides, existing instruction affected a variable inside a if, which break
Git's coding style
Signed-off-by: Célestin Matte celestin.ma...@ensimag.fr
Signed-off-by: Matthieu Moy matthieu@grenoble-inp.fr
---
Non-trivial numeric values (e.g., different from 0, 1 and 2) are placed in
constants at the top of the code to be easily modifiable and to make more sense
Signed-off-by: Célestin Matte celestin.ma...@ensimag.fr
Signed-off-by: Matthieu Moy matthieu@grenoble-inp.fr
---
Signed-off-by: Célestin Matte celestin.ma...@ensimag.fr
Signed-off-by: Matthieu Moy matthieu@grenoble-inp.fr
---
contrib/mw-to-git/git-remote-mediawiki.perl | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/contrib/mw-to-git/git-remote-mediawiki.perl
This follows the following rule:
InputOutput::RequireBracedFileHandleWithPrint (Severity: 1)
The `print' and `printf' functions have a unique syntax that supports an
optional file handle argument. Conway suggests wrapping this argument in
braces to make it visually stand out from the
Signed-off-by: Célestin Matte celestin.ma...@ensimag.fr
Signed-off-by: Matthieu Moy matthieu@grenoble-inp.fr
---
contrib/mw-to-git/git-remote-mediawiki.perl | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/contrib/mw-to-git/git-remote-mediawiki.perl
Option -2 launches perlcritic with level 2. Levels go from 5 (most pertinent)
to 1. Rules of level 1 are mostly a question of style, and are therefore
ignored.
Signed-off-by: Célestin Matte celestin.ma...@ensimag.fr
Signed-off-by: Matthieu Moy matthieu@grenoble-inp.fr
---
Signed-off-by: Célestin Matte celestin.ma...@ensimag.fr
Signed-off-by: Matthieu Moy matthieu@grenoble-inp.fr
---
contrib/mw-to-git/git-remote-mediawiki.perl | 26 +-
1 file changed, 13 insertions(+), 13 deletions(-)
diff --git
In this regexp, ' |\n' is used, whereas its equivalent '[ \n]', which is
clearer, is used elsewhere. Make the style coherent.
Signed-off-by: Célestin Matte celestin.ma...@ensimag.fr
Signed-off-by: Matthieu Moy matthieu@grenoble-inp.fr
---
contrib/mw-to-git/git-remote-mediawiki.perl | 2 +-
1
Signed-off-by: Célestin Matte celestin.ma...@ensimag.fr
Signed-off-by: Matthieu Moy matthieu@grenoble-inp.fr
---
contrib/mw-to-git/git-remote-mediawiki.perl | 1 -
1 file changed, 1 deletion(-)
diff --git a/contrib/mw-to-git/git-remote-mediawiki.perl
Signed-off-by: Célestin Matte celestin.ma...@ensimag.fr
Signed-off-by: Matthieu Moy matthieu@grenoble-inp.fr
---
contrib/mw-to-git/git-remote-mediawiki.perl | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/contrib/mw-to-git/git-remote-mediawiki.perl
m// and // is used randomly. It is better to use the m modifier only when
needed, e.g., when the regexp uses another separator than //.
Signed-off-by: Célestin Matte celestin.ma...@ensimag.fr
Signed-off-by: Matthieu Moy matthieu@grenoble-inp.fr
---
contrib/mw-to-git/git-remote-mediawiki.perl
Signed-off-by: Célestin Matte celestin.ma...@ensimag.fr
Signed-off-by: Matthieu Moy matthieu@grenoble-inp.fr
---
contrib/mw-to-git/git-remote-mediawiki.perl | 1 -
1 file changed, 1 deletion(-)
diff --git a/contrib/mw-to-git/git-remote-mediawiki.perl
@$var structures are re-written in the following way: @{$var}
It makes them more readable.
Signed-off-by: Célestin Matte celestin.ma...@ensimag.fr
Signed-off-by: Matthieu Moy matthieu@grenoble-inp.fr
---
contrib/mw-to-git/git-remote-mediawiki.perl | 8
1 file changed, 4
Empty strings are replaced by an $EMPTY constant.
Signed-off-by: Célestin Matte celestin.ma...@ensimag.fr
Signed-off-by: Matthieu Moy matthieu@grenoble-inp.fr
---
contrib/mw-to-git/git-remote-mediawiki.perl | 18 ++
1 file changed, 10 insertions(+), 8 deletions(-)
diff --git
Such a file allows to configure perlcritic.
Here, it is used to remove many unwanted rules and configure one to
remove unwanted warnings.
Signed-off-by: Célestin Matte celestin.ma...@ensimag.fr
Signed-off-by: Matthieu Moy matthieu@grenoble-inp.fr
---
contrib/mw-to-git/.perlcriticrc | 28
Change '[\n]' to '\n': brackets are useless here.
Signed-off-by: Célestin Matte celestin.ma...@ensimag.fr
Signed-off-by: Matthieu Moy matthieu@grenoble-inp.fr
---
contrib/mw-to-git/git-remote-mediawiki.perl | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
Put first parameter of map inside a block, for better readability.
Follow BuiltinFunctions::RequireBlockMap
Signed-off-by: Célestin Matte celestin.ma...@ensimag.fr
Signed-off-by: Matthieu Moy matthieu@grenoble-inp.fr
---
contrib/mw-to-git/git-remote-mediawiki.perl | 14 --
1 file
Hi,
sorry for being slow. Here's the attempt to change the German translation
from pure German to German+English. Because of the big changes in de.po,
I have splitted one big commit into these three parts and will squash
them after the review.
Thanks for reading and comments.
Ralf
Ralf Thielow
Antoine Pelisse apeli...@gmail.com writes:
Anyway, I'm starting to think that show blank lines changes near
other changes makes sense more and more sense.
Probably.
I'm glad to see how convinced you are ;)
That is not me not convinced.
It is merely I do not have a strong conviction that
Johan Herland jo...@herland.net writes:
When copying notes for a rewritten object, the resulting notes commit
would have the following hardcoded commit message:
Notes added by 'git notes copy'
This is obviously bogus when the notes rewriting is performed by
'git commit --amend'.
On Wed, Jun 12, 2013 at 2:10 AM, Johan Herland jo...@herland.net wrote:
On Wed, Jun 12, 2013 at 2:32 AM, Felipe Contreras
felipe.contre...@gmail.com wrote:
On Tue, Jun 11, 2013 at 7:13 PM, Johan Herland jo...@herland.net wrote:
This is a pure code movement of the machinery for copying notes to
Hi list,
I wonder if Git is multithreaded ? For example, during a commit, does it
process the files one after one, or does it use a set of threads, say
10, to process 10 files in parrallel ?
In the Git_Guide (http://wiki.sourcemage.org/Git_Guide.html), I can read
this :
T/o enable
When staging hunks interactively it is sometimes useful to use an
alternative diff algorithm which splits the changes into hunks in a more
logical manner. This is not possible because the plumbing commands
called by add--interactive ignore the diff.algorithm configuration
option (as they should).
On 06/12/2013 08:00 AM, Jakub Narebski wrote:
Charles McGarvey chazmcgarvey at brokenzipper.com writes:
It is convenient for the user to be able to customize the path to perl if
they
do not want to use the system perl. This may be the case, for example, if
the
user wants to use the
On Wed, Jun 12, 2013 at 8:28 PM, Felipe Contreras
felipe.contre...@gmail.com wrote:
On Wed, Jun 12, 2013 at 2:10 AM, Johan Herland jo...@herland.net wrote:
On Wed, Jun 12, 2013 at 2:32 AM, Felipe Contreras
felipe.contre...@gmail.com wrote:
On Tue, Jun 11, 2013 at 7:13 PM, Johan Herland
On Wed, Jun 12, 2013 at 07:44:10PM +0100, John Keeping wrote:
When staging hunks interactively it is sometimes useful to use an
alternative diff algorithm which splits the changes into hunks in a more
logical manner. This is not possible because the plumbing commands
called by
On Wed, Jun 12, 2013 at 08:28:52PM +0200, Laurent Alebarde wrote:
I wonder if Git is multithreaded ?
A few selected operations are multi-threaded if you compile with
thread support (i.e., do not set NO_PTHREADS when you build).
For example, during a commit, does it process the files one after
Johan Herland wrote:
On Wed, Jun 12, 2013 at 8:28 PM, Felipe Contreras
felipe.contre...@gmail.com wrote:
On Wed, Jun 12, 2013 at 2:10 AM, Johan Herland jo...@herland.net wrote:
On Wed, Jun 12, 2013 at 2:32 AM, Felipe Contreras
felipe.contre...@gmail.com wrote:
On Tue, Jun 11, 2013 at
On Wed, Jun 12, 2013 at 01:56:20PM +, Jakub Narebski wrote:
Junio C Hamano gitster at pobox.com writes:
* rr/remove-contrib-some (2013-06-02) 1 commit
(merged to 'next' on 2013-06-05 at fc15705)
+ contrib: remove continuous/ and patches/
Remove stale contrib/ material.
Johan Herland jo...@herland.net writes:
This is a pure code movement of the machinery for copying notes to
rewritten objects. This code was located in builtin/notes.c for
historical reasons. In order to make it available to builtin/commit.c
it was declared in builtin.h. This was more of an
Michael Haggerty mhag...@alum.mit.edu writes:
I would prefer a community standards document that looks more like this:
...
* Be welcoming to new community participants. Help them get oriented,
and be patient with their questions. Gently introduce them to our
community standards, above all
Johan Herland jo...@herland.net writes:
There is only one right solution. If a useful function is buried in
builtin/*.o as a historical accident (i.e. it started its life as a
helper for that particular command, and nobody else used it from
outside so far) and that makes it impossible to use
Célestin Matte celestin.ma...@ensimag.fr writes:
Not many changes since v3. This version is only for fixing last typos and
applying Junio's recommandations for [21/31], which I forgot in v4.
At this point, the only patch worth reviewing is [21/31].
Thanks. I went through the series again and
Le 12/06/2013 22:02, Matthieu Moy a écrit :
Célestin Matte celestin.ma...@ensimag.fr writes:
Not many changes since v3. This version is only for fixing last typos and
applying Junio's recommandations for [21/31], which I forgot in v4.
At this point, the only patch worth reviewing is [21/31].
From: Jakub Narebski jna...@gmail.com
Sent: Wednesday, June 12, 2013 3:49 PM
Philip Oakley philipoakley at iee.org writes:
From: Michael Haggerty mhagger at alum.mit.edu
Sent: Tuesday, June 11, 2013 7:52 PM
As my mother would say, politeness costs nothing
Does your mother program C? We
On Wed, Jun 12, 2013 at 8:48 PM, Charles McGarvey
chazmcgar...@brokenzipper.com wrote:
On 06/12/2013 08:00 AM, Jakub Narebski wrote:
Charles McGarvey chazmcgarvey at brokenzipper.com writes:
It is convenient for the user to be able to customize the path to perl if
they
do not want to use
Fredrik Gustafsson iv...@iveqy.com writes:
Bugg reported here:
http://thread.gmane.org/gmane.comp.version-control.git/218922/focus=226791
The URL is nice supplemental info as footnote, but please write log
message in a way that a reader can understand without going there.
In this case, it
Fredrik Gustafsson iv...@iveqy.com writes:
This is a work built on
http://thread.gmane.org/gmane.comp.version-control.git/198873/focus=198930
Basically git-submodule.sh needs to use something else than sh to handle
newline in filenames (and therefore needs to use a language that accepts
\0
Which commit is this based on?
git am fails, whether I try from Junio's master, next or pu. My best
try was from da608b124c76f8f7dba8a9d8f0bc8174b9744176. It seems you're
based on a local commit which doesn't exist on Junio's end.
Most commit in-flight are in next, so I guess rebasing the series
Le 12/06/2013 23:12, Matthieu Moy a écrit :
Which commit is this based on?
git am fails, whether I try from Junio's master, next or pu. My best
try was from da608b124c76f8f7dba8a9d8f0bc8174b9744176. It seems you're
based on a local commit which doesn't exist on Junio's end.
Indeed, we
On Wed, Jun 12, 2013 at 4:47 PM, Célestin Matte
celestin.ma...@ensimag.fr wrote:
Le 12/06/2013 22:34, Célestin Matte a écrit :
Célestin Matte celestin.ma...@ensimag.fr writes:
Actually, there still is a problem: the defined(@array) (introduced
with the change in [21/31]) produces a warning.
Am 12.06.2013 23:06, schrieb Junio C Hamano:
Fredrik Gustafsson iv...@iveqy.com writes:
Bugg reported here:
http://thread.gmane.org/gmane.comp.version-control.git/218922/focus=226791
The URL is nice supplemental info as footnote, but please write log
message in a way that a reader can
Mathieu Lienard--Mayor mathieu.lienard--ma...@ensimag.imag.fr
writes:
When 'git rm' fails, it now displays a single message
with the list of files involved, instead of displaying
a list of messages with one file each.
As an example, the old message:
error: 'foo.txt' has changes staged
Antoine Pelisse apeli...@gmail.com writes:
Maybe we can display previous and next commits to provide some
context. Like we do for diff.
For example:
$ git status
# HEAD detached from ecb9f3e
# Already applied 330 patches (displaying next 3):
# b170635... my_commit_message
#
This is somewhat minor but will confuse people who do not know how
to manually recover from. Start am without giving an input file
by mistake:
$ git am ;# no input file
^C
At this point, you are still in am session; completion will tell
you that you are in a middle of am/rebase. A more
Matthieu Moy matthieu@grenoble-inp.fr writes:
Which commit is this based on?
git am fails, whether I try from Junio's master, next or pu. My best
try was from da608b124c76f8f7dba8a9d8f0bc8174b9744176. It seems you're
based on a local commit which doesn't exist on Junio's end.
Most
Jens Lehmann jens.lehm...@web.de writes:
Hmm, I just came around to test that patch, and for me the new
test even succeeds without the changes to module_list(). So I'm
not convinced yet what we are fixing here ;-)
My guess is that you have core.quotepaths set to false.
The original poster
On Wed, Jun 12, 2013 at 03:57:52PM -0700, Junio C Hamano wrote:
Jens Lehmann jens.lehm...@web.de writes:
Hmm, I just came around to test that patch, and for me the new
test even succeeds without the changes to module_list(). So I'm
not convinced yet what we are fixing here ;-)
My guess
On 06/12/2013 03:03 PM, Jakub Narębski wrote:
On Wed, Jun 12, 2013 at 8:48 PM, Charles McGarvey wrote:
On 06/12/2013 08:00 AM, Jakub Narebski wrote:
Is it really necessary? There is always PERL5LIB if one wants to use Perl
modules installed in one's own home directory. If one is using
On Tue, Jun 11, 2013 at 7:04 PM, Fredrik Gustafsson iv...@iveqy.com wrote:
Bugg reported here:
http://thread.gmane.org/gmane.comp.version-control.git/218922/focus=226791
Note that newline (\n) is still not supported and will not be until the
sh-script is replaced by something in an other
On 06/12/2013 10:02 PM, Junio C Hamano wrote:
Michael Haggerty mhag...@alum.mit.edu writes:
I would prefer a community standards document that looks more like this:
...
* Be welcoming to new community participants. Help them get oriented,
and be patient with their questions. Gently
Junio C Hamano wrote:
$ git am ;# no input file
^C
$ git am --abort
Resolve operation not in progress, we are not resuming.
I tried it on git 1.8.3, and this only incidentally seems to half-work (?)
% git am
^C
% git am --abort
cat:
Ramkumar Ramachandra artag...@gmail.com writes:
I think this is a recent breakage after merging the topic to add
rebase --autostash, but I haven't looked into it yet.
Yes, I can confirm that my topic introduced this bug, and I apologize
for not having caught it. I'm looking into the
Michael Haggerty mhag...@alum.mit.edu writes:
On 06/12/2013 10:02 PM, Junio C Hamano wrote:
Coaching new contributors, like mentoring GSoC students, is often
more time consuming than scratching the same itch yourself for any
reviewer, but it is an investment, which hopefully yields dividend
On Thu, Jun 13, 2013 at 12:19 AM, Junio C Hamano gits...@pobox.com wrote:
Antoine Pelisse apeli...@gmail.com writes:
Maybe we can display previous and next commits to provide some
context. Like we do for diff.
For example:
$ git status
# HEAD detached from ecb9f3e
# Already applied 330
98 matches
Mail list logo