Mailing-List: contact dev-help@subversion.apache.org; run by ezmlm
Precedence: bulk
Received-SPF: pass (nike.apache.org: local policy)
Date: Wed, 5 Jan 2011 15:11:35 +0200
From: Daniel Shahaf <d.s@daniel.shahaf.name>
To: Prabhu Gnana Sundar <prabhugs@collab.net>
Cc: dev@subversion.apache.org
Subject: Re: [PATCH] '--version --quiet' should display only the version
	info in svnadmin, svnlook, svnsync, svndumpfilter, svnversion,
	svnserve
Message-ID: <20110105131135.GB12319@daniel3.local>
References: <1294227766.25028.10.camel@prabhugnanasundar>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <1294227766.25028.10.camel@prabhugnanasundar>
User-Agent: Mutt/1.5.18 (2008-05-17)

Prabhu Gnana Sundar wrote on Wed, Jan 05, 2011 at 17:12:46 +0530:
> Hi all,
> 
> Currently svnadmin, svnlook, svnsync, svndumpfilter, svnversion,
> svnserve show the entire version information even if '--version --quiet'
> is given as command.
> 
> The '--quiet' option has *not* been handled so far.
> 
> This patch would make svnadmin, svnlook, svnsync, svndumpfilter,
> svnversion, svnserve '--version --quiet' to display *only* the version
> number(info) just like 'svn --version --quiet' does.
> 

Heh.  This is one of the things I looked into, back before I was
formally involved with the project... (as patch manager and later
a committer)

But I never sent in a patch for this, because I reckoned that a patch
that just adds a --quiet option everywhere (and uses it just for
--version --quiet) would serve more as a 'practice for Daniel in sending
patches' and less as 'a patch that Subversion needs'.  So I looked
for more 'real' patches (eg, copy_tests 48) to work on.


Not to say that this patch is unnecessary (maybe you have a usecase
wherein it's required), but if you're looking to cut your teeth, I can
suggest a few more challening things to work on. :-)

> I have attached the patch and the log message with this mail. Please
> give your views on this.
> 
> 
> Thanks and regards
> Prabhu