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

shell: Enable setting acknowledgement request flag from shell #5523

Merged
merged 1 commit into from
Jun 7, 2016

Conversation

aeneby
Copy link
Member

@aeneby aeneby commented Jun 7, 2016

This PR adds functionality to the shell for setting/unsetting the acknowledgement request bit of sent packets, via the NETOPT_ACK_REQ option introduced in PR #5297.

@miri64 miri64 added the Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation label Jun 7, 2016
@miri64 miri64 self-assigned this Jun 7, 2016
@miri64
Copy link
Member

miri64 commented Jun 7, 2016

ACK and go when Murdock is happy.

@miri64 miri64 added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Jun 7, 2016
@kaspar030
Copy link
Contributor

pr and commit title are not optimal.

@miri64
Copy link
Member

miri64 commented Jun 7, 2016

True, @aeneby please prefix your PR and commit titles with the module(s) you change

@aeneby aeneby changed the title Enable setting acknowledgement request flag from shell shell: Enable setting acknowledgement request flag from shell Jun 7, 2016
@aeneby
Copy link
Member Author

aeneby commented Jun 7, 2016

Fixed.

@miri64
Copy link
Member

miri64 commented Jun 7, 2016

re-ACK

@kaspar030
Copy link
Contributor

thx! :)

@miri64 miri64 merged commit 7c3f393 into RIOT-OS:master Jun 7, 2016
@aeneby aeneby deleted the sc_ack_req branch June 7, 2016 20:11
@aeneby aeneby mentioned this pull request Jun 7, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants