Github user orhankislal commented on a diff in the pull request:

    https://github.com/apache/madlib/pull/286#discussion_r200793992
  
    --- Diff: pom.xml ---
    @@ -66,6 +66,7 @@
                   <exclude>deploy/preflight.sh</exclude>
                   <exclude>deploy/RPM/CMakeLists.txt</exclude>
                   <exclude>deploy/rpm_post.sh</exclude>
    +              <exclude>deploy/rpm_post_uninstall.sh</exclude>
    --- End diff --
    
    Since this is a new file, we should add the apache license instead of 
adding it to the exclude list.


---

Reply via email to