Issues with Xcode 12.3 / Redefinition of enumerator & Expected identifier or '('

[patrickbussmann]

Bug report

CHECKLIST

• [X] I have read the issue reporting guidelines
• [X] I confirm this is a suspected bug or issue that will affect other users
• [X] I have reproduced the issue using the example project or provided the necessary information to reproduce the issue.
• [X] I have read the documentation thoroughly and it does not help solve my issue.
• [X] I have checked that no similar issues (open or closed) already exist.

Current behavior:

❌  /Users/vagrant/git/ios/capacitor-cordova-ios-plugins/sources/Uk.co.workingedge.phonegap.plugin.launchnavigator/LN_Reachability.h:15:5: redefinition of enumerator 'NotReachable'
    NotReachable = 0,
        ^
❌  /Users/vagrant/git/ios/capacitor-cordova-ios-plugins/sources/Uk.co.workingedge.phonegap.plugin.launchnavigator/LN_Reachability.h:16:5: redefinition of enumerator 'ReachableViaWiFi'
    ReachableViaWiFi,
        ^
❌  /Users/vagrant/git/ios/capacitor-cordova-ios-plugins/sources/Uk.co.workingedge.phonegap.plugin.launchnavigator/LN_Reachability.h:17:5: redefinition of enumerator 'ReachableViaWWAN'
    ReachableViaWWAN
        ^
❌  /Users/vagrant/git/ios/capacitor-cordova-ios-plugins/sources/Uk.co.workingedge.phonegap.plugin.launchnavigator/LN_Reachability.h:24:18: expected identifier or '('
extern NSString *kReachabilityChangedNotification;

iOS build issue:

• Node JS version
  • 14
• XCode version
  • Xcode 12.3 (Build version 12C33)

There was an issue in the past with the redefinition thing.

248 #251

When I have time I'll fix this issue here, too.

[zieemer]

Hi Any ideas when fix might be ready ?

[blhylton]

Had to work through this in short order for a quick turnaround project. Created a PR with the updates.

[j3gb3rt]

Our team is facing this issue also. @blhylton Thanks for the patch! Looking forward to this being released

[gabides]

Hi @dpa99c, do you plan to release a new version soon on npm with these changes? That would be quite helpful to have a stable build on our ios version :) Thanks

[vivek365]

I am facing the same issue. Is there any chance to release a new version in near future with this fix? Or Can I do any work around to fix this issue now?

Thanks

[dpa99c]

The current master branch has been released to npm as uk.co.workingedge.phonegap.plugin.launchnavigator@5.0.6

[vivek365]

thanks @dpa99c