Skip to content

[AUTOMATIC] Release/4.17.1#2223

Merged
NachoSoto merged 1 commit into
mainfrom
release/4.17.1
Jan 18, 2023
Merged

[AUTOMATIC] Release/4.17.1#2223
NachoSoto merged 1 commit into
mainfrom
release/4.17.1

Conversation

@RCGitBot

Copy link
Copy Markdown
Contributor

This is an automatic release.

Other Changes

@RCGitBot RCGitBot requested a review from a team January 18, 2023 17:34
@NachoSoto

Copy link
Copy Markdown
Contributor

Sigh, purchase tester deployment failed because of this https://stackoverflow.com/a/67895931/401024 (since #2217)

@NachoSoto

Copy link
Copy Markdown
Contributor

I can't reproduce that locally though.

NachoSoto added a commit that referenced this pull request Jan 18, 2023
…eCat` and `ReceiptParser` (#2217)"

This reverts commit 02c2bce.

Unfortunately this breaks SPM integration under some conditions (see #2223):
> error: The package product 'RevenueCat' cannot be used as a dependency of this target because it uses unsafe build flags. (in target 'PurchaseTester' from project 'PurchaseTester')

I haven't been able to reproduce this locally, but better to remove this for now until `SPM` adds a way to set this flag without using unsafe flags.
@NachoSoto

Copy link
Copy Markdown
Contributor

Fixed #2225.
I'll merge this for now.

@NachoSoto NachoSoto merged commit 3489d10 into main Jan 18, 2023
@NachoSoto NachoSoto deleted the release/4.17.1 branch January 18, 2023 18:58
NachoSoto added a commit that referenced this pull request Jan 18, 2023
…eCat` and `ReceiptParser` (#2217)" (#2225)

This reverts commit 02c2bce.

Unfortunately this breaks SPM integration under some conditions (see
#2223):
> error: The package product 'RevenueCat' cannot be used as a dependency
of this target because it uses unsafe build flags. (in target
'PurchaseTester' from project 'PurchaseTester')

I haven't been able to reproduce this locally, but better to remove this
for now until `SPM` adds a way to set this flag without using unsafe
flags.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants