-
Notifications
You must be signed in to change notification settings - Fork 510
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
[Versions-ios.plist] watchOS started at 1.0 #1328
Conversation
VincentDondain
commented
Dec 9, 2016
- The WatchKit App on a watchOS 1 project has a version of 1.0 (the IDE needs that).
- We still have a watchOS 1 template.
Build failure |
Build failure |
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 don't think this is right, because code-wise we only care about WatchKit 2 extensions (since WatchKit 1 extensions are technically iOS extensions, not watchOS extensions).
IMHO the IDEs should hard-code whatever they need for WatchKit 1 extensions (also because nothing will ever change there, so hard-coding is still future-proof).
- The WatchKit App on a watchOS 1 project has a version of 1.0 (the IDE needs that). - We still have a watchOS 1 template. - Use MIN_WATCH_OS_VERSION instead of MIN_WATCHOS_SDK_VERSION. - iOS com.apple.watchkit is 8.2 not 8.0 - Added watchOS com.apple.watchkit (2.0). - Fixed typo in versions-check.csharp.
4284e9d
to
bb11c53
Compare
@rolfbjarne updated all the things, it should match our conversation, please re-check (; |
Build failure |
build |
Build failure |
Test failure is unrelated (ibtool hang due to bug #46097). |
- The WatchKit App on a watchOS 1 project has a version of 1.0 (the IDE needs that). - We still have a watchOS 1 template. - Use MIN_WATCH_OS_VERSION instead of MIN_WATCHOS_SDK_VERSION. - iOS com.apple.watchkit is 8.2 not 8.0 - Added watchOS com.apple.watchkit (2.0). - Fixed typo in versions-check.csharp.
- The WatchKit App on a watchOS 1 project has a version of 1.0 (the IDE needs that). - We still have a watchOS 1 template. - Use MIN_WATCH_OS_VERSION instead of MIN_WATCHOS_SDK_VERSION. - iOS com.apple.watchkit is 8.2 not 8.0 - Added watchOS com.apple.watchkit (2.0). - Fixed typo in versions-check.csharp.
Commit list for xamarin/maccore: * xamarin/maccore@83bd122dd3 [install-qa-provisioning-profiles] Make sure we find pencil_pusher. (#1328) * xamarin/maccore@2df4e9934d Merge pull request #1327 from xamarin/swift-o-matic-override-init * xamarin/maccore@a95796666b Bump to ignore stubbed constructors. Diff: xamarin/maccore@6324465...83bd122
* Bump maccore to get fix for maccore#1325. Commit list for xamarin/maccore: * xamarin/maccore@6324465797 [install-qa-provisioning-profiles] Make sure we find ruby and its related tools. (#1324) [...] Many irrelevant SoM commits [...] Diff: xamarin/maccore@e3d92b5...6324465 * Bump maccore to get an additional fix. Commit list for xamarin/maccore: * xamarin/maccore@83bd122dd3 [install-qa-provisioning-profiles] Make sure we find pencil_pusher. (#1328) * xamarin/maccore@2df4e9934d Merge pull request #1327 from xamarin/swift-o-matic-override-init * xamarin/maccore@a95796666b Bump to ignore stubbed constructors. Diff: xamarin/maccore@6324465...83bd122