atlantis icon indicating copy to clipboard operation
atlantis copied to clipboard

Support Carthage

Open NghiaTranUIT opened this issue 5 years ago • 8 comments

Description

As we've already supported Cocoapod and Swift Package Manager, we should also support Carthage 👍

NghiaTranUIT avatar Nov 01 '20 06:11 NghiaTranUIT

@NghiaTranUIT Not sure what is the status of that since README does mention Carthage https://github.com/ProxymanApp/atlantis#carthage

Regardless, it fails for me, please see details below.

Steps:

  1. echo 'github "ProxymanApp/atlantis"' > Cartfile

  2. carthage update --use-xcframeworks --platform iOS

Result:

*** Fetching atlantis
*** Checking out atlantis at "1.12.0"
*** xcodebuild output can be found in /var/folders/qw/5y40n2k973b2hklwn2_6rqqm0000gn/T/carthage-xcodebuild.GEP4ta.log
*** Skipped building atlantis due to the error:
Dependency "atlantis" has no shared framework schemes for any of the platforms: iOS

If you believe this to be an error, please file an issue with the maintainers at https://github.com/ProxymanApp/atlantis/issues/new

pokryfka avatar Nov 08 '21 23:11 pokryfka

👋

pokryfka avatar Jan 31 '22 00:01 pokryfka

I found this issue, too. Please support Carthage. I don't want to increase build time by adding more SPM packages.

saim80 avatar Jun 14 '22 19:06 saim80

Hey @saim80 @pokryfka since Atlantis is a pure SPM (no xcproject file), so Atlantis doesn't work well.

@saim80 please try this branch instead

github "ProxymanApp/atlantis" "feat/add-carthage"

I generated the xcproject file and it should work 👍

NghiaTranUIT avatar Jun 15 '22 02:06 NghiaTranUIT

It's quite tricky to set up Carthage work with xcframework, so I still highly suggest using the Swift Package Manager because it's straightforward to integrate Atlantis into your project.

NghiaTranUIT avatar Jun 15 '22 02:06 NghiaTranUIT

This is marked 'done', and the README has Carthage instructions, but I still get this error message. Here's my Cartfile line:

github "ProxymanApp/atlantis" == 1.18.0

Your branch feat/add-carthage works. Can it be merged?

stevelandeyasana avatar Jul 15 '22 21:07 stevelandeyasana

@stevelandeyasana It's merged and released on Atlantis 1.18.1 👍 : https://github.com/ProxymanApp/atlantis/releases/tag/1.18.1

NghiaTranUIT avatar Jul 16 '22 02:07 NghiaTranUIT

Thank you so much, works great!

stevelandeyasana avatar Jul 19 '22 17:07 stevelandeyasana