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

Drop WORKSPACE support from Bazel builds #1115

Closed
yukawa opened this issue Nov 2, 2024 · 0 comments · Fixed by #1116
Closed

Drop WORKSPACE support from Bazel builds #1115

yukawa opened this issue Nov 2, 2024 · 0 comments · Fixed by #1116
Assignees

Comments

@yukawa
Copy link
Collaborator

yukawa commented Nov 2, 2024

Description

As discussed in the following issue, Mozc has successfully migrated to bzlmod.

Given that no one is the Mozc team is actively testing and maintaining non-bzlmod build configuration, it would make much sense to make it clear that only bzlmod is now supported to build Mozc with Bazel by removing WORKSPACE support completely.

Version or commit-id

e1b40ed

@yukawa yukawa pinned this issue Nov 2, 2024
yukawa added a commit to yukawa/mozc that referenced this issue Nov 2, 2024
As Mozc has already migrated to bzlmod from WORKSPACE, let's remove
obsolete and unmaintained WORKSPACE-related files and configs to make it
clear that WORKSPACE is no longer supported to build Mozc with Bazel.

Closes google#1115.
@yukawa yukawa self-assigned this Nov 2, 2024
yukawa added a commit to yukawa/mozc that referenced this issue Nov 5, 2024
This commit removes the following bazel options that were originally
added [1] to support bzlmod for Android (google#1002).

 - experimental_enable_android_migration_apis
 - experimental_google_legacy_api

As far as we can see with Bazel 7.4.0, building Mozc with Bazel just
succeeds without the above two options in all the supported platforms.

This is a preparation to fully deprecate WORKSPACE support (google#1115).

Closes google#1117.

 [1]: 699c2ce
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 a pull request may close this issue.

1 participant