Skip to content

package import Foundation.NSData#231

Open
JanGorman wants to merge 1 commit into
pointfreeco:mainfrom
JanGorman:main
Open

package import Foundation.NSData#231
JanGorman wants to merge 1 commit into
pointfreeco:mainfrom
JanGorman:main

Conversation

@JanGorman

Copy link
Copy Markdown

This changes the package import Foundation in Deprecations.swift to a package import Foundation.NSData.

This solves the issue mentioned here #229 where the release build of our application was failing with errors like this one.

ambiguous implicit access level for import of 'Foundation'; it is imported as 'package' elsewhere
[11:36:26]: ▸ import class Foundation.Bundle

@mbrandonw

Copy link
Copy Markdown
Member

Hi @JanGorman, I think there is another solution. I think the problem is that the DocC bundle is being included in the package (there's even a warning in the logs about it), that causes a resource_bundle_accessor.swift file to be injected into the compilation with an import class Foundation.Bundle, and then that conflicts with our package import Foundation.

Can you point to the branch resource-docc-fix and see if it fixes it for you? There's a chance we may want both your fix and my fix, but first I would like to confirm that resource-docc-fix does fix it.

@JanGorman

Copy link
Copy Markdown
Author

Thanks @mbrandonw! I tested that branch on CI just now. Unfortunately it fails with same ambiguous implicit access level for import of 'Foundation'; it is imported as 'package' elsewhere error as before 😕

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.

2 participants