[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [RFE] Make 'svn patch' read from STDIN

From: Daniel Shahaf <d.s_at_daniel.shahaf.name>
Date: Tue, 09 Feb 2016 03:22:59 +0000

Bert Huijben wrote on Sun, Feb 07, 2016 at 13:56:12 +0100:
> Well.. Perhaps ^D works on your platform, but you will need other keys
> on other platforms.

If different platforms use different keys then the message can use
#ifdef to print the right advice. I expected readers to assume that,
at the level of formality I was writing at.

>
> And ^D doesn’t abort, but signals successful EOF, so you need
> a different description else.

So we can say "exit" or "cancel" instead. (Let's figure this out when
this becomes a patch.)

Daniel

> >
> > Let's not throw the baby out with the bathwater. Simply printing an
> > informative message "svn: Reading a patch from standard input; press ^D
> > to abort" should address the confusion without breaking the use-case of
> > people who run 'patch' without arguments and then paste a patch from the
> > clipboard.
> >
> > Cheers,
> >
> > Daniel
> >
Received on 2016-02-09 04:29:43 CET

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.