Skip to content

Improvements to generated rel/bin/app.bat #13679

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

Merged
merged 1 commit into from
Jun 21, 2024

Conversation

wojtekmach
Copy link
Member

  • Validate value of !RELEASE_DISTRIBUTION!

  • Validate value of !RELEASE_MODE!

  • Exit with ERRORLEVEL set to 1 on errors.

    Previously we did not and so apps that use releases and rely on exit codes to report errors were not doing so (e.g. Livebook).

  • Rename rpc node name from rem-(...) to rpc-(...)

Also updates generated rel/env.bat.exs.

  * Validate value of `!RELEASE_DISTRIBUTION!`

  * Validate value of `!RELEASE_MODE!`

  * Exit with `ERRORLEVEL` set to `1` on errors.

    Previously we did not and so apps that use releases and rely on exit
    codes to report errors were not doing so (e.g. Livebook).

  * Rename rpc node name from `rem-(...)` to `rpc-(...)`

Also updates generated `rel/env.bat.exs`.
@josevalim josevalim merged commit 134941e into elixir-lang:main Jun 21, 2024
9 checks passed
@josevalim
Copy link
Member

💚 💙 💜 💛 ❤️

@wojtekmach wojtekmach deleted the wm-app-bat branch June 21, 2024 14:17
josevalim pushed a commit that referenced this pull request Jul 2, 2024
  * Validate value of `!RELEASE_MODE!`

  * Exit with `ERRORLEVEL` set to `1` on errors.

    Previously we did not and so apps that use releases and rely on exit
    codes to report errors were not doing so (e.g. Livebook).

  * Rename rpc node name from `rem-(...)` to `rpc-(...)`

  * Update generated `rel/env.bat.exs`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants