-
Notifications
You must be signed in to change notification settings - Fork 757
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
[bazel,dvsim] remove gen-binaries.py's reliance on meson_init.sh #12634
[bazel,dvsim] remove gen-binaries.py's reliance on meson_init.sh #12634
Conversation
Signed-off-by: Timothy Trippel <[email protected]>
ea2aa31
to
1d6e944
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd rather you hadn't done the first commit: the project hasn't standardised on a Python formatter and to my eyes the results are much uglier than what was there already!
It's not a big deal and it seems silly to have to re-do the second patch, so don't worry too much. But please could you avoid re-formatting perfectly reasonable looking code until there's a project-level agreement on Python formatters (yapf isn't the only contender, and I also think that Python is a language where it might be better to mandate PEP8 compliance with, say, flake8 and then just expect people to format their code sensibly on their own).
Meson will soon be removed from our project and replaced entirely with bazel (lowRISC#12449). This updates the the OTBN `gen-binaries.py` script to not rely on the presence of the `.env` file that was produced by the `meson_init.sh` script to provide locations to the RV32 toolchain. Instead, the `gen-binaries.py` script now uses environment variables to get the locations of the RV32 toolchain tools, which are populated via queries to bazel. This fixes lowRISC#12447. Signed-off-by: Timothy Trippel <[email protected]>
1d6e944
to
ea9b55b
Compare
Meson will soon be removed from our project and replaced entirely with
bazel (#12449). This updates the the OTBN
gen-binaries.py
script tonot rely on the presence of the
.env
file that was produced by themeson_init.sh
script to provide locations to the RV32 toolchain.Instead, the
gen-binaries.py
script now uses environment variables toget the locations of the RV32 toolchain tools, which are populated via
queries to bazel.
This fixes #12447.
Signed-off-by: Timothy Trippel [email protected]