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.

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

cmake: Output executables to top-level /bin dir #601

Merged
merged 1 commit into from
May 21, 2021
Merged

Conversation

chfast
Copy link
Member

@chfast chfast commented May 21, 2021

This outputs all EVMC executables in top-level /bin build dir. So for parent projects like evmone e.g. evmc will be located in build/bin/evmc (build/evmc/bin/evmc previously).

The output dir can be changes with standard -DCMAKE_RUNTIME_OUTPUT_DIRECTORY.

evmone check: ethereum/evmone#330.

@chfast chfast force-pushed the cmake_output_bin_dir branch from 23110dd to c202aad Compare May 21, 2021 11:40
@chfast chfast force-pushed the cmake_output_bin_dir branch from c202aad to e28c15c Compare May 21, 2021 11:45
@chfast chfast requested a review from axic May 21, 2021 11:45
@codecov
Copy link

codecov bot commented May 21, 2021

Codecov Report

Merging #601 (e28c15c) into master (de0c5d1) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #601   +/-   ##
=======================================
  Coverage   95.68%   95.68%           
=======================================
  Files          24       24           
  Lines        3942     3942           
=======================================
  Hits         3772     3772           
  Misses        170      170           

@chfast chfast marked this pull request as ready for review May 21, 2021 12:35
@chfast chfast merged commit 38e65fc into master May 21, 2021
@chfast chfast deleted the cmake_output_bin_dir branch May 21, 2021 17:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant