Github user justinleet commented on the issue:

    https://github.com/apache/metron/pull/1013
  
    I didn't handle it particularly cleanly (I just nuked the dir if the user 
was okay with it), and the way you're doing the other stuff is cleaner anyway, 
so I'm fine with using this. 
    
    If we wanted to be more clean about it, we could have the directory just 
pull if it already exists.
    
    What I did was 
    ```
    # warn the user if the working directory exists
    if [ -d "$WORK" ]; then
      read -p "  directory exists [$WORK].  overwrite existing repo? [yN] " -n 
1 -r
      echo
      if [[ ! $REPLY =~ ^[Yy]$ ]]; then
        exit 1
      fi
    fi
    
    rm -rf "$WORK/metron-$VERSION"
    git clone $REPO "$WORK/metron-$VERSION"
    ```
    
    I'm pretty neutral on if we want to take care of that here, or we want to 
leave it as follow on.  I'll leave that up to you.
    
    +1


---

Reply via email to