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

Add build env's #184

Merged
merged 1 commit into from
Nov 5, 2024
Merged

Add build env's #184

merged 1 commit into from
Nov 5, 2024

Conversation

BlackDex
Copy link
Collaborator

@BlackDex BlackDex commented Nov 4, 2024

Sometimes you want to set specific environment variables during build. For example, on some systems nodejs needs some extra options to be able to run correctly on low memory systems. To make sure this will be loaded and works on both scripts and Dockerfile, you can now create a .build_env file. This .build_env file should contain all the variables (including an export command) you want to expose during build time.

The template file has a nodejs example.

Closes #183

Also fixed some shellcheck warnings.

@BlackDex BlackDex requested a review from dani-garcia November 4, 2024 21:26
Sometimes you want to set specific environment variables during build.
For example, on some systems nodejs needs some extra options to be able to run correctly on low memory systems.
To make sure this will be loaded and works on both scripts and Dockerfile, you can now create a `.build_env` file.
This `.build_env` file should contain all the variables (including an export command) you want to expose during build time.

The template file has a nodejs example.

Closes dani-garcia#183

Signed-off-by: BlackDex <black.dex@gmail.com>
@dani-garcia dani-garcia merged commit 2eb3311 into dani-garcia:master Nov 5, 2024
@BlackDex BlackDex deleted the add-build-envs branch November 5, 2024 23:04
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.

2 participants