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

Add support for serving a custom DevTools build #2748

Merged
merged 2 commits into from Feb 26, 2021
Merged

Conversation

bkonyi
Copy link
Contributor

@bkonyi bkonyi commented Feb 26, 2021

The Dart SDK will use a pinned version pre-built version of DevTools. This means that DDS requires a way to provide a path to the pinned build to devtools_server in order to correctly serve the application.

The Dart SDK will use a pinned version pre-built version of DevTools.
This means that DDS requires a way to provide a path to the pinned build
to devtools_server in order to correctly serve the application.
@jacob314
Copy link
Contributor

will there be possible version skew between the devtools_server and devtools_app in this mode. That should be fine but we should document that you cannot depend on the app and server to be the exact same version anymore if that is the plan.

@jacob314
Copy link
Contributor

lgtm

@bkonyi
Copy link
Contributor Author

bkonyi commented Feb 26, 2021

will there be possible version skew between the devtools_server and devtools_app in this mode. That should be fine but we should document that you cannot depend on the app and server to be the exact same version anymore if that is the plan.

Nope! The SDK will pull in devtools_server and devtools_shared alongside the devtools_app build output, all from the same revision (see the CIPD update script in $DART_SDK/third_party/devtools/).

@bkonyi bkonyi merged commit 7686580 into master Feb 26, 2021
@bkonyi bkonyi deleted the custom_devtools_support branch February 26, 2021 22:50
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.

None yet

2 participants