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

Fix for #478 Boot variable $ERTS_LIB_DIR not supplied in Windows scripts #479

Merged
merged 1 commit into from
Aug 24, 2016

Conversation

emtenet
Copy link
Contributor

@emtenet emtenet commented Apr 29, 2016

Proposed fix for #478. To test this fix, the following issues / pull requests need to be resolved first: #464 #467 #470 #471.

Add missing -boot_var argument to Windows scripts when starting erlang.

Use different boot variables on Windows vs non-Windows.

The built-in $ROOT boot variable points to the erts directory on Windows
(dictated by erl.ini [erlang] Rootdir=) and so a boot variable $RELEASE_DIR
is made pointing to the release directory instead of the $ERTS_LIB_DIR boot variable.

Add missing -boot_var argument to Windows scripts when starting erlang.

Use different boot variables on Windows vs non-Windows.

The built-in $ROOT boot variable points to the erts directory on Windows
(dictated by erl.ini [erlang] Rootdir=) and so a boot variable $RELEASE_DIR
is made pointing to the release directory
@gotthardp
Copy link

+1
This is an important patch. Releases made with {include_erts, false} wont start on Windows otherwise.

@tsloughter tsloughter merged commit 766a2fb into erlware:master Aug 24, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants