-
Notifications
You must be signed in to change notification settings - Fork 378
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
errors when trying to build with SPM #344
Comments
@mr-fixit I got a different error from branch
|
Bump! Any ideas on this? |
@mr-fixit Hi! Any ideas on this? |
i'm back to looking at this... no clues so far. |
i think this was fixed by pinterest/PINRemoteImage@93428e8 |
Also can't use a previous version because the Package file doesn't have manifests for previous versions |
I think SPM packaging of NYTPhotoViewer is going to be broken until PINRemoteImage can release a new version. Requested here: pinterest/PINRemoteImage#624 |
Sorry, I don't know what you're saying here. Is 'the Package file' a) package files in general, b) NYTPhotoViewer's, or c) PINRemoteImage's? Is 'the previous version' a) NYTPhotoViewer's, or b) PINRemoteImage's? |
One idea is to use NYTPhotoViewer via CocoaPods. |
Unfortunately we only rely on SPM as of today. |
I hear you. Put pressure on PINRemoteImage to fix #624. |
Thanks @mr-fixit I will probably do that now. |
@cassianodialpad reported errors when trying to build with SPM
#342 (comment)
i took a stab at fixing, see branch
bug/fix-build-error
haven't figured it out.The text was updated successfully, but these errors were encountered: