[PATCH] svn_load_dirs.pl: hide passwords printed to screen

2014-12-19 Thread Geoffrey Alary
Hello, I often use svn_load_dirs.pl in a script to load several huge third party libraries into our SVN repo. This repository uses LDAP authentication with https and I do not want my password popping up at times on the console executing the script (for several hours). Hence my second patch, that

[PATCH] svn_load_dirs.pl: fix rm temporary folder

2014-12-19 Thread Geoffrey Alary
Hello, I am contacting you to propose 2 patch files for the script svn_load_dirs.pl located at the URL below: http://svn.apache.org/repos/asf/subversion/trunk/contrib/client-side/svn_load_dirs/svn_load_dirs.pl.in This email discusses the first patch, which avoids failing to delete the temporary f

Re: [PATCH] svn_load_dirs.pl: do not print password to screen

2019-07-14 Thread Geoffrey Alary
Hi Daniel, > > CC: both of the most recent and biggest contributors to this file. > > In principle, same answer as in the other thread (ENOTIME unless it's a > regression I signed off on); but… Ok noted. Thank you for your replies. > > It implements a security feature: to hide the password when

Re: [PATCH] svn_load_dirs.pl: do not print password to screen

2019-07-14 Thread Geoffrey Alary
Hi Brane, Ok I understand, thank you. Best regards, Geoffrey Le dim. 14 juil. 2019 à 19:52, Branko Čibej a écrit : > > On 14.07.2019 09:47, Geoffrey Alary wrote: > > Hi Daniel, > > > >>> CC: both of the most recent and biggest contributors to this file. > &

[PATCH] svn_load_dirs.pl: do not print password to screen (v2)

2019-07-14 Thread geoffrey . alary
Hi, > > > > It implements a security feature: to hide the password when printing > > > > the command line to screen. > > > > > > I suggest to add a warning to usage() that passing the password in > > > a command-line argument may make it visible to other local OS users. > > > > Do you mean that sh

[PATCH] svn_load_dirs.pl: fix broken cleanup

2019-07-19 Thread Geoffrey Alary
Hi, When svn_load_dirs.pl is run without the -t option, cleanup on exit fails with the following error: "cannot remove path when cwd is /tmp/svn_load_dirs_ATrGGCJoWv/my_import_wc for /tmp/svn_load_dirs_ATrGGCJoWv" The only mention of the bug I have found was at the bottom of this forum post: http