search

klarna / klarna-mobile-sdk

Klarna Mobile SDK for iOS
https://docs.klarna.com/mobile-sdk/ios/
Apache License 2.0
29 stars 12 forks source link

[iOS] Umbrella header for module 'KlarnaMobileSDK' does not include header 'KlarnaMobileSDKPostfix.h' #202

Closed javiervg-itx closed 1 year ago

javiervg-itx commented 1 year ago

Describe the bug After upgrading from version 2.1.7 to 2.6.6 and compiling, the following errors are thrown:

To Reproduce Steps to reproduce the behavior:

  1. Changed version of the SPM Klarna package from 2.1.7 to 2.6.6
  2. Compile
  3. Also tested cleaning derived data and build folder before compilation

Expected behavior Correct compilation of the module

Device and version:

Additional context I found that the only way to compile is disabling the following compiler options:

polar133 commented 1 year ago

Hi @javiervg-itx Thanks for the report.

We try to replicate the same issue but for us it's working fine. What i can suggest is try to clean the SPM cache. Please give a try to the suggestions posted on https://stackoverflow.com/questions/72788019/where-is-swift-package-manager-cache-located-swift-5-7/72838880#72838880

alanperezoc commented 1 year ago

Hi! We are facing exactly the same issue when getting the SDK via Cocoapods. Same versions and device .

polar133 commented 1 year ago

Hi @alanperezoc. From what version are you updating ? Can you give a try and specify an older version and see if the error is the same ? Have you tried the same approach to clean the data ?

javiervg-itx commented 1 year ago

Hi @polar133, tried cleaning the SPM cache but the error persists

polar133 commented 1 year ago

@javiervg-itx thanks for the update. Can you please try again with a lower version? e.g 2.6.2

javiervg-itx commented 1 year ago

@polar133 Yes, tried now with 2.6.2 and same error.

Investigating, I've seen this: https://github.com/stripe/stripe-ios/issues/1772 where the same error ocurred with another library. It looks like they resolved with a fix inside the library but no other info was added to the issue...

polar133 commented 1 year ago

Thanks. We will investigate a little more and be back once we have some updates to share.

polar133 commented 1 year ago

We are working on a possible fix. It will be available on the next release early next week. 🤞

marc-op commented 1 year ago

Hello @polar133 ,

Do you have any updates on this matter?

Thanks.

polar133 commented 1 year ago

Hi @marc-op @javiervg-itx @alanperezoc, We just released 2.6.7 that should address this issue. Please give a try and let us know how it goes. 👍

alanperezoc commented 1 year ago

thanks @polar133 , we'll give it a try on next days!

pedronl commented 1 year ago

Hi @polar133, we´ve tried with the 2.6.7 version but we´re still finding these errors:

We´ve tried reseting the SPM cache, cleaning derived data and doing a clean build but the errors persist 😕

polar133 commented 1 year ago

Hi @pedronl, thank you for bringing this to our attention. We are actively working on a solution and aim to resolve it asap. We apologize for any inconvenience this may have caused.

polar133 commented 1 year ago

@pedronl Should be fixed on the new version 2.6.8. Please, give a try and let us know how it goes. Thanks and again sorry for any inconvenience. This should have been fixed on the prev. release.

javiervg-itx commented 1 year ago

Hi @polar133, version 2.6.8 solved the problem. Thank you!

polar133 commented 1 year ago

Thanks for the update @javiervg-itx. Since it's fixed, i will close the issue. If there is something else please don't hesitate to open it again. 👍