You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by pa...@apache.org on 2022/08/27 23:45:29 UTC

[arrow-nanoarrow] branch main updated: Make rm call a little more robust, fix typo (#37)

This is an automated email from the ASF dual-hosted git repository.

paleolimbot pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow-nanoarrow.git


The following commit(s) were added to refs/heads/main by this push:
     new d398929  Make rm call a little more robust, fix typo (#37)
d398929 is described below

commit d3989298af067b9bedd66d93ef1c0bd84e94c42d
Author: Dirk Eddelbuettel <ed...@debian.org>
AuthorDate: Sat Aug 27 18:45:24 2022 -0500

    Make rm call a little more robust, fix typo (#37)
    
    On an initial call, `rm` errors when the files are not (yet) present.  Adding `-f` fixes that.   And the final echo has a simple typo.
---
 r/configure | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/r/configure b/r/configure
index fe1fd1b..a8549bc 100755
--- a/r/configure
+++ b/r/configure
@@ -3,7 +3,7 @@
 # to generate the bundled header/source
 if [ -f "../src/nanoarrow/nanoarrow.h" ]; then
   echo "Bundling nanoarrow using ../CMakeLists.txt"
-  rm src/nanoarrow.h src/nanoarrow.c
+  rm -f src/nanoarrow.h src/nanoarrow.c
   if [ -d "cmake" ]; then
     rm -rf cmake
   fi
@@ -20,5 +20,5 @@ if [ -f "src/nanoarrow.h" ] && [ -f "src/nanoarrow.c" ]; then
 fi
 
 echo "Vendored src/nanoarrow.h and/or src/nanoarrow.c are missing"
-echo "Tge .tar.gz was probably built incorrectly and it's probably not your fault"
+echo "The .tar.gz was probably built incorrectly and it's probably not your fault"
 exit 1