SPM: SDWebImage.git could not be resolved

[RyuX51]

When trying to add the framework via Swift Package Manager I get the error: target 'SDWebImageMapKit' referenced in product 'SDWebImageMapKit' is empty.
Edit: This only happens in Xcode 11.4 beta. I just tried in in Xcode 11.3.1 and it works there. Maybe you want to ignore this issue.

[dreampiggy]

Please fill the issue template ? At least provide SDWebImage version.

New Issue Checklist

• I have read and understood the CONTRIBUTING guide
• I have read the Documentation
• I have searched for a similar issue in the project and found none

Issue Info

Info	Value
Platform Name	e.g. ios / macos / tvos / watchos
Platform Version	e.g. 12.0 / 10.14.0 / 12.0 / 5.0
SDWebImage Version	e.g. 5.0.0 / 4.4.0
Integration Method	e.g. carthage / cocoapods / manually
Xcode Version	e.g. Xcode 11 / Xcode 10
Repro rate	e.g. all the time (100%) / sometimes x% / only once
Repro with our demo prj	e.g. does it happen with our demo project?
Demo project link	e.g. link to a demo project that highlights the issue

Issue Description and Steps

Please fill in the detailed description of the issue (full output of any stack trace, compiler error, ...) and the steps to reproduce the issue.

[dreampiggy]

The Package.swift syntax match the official Swift PM standard, which does not use any hacks or tricks. If this does not works, it's Apple's Xcode bug. Currently I can not workaround this.

Maybe we need to fire Radars: https://feedbackassistant.apple.com/

[maundytime]

Same issue, I use sub-xcode-project for now. SDWebImage's guide is very clear, thanks.

https://github.com/SDWebImage/SDWebImage/wiki/Installation-Guide#using-sdwebimage-as-sub-xcode-project

[dreampiggy]

Did you fired radar ? If not, I fire one.

If we don't fire radar, maybe Apple's Xcode team does not realize they break SwiftPM's standard and will ship this behavior in official Xcode 11.4.

[dreampiggy]

Fired FB7571208

[kylebrowning]

https://github.com/apple/swift-evolution/blob/master/proposals/0273-swiftpm-conditional-target-dependencies.md

[dreampiggy]

@kylebrowning Hi. Does this proposal have some in-compatbile issue with Swift-5.1 Package.swift behavior ?

I read the poposal, it's about the conditional dependency (Xcode configuration or platform). Does this related to this issue ?

For example, do we need to turn SDWebImageMapKit target, to dependency the Product type of SDWebImage, but not Library type of SDWebImage ?

[dreampiggy]

Another more radical solution. We can move the total MapKit related thing, into a standalone Git Repo.

Just like all what we've done in SDWebImagePhotosPlugin, SDWebImageLinkPlugin, and Coder Plugins.

The MapKit seems not a core feature of SDWebImage, not so many people use that dependency.

[kylebrowning]

I thought it may be of relevance, but I suppose not.

@dreampiggy I would vote for a standalone repo as you've suggested.

[dreampiggy]

@kylebrowning @RyuX51 @maundytime Does Xcode 11.4 Beta 2 solve this issue ?
https://developer.apple.com/documentation/xcode_release_notes/xcode_11_4_beta_2_release_notes?language=objc

If not, maybe it's time to totally move the MapKit integration into another Git Repo. I can do this because this need new Git Repo creation of SDWebImage GitHub organization.

[kiliankoe]

11.4 Beta 2 unfortunately didn't resolve the issue in my tests.

[RyuX51]

@dreampiggy Unfortunately it doesn't. Still getting target 'SDWebImageMapKit' referenced in product 'SDWebImageMapKit' is empty.