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

dist: ng_sniffer: read interface from device #3150

Merged
merged 1 commit into from Jun 5, 2015

Conversation

miri64
Copy link
Member

@miri64 miri64 commented Jun 2, 2015

Reads first interface from the device to choose it for setting the options and also returns an error if the application on the device doesn't has any interfaces.

@miri64 miri64 added the NSTF label Jun 2, 2015
@miri64 miri64 added this to the Release 2015.06 milestone Jun 2, 2015
@OlegHahm
Copy link
Member

OlegHahm commented Jun 3, 2015

Untested, but 👍 for the change.

@haukepetersen
Copy link
Contributor

+1

@PeterKietzmann PeterKietzmann added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Jun 5, 2015
@PeterKietzmann
Copy link
Member

I see a 👍 and a +1 . Travis and me also think that the change is reasonable but I did not test it. Someone willing to push the green button?

@miri64
Copy link
Member Author

miri64 commented Jun 5, 2015

If you give an ACK :D

@haukepetersen
Copy link
Contributor

ACK and go.

haukepetersen added a commit that referenced this pull request Jun 5, 2015
dist: ng_sniffer: read interface from device
@haukepetersen haukepetersen merged commit b2413f7 into RIOT-OS:master Jun 5, 2015
@miri64 miri64 deleted the dist/enh/sniffer-read-iface branch June 5, 2015 14:35
@miri64 miri64 added the Area: network Area: Networking label Sep 30, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: network Area: Networking CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants