-
-
Notifications
You must be signed in to change notification settings - Fork 33
[Bug]: shared.sh Uses Bash-Specific Syntax #224
Copy link
Copy link
Open
Labels
bugSomething isn't workingSomething isn't working
Description
Version
Version 0.10.4.1 (Official Build, Chromium 146.0.7680.75) Arch Linux (64-bit)
Have you tested that this is not an upstream issue or an issue with your configuration?
- I have tried reproducing this issue in Chrome and it could not be reproduced there
- I have tried reproducing this issue in ungoogled-chromium and it could not be reproduced there
- I have tried reproducing this issue in Helium with a new and empty profile using
--user-data-dircommand line argument and it could not be reproduced there
Description
The setup_toolchain() function in scripts/shared.sh uses the export CXXFLAGS+="..." syntax which is specific to bash shell and fails when it is used under another shell like zsh. The failure occurs because when sourcing dev.sh it in turn sources shared.sh where the incompatible syntax gets loaded into the active shell.
How to Reproduce?
- Open a zsh shell.
- (Optionally) Activate your Python environment.
- Run
source dev.sh. - Run
he setup.
Actual behavior
The script fails with
setup_toolchain:export not valid in this context: CXXFLAGS+
Expected behavior
The script should setup the helium development environment successfully.
Additional context
- Although the
shared.shscript has a bash shebang, however it has no effect because it is sourced bydev.shwhich itself is sourced in the current user shell. - We should support Zsh considering a lot of devs prefer it over bash (Not relevant here but MacOS also defaults to Zsh)
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
bugSomething isn't workingSomething isn't working