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

Adding cmd line args to tv-casting-app for device type of commissioners to be discovered #9572

Merged
merged 9 commits into from
Sep 14, 2021

Conversation

sharadb-amazon
Copy link
Contributor

@sharadb-amazon sharadb-amazon commented Sep 9, 2021

Problem

The tv-casting-app could only discover a fixed, hard coded type of commissioner to request commissioning from.

Change overview

This change allows passing in a device type ("all" or a specific device type id) as a command line argument, so that the user can choose to request commissioning from a non-video player device (video player commissioner is still the default)

Testing

Tested against minimal-mdns to make sure the tv-casting-app is able to discover non-video player device type commissioners.

@andy31415
Copy link
Contributor

Please update the summary to not include the 'example text'

examples/tv-casting-app/linux/main.cpp Outdated Show resolved Hide resolved
examples/tv-casting-app/linux/main.cpp Outdated Show resolved Hide resolved
examples/tv-casting-app/linux/main.cpp Outdated Show resolved Hide resolved
examples/tv-casting-app/linux/main.cpp Outdated Show resolved Hide resolved
examples/tv-casting-app/linux/main.cpp Outdated Show resolved Hide resolved
examples/tv-casting-app/linux/main.cpp Outdated Show resolved Hide resolved
examples/tv-casting-app/linux/main.cpp Outdated Show resolved Hide resolved
@woody-apple
Copy link
Contributor

@sharadb-amazon ?

@sharadb-amazon
Copy link
Contributor Author

@sharadb-amazon ?

Updated the code. Waiting for andy31415@ to review/approve. :)

@andy31415
Copy link
Contributor

I have validated that the 2 device types added as an example match the spec received (not yet in official spec, but the list I received from the spec writers)

@sharadb-amazon
Copy link
Contributor Author

I have validated that the 2 device types added as an example match the spec received (not yet in official spec, but the list I received from the spec writers)

Looks like all checks passed. Could you (or someone else who has permissions) please merge this PR?

@andy31415 andy31415 merged commit cfacfda into project-chip:master Sep 14, 2021
@sharadb-amazon sharadb-amazon deleted the tv-casting-app-dt branch September 14, 2021 21:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants