Re: [PATCH 1/2] Add "--only-untracked" flag to status commands.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Aug 23, 2007, at 23:50, Matthieu Moy wrote:

Väinö Järvelä <v@xxxxxxxxxx> writes:

Subject: Re: [PATCH 1/2] Add "--only-untracked" flag to status commands.
                                       ^^

I suppose you meant --only-tracked.

Yes, a stupid typo. It should be --only-tracked.

 static const char runstatus_usage[] =
-"git-runstatus [--color|--nocolor] [--amend] [--verbose] [-- untracked]";
+"git-runstatus [--color|--nocolor] [--amend] [--verbose]
[--untracked]" "[--only-tracked]";
               ^^^

The compiler will concatenate the strings, but won't add a space. That
results in "... [--untracked][--only-tracked]".

Should have checked it more properly, sorry about that.


Otherwise, the patch sounds fine to me. Not /terribly/ usefull, but
why not.

This is something that has come up many times on our project, there might be a lot of temporary files that are used inside the project tree, to test the code, and the user just wants to see the modifications to tracked files without using a pager. One could use gitignore, but in our case, it would be hard to create a gitignore which would ignore all of the test files, and none of the actual project files, so it's quite convenient to use --only-tracked.

A better behavior might be to test outside of the project tree.


--
Matthieu

--
Väinö-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux