Skip to content

pkg: tools: ensure cmake based packages/tools are built out-of-source#14286

Merged
kaspar030 merged 8 commits intoRIOT-OS:masterfrom
aabadie:pr/pkg/cmake_build_source_dir
Jun 23, 2020
Merged

pkg: tools: ensure cmake based packages/tools are built out-of-source#14286
kaspar030 merged 8 commits intoRIOT-OS:masterfrom
aabadie:pr/pkg/cmake_build_source_dir

Conversation

@aabadie
Copy link
Copy Markdown
Contributor

@aabadie aabadie commented Jun 15, 2020

Contribution description

This PR adapt all packages and tools that are built using CMake so that source files and generated files are not stored in the same directory. Affected packages are ccn-lite, cryptoauthlib, jerryscript and relic. Affected tool is only flatc.

The jerryscript Makefile is also reworked a bit: all cmake related target are in the main package makefile, instead of another sub-makefile.

Testing procedure

  • A green Murdock with tests enabled should be enough
  • Functional tests of affected packages would be better

Issues/PRs references

Depends on #14268

@aabadie aabadie added Area: build system Area: Build system Area: pkg Area: External package ports CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Area: tools Area: Supplementary tools labels Jun 15, 2020
@aabadie aabadie requested a review from PeterKietzmann as a code owner June 15, 2020 08:01
@aabadie aabadie changed the title pkg: tools: ensure cmake based packages are built out-of-source pkg: tools: ensure cmake based packages/tools are built out-of-source Jun 15, 2020
@aabadie aabadie force-pushed the pr/pkg/cmake_build_source_dir branch from f628ff5 to f869354 Compare June 19, 2020 12:24
@aabadie
Copy link
Copy Markdown
Contributor Author

aabadie commented Jun 19, 2020

rebased

Copy link
Copy Markdown
Contributor

@kaspar030 kaspar030 left a comment

Choose a reason for hiding this comment

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

ACK.
Tested local build of all affected packages.
PKG_SOURCE_LOCAL still works as intended.

@kaspar030 kaspar030 merged commit b4e2684 into RIOT-OS:master Jun 23, 2020
@aabadie aabadie deleted the pr/pkg/cmake_build_source_dir branch June 23, 2020 14:18
@miri64 miri64 added this to the Release 2020.07 milestone Jun 24, 2020
@miri64 miri64 added the Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation label Jul 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: build system Area: Build system Area: pkg Area: External package ports Area: tools Area: Supplementary tools CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants