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

Bump minimum cmake and stop creating build directory #2622

Merged
merged 3 commits into from
May 17, 2023

Conversation

Ryanf55
Copy link
Contributor

@Ryanf55 Ryanf55 commented May 16, 2023

This upgrades the cmake version to match the version Ubuntu 20.04 now that Ubuntu 18.04 is EOL.

I updated the build instructions to use the method that doesn't require you to delete, create and change directories into a build directory to compile and test the code. The other advantage is you can now have multiple build directories while continuing to work in the same root directory.

Closes #2617

Signed-off-by: Ryan Friedman <ryanfriedman5410+github@gmail.com>
@ghost
Copy link

ghost commented May 16, 2023

👇 Click on the image for a new way to code review

Review these changes using an interactive CodeSee Map

Legend

CodeSee Map legend

README.md Outdated Show resolved Hide resolved
@codecov
Copy link

codecov bot commented May 16, 2023

Codecov Report

Merging #2622 (34fddc8) into main (c319699) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #2622   +/-   ##
=======================================
  Coverage   63.93%   63.93%           
=======================================
  Files         103      103           
  Lines       22296    22298    +2     
  Branches    10787    10789    +2     
=======================================
+ Hits        14255    14257    +2     
  Misses       5823     5823           
  Partials     2218     2218           
Impacted Files Coverage Δ
src/bmffimage.cpp 72.67% <100.00%> (+0.11%) ⬆️

Signed-off-by: Ryan Friedman <ryanfriedman5410+github@gmail.com>
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@kmilos kmilos added the CMake Configuration issues related with CMake label May 17, 2023
Signed-off-by: Ryan Friedman <ryanfriedman5410+github@gmail.com>
@neheb neheb merged commit 445757a into Exiv2:main May 17, 2023
108 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CMake Configuration issues related with CMake
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Consider bumping CMake minimum version to 3.13 for easier build workflow
3 participants