Open phillipplum opened 1 year ago
Hi,
I wonder where those quotes come from.
Can you try to:
grep -F -e '"com.app.demo"' -R .
;
If the issues comes back can you find a reproduction to make those quotes appear, we'll then be able to fix our hook.
Hello, I've finally been able to do some testing.
If I add the iOS platform completely new and do not install Wonderpush Cordova SDK I find the following results:
./platforms/ios/Demo.xcodeproj/project.pbxproj: PRODUCT_BUNDLE_IDENTIFIER = "com.app.demo";
./platforms/ios/Demo.xcodeproj/project.pbxproj: PRODUCT_BUNDLE_IDENTIFIER = "com.app.demo";
If I remove the quotes and then run cordova plugin add
, the quotes are added again. The search then returns the following result.
./platforms/ios/Demo.xcodeproj/project.pbxproj: PRODUCT_BUNDLE_IDENTIFIER = "com.app.demo";
./platforms/ios/Demo.xcodeproj/project.pbxproj: PRODUCT_BUNDLE_IDENTIFIER = "com.app.demo";
./platforms/ios/Demo.xcodeproj/project.pbxproj: PRODUCT_BUNDLE_IDENTIFIER = "com.app.demo".WonderPushNotificationServiceExtension;
./platforms/ios/Demo.xcodeproj/project.pbxproj: PRODUCT_BUNDLE_IDENTIFIER = "com.app.demo".WonderPushNotificationServiceExtension;
I cannot reproduce the quotes.
Xcode accepts creating a project with a dash in it's bundle identifier and then uses quotes. Otherwise I have no quotes.
But since we're talking cordova I have to make cordova create the project and the cordova create
command refuses dashes, underscores or accentuated characters.
Can you give me a reproduction with a few commands I would run?
I've been fiddling a bit and managed to get Xcode to add quotes. Can you test our plugin using the latest commit?
# Remove the plugin if necessary
cordova plugin rm wonderpush-cordova-sdk --variable CLIENT_ID=… --variable CLIENT_SECRET=…
# Checkout out the WonderPush Cordova SDK at the right commit
git clone https://github.com/wonderpush/wonderpush-cordova-sdk.git ../wonderpush-cordova-sdk
cd ../wonderpush-cordova-sdk
git checkout e03152cf950d4837ce3f31cce5c1982f134985af
# Go back to your project
cd -
# Add the WonderPush Cordova SDK from the local checkout
cordova plugin add ../wonderpush-cordova-sdk --variable CLIENT_ID=… --variable CLIENT_SECRET=…
Note the way to add the WonderPush Cordova SDK now uses a (relative) path.
If it works fine for you I'll release it. I'm still interested in your reproduction commands if you can provide them.
Hey, I apologise for my delay.
Now I get this error:
/platforms/ios/WonderPushNotificationServiceExtension/wonderpushnotificationserviceextension-debug.xcconfig:7:1: error: could not find included file '../Pods/Target Support Files/Pods-WonderPushNotificationServiceExtension/Pods-WonderPushNotificationServiceExtension.debug.xcconfig' in search paths
If I create a new clean cordova project, the problem does not exist. So, I think I have a problem with other plugin(s). I will start with a new project and gradually add more plugins. Maybe I'll find the problem that way.
Hey I want to give a small update.
Your fix works but cordova-ios version >= 7 breaks Podfile on build. I create a issue for this: https://github.com/apache/cordova-ios/issues/1389
Hey, I apologise for my delay.
Now I get this error:
/platforms/ios/WonderPushNotificationServiceExtension/wonderpushnotificationserviceextension-debug.xcconfig:7:1: error: could not find included file '../Pods/Target Support Files/Pods-WonderPushNotificationServiceExtension/Pods-WonderPushNotificationServiceExtension.debug.xcconfig' in search paths
If I create a new clean cordova project, the problem does not exist. So, I think I have a problem with other plugin(s). I will start with a new project and gradually add more plugins. Maybe I'll find the problem that way.
Hi, I'm also trying to deal with this error. Did you find any solution?
When I install woderpush-cordova-sdk and running cordova-ios with latest version 7.0.0 I gettings this error: