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

Add log message containing Xcode version #4863

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

Conversation

low-batt
Copy link
Contributor

@low-batt low-batt commented Apr 2, 2024

This commit will:

  • Add a key com.colliderli.iina.build.sdk to Info.plist with a value giving the macOS SDK version used when generating the executable
  • Add a key com.colliderli.iina.build.xcode to Info.plist with a value giving the Xcode version used when generating the executable
  • Add buildSDK and buildXcode properties to InfoDictionary to provide access to this information
  • Add logging of the Xcode and macOS SDK versions to logBuildDetails in AppDelegate

This information is useful as AppKit behavior can change based on the version of the macOS SDK an application was built with.


Description:

This commit will:
- Add a key com.colliderli.iina.build.sdk to Info.plist with a value
  giving the macOS SDK version used when generating the executable
- Add a key com.colliderli.iina.build.xcode to Info.plist with a value
  giving the Xcode version used when generating the executable
- Add buildSDK and buildXcode properties to InfoDictionary to provide
  access to this information
- Add logging of the Xcode and macOS SDK versions to logBuildDetails in
  AppDelegate

This information is useful as AppKit behavior can change based on the
version of the macOS SDK an application was built with.
@low-batt
Copy link
Contributor Author

low-batt commented Apr 2, 2024

The startup log message now looks like:

21:18:42.352 [iina][d] Built using Xcode 15.2 and macOS SDK 14.2 on Apr 1, 2024 at 9:18:39 PM
21:18:42.352 [iina][d] From branch xcode-version, commit 73c960842cc0002eaa0783f893802056f7638c50

@low-batt low-batt self-assigned this Apr 2, 2024
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.

None yet

1 participant