Skip to content

info-build: add INCLUDES to the output#8714

Merged
aabadie merged 1 commit intoRIOT-OS:masterfrom
cladmi:pr/info/add_includes
Mar 2, 2018
Merged

info-build: add INCLUDES to the output#8714
aabadie merged 1 commit intoRIOT-OS:masterfrom
cladmi:pr/info/add_includes

Conversation

@cladmi
Copy link
Copy Markdown
Contributor

@cladmi cladmi commented Mar 1, 2018

Contribution description

Add INCLUDES to the output of info-build

When typing make info-build I added an output for INCLUDES variable.
It will help for debugging INCLUDES resolution.

The added output when doing make -C examples/default BOARD=native

INCLUDES: 
        -I/home/harter/work/git/RIOT/core/include  
        -I/home/harter/work/git/RIOT/drivers/include  
        -I/home/harter/work/git/RIOT/sys/include  
        -I/home/harter/work/git/RIOT/cpu/native/include  
        -I/home/harter/work/git/RIOT/boards/native/include  
        -DNATIVE_INCLUDES  
        -I/home/harter/work/git/RIOT/boards/native/include/  
        -I/home/harter/work/git/RIOT/core/include/  
        -I/home/harter/work/git/RIOT/drivers/include/  
        -I/home/harter/work/git/RIOT/cpu/native/include  
        -I/home/harter/work/git/RIOT/sys/include

Issues/PRs references

None

@cladmi
info-build: add INCLUDES to the output
1c816a9 
Add INCLUDES to the output of info-build
@cladmi cladmi added Area: build system Area: Build system Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Mar 1, 2018
This was referenced Mar 1, 2018
Merged
Merged
@cladmi cladmi requested review from aabadie and kYc0o March 2, 2018 14:05
aabadie
aabadie approved these changes Mar 2, 2018
View reviewed changes
Copy link
Copy Markdown
Contributor

@aabadie aabadie left a comment

Choose a reason for hiding this comment

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

ACK

I think it's useful, the changes are ok and work

@aabadie aabadie merged commit 3d3166e into RIOT-OS:master Mar 2, 2018
@cladmi cladmi deleted the pr/info/add_includes branch March 5, 2018 14:27
@cladmi
Copy link
Copy Markdown
Contributor Author

cladmi commented Mar 5, 2018

Thanks

@cladmi cladmi added this to the Release 2018.04 milestone Nov 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@aabadie aabadie aabadie approved these changes

@kYc0o kYc0o Awaiting requested review from kYc0o

Assignees

No one assigned

Labels

Area: build system Area: Build system CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer

Projects

None yet

Milestone

Release 2018.04

Development

Successfully merging this pull request may close these issues.

2 participants

@cladmi @aabadie