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
[yamahamusiccast] Add discovery information #16231
[yamahamusiccast] Add discovery information #16231
Conversation
Seems like Yamaha is using uppercase `YAMAHA CORPORATION` for the manufacturer info of non-MusicCast devices and normal case `Yamaha Corporation` for MusicCast devices.
<match-properties> | ||
<match-property> | ||
<name>manufacturer</name> | ||
<regex>(?i).*Yamaha.*</regex> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As I understand, (?i)
turns on case-insensitive mode. So if you keep it, you will catch value containing either Yamaha or YAMAHA.
Exactly the same result as in the "Yamaha Receiver" binding.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
True, I will fix that.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have just commited that @lolodomo .
…ption Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
# channel types | ||
|
||
channel-type.yamahareceiver.surroundProgram.state.option.Neo:6 Music = Neo:6 Music | ||
channel-type.yamahareceiver.surroundProgram.state.option.Neo:6 Cinema = Neo:6 Cinema |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes see #11828
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed.
Sorry, I did not check what the i18n plugin generated - I should probably don't trust it without checking what it does.
Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thank you
* [yamahamusiccast] Add discovery information Seems like Yamaha is using uppercase `YAMAHA CORPORATION` for the manufacturer info of non-MusicCast devices and normal case `Yamaha Corporation` for MusicCast devices. * [yamaha] Fix REGEXes for Yamaha binding suggestion and improve description --------- Signed-off-by: Florian Hotze <florianh_dev@icloud.com> Signed-off-by: Andras Uhrin <andras.uhrin@gmail.com>
* [yamahamusiccast] Add discovery information Seems like Yamaha is using uppercase `YAMAHA CORPORATION` for the manufacturer info of non-MusicCast devices and normal case `Yamaha Corporation` for MusicCast devices. * [yamaha] Fix REGEXes for Yamaha binding suggestion and improve description --------- Signed-off-by: Florian Hotze <florianh_dev@icloud.com> Signed-off-by: Jørgen Austvik <jaustvik@acm.org>
Seems like Yamaha is using uppercase
YAMAHA CORPORATION
for the manufacturer info of non-MusicCast devices and normal caseYamaha Corporation
for MusicCast devices.Signed-off-by: Florian Hotze florianh_dev@icloud.com