This repository was archived by the owner on Sep 7, 2020. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 33
PPM-249 - independent unit to deal with hostapd configuration #1514
Closed
Changes from all commits
Commits
Show all changes
27 commits
Select commit
Hold shift + click to select a range
b8ff8d6
prplmesh-hostapd: info: added README.md file
RanRegev 0ed7fba
prplmesh-hostapd: added configuration class
RanRegev d936489
prplmesh-hostapd: added a unit test file
RanRegev 328d2c9
prplmesh-hostapd: added cmake for prplmesh hostapd
RanRegev 4c53b57
prplmesh-hostapd: added set_create_vap_value()
RanRegev b4c53d8
prplmesh-hostapd: added set_create_vap_value() overload with int
RanRegev bdcd28d
prplmesh-hostapd: added store()
RanRegev 9c4327e
prplmesh-hostapd: added get_vap_value()
RanRegev 9beae66
prplmesh-hostapd: added disable_all_ap_vaps()
RanRegev 955cfed
prplmesh-hostapd: added private help functions
RanRegev ead6021
prplmesh-hostapd: added comment_vap()
RanRegev edd5bc2
prplmesh-hostapd: added comment ability
RanRegev 03adfc7
prplmesh-hostapd: added uncomment_vap()
RanRegev 39b1f33
prplmesh-hostapd: added for_all_ap_vaps()
RanRegev 15ee90f
prplmesh-hostapd: added for_all_ap_vaps() with user iterator
RanRegev 10775b8
prplmesh-hostapd: added test enable_all_vaps()
RanRegev 9fdd0c3
prplmesh-hostapd: added test iterate-both-containers
RanRegev 6399e7b
prplmesh-hostapd: few cleanups after rebase
RanRegev 6bd96dd
prplmesh-hostapd: added set_create_xx_head_value
RanRegev f2f33cd
prplmesh-hostapd: added get_head_value()
RanRegev 933fe63
prplmesh-hostapd: added vap_indication to load()
RanRegev 72cc86f
prplmesh-hostapd: added ap predicate to for_all_ap_vaps
RanRegev c21d2be
prplmesh-hostapd: PR 1514 review fixups
RanRegev 52b7859
prplmesh-hostapd: added ap predicate to for_all_ap_vaps
RanRegev 629bf66
nl80211: hostapd configuration: cmake
RanRegev 4c321eb
nl80211: hostapd configuration update_vap_credentials()
RanRegev b87c6f4
nl80211: hostapd configuration: close issues found in PR 1550
RanRegev File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,10 +1,11 @@ | ||
| add_subdirectory(bcl) | ||
| add_subdirectory(bwl) | ||
| add_subdirectory(tlvf) | ||
| add_subdirectory(btl) | ||
| add_subdirectory(bwl) | ||
| add_subdirectory(hostapd) | ||
| add_subdirectory(scripts) | ||
| add_subdirectory(tlvf) | ||
|
|
||
| # BWL depends on BCL, so make sure it's built first | ||
| add_dependencies(bwl bcl) | ||
| add_dependencies(btlvf bcl) | ||
| add_dependencies(btl btlvf) | ||
| add_dependencies(btl btlvf) | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,48 @@ | ||
|
|
||
| project(prplmesh_hostapd VERSION ${prplmesh_VERSION}) | ||
|
|
||
| # Set the base path for the current module | ||
| set(MODULE_PATH ${CMAKE_CURRENT_LIST_DIR}) | ||
|
|
||
| # Build the library | ||
| set(hostapd_sources source/configuration.cpp) | ||
| add_library(${PROJECT_NAME} ${hostapd_sources}) | ||
| set_target_properties(${PROJECT_NAME} PROPERTIES VERSION ${prplmesh_VERSION} SOVERSION ${prplmesh_VERSION_MAJOR}) | ||
| target_link_libraries(${PROJECT_NAME} elpp ${PRPLMESH_HOSTAPD_LIBS}) | ||
| target_include_directories(${PROJECT_NAME} | ||
| PRIVATE | ||
| ${PLATFORM_INCLUDE_DIR} | ||
| PUBLIC | ||
| $<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/include> | ||
| ) | ||
|
|
||
| install( | ||
| TARGETS ${PROJECT_NAME} | ||
| ARCHIVE DESTINATION ${CMAKE_INSTALL_LIBDIR} | ||
| LIBRARY DESTINATION ${CMAKE_INSTALL_LIBDIR} | ||
| RUNTIME DESTINATION ${CMAKE_INSTALL_BINDIR} | ||
| ) | ||
|
|
||
| if (BUILD_TESTS) | ||
| set(TEST_PROJECT_NAME ${PROJECT_NAME}_unit_tests) | ||
| set(unit_tests_sources | ||
| ${MODULE_PATH}/unit_tests/configuration_test.cpp | ||
| ) | ||
| add_executable(${TEST_PROJECT_NAME} | ||
| ${unit_tests_sources} | ||
| ) | ||
| if (COVERAGE) | ||
| set_target_properties(${TEST_PROJECT_NAME} PROPERTIES COMPILE_FLAGS "--coverage -fPIC -O0") | ||
| set_target_properties(${TEST_PROJECT_NAME} PROPERTIES LINK_FLAGS "--coverage") | ||
| endif() | ||
| target_include_directories(${TEST_PROJECT_NAME} | ||
| PRIVATE | ||
| ${PLATFORM_INCLUDE_DIR} | ||
| PUBLIC | ||
| $<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/include> | ||
| ) | ||
| target_link_libraries(${TEST_PROJECT_NAME} ${PROJECT_NAME} ${PRPLMESH_HOSTAPD_LIBS}) | ||
| target_link_libraries(${TEST_PROJECT_NAME} gtest_main) | ||
| install(TARGETS ${TEST_PROJECT_NAME} DESTINATION bin/tests) | ||
| add_test(NAME ${TEST_PROJECT_NAME} COMMAND $<TARGET_FILE:${TEST_PROJECT_NAME}>) | ||
| endif() |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Commit message says "created CMakeLists.txt" but the commit actually adds the CMakeLists.txt created in the previous commit to the parent, and sorts the add_subdirectory calls alphabetically. The latter should be mentioned in the commit message as well.
But there's really no need to split off the creation of CMakeLists.txt or adding it to the parent in a separate commit.
Similarly, when a unit test is added, it's better to add it to CI in the same commit.
By the way, you may want to re-read https://github.com/prplfoundation/prplMesh/blob/master/CONTRIBUTING.md#commits. About the subject line:
And about the body:
And it's not written explicilty there, but the examples show that the JIRA URL should be a separate paragraph.
All of this is not very important, i.e. not worth spending your time on fixing all the commit messages are commits. Just something to take into account for the future.