threads / patch / 16372

patchRetain multiple -q/-v occurrences in git pull

Subject: [PATCH] Retain multiple -q/-v occurrences in git pull

## tl;dr

5 messages between Nov 17, 2008 and Nov 19, 2008. Diffs are folded; open one to read it.

replies: 4people: 3as markdown or json

Tuncer Ayaz· Nov 17, 2008, 22:09 UTC · lore

To support counting -q/-v options in git pull retain them by concatenating.

Signed-off-by: Tuncer Ayaz <tuncer.ayaz@gmail.com>
---
 git-pull.sh |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)
Show changes to git-pull.sh +2 −2
diff --git a/git-pull.sh b/git-pull.sh
index 8866f2a..1cac898 100755
--- a/git-pull.sh
+++ b/git-pull.sh
@@ -24,9 +24,9 @@ while :
 do
 	case "$1" in
 	-q|--quiet)
-		verbosity=-q ;;
+		verbosity="$verbosity -q" ;;
 	-v|--verbose)
-		verbosity=-v ;;
+		verbosity="$verbosity -v" ;;
 	-n|--no-stat|--no-summary)
 		no_stat=-n ;;
 	--stat|--summary)

-- 
1.6.0.2.GIT
Constantine Plotnikov· Nov 19, 2008, 11:46 UTC · re: Tuncer Ayaz · lore

Re: [PATCH] Retain multiple -q/-v occurrences in git pull

On Tue, Nov 18, 2008 at 1:09 AM, Tuncer Ayaz <tuncer.ayaz@gmail.com> wrote:
> To support counting -q/-v options in git pull retain
> them by concatenating.
>
[rest of message cut]

By the way, there is yet another way to invoke git fetch. It is "git remote update". Possibly it should support "-v" and "-q" options for consistency as well.

Regards, Constantine

Tuncer Ayaz· Nov 19, 2008, 22:47 UTC · re: Constantine Plotnikov · lore

Re: [PATCH] Retain multiple -q/-v occurrences in git pull

On Wed, Nov 19, 2008 at 12:46 PM, Constantine Plotnikov <constantine.plotnikov@gmail.com> wrote:

Show 9 quoted lines
> On Tue, Nov 18, 2008 at 1:09 AM, Tuncer Ayaz <tuncer.ayaz@gmail.com> wrote:
>> To support counting -q/-v options in git pull retain
>> them by concatenating.
>>
> [rest of message cut]
>
> By the way, there is yet another way to invoke git fetch. It is "git
> remote update". Possibly it should support "-v" and "-q" options for
> consistency as well.

Yes, we could add -q to builtin-remote and also make sure that it passes the already existing -v and maybe a yet-to-be-added -q to the fetch command. I want to be first sure what the overall strategy regarding -q/-v is.

Right now we have some code which uses OPT__VERBOSE/OPT__QUIET and some new code which uses the newly-added OPT__VERBOSITY.

These are the options I can see:
1) Use the old and new macros and decide in each module which one to use
2) Get rid off OPT__VERBOSE and OPT__QUIET and use OPT__VERBOSITY only
3) Think about new ways to handle all of this. Possibly some new print_* macros?
Any opinions?
Tuncer Ayaz· Nov 19, 2008, 23:00 UTC · lore

Re: [PATCH] Retain multiple -q/-v occurrences in git pull

[re-sent due to propagated wrong Junio address. sorry.]

On Wed, Nov 19, 2008 at 12:46 PM, Constantine Plotnikov <constantine.plotnikov@gmail.com> wrote:

Show 9 quoted lines
> On Tue, Nov 18, 2008 at 1:09 AM, Tuncer Ayaz <tuncer.ayaz@gmail.com> wrote:
>> To support counting -q/-v options in git pull retain
>> them by concatenating.
>>
> [rest of message cut]
>
> By the way, there is yet another way to invoke git fetch. It is "git
> remote update". Possibly it should support "-v" and "-q" options for
> consistency as well.

Yes, we could add -q to builtin-remote and also make sure that it passes the already existing -v and maybe a yet-to-be-added -q to the fetch command. I want to be first sure what the overall strategy regarding -q/-v is.

Right now we have some code which uses OPT__VERBOSE/OPT__QUIET and some new code which uses the newly-added OPT__VERBOSITY.

These are the options I can see:
1) Use the old and new macros and decide in each module which one to use
2) Get rid off OPT__VERBOSE and OPT__QUIET and use OPT__VERBOSITY only
3) Think about new ways to handle all of this. Possibly some new print_* macros?
Any opinions?

← back to recent threads