Skip to content

Conversation

@colemancda
Copy link
Contributor

Build as .so instead of .a on Linux for use across different libraries and executables.

@codecov
Copy link

codecov bot commented Aug 31, 2020

Codecov Report

Merging #200 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #200   +/-   ##
=======================================
  Coverage   73.71%   73.71%           
=======================================
  Files          43       43           
  Lines        2359     2359           
=======================================
  Hits         1739     1739           
  Misses        620      620           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 524de45...3b16403. Read the comment docs.

@MaxDesiatov
Copy link
Collaborator

Thanks for the PR. Could you clarify please how is this supposed to work if our Linux users want a static library? Last time I talked to SwiftPM folks, their opinion was that this should be controlled on library consumer side, not library vendor side, otherwise consumers lose all flexibility.

@MaxDesiatov
Copy link
Collaborator

With no updates from the OP I'm closing this as stale.

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.

2 participants