Skip to content
This repository was archived by the owner on Apr 21, 2025. It is now read-only.

Conversation

@stuartmorgan-g
Copy link
Contributor

If no explicit sysroot is set, and it's not disabled, use a sysroot for
Linux host builds. This means we will no longer require specific
libraries to be on CI images for host builds, and developers won't need
as much local setup.

Fixes flutter/flutter#53176

If no explicit sysroot is set, and it's not disabled, use a sysroot for
Linux host builds. This means we will no longer require specific
libraries to be on CI images for host builds, and developers won't need
as much local setup.

Fixes flutter/flutter#53176
@stuartmorgan-g
Copy link
Contributor Author

Do the conditions on this look right? The goal is for it to apply only to the Linux host build, not variants (e.g., Fuchsia) or non-host builds (e.g., Android).

I can't test this against the engine CI without landing it here first AFAICT, unfortunately, due to the split buildroot/engine repo situation.

@stuartmorgan-g
Copy link
Contributor Author

Actually, I should be able to do a canary PR against this branch, so I'll do that before landing here.

@cbracken
Copy link
Member

cbracken commented Apr 8, 2020

/cc @iskakaushik because I can never remember the exact is_fuchsia vs is_fuchsia_host and which one is set when :)

@stuartmorgan-g
Copy link
Contributor Author

Well, is_fuchsia_host is always false in our buildroot, so I assume that's something that it only makes sense to check in flutter/engine (which I assume uses a different buildroot for fuchsia host builds?).

Test PR passed compilation, so I'll go ahead and land this.

@stuartmorgan-g stuartmorgan-g merged commit 0c64fd2 into flutter:master Apr 8, 2020
@stuartmorgan-g stuartmorgan-g deleted the linux-host-default-sysroot branch April 8, 2020 02:22
chinmaygarde pushed a commit to chinmaygarde/flutter_buildroot that referenced this pull request May 4, 2021
If no explicit sysroot is set, and it's not disabled, use a sysroot for
Linux host builds. This means we will no longer require specific
libraries to be on CI images for host builds, and developers won't need
as much local setup.

Fixes flutter/flutter#53176
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Use a sysroot for Linux host builds on CI

2 participants