On 5 June 2013 16:45, Felipe Contreras wrote:
> On Tue, Jun 4, 2013 at 7:04 PM, Junio C Hamano wrote:
> That might make sense for the shorter term, but in longer term I see
> Perl as declining in favor of other languages. It's only a matter of
> time before Ruby surpasses Perl in popularity, and
On Thu, Jun 6, 2013 at 2:26 AM, demerphq wrote:
> On 5 June 2013 16:45, Felipe Contreras wrote:
>> On Tue, Jun 4, 2013 at 7:04 PM, Junio C Hamano wrote:
>> That might make sense for the shorter term, but in longer term I see
>> Perl as declining in favor of other languages. It's only a matter of
On 06/06/2013 12:40 AM, Jeff King wrote:
> We cannot create an archive from a blob object, so we would
> not expect anyone to provide one to us. And if they do, we
> will fail anyway just after the reachability check. We can
> therefore optimize our reachability check to ignore blobs
> completely,
On Mac OS X, any application that is started from the Terminal will open
behind all running applications; as a work-around, manually bring ourselves
to the front. (Stolen from gitk, commit 76bf6ff93e.)
We do this as the very first thing, so that any message boxes that might pop
up during the rest
Hey,
What I want to do:
* ignore certain patterns completely
* but keep one important directoy
* but not the patterns from the first step that are located within
that directory.
Concrete example: I'm tracking certain parts of my $HOME including
~/.emacs.d/ -- but I don't want the compiled Emacs
Signed-off-by: Felipe Contreras
---
sequencer.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/sequencer.c b/sequencer.c
index dd9de49..c3a17f8 100644
--- a/sequencer.c
+++ b/sequencer.c
@@ -754,8 +754,8 @@ static void prepare_revs(struct replay_opts *opts)
if (pr
Will be useful for the next commits.
Signed-off-by: Felipe Contreras
---
sequencer.c | 102 +++-
sequencer.h | 1 +
2 files changed, 102 insertions(+), 1 deletion(-)
diff --git a/sequencer.c b/sequencer.c
index d3c7d72..76ff2ff 100644
--
We should free objects before leaving.
Signed-off-by: Felipe Contreras
---
sequencer.c | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/sequencer.c b/sequencer.c
index b4989ba..f7be7d8 100644
--- a/sequencer.c
+++ b/sequencer.c
@@ -628,8 +628,10 @@ static int do_pick_com
Akin to 'am --skip' and 'rebase --skip'.
Signed-off-by: Felipe Contreras
---
Documentation/git-cherry-pick.txt | 1 +
Documentation/git-revert.txt | 1 +
Documentation/sequencer.txt | 3 +++
builtin/revert.c | 6 ++
sequencer.c | 32 +
Pretty much what it says on the tin.
Signed-off-by: Felipe Contreras
---
Documentation/git-cherry-pick.txt | 3 +++
builtin/revert.c| 8
sequencer.c | 6 ++
sequencer.h | 1 +
t/t3508-cherry-pick-many-commits.
Signed-off-by: Felipe Contreras
---
sequencer.c | 33 +
1 file changed, 29 insertions(+), 4 deletions(-)
diff --git a/sequencer.c b/sequencer.c
index 74480d7..6d13e63 100644
--- a/sequencer.c
+++ b/sequencer.c
@@ -14,6 +14,7 @@
#include "merge-recursive.h"
#incl
As we should.
Signed-off-by: Felipe Contreras
---
sequencer.c | 42 +-
1 file changed, 41 insertions(+), 1 deletion(-)
diff --git a/sequencer.c b/sequencer.c
index 76ff2ff..74480d7 100644
--- a/sequencer.c
+++ b/sequencer.c
@@ -127,6 +127,37 @@ static voi
Signed-off-by: Felipe Contreras
---
Documentation/git-cherry-pick.txt | 6 +-
Documentation/git-revert.txt | 6 +-
builtin/revert.c | 1 +
sequencer.c | 9 ++---
sequencer.h | 1 +
5 files changed, 18 insertions(+), 5 d
By using good ol' goto.
Signed-off-by: Felipe Contreras
---
sequencer.c | 16 +---
1 file changed, 9 insertions(+), 7 deletions(-)
diff --git a/sequencer.c b/sequencer.c
index ab6f8a7..b4989ba 100644
--- a/sequencer.c
+++ b/sequencer.c
@@ -474,7 +474,7 @@ static int do_pick_commit(s
Hi,
Sames as v2, plus a few changes from the feedback, and cleanups.
Felipe Contreras (9):
sequencer: remove useless indentation
sequencer: trivial fix
cherry-pick: add --skip-empty option
cherry-pick: store rewritten commits
sequencer: run post-rewrite hook
cherry-pick: add support t
Ramkumar Ramachandra:
Yeah. We generally prefer the long-form equivalents while doing
completions, but these blame options do not have equivalent
long-forms.
Perhaps that is the real bug, then. -M and -C already have long names
for diff (and its friends), perhaps blame should have the same
On Thu, Jun 6, 2013 at 3:59 AM, Felipe Contreras
wrote:
> Signed-off-by: Felipe Contreras
> ---
> sequencer.c | 33 +
> 1 file changed, 29 insertions(+), 4 deletions(-)
>
> diff --git a/sequencer.c b/sequencer.c
> index 74480d7..6d13e63 100644
> --- a/sequencer.c
On 2013-06-06 03:46:59 EDT, Felipe Contreras wrote:
> On Thu, Jun 6, 2013 at 2:26 AM, demerphq wrote:
>> Good thing you are being objective and leaving out the Python 3.0
>> mess, the long legacy of backwards compatibility in the Perl
>> community, the active community behind it, its extensive po
Michael Haggerty writes:
> * at the source-code level, a tool in contrib can take advantage of some
> of the Git build/test infrastructure, though I don't know whether they
> currently do.
They do not do much AFAICT. For example, contrib/subtree/t/Makefile is
essentially copy-pasted from Git's e
On Thu, Jun 6, 2013 at 7:24 AM, Barry Fishman wrote:
>
> On 2013-06-06 03:46:59 EDT, Felipe Contreras wrote:
>> On Thu, Jun 6, 2013 at 2:26 AM, demerphq wrote:
>>> Good thing you are being objective and leaving out the Python 3.0
>>> mess, the long legacy of backwards compatibility in the Perl
>>
On 2013-06-06 09:01:48 EDT, Felipe Contreras wrote:
> On Thu, Jun 6, 2013 at 7:24 AM, Barry Fishman wrote:
>>
>> On 2013-06-06 03:46:59 EDT, Felipe Contreras wrote:
>>> On Thu, Jun 6, 2013 at 2:26 AM, demerphq wrote:
Good thing you are being objective and leaving out the Python 3.0
mes
On Thu, Jun 6, 2013 at 8:46 AM, Barry Fishman wrote:
>
> On 2013-06-06 09:01:48 EDT, Felipe Contreras wrote:
>> Nobody is judging the usefulness of a language, I have plenty of
>> arguments for that, but this is about popularity.
>
> I used "usefulness" in its general vague sense. It is useful t
On 6 June 2013 09:17, Stefan Haller wrote:
> On Mac OS X, any application that is started from the Terminal will open
> behind all running applications; as a work-around, manually bring ourselves
> to the front. (Stolen from gitk, commit 76bf6ff93e.)
>
> We do this as the very first thing, so that
On 2013-06-06 10:09:21 EDT, Felipe Contreras wrote:
> I don't know what you are saying, but it clearly has nothing to do
> with the point.
>
> Perl is declining, and it would be wise to use another language
> instead of it.
You want a simple statement. I don't particulary like Perl, but it has
wo
On Thu, Jun 6, 2013 at 9:41 AM, Barry Fishman wrote:
> On 2013-06-06 10:09:21 EDT, Felipe Contreras wrote:
>> I don't know what you are saying, but it clearly has nothing to do
>> with the point.
>>
>> Perl is declining, and it would be wise to use another language
>> instead of it.
>
> You want a
As one of the people who helps maintain git packages in pkgsrc, my
initial reaction is negative to adding a ruby dependency. There are
several not-entirely-related reasons:
git is a core tool that people use on almost the smallest of boxes,
perhaps even replacing rcs for managing local config fi
On Thu, Jun 6, 2013 at 9:54 AM, Greg Troxel wrote:
>
> As one of the people who helps maintain git packages in pkgsrc, my
> initial reaction is negative to adding a ruby dependency. There are
> several not-entirely-related reasons:
>
> git is a core tool that people use on almost the smallest of
Hi Greg,
On Thu, 6 Jun 2013, Greg Troxel wrote:
> As one of the people who helps maintain git packages in pkgsrc, my
> initial reaction is negative to adding a ruby dependency.
My initial reaction, too. It was hard enough to get Perl included with Git
for Windows (because of that pesky Subversio
Pat Thoyts wrote:
> On 6 June 2013 09:17, Stefan Haller wrote:
> > +## On Mac, bring the current Wish process window to front
> > +
> > +if {[tk windowingsystem] eq "aqua"} {
> > + exec osascript -e [format {
> > + tell application "System Events"
> > +
On Thu, 6 Jun 2013, Felipe Contreras wrote:
In the end my point remains unchanged; Perl is declining, so it would
be wise for the future to use another scripting language instead.
Perl use may or may not be declining (depending on how you measure it), but are
you really willing to take on the
Felipe Contreras writes:
> On Thu, Jun 6, 2013 at 9:54 AM, Greg Troxel wrote:
>>
>> git is a core tool that people use on almost the smallest of boxes,
>> perhaps even replacing rcs for managing local config files. On such
>> machines, even perl may be large, but a second scripting language se
Jeff King writes:
> On Thu, Jun 06, 2013 at 01:41:05AM -0500, Felipe Contreras wrote:
>
>> > Thanks. I wasn't quite clear on how the signal handling worked on
>> > Windows, but from your description, I agree there is not any point in
>> > running the test at all.
>>
>> Shouldn't we clarify that
Jeff King writes:
> On Wed, Jun 05, 2013 at 12:38:23PM -0400, Jeff King wrote:
>
>> 2. Actually do a reachability check. Doing a full object check to
>> allow fetching an arbitrary tree by sha1 is probably prohibitively
>> expensive[2], but we could allow the form "[:]", check
>>
On Thu, Jun 06, 2013 at 10:21:47AM -0700, Junio C Hamano wrote:
> > The particular deficiency is that when a signal is raise()d whose SIG_DFL
> > action will cause process death (SIGTERM in this case), the
> > implementation of raise() just calls exit(3).
>
> After a bit of web searching, it seem
Jeff King writes:
> Many code paths will free a tree object's buffer and set it
> to NULL after finishing with it in order to keep memory
> usage down during a traversal. However, out of 8 sites that
> do this, only one actually unsets the "parsed" flag back.
> Those sites that don't are setting
On Tue, Jun 4, 2013 at 2:53 PM, Junio C Hamano wrote:
> After reading the subject alone, my reaction was "is this sorting
> commits by the name of the author"?
>
> That is one of the expected natural reactions when people hear about
> this option, which is not what you want.
>
> Perhaps naming it
Pat Thoyts writes:
> Seems fine to me. I can't test this as I have no access to this
> platform. Possibly you should run this in a catch statement so it can
> ignore any errors and I would tend to use the 'auto_execok' command to
> ensure that osascript actually exists. Something like
>
> set a
On Thu, Jun 6, 2013 at 11:09 AM, David Lang wrote:
> On Thu, 6 Jun 2013, Felipe Contreras wrote:
>
>> In the end my point remains unchanged; Perl is declining, so it would
>> be wise for the future to use another scripting language instead.
>
>
> Perl use may or may not be declining (depending on
Moritz Bunkus writes:
> This used to work until recently, though I'm not sure up to which
> version excatly. I'm on Arch Linux and therefore usually pretty much
> up to date. My current git version is 1.8.3.
Could you test if 'master' at 3684101a654d (Merge branch
'kb/status-ignored-optim-2', 20
On Thu, Jun 6, 2013 at 12:16 PM, Greg Troxel wrote:
>
> Felipe Contreras writes:
>
>> On Thu, Jun 6, 2013 at 9:54 AM, Greg Troxel wrote:
>>>
>>> git is a core tool that people use on almost the smallest of boxes,
>>> perhaps even replacing rcs for managing local config files. On such
>>> machin
Felipe Contreras writes:
> Pretty much what it says on the tin.
And a bit more, isn't it?
The --keep-redundant-commits option implies the --allow-empty option
and it was perfectly acceptable to give both. By making sure that
only at most one of -k-r-d, -a-e or -s-e is given, this forbids that
On Thu, Jun 6, 2013 at 12:21 PM, Junio C Hamano wrote:
> Jeff King writes:
>> If somebody wants to write a note somewhere in the git
>> documentation, that's fine with me, but I'm not clear on exactly
>> what it would even say.
>
> I agree with both points. I can suggest to clarify the log mess
On Thu, Jun 6, 2013 at 1:30 PM, Junio C Hamano wrote:
> Felipe Contreras writes:
>
>> Pretty much what it says on the tin.
>
> And a bit more, isn't it?
>
> The --keep-redundant-commits option implies the --allow-empty option
> and it was perfectly acceptable to give both. By making sure that
>
Felipe Contreras writes:
> As we should.
>
> Signed-off-by: Felipe Contreras
> ---
> sequencer.c | 42 +-
> 1 file changed, 41 insertions(+), 1 deletion(-)
>
> diff --git a/sequencer.c b/sequencer.c
> index 76ff2ff..74480d7 100644
> --- a/sequencer.c
> ++
On Tue, Jun 4, 2013 at 3:14 PM, Junio C Hamano wrote:
> elliottcable writes:
>> Thus, I've added an --authorship-order version of --date-order, which relies
>> upon the AUTHOR_DATE instead of the COMMITTER_DATE; this means that old
>> commits
>> will continue to show up chronologically in-order
Felipe Contreras writes:
> On Thu, Jun 6, 2013 at 1:30 PM, Junio C Hamano wrote:
>> Felipe Contreras writes:
>>
>>> Pretty much what it says on the tin.
>>
>> And a bit more, isn't it?
>>
>> The --keep-redundant-commits option implies the --allow-empty option
>> and it was perfectly acceptable
Felipe Contreras writes:
> On Thu, Jun 6, 2013 at 1:30 PM, Junio C Hamano wrote:
>> Felipe Contreras writes:
>>
>>> Pretty much what it says on the tin.
>>
>> And a bit more, isn't it?
>>
>> The --keep-redundant-commits option implies the --allow-empty option
>> and it was perfectly acceptable
Elliott Cable writes:
> On Tue, Jun 4, 2013 at 3:14 PM, Junio C Hamano wrote:
>> elliottcable writes:
>>> Thus, I've added an --authorship-order version of --date-order, which relies
>>> upon the AUTHOR_DATE instead of the COMMITTER_DATE; this means that old
>>> commits
>>> will continue to sh
Wow. That's my bad entirely. I apparently hallucinated a section
suggesting that you “sign-off” commits that you'd reviewed, or
something; and I'd completely skipped the section on certifying that
you have legal rights to the work, because I'd *written* it, and
didn't think it'd be relevant.
I fee
Fix warnings from perlcritic's level 5 and 4. They correspond to the following
cases:
- always end a submodule with a return
- don't use the constant pragma, use the Readonly module instead
- some syntax details for maps, and others.
Signed-off-by: Célestin Matte
Signed-off-by: Matthieu Moy
---
Hi,
This series of commits intends to follow most of perlcritic's recommandations
in order to make the code more maintainable and readable.
I followed most recommandations from level 5 (most critical ones )to 2, but
left a great part of the level 1 ones, as they were more about personal choices
of
- Remove m modifier when useless (m// and // was used randomly; this makes the
code more coherent)
- Remove stringy split (split('c', ...) instead of split(/c/, ...))
- Use {}{} instead of /// when slashes or used inside the regexp so as not to
escape it.
Signed-off-by: Célestin Matte
Signed-off-
Signed-off-by: Célestin Matte
Signed-off-by: Matthieu Moy
---
contrib/mw-to-git/git-remote-mediawiki.perl | 44 ---
1 file changed, 26 insertions(+), 18 deletions(-)
diff --git a/contrib/mw-to-git/git-remote-mediawiki.perl
b/contrib/mw-to-git/git-remote-mediawiki.perl
There is already a global variable called $url. Changing the name of the local
variable prevents future possible misunderstanding.
Signed-off-by: Célestin Matte
Signed-off-by: Matthieu Moy
---
contrib/mw-to-git/git-remote-mediawiki.perl |6 +++---
1 file changed, 3 insertions(+), 3 deletion
Signed-off-by: Célestin Matte
Signed-off-by: Matthieu Moy
---
contrib/mw-to-git/git-remote-mediawiki.perl | 26 +-
1 file changed, 13 insertions(+), 13 deletions(-)
diff --git a/contrib/mw-to-git/git-remote-mediawiki.perl
b/contrib/mw-to-git/git-remote-mediawiki.perl
Signed-off-by: Célestin Matte
Signed-off-by: Matthieu Moy
---
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
b/contrib/mw-to-git/git-remote-mediawiki.perl
index 757a7a3..b
@$var structures are re-written in the following way: @{ $var }
It makes them more readable.
Signed-off-by: Célestin Matte
Signed-off-by: Matthieu Moy
---
contrib/mw-to-git/git-remote-mediawiki.perl | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/contrib/mw-to
Signed-off-by: Célestin Matte
Signed-off-by: Matthieu Moy
---
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
b/contrib/mw-to-git/git-remote-mediawiki.perl
index ae6dd2e..1c34ada 1
Placing the open() call inside the do{} struct will automatically close the
filehandle if possible.
Placing the close() call outside the do{} struct is useless and will make it
fail systematically
Change the error message to state that what fails is a fork(), not a file
opening.
Use autodie to prop
Signed-off-by: Célestin Matte
Signed-off-by: Matthieu Moy
---
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
b/contrib/mw-to-git/git-remote-mediawiki.perl
index 68fd129..a6c7d
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 ot
Signed-off-by: Célestin Matte
Signed-off-by: Matthieu Moy
---
contrib/mw-to-git/git-remote-mediawiki.perl | 20 ++--
1 file changed, 14 insertions(+), 6 deletions(-)
diff --git a/contrib/mw-to-git/git-remote-mediawiki.perl
b/contrib/mw-to-git/git-remote-mediawiki.perl
index 0
Signed-off-by: Célestin Matte
Signed-off-by: Matthieu Moy
---
contrib/mw-to-git/git-remote-mediawiki.perl |5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/contrib/mw-to-git/git-remote-mediawiki.perl
b/contrib/mw-to-git/git-remote-mediawiki.perl
index 7fbc998..ae6dd2e 1
- 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 w
Signed-off-by: Célestin Matte
Signed-off-by: Matthieu Moy
---
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
b/contrib/mw-to-git/git-remote-mediawiki.perl
index a6c7de2..cf8dfc8 100755
--- a/contrib/m
Signed-off-by: Célestin Matte
Signed-off-by: Matthieu Moy
---
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
b/contrib/mw-to-git/git-remote-mediawiki.perl
index cf8dfc8..7fbc9
Empty strings are replaced by an $EMPTY constant.
Signed-off-by: Célestin Matte
Signed-off-by: Matthieu Moy
---
contrib/mw-to-git/git-remote-mediawiki.perl | 18 ++
1 file changed, 10 insertions(+), 8 deletions(-)
diff --git a/contrib/mw-to-git/git-remote-mediawiki.perl
b/co
Signed-off-by: Célestin Matte
Signed-off-by: Matthieu Moy
---
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
b/contrib/mw-to-git/git-remote-mediawiki.perl
index ff9fd8f..952ddcc 1
Elliott Cable writes:
>> And update revs->lifo to use that same enum, without adding
>> use_author_date bit to rev_info.
>
> I'll look into replacing lifo with an enum as soon as I can sit back
> down to update this patch. For the moment, nothing more than
> committer_date_sort and author_date_so
On Wed, Jun 5, 2013 at 6:13 AM, Michael Haggerty wrote:
>
> But my main point is that I think it would be easier to phase out
> contrib/ if there were a good alternate way of providing visibility to
> "satellite" projects. The relevant Git wiki page [1] is the most likely
> candidate, but it is a
David Lang wrote:
> Perl use may or may not be declining (depending on how you measure it), but
> are you really willing to take on the task of re-writing everything that's
> in Perl into another language and force all developers of scripts to learn
> that other language? what's the ROI of this?
L
Отправлено с iPhonegmail--
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 http://vger.kernel.org/majordomo-info.html
Johannes Schindelin wrote:
> My initial reaction, too. It was hard enough to get Perl included with Git
> for Windows (because of that pesky Subversion dependency).
Nevertheless, we had to do it, and we did it. We will do it again, if
we get enough important code written in Ruby.
> As you can se
On 06/06/2013 01:46 AM, Felipe Contreras wrote:
> On Thu, Jun 6, 2013 at 2:26 AM, demerphq wrote:
>>
>> Good thing you are being objective and leaving out the Python 3.0
>> mess, the long legacy of backwards compatibility in the Perl
>> community, the active community behind it, its extensive port
Greg Troxel wrote:
> It's not about what I want. It's about making choices that affect other
> people, and trying to find a plan that will be overall reasonable;
> that's the essence of stewardship in packaging. Compiling for just
> myself is far easier.
Have you asked the SBCL or Google-Chrome
Hey,
yes, the problem is gone at 3684101a654d. Thanks.
Kind regards,
mosu
--
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 http://vger.kernel.org/majordomo-info.html
Célestin Matte writes:
> Placing the open() call inside the do{} struct will automatically close the
> filehandle if possible.
> Placing the close() call outside the do{} struct is useless and will make it
> fail systematically
> Change the error message to state that what fails is a fork(), not
On Fri, 7 Jun 2013, Ramkumar Ramachandra wrote:
David Lang wrote:
Perl use may or may not be declining (depending on how you measure it), but
are you really willing to take on the task of re-writing everything that's
in Perl into another language and force all developers of scripts to learn
tha
Hello All,
If I did 'git diff HEAD^..HEAD -- file' should git not report some
kind of warning if it could not match the file? For example, if 'file'
were infact 'dir/file' and 'file' were unique, would it not be a good
idea to report that in the present working directory 'file' were not
found but
Le 06/06/2013 23:13, Junio C Hamano a écrit :
> Confused. Which part of this patch moves open inside a do{} block?
> This was last touched by [9/18] but it doesn't do any such thing,
> either.
I must have failed the rebase, as the first part of the commit moved to
[14/18] because it modifies a pa
Ramkumar Ramachandra wrote:
> Git is probably the _last_ thing
> to be complaining about when it comes to packaging.
It would be nice if contrib/ files supported the usual "make; make
install; make clean" targets. That's missing functionality that does
matter
Le 06/06/2013 23:26, Sarma Tangirala a écrit :
> Hello All,
>
> If I did 'git diff HEAD^..HEAD -- file' should git not report some
> kind of warning if it could not match the file? For example, if 'file'
> were infact 'dir/file' and 'file' were unique, would it not be a good
> idea to report that
Célestin Matte writes:
> So using autodie may not be a good idea.
> But the problem is that in the current state, open() return values are
> checked, but print ones are not, although it should be.
I tried "man autodie" and tried to spot 'print' in the categories
list that shows things like ":all
Célestin Matte writes:
> Le 06/06/2013 23:26, Sarma Tangirala a écrit :
>> Hello All,
>>
>> If I did 'git diff HEAD^..HEAD -- file' should git not report some
>> kind of warning if it could not match the file? For example, if 'file'
>> were infact 'dir/file' and 'file' were unique, would it not
Le 06/06/2013 23:58, Junio C Hamano a écrit :
> Célestin Matte writes:
>
>> So using autodie may not be a good idea.
>> But the problem is that in the current state, open() return values are
>> checked, but print ones are not, although it should be.
>
> I tried "man autodie" and tried to spot 'p
Here are the topics that have been cooking. Commits prefixed with
'-' are only in 'pu' (proposed updates) while commits prefixed with
'+' are in 'next'.
You can find the changes described here in the integration branches
of the repositories listed at
http://git-blame.blogspot.com/p/git-publi
Junio C Hamano writes:
> If you want to do this in a multi-step series (which may not be a
> bad idea), I would imagine that the enum starts as a choice between
> the two: traversal-order vs committer-date-order. The first patch
> would change nothing else.
>
> And then you would add the third c
When sorting commits topologically, the primary invariant is to emit
all children before its parent is emitted. When traversing a forked
history like this with "git log C E":
ABC
\
DE
we ensure that A is emitted after all of B, C, D, and E are done, B
has to wait until
On Thu, Jun 06, 2013 at 03:41:08PM -0700, Junio C Hamano wrote:
> * rr/complete-difftool (2013-06-03) 2 commits
> (merged to 'next' on 2013-06-04 at 01c7611)
> + completion: clarify ls-tree, archive, show completion
> + completion: difftool takes both revs and files
>
> Update command line co
On Wed, Jun 5, 2013 at 6:40 PM, Jeff King wrote:
> We cannot create an archive from a blob object, so we would
> not expect anyone to provide one to us. And if they do, we
> will fail anyway just after the reachability check. We can
> therefore optimize our reachability check to ignore blobs
> co
SZEDER Gábor writes:
> On Thu, Jun 06, 2013 at 03:41:08PM -0700, Junio C Hamano wrote:
>> * rr/complete-difftool (2013-06-03) 2 commits
>> (merged to 'next' on 2013-06-04 at 01c7611)
>> + completion: clarify ls-tree, archive, show completion
>> + completion: difftool takes both revs and files
On Thu, Jun 6, 2013 at 3:34 PM, Célestin Matte
wrote:
> [PATCH 08/18] Explicitely assign local variable as undef and make a proper
> one-instruction-by- line indentation
s/Explicitely/Explicitly/
s/by- /by-/
> Signed-off-by: Célestin Matte
> Signed-off-by: Matthieu Moy
--
To unsubscribe from
Junio C Hamano writes:
> When sorting commits topologically, the primary invariant is to emit
> all children before its parent is emitted. When traversing a forked
s/its/their/;
>> As I needed to have an excuse to push jk/commit-info-slab topic
>> further (I have an unpublished show-branch rew
Hi,
On a relatively-empty Intel Core i7 975 @ 3.33GHz (quad-core):
Cns# cd DragonFly/
Cns# time git log sys/sys/sockbuf.h >/dev/null
0.540u 0.140s 0:04.30 15.8% 0+0k 2754+55io 6484pf+0w
Cns# time git log sys/sys/sockbuf.h > /dev/null
0.000u 0.030s 0:00.52 5.7% 0+0k 0+0io 0pf+0w
Cns# tim
On Thu, Jun 6, 2013 at 3:34 PM, Célestin Matte
wrote:
> Fix warnings from perlcritic's level 5 and 4. They correspond to the following
> cases:
> - always end a submodule with a return
> - don't use the constant pragma, use the Readonly module instead
> - some syntax details for maps, and others.
On Thu, Jun 6, 2013 at 3:34 PM, Célestin Matte
wrote:
> - Remove m modifier when useless (m// and // was used randomly; this makes the
> code more coherent)
> - Remove stringy split (split('c', ...) instead of split(/c/, ...))
> - Use {}{} instead of /// when slashes or used inside the regexp so a
Eric Sunshine writes:
>> - if (my ($child, $parents) = $line =~
>> m/^-?([a-f0-9]+) ([a-f0-9 ]+)/) {
>> - foreach my $parent (split(' ', $parents)) {
>> + if (my ($child, $parents) = $line =~ /^-?([a-f0-9]+)
>> ([a-f0-9 ]
Hi git,
I want to work on a visualization program for git. I was hoping there
was a library that would allow me to monitor a git repo for changes.
Consider it like inotify, but for a git repository (in fact, I think
it would probably have inotify under the hood).
This hypothetical library would t
Hi Ram,
On Fri, 7 Jun 2013, Ramkumar Ramachandra wrote:
> Johannes Schindelin wrote:
> > My initial reaction, too. It was hard enough to get Perl included with Git
> > for Windows (because of that pesky Subversion dependency).
>
> Nevertheless, we had to do it, and we did it.
That is not quite
On Thu, Jun 6, 2013 at 3:34 PM, Célestin Matte
wrote:
> Signed-off-by: Célestin Matte
> Signed-off-by: Matthieu Moy
> ---
> 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.
1 - 100 of 127 matches
Mail list logo
