subversion-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Daniel Shahaf <...@daniel.shahaf.name>
Subject Re: [RFE] Make 'svn patch' read from STDIN
Date Tue, 09 Feb 2016 03:22:59 GMT
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
> > 

Mime
View raw message