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

[meson] delete meson.build files in favor of bazel #12449

Closed
17 of 18 tasks
timothytrippel opened this issue May 3, 2022 · 2 comments · Fixed by #13156
Closed
17 of 18 tasks

[meson] delete meson.build files in favor of bazel #12449

timothytrippel opened this issue May 3, 2022 · 2 comments · Fixed by #13156
Assignees
Labels

Comments

@timothytrippel
Copy link
Contributor

timothytrippel commented May 3, 2022

Once the following issues have been addressed, all meson.build files can be removed (with minimal regressions):

Additionally the following items will need to be fixed after the meson deletion to cleanup some minor regressions:

@timothytrippel
Copy link
Contributor Author

This is partially accomplished in #12394, but it was determined it would be easier to remove all meson.build files at once, than to stage them (i.e., ROM build files first, then all sw/ after).

@arunthomas arunthomas added the Priority:P0 Priority: critical label May 3, 2022
@timothytrippel timothytrippel added Priority:P1 Priority: high and removed Priority:P0 Priority: critical labels May 6, 2022
@timothytrippel
Copy link
Contributor Author

CC: @cfrantz @arunthomas As I mentioned earlier this week, this issue contains the list of items that must be addressed to delete meson from the repository. CCing you both for visibility.

@lowRISC lowRISC deleted a comment from milesdai May 10, 2022
timothytrippel added a commit to timothytrippel/opentitan that referenced this issue May 12, 2022
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]>
timothytrippel added a commit to timothytrippel/opentitan that referenced this issue May 12, 2022
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]>
timothytrippel added a commit to timothytrippel/opentitan that referenced this issue May 13, 2022
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]>
timothytrippel added a commit to timothytrippel/opentitan that referenced this issue May 13, 2022
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]>
timothytrippel added a commit to timothytrippel/opentitan that referenced this issue May 13, 2022
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]>
timothytrippel added a commit that referenced this issue May 13, 2022
Meson will soon be removed from our project and replaced entirely with
bazel (#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 #12447.

Signed-off-by: Timothy Trippel <[email protected]>
@lowRISC lowRISC deleted a comment from drewmacrae May 24, 2022
timothytrippel added a commit to timothytrippel/opentitan that referenced this issue Jun 10, 2022
This removes all `meson.build` files in favor of using Bazel as the
OpenTitan build system. This fixes lowRISC#12449.

Signed-off-by: Timothy Trippel <[email protected]>
timothytrippel added a commit to timothytrippel/opentitan that referenced this issue Jun 10, 2022
This removes all `meson.build` files in favor of using Bazel as the
OpenTitan build system. This fixes lowRISC#12449.

Signed-off-by: Timothy Trippel <[email protected]>
timothytrippel added a commit that referenced this issue Jun 10, 2022
This removes all `meson.build` files in favor of using Bazel as the
OpenTitan build system. This fixes #12449.

Signed-off-by: Timothy Trippel <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants