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

chore: makefile linking fix #197

Merged
merged 1 commit into from
Oct 14, 2021
Merged

chore: makefile linking fix #197

merged 1 commit into from
Oct 14, 2021

Conversation

Roznovjak
Copy link
Collaborator

make build throws error when the symbolic link already exists.
This fix forces the ln command to replace the existing link.

@Roznovjak Roznovjak requested a review from mrq1911 October 14, 2021 11:02
@codecov
Copy link

codecov bot commented Oct 14, 2021

Codecov Report

Merging #197 (51616ad) into master (edec51c) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #197   +/-   ##
=======================================
  Coverage   85.22%   85.22%           
=======================================
  Files          21       21           
  Lines        2132     2132           
=======================================
  Hits         1817     1817           
  Misses        315      315           

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 edec51c...51616ad. Read the comment docs.

@jak-pan jak-pan changed the title fix: makefile chore: makefile Oct 14, 2021
@jak-pan jak-pan merged commit 431ce5e into master Oct 14, 2021
@Roznovjak Roznovjak deleted the fix/makefile branch October 14, 2021 21:37
@jak-pan jak-pan changed the title chore: makefile chore: makefile linking fix Oct 14, 2021
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