Skip to content

gcc: pass hostEqualsTarget to builder#210097

Closed
ghost wants to merge 1 commit intostagingfrom
unknown repository
Closed

gcc: pass hostEqualsTarget to builder#210097
ghost wants to merge 1 commit intostagingfrom
unknown repository

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Jan 10, 2023

Description of changes

Currently gcc's builder.sh (ab)uses targetConfig to determine if host==target. For external bootstrap we may need to pass a targetConfig even for native builds. Therefore this commit passes a separate flag to indicate if host==target.

Things done
  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • Fits CONTRIBUTING.md.

Currently gcc's builder.sh (ab)uses `targetConfig` to determine if
host==target.  For external bootstrap we may need to pass a
`targetConfig` even for native builds.  Therefore this commit passes
a separate flag to indicate if host==target.
@ghost ghost requested a review from matthewbauer as a code owner January 10, 2023 20:37
@ghost ghost mentioned this pull request Jan 10, 2023
4 tasks
@ofborg ofborg bot added the 10.rebuild-linux-stdenv This PR causes stdenv to rebuild on Linux and must target a staging branch. label Jan 10, 2023
@ofborg ofborg bot added 10.rebuild-darwin: 501+ This PR causes many rebuilds on Darwin and should normally target the staging branches. 10.rebuild-darwin: 5001+ This PR causes many rebuilds on Darwin and must target the staging branches. 10.rebuild-linux: 501+ This PR causes many rebuilds on Linux and should normally target the staging branches. 10.rebuild-linux: 5001+ This PR causes many rebuilds on Linux and must target the staging branches. labels Jan 10, 2023
@ghost
Copy link
Copy Markdown
Author

ghost commented Jan 11, 2023

This turns out to be unnecessary, and in fact a bad idea. We should be sending more builds through the cross-codepath rather than the native-codepath.

@ghost ghost closed this Jan 11, 2023
@ghost ghost deleted the pr/gcc/hostEqualsTarget branch January 23, 2024 06:48
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

10.rebuild-darwin: 501+ This PR causes many rebuilds on Darwin and should normally target the staging branches. 10.rebuild-darwin: 5001+ This PR causes many rebuilds on Darwin and must target the staging branches. 10.rebuild-linux: 501+ This PR causes many rebuilds on Linux and should normally target the staging branches. 10.rebuild-linux: 5001+ This PR causes many rebuilds on Linux and must target the staging branches. 10.rebuild-linux-stdenv This PR causes stdenv to rebuild on Linux and must target a staging branch.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants