login
Header Space

 
 

Re: Two crazy proposals for changing git's diff commands

Score:
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Carl Worth <cworth@...>
Cc: <git@...>
Date: Thursday, February 9, 2006 - 8:13 pm

Carl Worth <cworth@cworth.org> writes:


I see where you are coming from and I personally kind of like
this consistency.  But I am hesitant to declare these two
workflows as the _only_ ones officially supported by the tool at
this moment.  The collective use pattern of git is still young
and leaving the door open would help us to evolve more useful
workflows around the core in the future.  One of our first goals
would be to have good set of introductory documentations for
best current practices -- if the project you work on fits this
workflow, here is a way to do it.  With that workflow, there is
this way.  By that time, hopefully many useless workflows (my
"constantly rewinding pu branch" pattern _could_ fall into that
category) would be withered away and it would be a good time to
streamline the tool around officially supported workflows.

My feeling is that it is a bit premature to do that right now; I
do not think we are there yet.


Sorry, I am not convinced about separate command names, but in
the meantime you could have small wrappers around "git diff":
"cw-diff-files" and "cw-diff-index" would be one liner each,
wouldn't they?

And that is a _good_ thing about git.  If the sample set of
barebone Porcelains do not fit your needs, you can mix and match
using lowlevel commands to quickly script your customized ones.
If that is useful in general, that would become one of the best
current practices.


I think that is where we differ.  I think even for index-lover
"diff HEAD" is useful in certain cases (obviously index-ignorer
would not see much useful output from "diff --cached", though).
In fact, I fall into "index-lover" camp but I use both depending
on occasion.  And as I said, I do not think "index-lover" and
"index-ignorer" distinction above would be the only two valid
workflows anyway, so I feel partitioning the command set along
those lines is at least premature if not wrong.


-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
Re: Two crazy proposals for changing git's diff commands, Junio C Hamano, (Wed Feb 8, 9:21 pm)
Re: Two crazy proposals for changing git's diff commands, J. Bruce Fields, (Sat Feb 11, 11:15 pm)
Re: Two crazy proposals for changing git's diff commands, Junio C Hamano, (Sat Feb 11, 11:48 pm)
Re: Two crazy proposals for changing git's diff commands, Junio C Hamano, (Thu Feb 9, 8:13 pm)
Re: Two crazy proposals for changing git's diff commands, Junio C Hamano, (Thu Feb 9, 10:24 pm)
Re: Two crazy proposals for changing git's diff commands, Linus Torvalds, (Wed Feb 8, 9:30 pm)
Re: Two crazy proposals for changing git's diff commands, Junio C Hamano, (Fri Feb 10, 5:05 am)
Re: Comments on "status -v", Junio C Hamano, (Fri Feb 10, 5:09 pm)
Re: Comments on "status -v", Petr Baudis, (Fri Feb 10, 6:51 pm)
Re: Comments on "status -v", Junio C Hamano, (Fri Feb 10, 7:26 pm)
Re: Comments on "status -v", Linus Torvalds, (Fri Feb 10, 5:35 pm)
Re: Comments on "status -v", Junio C Hamano, (Fri Feb 10, 6:12 pm)
Re: Two crazy proposals for changing git's diff commands, Junio C Hamano, (Wed Feb 8, 9:37 pm)
Re: Two crazy proposals for changing git's diff commands, Linus Torvalds, (Wed Feb 8, 9:05 pm)
Re: Two crazy proposals for changing git's diff commands, Junio C Hamano, (Wed Feb 8, 9:35 pm)
Re: Two crazy proposals for changing git's diff commands, Linus Torvalds, (Wed Feb 8, 9:21 pm)
Re: Two crazy proposals for changing git's diff commands, Junio C Hamano, (Thu Feb 9, 7:40 pm)
speck-geostationary