Skip to content
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

Open
mr-fixit opened this issue Jun 21, 2021 · 13 comments
Open

errors when trying to build with SPM #344

mr-fixit opened this issue Jun 21, 2021 · 13 comments

Comments

@mr-fixit
Copy link
Contributor

@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.

@cassianodialpad
Copy link

cassianodialpad commented Jun 21, 2021

@mr-fixit I got a different error from branch bug/fix-build-error

In file included from /Users/cassiano/Library/Developer/Xcode/DerivedData/Dialpad-dcrsnbdqvqefaudlnsocmkwonmef/SourcePackages/checkouts/NYTPhotoViewer/NYTPhotoViewerGIF/NYTPhotosViewController.m:15:
/Users/cassiano/Library/Developer/Xcode/DerivedData/Dialpad-dcrsnbdqvqefaudlnsocmkwonmef/SourcePackages/checkouts/NYTPhotoViewer/NYTPhotoViewerGIF/include/NYTScalingImageView.h:27:33: error: unknown type name 'PINAnimatedImageView'; did you mean 'PINAnimatedImage'?
@property (nonatomic, readonly) PINAnimatedImageView *imageView;
                                ^~~~~~~~~~~~~~~~~~~~
                                PINAnimatedImage
In module 'PINRemoteImage' imported from /Users/cassiano/Library/Developer/Xcode/DerivedData/Dialpad-dcrsnbdqvqefaudlnsocmkwonmef/SourcePackages/checkouts/NYTPhotoViewer/NYTPhotoViewerGIF/include/NYTScalingImageView.h:13:
/Users/cassiano/Library/Developer/Xcode/DerivedData/Dialpad-dcrsnbdqvqefaudlnsocmkwonmef/SourcePackages/checkouts/PINRemoteImage/Source/Classes/include/PINAnimatedImage.h:67:12: note: 'PINAnimatedImage' declared here
@interface PINAnimatedImage : NSObject
           ^
/Users/cassiano/Library/Developer/Xcode/DerivedData/Dialpad-dcrsnbdqvqefaudlnsocmkwonmef/SourcePackages/checkouts/NYTPhotoViewer/NYTPhotoViewerGIF/NYTPhotosViewController.m:133:20: warning: incompatible pointer types assigning to 'UIView *' from 'PINAnimatedImage *' [-Wincompatible-pointer-types]
        endingView = self.currentPhotoViewController.scalingImageView.imageView;
                   ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/Users/cassiano/Library/Developer/Xcode/DerivedData/Dialpad-dcrsnbdqvqefaudlnsocmkwonmef/SourcePackages/checkouts/NYTPhotoViewer/NYTPhotoViewerGIF/NYTPhotosViewController.m:429:22: warning: incompatible pointer types assigning to 'UIView *' from 'PINAnimatedImage *' [-Wincompatible-pointer-types]
        startingView = self.currentPhotoViewController.scalingImageView.imageView;
                     ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

/Users/cassiano/Library/Developer/Xcode/DerivedData/Dialpad-dcrsnbdqvqefaudlnsocmkwonmef/SourcePackages/checkouts/NYTPhotoViewer/NYTPhotoViewerGIF/include/NYTScalingImageView.h:27:33: Unknown type name 'PINAnimatedImageView'; did you mean 'PINAnimatedImage'?

@cassianodialpad
Copy link

Bump! Any ideas on this?

@cassianodialpad
Copy link

@mr-fixit Hi! Any ideas on this?

@mr-fixit
Copy link
Contributor Author

i'm back to looking at this... no clues so far.

@mr-fixit
Copy link
Contributor Author

i think this was fixed by pinterest/PINRemoteImage@93428e8
merged may 18, 2021
as yet unreleased

@Xendrez
Copy link

Xendrez commented Oct 25, 2022

Also can't use a previous version because the Package file doesn't have manifests for previous versions

@mr-fixit
Copy link
Contributor Author

I think SPM packaging of NYTPhotoViewer is going to be broken until PINRemoteImage can release a new version.

Requested here: pinterest/PINRemoteImage#624

@mr-fixit
Copy link
Contributor Author

Also can't use a previous version because the Package file doesn't have manifests for previous versions

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?

@PranotiK523
Copy link

PranotiK523 commented Nov 7, 2022

Hi, I'm currently noticing similar errors when i try to bump up the version to 5.0.8 via SPM, my project is using 5.0.6 as of today.
Screen Shot 2022-11-01 at 5 19 51 PM

Any ideas here, since this is avoiding us to use the latest version?

@mr-fixit
Copy link
Contributor Author

mr-fixit commented Nov 8, 2022

One idea is to use NYTPhotoViewer via CocoaPods.

@PranotiK523
Copy link

Unfortunately we only rely on SPM as of today.

@mr-fixit
Copy link
Contributor Author

mr-fixit commented Nov 8, 2022

I hear you. Put pressure on PINRemoteImage to fix #624.

@PranotiK523
Copy link

Thanks @mr-fixit I will probably do that now.

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

No branches or pull requests

4 participants