Skip to content

Commit

Permalink
Don't use declare -x
Browse files Browse the repository at this point in the history
  • Loading branch information
lalten committed Jul 17, 2023
1 parent 9328323 commit 6a446b2
Showing 1 changed file with 7 additions and 1 deletion.
8 changes: 7 additions & 1 deletion appimage/appimage.bzl
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,13 @@ def _appimage_impl(ctx):
ctx.actions.run_shell(
outputs = [env_file],
env = env,
command = "export -p > " + env_file.path,
command = "".join([
"export -p",
# Some shells like to use `declare -x` instead of `export`. However there is no guarantee that `declare` is
# available at runtime. So let's use `export` instead.
" | sed 's/^declare -x/export/'",
" > " + env_file.path,
]),
)

# Run our tool to create the AppImage
Expand Down

0 comments on commit 6a446b2

Please sign in to comment.