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

Network manager integration #7

Closed
wants to merge 18 commits into from

Conversation

jpds-zz
Copy link

@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.

Verified

This commit was signed with the committer’s verified signature. The key has expired.

Verified

This commit was signed with the committer’s verified signature.
isaacs isaacs

Verified

This commit was signed with the committer’s verified signature.
isaacs isaacs
… exists

before attempting providers.

Verified

This commit was signed with the committer’s verified signature. The key has expired.

Verified

This commit was signed with the committer’s verified signature. The key has expired.

Verified

This commit was signed with the committer’s verified signature. The key has expired.

Verified

This commit was signed with the committer’s verified signature.
isaacs isaacs

Partially verified

This commit is signed with the committer’s verified signature.
isaacs’s contribution has been verified via GPG key.
We cannot verify signatures from co-authors, and some of the co-authors attributed to this commit require their commits to be signed.
network_manager_connected().

Verified

This commit was signed with the committer’s verified signature.
nlf nlf

Verified

This commit was signed with the committer’s verified signature.
nlf nlf
…ate.

Verified

This commit was signed with the committer’s verified signature.
nlf nlf
connections are available to retrieve location information.
@jonls
Copy link
Owner

jonls commented Nov 23, 2013

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

@jpds-zz
Copy link
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
Owner

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

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
Owner

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 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.

None yet

2 participants