Linking: Remove link_SDL2*
methods, update README, and fix race condition
#26
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I've split this out from #25 to compare/contrast the approach.
Changes:
build.zig
: The run step did not depend on install step, causing the tests to be run fromzig-cache
instead ofzig-out
. That meant that even with a correct rpath, it would still fail.link_SDL2*
methods: The methods cause a compile error when used directing the user to the readme.When using any of the old
link_SDL2*
methods, the user will see this:To Do: