[swift-evolution] [DRAFT] Aliasing the OS X Platform Configuration Test

[Charlie Monroe]

Have you considered just #if os(mac)? I mean the fact that you're testing against "OS" is already expressed. I know it goes against the other values (watchOS, tvOS), but I'd personally drop those as well in favor of simplier os(watch), os(tv).

This would go along the various Swift 3 improvements where e.g. NSFileManager.defaultManager() became simply FileManager.default (thus dropping the unnecessary "Manager")...

But I understand if you're against it, considering that these are really trademarks and official OS names...


> On Jun 16, 2016, at 4:29 PM, Erica Sadun via swift-evolution <swift-evolution at swift.org> wrote:
> 
> Starting in Sierra, Apple's Mac-based OS is renamed to macOS.
> 
> All user-facing Swift APIs must go through Swift Evolution. While this is a trivial API change, I have put together a formal proposal as is normal and usual for this process. Here is a draft for public comment.
> 
>  -- Erica
> 
> Gist:  https://gist.github.com/erica/f53fa6cfef9e5cf17ab139f7528edde2 <https://gist.github.com/erica/f53fa6cfef9e5cf17ab139f7528edde2>
> 
> Aliasing the OS X Platform Configuration Test
> 
> Proposal: TBD
> Author: Erica Sadun <http://github.com/erica>
> Status: TBD
> Review manager: TBD
>  <https://gist.github.com/erica/f53fa6cfef9e5cf17ab139f7528edde2#introduction>Introduction
> 
> This proposal adds a #if os(macos) platform configuration test to Swift that acts like the current if os(osx)
> 
> Swift Evolution Discussion: TBD <applewebdata://591A7BC6-6FAB-4AA8-83B9-6A6B7D0E1FCA>
>  <https://gist.github.com/erica/f53fa6cfef9e5cf17ab139f7528edde2#motivation>Motivation
> 
> Apple renamed its Mac operating system from OSX to macOS, starting in macOS Sierra. Adding rather than replacing "OSX" enables this adoption to be purely additive and supports the notion that Swift-based applications could be deployed to operating systems earlier than Sierrra.
> 
>  <https://gist.github.com/erica/f53fa6cfef9e5cf17ab139f7528edde2#current-art>Current Art
> 
> Swift currently supports the following platform configuration tests, defined in lib/Basic/LangOptions.cpp.
> 
> The literals true and false
> The os() function that tests for OSX, iOS, watchOS, tvOS, Linux, Windows, Android, and FreeBSD
> The arch() function that tests for x86_64, arm, arm64, i386, powerpc64, s390x, and powerpc64le
> The swift() function that tests for specific Swift language releases, e.g. swift(>=2.2)
>  <https://gist.github.com/erica/f53fa6cfef9e5cf17ab139f7528edde2#detailed-design>Detailed Design
> 
>   if (Target.isMacOSX()) {
>     addPlatformConditionValue("os", "OSX");
>     addPlatformConditionValue("os", "macOS");
>   }
>  <https://gist.github.com/erica/f53fa6cfef9e5cf17ab139f7528edde2#impact-on-existing-code>Impact on Existing Code
> 
> This is purely additive
> 
>  <https://gist.github.com/erica/f53fa6cfef9e5cf17ab139f7528edde2#alternatives-considered>Alternatives Considered
> 
> I think it's unwise to replace OSX instead of adding macOS due to Apple's large and active userbase that may install Swift-sourced software and not upgrade to the latest operating system.
> 
> 
> _______________________________________________
> swift-evolution mailing list
> swift-evolution at swift.org
> https://lists.swift.org/mailman/listinfo/swift-evolution

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.swift.org/pipermail/swift-evolution/attachments/20160616/d424fe66/attachment.html>