martin-olivier / dylib

C++ cross-platform wrapper around dynamic loading of shared libraries (dll, so, dylib)

https://conan.io/center/recipes/dylib

MIT License

293 stars 44 forks source link

fix: mingw #64

Closed martin-olivier closed 2 years ago

martin-olivier commented 2 years ago

Signed-off-by: Martin Olivier martin.olivier@live.fr Fixes https://github.com/martin-olivier/dylib/issues/62

Description

Fixed build with MinGW by changing shared library prefix back to Windows' default. Added MinGW build and tests in the CI

Changes include

[X] Bugfix (non-breaking change that solves an issue)
[ ] New feature (non-breaking change that adds functionality)
[ ] Breaking change (change that is not backwards-compatible and/or changes current functionality)

Checklist

[X] I have assigned this PR to myself
[X] I have added at least 1 reviewer
[X] I have tested this code
[X] I have added sufficient documentation in the code

codecov[bot] commented 2 years ago

Codecov Report

Merging #64 (cc3c204) into main (f951429) will not change coverage. The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #64   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            1         1           
  Lines           59        59           
  Branches         3         3           
=========================================
  Hits            59        59

Impacted Files	Coverage Δ
include/dylib.hpp	`100.00% <ø> (ø)`

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.