Skip to content
This repository was archived by the owner on Apr 1, 2026. It is now read-only.

Network manager integration#7

Closed
jpds-zz wants to merge 18 commits intosharpbracket:masterfrom
jpds-zz:network-manager-integration
Closed

Network manager integration#7
jpds-zz wants to merge 18 commits intosharpbracket:masterfrom
jpds-zz:network-manager-integration

Conversation

@jpds-zz
Copy link
Copy Markdown

@jpds-zz jpds-zz commented Nov 9, 2013

Use NetworkManager over DBus to detect if a network connection is available before we attempt to retrieve location information.

connections are available to retrieve location information.
@jonls
Copy link
Copy Markdown
Collaborator

jonls commented Nov 23, 2013

What if the geoclue provider that the user has chosen does not require network connection?

@jpds-zz
Copy link
Copy Markdown
Author

jpds-zz commented Nov 23, 2013

This code executes after the "if (... mode != PROGRAM_MODE_MANUAL)" bit, so I assumed that everything non-manual would need a connection?

@jonls
Copy link
Copy Markdown
Collaborator

jonls commented Nov 23, 2013

No, it may be needed in most cases but geoclue does not strictly need a network connection (for example it can fetch the position from an attached gps device). I agree that the current situation where redshift sometimes fails because there is no connection at startup should be fixed, but I don't think this is the right way to do it.

@ghost
Copy link
Copy Markdown

ghost commented Dec 7, 2014

without looking at the code, it also should be noted that many DE's and linux users do NOT use networkmanager!, their are simpler lower level ways to determine connectivity. The most common technique would be to wget / curl some well know url.. if its responds your online! (ICMP won't work on many networks for this)

(UGG i'm so sorry guys, i just noticed this is a year old... nevermind.. sorry)

@jonls
Copy link
Copy Markdown
Collaborator

jonls commented Aug 19, 2017

With #500, missing connectivity on startup should no longer cause Redshift to fail. If connectivity is eventually established, GeoClue2 will discover a location and report it to Redshift.

@jonls jonls closed this Aug 19, 2017
minus7 pushed a commit to minus7/redshift that referenced this pull request Jul 22, 2018
gamma-wl: wait for sync callback on destroy
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants