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.

Sign up for GitHub

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

Jump to bottom

treewide: use ifndef guards for config.h variables + remove deprecated options, e.g., FreeRTOS #518

Merged
merged 20 commits into from
Aug 31, 2020
Merged

treewide: use ifndef guards for config.h variables + remove deprecated options, e.g., FreeRTOS #518

merged 20 commits into from
Aug 31, 2020

Conversation

fjmolinas
Copy link
Contributor

This PR replaces:

// #define OPENWSN_% (0)

by

#ifndef OPENWSN_%
#define OPENWSN_% (0)
#endif

It then replaces all the conditionals accordingly to not check if the Macro is defined but if it is "active" (value 1)

This PR is based on top of #514.

@fjmolinas
Copy link
Contributor Author

@TimothyClaeys might want to take a look :)

@TimothyClaeys
Copy link
Member

TimothyClaeys commented Aug 27, 2020 •
edited
Loading

@fjmolinas @changtengfei I updated this PR:

  • Remove some deprecated code (e.g., FreeRTOS)
  • Remove deprecated options (e.g., ide option)
  • Remove tools folder and .project file
  • Add compile flags to include apps, software modules, stack configuration and board options from the command line

Example: scons board=openmote-cc2538 toolchain=armgcc modules=udp,coap apps=cjoin boardopt=hw-crypto stackcfg=adaptive-msf oos_openwsn.

@fjmolinas
Copy link
Contributor Author

@fjmolinas @changtengfei I updated this PR:

Feel free to force push if you want, or to open another PR that includes these changes!

@TimothyClaeys TimothyClaeys changed the title treewide: use ifndef guards for config.h variables treewide: use ifndef guards for config.h variables + remove deprecated options, e.g., FreeRTOS Aug 31, 2020
@fjmolinas fjmolinas mentioned this pull request Aug 31, 2020
Closed
fjmolinas and others added 17 commits August 31, 2020 17:12
@fjmolinas
treewide: use ifndef guards for config.h variables
0410244
@TimothyClaeys @fjmolinas
Remove unused IDE options
3d518ec
@TimothyClaeys @fjmolinas
Move FASTSIM switch to config.h + add checks to check_config.h
1cd6a3c
@TimothyClaeys @fjmolinas
Move DEADLINE_OPTION to config.h
dd2c705
@TimothyClaeys @fjmolinas
Move DAGROOT switch to config.h
b833f67
@TimothyClaeys @fjmolinas
Deprecate FREERTOS
1e3cf23
@TimothyClaeys @fjmolinas
Remove eclipse project file
06fdefe
@TimothyClaeys @fjmolinas
Remove unused debug flag
34e7281
@TimothyClaeys @fjmolinas
Move PANID to config.h
eae24a9
@TimothyClaeys @fjmolinas
Docs update
181a48c
@TimothyClaeys @fjmolinas
Update travis yaml file
264e200
@TimothyClaeys @fjmolinas
fix warnings in doxygen
1745e59
@TimothyClaeys @fjmolinas
Add compile flags for apps and modules
d8c2a40
@TimothyClaeys @fjmolinas
Split stack modules and stack configuration
4210b12 
Move ADAPTIVE-MSF to stack conf
@TimothyClaeys @fjmolinas
Add compile flags for apps, modules, stack conf., and board options
38d4939
@TimothyClaeys @fjmolinas
Move deadline option to stack configuration in config.h + fix ifdef bug
83560a2
@TimothyClaeys @fjmolinas
Move python build scripts to "Scripts" folder + announce footprint sc…
60c4c12 
…ripts
@changtengfei changtengfei merged commit 94997d0 into openwsn-berkeley:develop Aug 31, 2020
@fjmolinas fjmolinas deleted the pr_config_guards branch September 3, 2020 10:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@fjmolinas @TimothyClaeys @changtengfei