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

allow DBus data type='aay' #368

Closed
wants to merge 3 commits into from

Conversation

crussell42
Copy link

These changes allow use of dbus data type='aay' or 'aas' or 'aai' etc.
The idea is DBusArra(DBusSignature('a'),DBusArray.byte([42,42,42]).

This is needed in order to use Avahi daemon (via dbus) on linux to register a Service in an EntryGroup.
The included test-aay.dart does this by creating an NearbyShare advertisment service.
Upon successfully adding the service any NearbyShare on android or Mac or Win box will see our service advertisment.
The structure of the service is pulled from the mac NearDrop repo.
The full intent is to use dbus/avahi as the advertising mechanism in a NearbyShare server on Linux.

* test-aay.dart actually creates an Avahi service for NearbyShare advertising.
  If successful, your machine shows up as a Nearby Share device.
  I notice that canonical Avahis is a very limited implementation via dbus.
  This test would work nicely there if that package used this new DBus with aay.
* test-array-of-arrays.dart is just a rudimentary regression.

This is ready for a pull request as I would like to get eyes on as soon as possible.
If it gains any traction I will fork the avahi project and add examples from here.
…not versed on dbus byte ordering so am hoping for some eyes on help
@github-actions
Copy link

github-actions bot commented Oct 4, 2023

Hey! crussell42 has not signed the Canonical CLA which is required to get this contribution merged on this project.

Please head over to https://ubuntu.com/legal/contributors to read more about it.

@@ -3,3 +3,4 @@
pubspec.lock
test/.test_cov.dart
coverage/
/test/widget_test.dart
Copy link
Contributor

Choose a reason for hiding this comment

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

Doesn't seem to be used.

Copy link
Author

Choose a reason for hiding this comment

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

This was me trying to learn AndroidStudio (click this, click that, suddenly files show up) to run the tests.

@crussell42
Copy link
Author

I left a comment in the issue, basically this was a mistake on my part and I humbly withdraw this pull request.

@crussell42 crussell42 closed this Oct 5, 2023
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.

2 participants