Skip to content

Kconfig: Expose gnrc/ipv6/blacklist configurations#12922

Merged
jia200x merged 2 commits intoRIOT-OS:masterfrom
leandrolanzieri:pr/kconfig_migrate/gnrc/ipv6_blacklisting
Dec 20, 2019
Merged

Kconfig: Expose gnrc/ipv6/blacklist configurations#12922
jia200x merged 2 commits intoRIOT-OS:masterfrom
leandrolanzieri:pr/kconfig_migrate/gnrc/ipv6_blacklisting

Conversation

@leandrolanzieri
Copy link
Copy Markdown
Contributor

Contribution description

Like #12893, this moves the gnrc_ipv6_blacklist module configuration macros to the CONFIG_ namespace and exposes them to Kconfig.

Testing procedure

  • Compile gnrc_networking example application using this module (USEMODULE=gnrc_ipv6_blacklist make all term), the default configuration should still be the same.

  • Call menuconfig using this module (USEMODULE=gnrc_ipv6_blacklist make menuconfig), you should the list size as an option. The default list size should still be the same. If you change the value it should change in the app.

Issues/PRs references

Part of #12888

@leandrolanzieri leandrolanzieri added Area: network Area: Networking Type: new feature The issue requests / The PR implemements a new feature for RIOT CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR TF: Config Marks issues and PRs related to the work of the Configuration Task Force labels Dec 11, 2019
@leandrolanzieri leandrolanzieri added this to the Release 2020.01 milestone Dec 11, 2019
@jia200x
Copy link
Copy Markdown
Member

jia200x commented Dec 20, 2019

Works as expected. Thanks!

Copy link
Copy Markdown
Member

@jia200x jia200x left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All good. The symbol is generated as expected (this is analog to #12893 so I used the same test procedure). ACK

@jia200x jia200x merged commit 7d4082a into RIOT-OS:master Dec 20, 2019
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 TF: Config Marks issues and PRs related to the work of the Configuration Task Force Type: new feature The issue requests / The PR implemements a new feature for RIOT

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants