sgrebnov / cordova-plugin-azure-notificationhub

Apache License 2.0
11 stars 36 forks source link

Plugin not working on 64-bit architecture #4

Open kant2002 opened 9 years ago

kant2002 commented 9 years ago

XCode complain that it ignores the WindowsAzureMessaging.framework since it does not have symbols for x86_64 platform.

emtopping commented 9 years ago

Hi, can you update the plugin for 64 bit support so it can be used on iOS8 and PhoneGap Build?

kant2002 commented 9 years ago

Essentially, I just recompile WindowsAzureMessaging.framework to support 64-bit arch.

@emtopping If you need this, you could install from my repo, until fix would be merged.

cordova plugin add https //github.com/kant2002/cordova-plugin-azure-notificationhub.git
emtopping commented 9 years ago

@kant2002 I appreciate the offer, Andrey. Unfortunately I'm using PhoneGap Build so I need srebnov to update it in PGB. I'm trying to setup my own environment in case the author has abandoned the project but having trouble updating my version of cordova in Visual Studio Tools for Apache Cordova from the default - but that's off-topic! ;)

sgrebnov commented 9 years ago

merged and thank you a lot for the effort - please let me know if it does not work.

sgrebnov commented 9 years ago

I believe that PG Build uses the following fork so this fix must be landed here to make it working https://github.com/PhonegapProjects/cordova-plugin-azure-notificationhub

emtopping commented 9 years ago

@sgrebnov indeed it does - does that mean PG Build won't be updated? (Sorry I'm a bit of a newbie with this)?

sgrebnov commented 9 years ago

@PhonegapProjects hey guys, could you merge or cherry pick this fix to your fork? https://github.com/sgrebnov/cordova-plugin-azure-notificationhub/issues/10

emtopping commented 9 years ago

Thanks, Sergey!

mvanhalen commented 9 years ago

Hi @sgrebnov ,

Can it be that the PhoneGap Build is using the old version still? Got a 64 bit error when compiling with a PhoneGap ios build. I did not specify a in the config file version so it should get the latest plugin version right?

Thanks

emtopping commented 9 years ago

@mvanhalen in the end I used the Azure javascript library not the plugin (in conjunction with pushplugin)....

mvanhalen commented 9 years ago

@emtopping That's another option indeed. But I would like to have a clean integration with notification hubs directly. This seems the best way to do it. But always good to have a fallback thanks. I'll first try it out with Android.

Lanayx commented 9 years ago

The issue is still open, I'm getting the error

Link: ld: warning: ignoring file Cordova370/Plugins/msopentech.azure.NotificationHub/WindowsAzureMessaging.framework/WindowsAzureMessaging, missing required architecture arm64 in file Cordova370/Plugins/msopentech.azure.NotificationHub/WindowsAzureMessaging.framework/WindowsAzureMessaging (3 slices) Undefined symbols for architecture arm64: "_OBJCCLASS$_SBNotificationHub", referenced from: objc-class-ref in NotificationHub.o ld: symbol(s) not found for architecture arm64 clang: error: linker command failed with exit code 1 (use -v to see invocation)

UPDATE:

I've updated the WindowsAzureMessaging.framework from the official repository https://github.com/Azure/azure-notificationhubs/blob/master/iOS/bin/WindowsAzureMessaging.framework.zip Build worked, but when using test code, it just shuts down the app...

svenrobin commented 9 years ago

my build is faling with the zip you linked to: im getting the error: file is universal (4 slices) but does not contain a(n) armv7s slice:

Lanayx commented 9 years ago

I've also come up with PushPlugin and Azure REST for notifications hub

mvanhalen commented 9 years ago

Hi @svenrobin and @Lanayx , I made a fork to this git especially for PhoneGap Build projects. It´s has the push notification integrated on the native side. The build is working for iOS8 64 as well. iOS and Android work. https://github.com/mvanhalen/cordova-plugin-azure-notificationhub

Good luck

svenrobin commented 9 years ago

@mvanhalen Hey, Good fork but i still got the same error when trying to build it for my ipad: : file is universal (4 slices) but does not contain a(n) armv7s slice: Plugins/msopentech.azure.NotificationHub.Extended/WindowsAzureMessaging.framework/WindowsAzureMessaging file

svenrobin commented 8 years ago

do you have any idea in how i can fix this @mvanhalen ?

mvanhalen commented 8 years ago

Odd. I have used the latest version a couple of times now. Compiled without issues in PhoneGap Build. Even got a new App approved in Apple Store with it. Did you specify a version number maybe? Better leave that out to use the last version.

svenrobin commented 8 years ago

I have left it out, do you know if you built the framework with the armv7s architecture @mvanhalen ? it's crashing only for that architecture.

mvanhalen commented 8 years ago

Ah ok. It´s possible that was left out. I tested only with ios8

But still PhoneGap build does not give any errors. Did you add the plugin to your PhoneGap project directly? Without using PhoneGap build?

Sent from Windows Mail

From: svenrobin Sent: ‎Sunday‎, ‎May‎ ‎24‎, ‎2015 ‎3‎:‎18‎ ‎PM To: sgrebnov/cordova-plugin-azure-notificationhub Cc: Martijn van Halen

I have left it out, do you know if you built the framework with the armv7s architecture @mvanhalen ? it's crashing only for that architecture.

— Reply to this email directly or view it on GitHub.

svenrobin commented 8 years ago

i added it with a build and that works, its just when i try to build it for armv7s, is it possible for you to recompile it for armv7s also @mvanhalen ?

svenrobin commented 8 years ago

bump @mvanhalen !