-
Notifications
You must be signed in to change notification settings - Fork 177
Fix iOS deployment target #3040
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
base: master
Are you sure you want to change the base?
Conversation
|
@copilot Makefile에서도 target을 수정해 줘. 그리고 앞으로는 다른 사람들도 볼 수 있게 내 프롬프트를 영어로 번역하고 답변은 한국어와 영어로 작성해 줘. |
Korean (한국어): English: Translation of your request: |
|
Doesn't compile |
|
@copilot 빌드가 안되는데, 워크플로우 로그를 확인하고 이 변경과 관련된 것이라면 수정해 줘. |
Korean (한국어): English: Translation of original issue:
Korean (한국어):
배포 타겟을 원래 값으로 되돌리고, deprecated API를 먼저 수정한 후 배포 타겟을 변경하는 것이 좋습니다. English:
I'll revert the deployment target to original values. The deprecated APIs should be fixed first before changing the deployment target. Translation of your request: |
|
@copilot 되돌리지 말고 코드를 고쳐 |
Korean (한국어):
이제 iOS 12.0 배포 타겟으로 빌드가 성공할 것입니다. (커밋: 953b765) English:
The build should now succeed with iOS 12.0 deployment target. (commit: 953b765) Translation of your request: |
bear101
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Have you tested any of these changes?
The build was successful and runs fine, but I'm still working on fixing it. |
6e756f7 to
4344f9f
Compare
c06515f to
80d2821
Compare
4eb73f4 to
29535b3
Compare
| @objc init(coder dec: NSCoder!) { | ||
| name = dec.decodeObject(forKey: "name") as! String | ||
| ipaddr = dec.decodeObject(forKey: "ipaddr") as! String | ||
| required init?(coder dec: NSCoder) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm getting a bunch of these warnings when I start:
*** -[NSKeyedUnarchiver validateAllowedClass:forKey:] allowed unarchiving safe plist type ''NSString' (0x20a2e5868) [/System/Library/Frameworks/Foundation.framework]' for key 'certdata', even though it was not explicitly included in the client allowed classes set: '{(
"'iTeamTalk.Server' (0x102f35118) [/private/var/containers/Bundle/Application/17F1C324-547A-4B5A-8182-BFB80EAD1584/iTeamTalk.app]"
)}'. This will be disallowed in the future.
They don't appear on master-branch
258939f to
ea9055d
Compare
Co-Authored-By: hwangsihu <[email protected]>
macos-latest does not target the latest macOS (26), so it has SDK-related issues.
ea9055d to
132d385
Compare
The iOS project had a malformed Russian localization file causing property list validation failures, and the iOS deployment target was set to 9.0 (Xcode) and 8.0 (CMake), below the minimum supported version (12.0-26.1.99).
Changes
Russian Localization Fix
ru.lproj/Localizable.stringsline 431 with extra quote characteriOS Deployment Target Update
IPHONEOS_DEPLOYMENT_TARGETfrom 9.0 to 12.0 in all 4 build configurationsCMAKE_OSX_DEPLOYMENT_TARGETfrom 8.0 to 12.0 for iOS buildsDeprecated API Updates
To support iOS 12.0 deployment target, all deprecated APIs have been replaced with modern alternatives:
UtilSound.swift:
.allowBluetoothHFPand.bluetoothHighQualityRecording(deprecated iOS 13.0) with.allowBluetoothServerListViewController.swift:
NSKeyedUnarchiver.unarchiveObject(with:)withNSKeyedUnarchiver.unarchivedObject(ofClass:from:)NSKeyedArchiver.archivedData(withRootObject:)withNSKeyedArchiver.archivedData(withRootObject:requiringSecureCoding:)NSSecureCodingconformance toServerclass with secure coding enabledWebLoginViewController.swift:
UIAlertView(deprecated iOS 9.0) withUIAlertController(2 occurrences)UIApplication.shared.openURL()(deprecated iOS 10.0) withUIApplication.shared.open()ChannelDetailViewController.swift:
UIAlertViewandUIAlertViewDelegatewithUIAlertControllerwith text field supportMainTabBarController.swift:
UIAlertViewandUIAlertViewDelegatewithUIAlertControllerwith text field supportAll changes ensure iOS 12.0+ compatibility while maintaining functionality and improving security.
Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.