Skip to content

Conversation

@JustArchi
Copy link
Contributor

See Homebrew/homebrew-core#206331 (comment) for detailed reasoning and analysis.

If you could schedule a bugfix release with it (3.0.2), I'd appreciate that. Thanks in advance.

References:

@JustArchi JustArchi changed the title Fix macOS incompatibility related to IOKit Fix macOS incompatibilies related to system libraries Feb 2, 2025
@JustArchi
Copy link
Contributor Author

Added unit test and fixed another occurances of the same issue. Unit test was failing before, so it definitely proves that there was an issue.

Thanks in advance for merging.

@yaakov-h
Copy link
Member

yaakov-h commented Feb 3, 2025

By reverting the changes to the library paths and keeping the tests, I cannot get this to fail on my own MBP with macOS Sequoia 15.3 (24D60) with either .NET 8 or .NET 9 SDKs. 🤔

@JustArchi
Copy link
Contributor Author

JustArchi commented Feb 3, 2025

I'm not an expert in macOS so I don't know why you can't reproduce it locally, but it definitely did fail in CI:

(Those were the other two issues that I discovered accidentally after writing unit test for verification of the first one)

@yaakov-h
Copy link
Member

yaakov-h commented Feb 3, 2025

Yeah, I saw that. I still have Questions.

Will have a think about this - seems harmless atm but not sure what the wider impact is exactly.

@yaakov-h
Copy link
Member

yaakov-h commented Feb 3, 2025

ah, got it.

This only seems to affect .NET 9.0.1 / SDK 9.0.102 or higher. Iiiiiinteresting.

@yaakov-h
Copy link
Member

yaakov-h commented Feb 3, 2025

If we have global.json for 8.0.100 latestMinor then why is dlopen on GitHub Actions looking in /Users/runner/.dotnet/shared/Microsoft.NETCore.App/9.0.1?

@yaakov-h
Copy link
Member

yaakov-h commented Feb 3, 2025

https://github.com/SteamRE/SteamKit/blob/master/SteamKit2/Tests/Tests.csproj#L5

sigh.

ok I think I've wrapped up the loose ends here. and I'm not exactly happy with where we are but I think a hotfix makes sense.

@yaakov-h yaakov-h merged commit f164e14 into SteamRE:master Feb 3, 2025
17 checks passed
@xPaw
Copy link
Member

xPaw commented Feb 3, 2025

rollforward was specified in csprojs before global.json existed
ad531d9

@JustArchi JustArchi deleted the patch-1 branch February 3, 2025 10:13
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.

3 participants