-
Notifications
You must be signed in to change notification settings - Fork 1.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
HOME vs -Duser.home + -Dmaven.repo.local #2318
Comments
Thanks, that's good to know! Some random thoughts for now (as a user): a) I think 2.8.0-RC1 being stricter and just exiting is good. I think builds can usually be fixed and before it was implicitly fragile anyway, right? b) Regarding the 2 methods/values... I guess we should document the current order and/or add some fallback mechanism. c) Regarding |
It's definitely intentional that we don't just follow settings that are for Maven. The point of the Aether library is that while many programs can share the same underlying implementation as Maven, they are distinct from Maven and shouldn't be expected to honor Maven's settings. The problem with
Can you be more specific? I can run some |
Given that both Maven and Leiningen use the same path to local Maven repository, they are compatible, and there should be an option to switch both to another location. |
I'd take a patch for a new setting to look in a different location than |
Just noting that such a fix/feature would have to be implemented in Pomegranate, not Leiningen. |
Currently Leiningen uses two different values for the home directory:
HOME
environment variable for~/.lein
user.home
JVM system property for~/.m2
To move home (if the home directory is read-only, e.g.
/var/empty
) one currently needs to set bothHOME
and_JAVA_OPTIONS=-Duser.home=<value>
. Before 2.8.0-RC1, ifHOME
is not writable~/.lein
just won't be created, but since 2.8.0-RC1 Leiningen exits when that happens, which might break some builds.Also, Leiningen doesn't honor
maven.repo.local
JVM system property (~/.m2/repository
by default).The text was updated successfully, but these errors were encountered: