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

November issues #18

Merged
merged 6 commits into from Jan 3, 2014
Merged

November issues #18

merged 6 commits into from Jan 3, 2014

Conversation

technosopher
Copy link

Addresses #12 and #13 , and provides a stopgap for #9. Depends on opentechinstitute/commotion-linux-py@d148f59
To test:

  1. Install on a test machine with two wireless adapters, one which supports both ad-hoc mode and ibss-rsn (via nl80211 support), and one which does not. Ensure that both adapters are active. Attempt to connect to a Commotion mesh using the applet, preferably several times. The code passes this test as long as the wireless adapter that does not support ad-hoc mode and ibss-rsn is NEVER invoked by Commotion. (Check syslog to confirm).
  2. Edit a profile using the new built-in editor in the applet menu. Optimally, you should see a very minimalist text editor with nothing but a Save button; however, the code still passes this test if you get a gedit window instead (the fallback). You should be able to save the profile without encountering any permissions errors.

Jordan McCarthy added 2 commits November 22, 2013 20:29
…ommotionc.getInterface; added disconnect call prior to connect logic
…where gtk errors out, and separate edit menu options for each profile
@ghost ghost assigned natmey Dec 12, 2013
@glamrock
Copy link
Collaborator

glamrock commented Jan 2, 2014

What's the status of this pull request?

@technosopher
Copy link
Author

My own tests indicate that this pull request should be ready to be merged, but it'd be great to get independent verification that everything is working properly.On Thursday, January 2, 2014 11:12 EST, Griffin Boyce notifications@github.com wrote: What's the status of this pull request?—Reply to this email directly or view it on GitHub.--Jordan McCarthyOpen Technology Initiative | New America FoundationMy Public Key: C08D8042

technosopher pushed a commit that referenced this pull request Jan 3, 2014
Confirmed that Linux client can successfully mesh with a node running R1 on a system with multiple active wireless adapters
@technosopher technosopher merged commit 351ecc9 into master Jan 3, 2014
@technosopher technosopher deleted the november branch January 3, 2014 21:39
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

3 participants