Skip to content

Conversation

TheJayas
Copy link

Hi @tillkamppeter and @zdohnal , I have created new pull request with all the changes required by you. Please look into it and suggest if any other changes are required.

Copy link
Member

@zdohnal zdohnal left a comment

Choose a reason for hiding this comment

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

Thank you for the PR!

Please check my comment and do let me know what do you think - IMO we call Avahi there as well, so if the module is missing, we get traceback, so we should check the AVAHI_AVAILABLE again.

Thank you in advance!


self.PrintersWindow.show()
self.builder = Gtk.Builder()
self.service_browser = AvahiServiceBrowser()
Copy link
Member

Choose a reason for hiding this comment

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

Shouldn't these three lines be guarded by AVAHI_AVAILABLE as well? We call objects from Avahi at the method run().

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