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

Fix macOS linker warnings #6095

Draft
wants to merge 1 commit into
base: develop
Choose a base branch
from
Draft

Conversation

fasmat
Copy link
Member

@fasmat fasmat commented Jul 3, 2024

Motivation

This should fix linker warnings we keep seeing on macOS.

Merge after spacemeshos/post#297

Description

Update post library and remove duplicate LDFLAGS definition in Makefile.

Test Plan

  • existing tests pass.

TODO

  • Explain motivation or link existing issue(s)
  • Test changes and document test plan
  • Update documentation as needed
  • Update changelog as needed

@fasmat fasmat self-assigned this Jul 3, 2024
Copy link

codecov bot commented Jul 3, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 79.7%. Comparing base (517c305) to head (dc10746).

Additional details and impacted files
@@            Coverage Diff            @@
##           develop   #6095     +/-   ##
=========================================
- Coverage     79.7%   79.7%   -0.1%     
=========================================
  Files          328     328             
  Lines        42904   42904             
=========================================
- Hits         34224   34208     -16     
- Misses        6752    6764     +12     
- Partials      1928    1932      +4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@fasmat fasmat force-pushed the fix-macos-linker-warnings branch from 736726e to e1e110a Compare July 4, 2024 15:17
@fasmat fasmat force-pushed the fix-macos-linker-warnings branch from e1e110a to ddbcb0a Compare July 13, 2024 23:41
@fasmat
Copy link
Member Author

fasmat commented Jul 13, 2024

Since the fix only works after we update our CI runners, I'll set this to draft for now.

@fasmat fasmat marked this pull request as draft July 13, 2024 23:41
@fasmat fasmat force-pushed the fix-macos-linker-warnings branch from ddbcb0a to da49306 Compare July 18, 2024 09:04
@fasmat fasmat force-pushed the fix-macos-linker-warnings branch 4 times, most recently from 2db5f46 to e5ba3e0 Compare August 1, 2024 15:04
@fasmat fasmat force-pushed the fix-macos-linker-warnings branch 5 times, most recently from 14f3cab to 765544a Compare August 8, 2024 15:15
@fasmat fasmat force-pushed the fix-macos-linker-warnings branch 6 times, most recently from 3a5dcf9 to f214e52 Compare August 17, 2024 12:26
@fasmat fasmat force-pushed the fix-macos-linker-warnings branch 3 times, most recently from ea3c77c to baa7aa7 Compare August 23, 2024 10:33
@fasmat fasmat force-pushed the fix-macos-linker-warnings branch 2 times, most recently from dd166e1 to 9d6c587 Compare August 29, 2024 19:41
@fasmat fasmat force-pushed the fix-macos-linker-warnings branch 2 times, most recently from 6f46f2b to 193a736 Compare October 9, 2024 08:24
@fasmat fasmat force-pushed the fix-macos-linker-warnings branch 2 times, most recently from 78dc906 to f9d0cf5 Compare October 15, 2024 18:53
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