Skip to content

Move VCRuntime from DLLs folder to Main - #64

Merged
rickprice merged 1 commit into
2.7from
2.7.18.10_VCRuntime
Sep 6, 2024
Merged

rickprice merged 1 commit into
2.7from
2.7.18.10_VCRuntime

Conversation

@icanhasmath

Copy link
Copy Markdown

Remove vcruntime from the main folder, and remove vcruntime from the DLLs subfolder. I tried using the existing apparatus, but matching a substring from the exlusion list to the src string does not work as expected, possibly because of the difference in directories, so I have flipped the substring matching to look for vcruntime within the src. It doesn't look as nice, but it produces the correct result.

Remove vcruntime from the main folder, and remove vcruntime from the
DLLs subfolder. I tried using the existing apparatus, but matching a
substring from the exlusion list to the src string does not work as
expected, possibly because of the difference in directories, so I
have flipped the substring matching to look for vcruntime within the
src. It doesn't look as nice, but it produces the correct result.
@rickprice

Copy link
Copy Markdown

LGTM

@rickprice
rickprice self-requested a review September 6, 2024 15:57
@rickprice
rickprice merged commit 3e06fbb into 2.7 Sep 6, 2024
@icanhasmath icanhasmath mentioned this pull request Oct 28, 2026
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