Skip to content

Use the frontend server to compile pub executables #3008

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

Merged
merged 1 commit into from
May 21, 2021

Conversation

jonasfj
Copy link
Member

@jonasfj jonasfj commented May 20, 2021

Reverts #3006 which reverted #2968, hence, introducing: Use the frontend server to compile pub executables.

cc @jakemac53

We can merge this once we've figured out how to use the frontend_server_client in the Dart SDK.

@google-cla google-cla bot added the cla: yes label May 20, 2021
@jakemac53
Copy link
Contributor

Interesting, I assumed this had been worked out because the test package does use it... but maybe it hasn't been updated either yet or something. I can look into it.

@jakemac53
Copy link
Contributor

https://dart-review.googlesource.com/c/sdk/+/200920

dart-bot pushed a commit to dart-lang/sdk that referenced this pull request May 20, 2021
unblocks dart-lang/pub#3008

Change-Id: I14ed70371001b0ca709469172afd2b97679a1e2b
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/200920
Auto-Submit: Jake Macdonald <[email protected]>
Commit-Queue: Jonas Jensen <[email protected]>
Reviewed-by: Jonas Jensen <[email protected]>
@jonasfj jonasfj marked this pull request as ready for review May 21, 2021 10:37
@jonasfj jonasfj merged commit def32ce into master May 21, 2021
@jonasfj jonasfj deleted the revert-3006-revert-2968-fe-server branch May 21, 2021 10:37
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 this pull request may close these issues.

2 participants