Re: Update of missing on non-working directory

On Wednesday 12 May 2010, Andy Levy wrote:
> On Wed, May 12, 2010 at 09:23, Vadym Chepkov <vchepkov_at_gmail.com> wrote:
> > I think the exit code subversion returns isn't right, it should indicate
> > an error
[...]
> > $ svn update /etc
> > Skipped '/etc'
> > $ echo $?
> > 0
>
> Why? The command executed successfully and reported what it did.
> Skipping something which is not a working copy isn't a program error;
> attempting to update a something which is not a WC is a user error.

Actually I'm with Vadym here. The docs clearly say that "update" wants a path
to a working copy. If the given path isn't a working copy, that is reason to
return EXIT_FAILURE. Claiming that an update on a non-working copy was
successful is bad.

Uli

-- 
ML: http://subversion.tigris.org/mailing-list-guidelines.html
FAQ: http://subversion.tigris.org/faq.html
Docs: http://svnbook.red-bean.com/
Sator Laser GmbH, Fangdieckstraße 75a, 22547 Hamburg, Deutschland
Geschäftsführer: Thorsten Föcking, Amtsgericht Hamburg HR B62 932
**************************************************************************************
Sator Laser GmbH, Fangdieckstraße 75a, 22547 Hamburg, Deutschland
Geschäftsführer: Thorsten Föcking, Amtsgericht Hamburg HR B62 932
**************************************************************************************
           Visit our website at <http://www.satorlaser.de/>
**************************************************************************************
Diese E-Mail einschließlich sämtlicher Anhänge ist nur für den Adressaten bestimmt und kann vertrauliche Informationen enthalten. Bitte benachrichtigen Sie den Absender umgehend, falls Sie nicht der beabsichtigte Empfänger sein sollten. Die E-Mail ist in diesem Fall zu löschen und darf weder gelesen, weitergeleitet, veröffentlicht oder anderweitig benutzt werden.
E-Mails können durch Dritte gelesen werden und Viren sowie nichtautorisierte Änderungen enthalten. Sator Laser GmbH ist für diese Folgen nicht verantwortlich.
**************************************************************************************