[GitHub] incubator-zeppelin pull request: Make path for .spark-dist folder always acc...

[bzz <gi...@git.apache.org>]

Github user bzz commented on the pull request:

    https://github.com/apache/incubator-zeppelin/pull/930
  
    Thanks for explaining! 
    
    So as far as I have understood, the pain point is: after `mvn clean`, on the same repo `mvn validate` faild due to  `maven-download-plugin` exception
    ```
    iO Error: Destination '/opt/zeppelin-master/spark-dependencies/target/../../.spark-dist/spark-1.6.1.tgz' directory cannot be created 
    ```
    who presumably complaining about non-existant `/target/` part of it's destination path?
    
    and what you suggest is that we put there some different path, that is not target-related, and what I suggest is that we better make it still pointing to the same `.spark-dist` dir, one level above the repo root.
    
    `${session.executionRootDirectory}/../.spark-dist` sounds as a good solution to me. 
    
    Or another, harder way could be: if you could provide the stack trace of the same error (running `mvn -X ...`) we could look into the implementation of [maven-download-plugin](https://github.com/maven-download-plugin/maven-download-plugin) and see if we could improve it to behave like `mkdir -p` in this case.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---