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

fix issue where rules_perl could not used for cross compilation #31

Merged
merged 1 commit into from
Dec 20, 2021

Conversation

jheaff1
Copy link
Contributor

@jheaff1 jheaff1 commented Nov 6, 2021

When using the perl toolchain and specifying a platform on the bazel
command line (e.g. --platforms=:android), the command would fail as
bazel required a perl toolchain that is "target compatible" with the
":android" platform. Unlike a C++ toolchain, the perl toolchain is
compatible with any target platform therefore no target_compatible_with should be specified

When using the perl toolchain and specifying a platform on the bazel
command line (e.g. --platforms=:android), the command would fail as
bazel required a perl toolchain that is "target compatible" with the
":android" platform. Unlike a C++ toolchain, the perl toolchain is
compatible with any target platform.
@jheaff1
Copy link
Contributor Author

jheaff1 commented Dec 14, 2021

@PiotrSikora @skeletonkey Would it be possible for one of you guys to review this PR please?

@skeletonkey
Copy link
Collaborator

@jheaff1 I will be able to take a look at this this weekend (Monday at the latest). Sorry for the delay - end of year has been a little crazy.

@skeletonkey skeletonkey merged commit fc55108 into bazel-contrib:master Dec 20, 2021
@jheaff1 jheaff1 deleted the support_cross_compilation branch December 20, 2021 21:58
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 this pull request may close these issues.

2 participants