Skip to content

Re-enable the Dart Development Service (DDS)#64671

Merged
bkonyi merged 4 commits into
masterfrom
enable_dds
Aug 27, 2020
Merged

Re-enable the Dart Development Service (DDS)#64671
bkonyi merged 4 commits into
masterfrom
enable_dds

Conversation

@bkonyi

@bkonyi bkonyi commented Aug 26, 2020

Copy link
Copy Markdown
Contributor

This change re-enables DDS and outputs the DDS URI in place of the VM service URI on the console. If --disable-dds is not provided, --host-vmservice-port will be used to determine the port for DDS rather than the host port for the VM service, which will instead be randomly chosen.

@flutter-dashboard flutter-dashboard Bot added the tool Affects the "flutter" command-line tool. See also t: labels. label Aug 26, 2020
Comment thread packages/flutter_tools/lib/src/base/dds.dart Outdated
@bkonyi bkonyi requested a review from jonahwilliams August 26, 2020 22:04
@jonahwilliams

Copy link
Copy Markdown
Contributor

what remains to be done after this PR? Just web support?

@jonahwilliams jonahwilliams left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@bkonyi

bkonyi commented Aug 27, 2020

Copy link
Copy Markdown
Contributor Author

what remains to be done after this PR? Just web support?

Yup, that should be it. DDS has SSE support now, so it's just a matter of wiring things up hopefully.

bkonyi added 4 commits August 27, 2020 10:26
This change re-enables DDS and outputs the DDS URI in place of the VM
service URI on the console. If --disable-dds is not provided,
--host-vmservice-port will be used to determine the port for DDS rather
than the host port for the VM service, which will instead be randomly
chosen.
@bkonyi bkonyi merged commit d7d1241 into master Aug 27, 2020
@bkonyi bkonyi deleted the enable_dds branch August 27, 2020 23:35
bkonyi added a commit that referenced this pull request Aug 28, 2020
bkonyi added a commit that referenced this pull request Aug 28, 2020
bkonyi added a commit that referenced this pull request Aug 28, 2020
bkonyi added a commit that referenced this pull request Aug 28, 2020
* Reland "Re-enable the Dart Development Service (DDS) (#64671)"

This reverts commit 2ae25cc.

* Fix MDNS building Observatory URI with port 0 instead of forwarding the device port

* Added MDNS test
jonahwilliams pushed a commit that referenced this pull request Aug 29, 2020
jonahwilliams pushed a commit that referenced this pull request Aug 29, 2020
bkonyi added a commit that referenced this pull request Aug 29, 2020
bkonyi added a commit that referenced this pull request Aug 31, 2020
mehmetf added a commit that referenced this pull request Aug 31, 2020
mehmetf added a commit that referenced this pull request Aug 31, 2020
mingwandroid pushed a commit to mingwandroid/flutter that referenced this pull request Sep 6, 2020
This change re-enables DDS and outputs the DDS URI in place of the VM
service URI on the console. If --disable-dds is not provided,
--host-vmservice-port will be used to determine the port for DDS rather
than the host port for the VM service, which will instead be randomly
chosen.
mingwandroid pushed a commit to mingwandroid/flutter that referenced this pull request Sep 6, 2020
mingwandroid pushed a commit to mingwandroid/flutter that referenced this pull request Sep 6, 2020
…flutter#64802)

* Reland "Re-enable the Dart Development Service (DDS) (flutter#64671)"

This reverts commit 2ae25cc.

* Fix MDNS building Observatory URI with port 0 instead of forwarding the device port

* Added MDNS test
mingwandroid pushed a commit to mingwandroid/flutter that referenced this pull request Sep 6, 2020
mingwandroid pushed a commit to mingwandroid/flutter that referenced this pull request Sep 6, 2020
mingwandroid pushed a commit to mingwandroid/flutter that referenced this pull request Sep 6, 2020
bkonyi added a commit that referenced this pull request Sep 15, 2020
bkonyi added a commit that referenced this pull request Sep 16, 2020
This reverts commit 66b01c1.

* Add DDS ipv6 support

* Use --dds-port for DDS instead of hijacking --host-vmservice-port
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Aug 28, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

tool Affects the "flutter" command-line tool. See also t: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants