Skip to content

fix: #1144 by forcing misc_tools to be a static lib#2341

Merged
sudden6 merged 1 commit intoTokTok:masterfrom
Green-Sky:fix_misc_tools_extra_so_on_dyn_builds
Dec 8, 2022
Merged

fix: #1144 by forcing misc_tools to be a static lib#2341
sudden6 merged 1 commit intoTokTok:masterfrom
Green-Sky:fix_misc_tools_extra_so_on_dyn_builds

Conversation

@Green-Sky
Copy link
Member

@Green-Sky Green-Sky commented Oct 4, 2022

not sure if this works as intended, but i saw a .patch file in the fedora pkg source and was displeased.
an alternative would be to use a cmake OBJECT library.


This change is Reviewable

@auto-add-label auto-add-label bot added the bug Bug fix for the user, not a fix to a build script label Oct 4, 2022
@Green-Sky Green-Sky marked this pull request as ready for review October 7, 2022 12:08
@Green-Sky Green-Sky force-pushed the fix_misc_tools_extra_so_on_dyn_builds branch from 39f5c24 to 2187893 Compare October 7, 2022 12:08
@JFreegman JFreegman added this to the v0.2.19 milestone Oct 7, 2022
Copy link

@sudden6 sudden6 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm_strong: please rebase so it can be merged

Reviewed 1 of 1 files at r1, all commit messages.
Reviewable status: :shipit: complete! 1 of 1 approvals obtained

@Green-Sky Green-Sky force-pushed the fix_misc_tools_extra_so_on_dyn_builds branch from 2187893 to e1c38b4 Compare December 8, 2022 13:43
@codecov
Copy link

codecov bot commented Dec 8, 2022

Codecov Report

Merging #2341 (e1c38b4) into master (8738960) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master   #2341      +/-   ##
=========================================
- Coverage    9.95%   9.94%   -0.02%     
=========================================
  Files         141     141              
  Lines       32503   32503              
=========================================
- Hits         3236    3232       -4     
- Misses      29267   29271       +4     
Impacted Files Coverage Δ
toxcore/TCP_client.c 5.66% <0.00%> (-0.23%) ⬇️
toxcore/TCP_connection.c 10.37% <0.00%> (-0.14%) ⬇️
toxcore/group_chats.c 7.24% <0.00%> (-0.08%) ⬇️
auto_tests/tox_many_test.c 13.55% <0.00%> (+1.69%) ⬆️

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

@sudden6 sudden6 merged commit e1c38b4 into TokTok:master Dec 8, 2022
@Green-Sky Green-Sky deleted the fix_misc_tools_extra_so_on_dyn_builds branch March 8, 2023 12:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Bug fix for the user, not a fix to a build script

Development

Successfully merging this pull request may close these issues.

3 participants