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

make: s/PROJDEPS/APPDEPS/ #1339

Merged
merged 2 commits into from Jun 21, 2014
Merged

make: s/PROJDEPS/APPDEPS/ #1339

merged 2 commits into from Jun 21, 2014

Conversation

LudwigKnuepfer
Copy link
Member

fixes #1284

@LudwigKnuepfer
Copy link
Member Author

update when #1338 is merged

Kijewski
Kijewski reviewed Jun 21, 2014
View reviewed changes
dist/Makefile
@@ -49,7 +49,7 @@ export QUIET ?= 1
#export INCLUDES += -Iapplication_include

# Specify custom dependencies for your application here ...
# export PROJDEPS = proj_data.h
# export APPDEPS = app_data.h
Copy link
Contributor

Choose a reason for hiding this comment

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

While you are at it: maybe change to app_data.h config.h ... to better display that this is a list?

Copy link
Member Author

Choose a reason for hiding this comment

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

good point

LudwigKnuepfer added a commit to LudwigKnuepfer/RIOT that referenced this pull request Jun 21, 2014
@LudwigKnuepfer
doc: improve APPDEPS example
75e50e2 
Addresses RIOT-OS#1339 (comment)
@LudwigKnuepfer
Copy link
Member Author

rebased

@Kijewski
Copy link
Contributor

test_bloom, our only user of the variable, still compiles.

ACK & go.

Kijewski added a commit that referenced this pull request Jun 21, 2014
@Kijewski
Merge pull request #1339 from LudwigOrtmann/issue-1284
e0b3e84 
make: s/PROJDEPS/APPDEPS/
@Kijewski Kijewski merged commit e0b3e84 into RIOT-OS:master Jun 21, 2014
@LudwigKnuepfer LudwigKnuepfer deleted the issue-1284 branch June 22, 2014 12:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@Kijewski Kijewski

Labels
Area: build system Area: Build system Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

make : s/PROJDEPS/APPDEPS/ ?
2 participants
@LudwigKnuepfer @Kijewski