Skip to content
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

make environment variable BOOT_LOCAL_REPO usable on windows #246

Merged
merged 1 commit into from
Jul 9, 2015

Conversation

schmir
Copy link
Contributor

@schmir schmir commented Jul 9, 2015

use md5 hash when building path components when the environment variable
BOOT_LOCAL_REPO is set.

Previously we ended up trying to use illegal filenames like
C:\home.boot\cache\boot\customC:\home.m2... on windows.

We now use a single directory under boot/custom/ instead of a deep
directory structure.

The conversion to a hex string is a bit awkward, see
https://stackoverflow.com/questions/9655181/how-to-convert-a-byte-array-to-a-hex-string-in-java
for a discussion.

fixes #243

use md5 hash when building path components when the environment variable
BOOT_LOCAL_REPO is set.

Previously we ended up trying to use illegal filenames like
C:\home\.boot\cache\boot\customC:\home\.m2\... on windows.

We now use a single directory under boot/custom/ instead of a deep
directory structure.

The conversion to a hex string is a bit awkward, see
https://stackoverflow.com/questions/9655181/how-to-convert-a-byte-array-to-a-hex-string-in-java
for a discussion.

fixes #243
@micha
Copy link
Contributor

micha commented Jul 9, 2015

👍 Awesome!

micha added a commit that referenced this pull request Jul 9, 2015
make environment variable BOOT_LOCAL_REPO usable on windows
@micha micha merged commit b5e78fd into master Jul 9, 2015
@micha micha removed the in progress label Jul 9, 2015
@micha micha added this to the 2.2.0 milestone Jul 10, 2015
@schmir schmir deleted the issue-243-boot-local-repo-md5 branch July 21, 2015 19:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Can't use BOOT_LOCAL_REPO on windows
2 participants