I created a pull request here:

https://github.com/apple/swift/pull/3476


> On Jul 12, 2016, at 1:05 PM, Mishal Shah via swift-dev <swift-dev@swift.org> 
> wrote:
> 
> We are also seeing these failures in swift-3.0-preview-3-branch, can we pull 
> this in to swift-3.0-preview-3-branch?
> 
> https://ci.swift.org/view/swift-3.0-preview-3/job/oss-swift-3.0-preview-3-incremental-RA-osx/34/console
> 
> Thanks, 
> Mishal Shah
>> On Jul 12, 2016, at 12:44 PM, Devin Coughlin via swift-dev 
>> <swift-dev@swift.org> wrote:
>> 
>>> 
>>> On Jul 12, 2016, at 11:56 AM, Manman <m...@apple.com> wrote:
>>> 
>>>> 
>>>> On Jul 12, 2016, at 11:51 AM, Devin Coughlin <dcough...@apple.com> wrote:
>>>> 
>>>>> 
>>>>> On Jul 12, 2016, at 11:10 AM, Manman via swift-dev <swift-dev@swift.org> 
>>>>> wrote:
>>>>> 
>>>>> 
>>>>>> On Jul 12, 2016, at 10:14 AM, Jordan Rose <jordan_r...@apple.com> wrote:
>>>>>> 
>>>>>> It’s possible we’re not treating “macos” the same as “macosx”, and thus 
>>>>>> changing the way that methods with the same basename get imported. So it 
>>>>>> probably is that change.
>>>>>> 
>>>>>> …yep, it looks like we look for literal platform name strings in 
>>>>>> ClangImporter.cpp and ImportDecl.cpp.
>>>>> 
>>>>> So we should just change the string match from “macosx” to “macos” in 
>>>>> ClangImporter.cpp.
>>>>> Are you working on fixing this? Or should I look into it? I am still 
>>>>> trying to get a successful local build of swift.
>>>> 
>>>> I have a fix for this that I’ve been working on to fix this for 
>>>> master-next — I’ll pull it into master instead.
>>> 
>>> Thanks Devin!
>>> 
>>> @Adrian, thanks for offering to help, Devin will fix the bot failure caused 
>>> by my change.
>> 
>> Pushed to master in 4b566df9f3eeb8a5255afcd265722dd9e8fc37d0
>> 
>> Devin
>> 
>> _______________________________________________
>> swift-dev mailing list
>> swift-dev@swift.org
>> https://lists.swift.org/mailman/listinfo/swift-dev
> 
> _______________________________________________
> swift-dev mailing list
> swift-dev@swift.org
> https://lists.swift.org/mailman/listinfo/swift-dev

_______________________________________________
swift-dev mailing list
swift-dev@swift.org
https://lists.swift.org/mailman/listinfo/swift-dev

Reply via email to