SpectrumKit in cocoapod use_modulars_headers!

[scutletbutt]

i am using Cocoapods to use Spectrum
pod 'SpectrumKit/Plugins/JPEG
Using use_modular_headers!, pod installation completed
Xcode compile error as below

on file: ‘LibJpegDctTransformer.h’

Warning appear on code:
extern "C" {
#include <mozjpeg/transupp.h> // doesn't play nice with C++
}

the warning message is:
import of C++ module 'mozjpeg.mozjpeg.transupp' appears within extern "C" language linkage specification

[cuva]

Thanks for submitting an issue @scutletbutt - I believe this is linked to #25 but I will have a look into it shortly and report back!

[scutletbutt]

Hi Cuva, thanks for respond
In cocoapod I tried option as below, switching back and forth many time, the conclusion is:
Option 1) use_framework, Spectrum does not support framework yet
Option 2) use_modular_headers, error come out from a Spectrum code ‘import of C++ module 'mozjpeg.mozjpeg.transupp' appears within extern "C" language linkage specification’
Option 3) without use_framwork and use_modular_headers, some modules does not recognize by Xcode one after another. Error mesage “no such modules X”, X = modules such as FBSDKLoginKit or FirebaseAuth

Now in my cocoapod I am using use_modular_header without pod Spectrum, others modules work well, so i temporary remove spectrum, would like to use it if possible

[cuva]

@scutletbutt I'm working on adding support for modular headers. In the meantime, feel free to exclude modular headers from this pod like so:

pod 'SpectrumKit', :modular_headers => false

[scutletbutt]

@cuva I tried that, the end result is same as only use_modular_header!
Somehow cocoapod won’t recognize the code :modular_headers => false
Search for cocopod github issues, seem like a bug
Good to know u are working on support for modular headers, may I ask about roughly how long it will take

[cuva]

Hi @scutletbutt. I've got a fix locally which enables support for modular header - but I haven't had the time to push it yet (it requires pushing an update to the mozjpeg pod too).

I'm hopping to get it merged before the end of the week - I'll report back then!

[scutletbutt]

Thank for the work @cuva , appreciate it.

[cuva]

Hey @scutletbutt. Just a quick heads up - some changes to our internal build system has made this fix slightly more complicated than anticipated. I'm still working on it, I don't anticipate it taking much longer.

[scutletbutt]

Thanks for the heads up @cuva

[tarekabdallah]

Hey @cuva any updates concerning this issue?

[cuva]

@scutletbutt a fix for this will be released in the next few days - thanks for your patience!

[diegosanchezr]

Hey @scutletbutt, we've just pushed v1.1.0, which adds support for :modular_headers.

Can you update your Podfile to use v.1.1.0 and check whether it works for you?

[rahulvyas]

It still doesn't work