admob-google / admob-cordova

Google AdMob plugin for Cordova, Phonegap and Intel XDK ,support ios and android,support admob v1 and admob v2 ,Monetize your html5 app with one javascript line.api is easy to use.
65 stars 50 forks source link

Intel XDK not building, xdk: early access 2866. commit: Master #18

Closed arjankuijpers closed 8 years ago

arjankuijpers commented 8 years ago

i'm currently using a early access version of XDK (2866) and this plugin is not building (with my settings). gettting an error while building:

file is universal (2 slices) but does not contain a(n) armv7s slice.

/Plugins/com.admob.plugin/AdmobAPI.framework/AdmobAPI file 'Appname/Plugins/com.admob.plugin/AdmobAPI.framework/AdmobAPI' for architecture armv7s clang: error: linker command failed with exit code 1 (use -v to see invocation)

\ BUILD FAILED **

gregstein commented 8 years ago

Hello,

Remove the plugin and reinstall it by only inputting the plugin id: com.admob.plugin

(Do not input version)

Now rebuild your app.

If the build fails create a new project , copy your app files in it and install admob as mentioned above. At this point, It can not fail, but if it does there is a plugin that causing the issue, try to remove them one by one and rebuild.

admob-google commented 8 years ago

remove armv7s from your xcode project setting ,it is not reqired.

gregstein commented 8 years ago

Go with admob-google solution first :)

arjankuijpers commented 8 years ago

Not seeing a option in Intel XDK to disable armv7s.

gregstein commented 8 years ago

In Project tab>Plugin Managements

You just remove it.

arjankuijpers commented 8 years ago

the whole plugin ..?

schermafbeelding 2016-02-12 om 00 34 57
gregstein commented 8 years ago

I think it's AppVersion that you need to delete

gregstein commented 8 years ago

by the way did you touch Launch Icons and Splash Screens ? You can fill in the icons but avoid Splash screen because whenever i try to setup splash screens I get error compiling the app, so I had to create new project and drag my html files into it and compiling works.

ugierth commented 8 years ago

I am facing the same error and tried to remove and reinstall as admob-google mentioned, but without luck. I can build the iOS app without errors if the admit-plugin is not part of the app. So I think the armv7s could be at admob plugin. Is there any way to disable it? Thanks, Uwe

gregstein commented 8 years ago

Create a new fresh project and add only admob, and also do not fill the Splash screens.

ugierth commented 8 years ago

Do you think that this is a solution? I need the plugin working at my app Cheers Uwe

gregstein commented 8 years ago

I also use intel xdk and it is buggy as hell, and at least that seems to solve for me.

arjankuijpers commented 8 years ago

@gregstein which version of XDK do you use and so using Early Access version .. the versioning of EA is different.

gregstein commented 8 years ago

@arjankuijpers I always update to the latest one and make sure i use higher version of cordova v.4.1.1 or v.5 or above. Do not use lower versions sooner google play will stop the support for that.

arjankuijpers commented 8 years ago

new blank project without plugins succeeding iOS builds, com.admob.plugin added trough git (master) fails log: https://gist.github.com/arjankuijpers/14c810d14992d4968e6b

arjankuijpers commented 8 years ago

screen_failed_blank

gregstein commented 8 years ago

Please change to cordova CLI 4.1.2 version: http://i.imgur.com/ugY1S0C.png

and try to re-upload and re-build

arjankuijpers commented 8 years ago

it is no longer supported to use that version image

gregstein commented 8 years ago

Odd! Remove admob google and try this plugin : https://github.com/appfeel/admob-google-cordova instead, see if it works or u get same results. @

gregstein commented 8 years ago

install like this: http://i.imgur.com/oOoWoXh.png

arjankuijpers commented 8 years ago

fails aswell screen_failed_blank_appfeel

arjankuijpers commented 8 years ago

https://gist.github.com/arjankuijpers/c3815c213c9ffb6a6ec5

gregstein commented 8 years ago

create another empty project and try to rebuild with that plugin again.

if it doesn't work, try this plugin too in another project: https://github.com/floatinghotpot/cordova-plugin-admob. Also keep the name of the app as default so u know what's the problem.

gregstein commented 8 years ago

If all these plugins fail. 100% the early access version that is causing it.

arjankuijpers commented 8 years ago

Had the same thing with the stable version, but I will try the normal before confirming that.

arjankuijpers commented 8 years ago

if its not to much to ask: could you create an XDK project with com.admob.plugin ? and zip this (and attach file). then I can validate if this problem is my setup.

gregstein commented 8 years ago

sure

gregstein commented 8 years ago

Here is the link to the sample project:http://www.mediafire.com/download/v526rr6hf4cs345/admobsample.zip

and here is a success android build screen with that project: http://prntscr.com/a2euvi

arjankuijpers commented 8 years ago

normal fails as well at xdk 2893, havnt tried your zip yet.

app-blank-with-plugin-fail app-blank-withoutplugin-suc app-blank-with-plugin-appfeel-fail

https://gist.github.com/arjankuijpers/28028e0e2c8e2b04aff7

arjankuijpers commented 8 years ago

your Zip also fails, have you tried building iOS ?, because the screenshot you added is from Android and my plugins work on Android aswell but it need to be working both platforms.

admob-google commented 8 years ago

We may add armv7s support

arjankuijpers commented 8 years ago

that would be great, because this is really a plugin I want to use before the others

arjankuijpers commented 8 years ago

if its not that hard, I could take a look to help, but I dont know where to begin :)

ugierth commented 8 years ago

Hi, if I understand the error message correct there is a compiler/linker flag to create also for armv7s architecture. But with using xcode 6 and iOS 8 apple removed armv7s. Please take a look at https://github.com/openframeworks/openFrameworks/issues/3423 if you update the flag to armv7 it should build without error. Cheers, Uwe

arjankuijpers commented 8 years ago

Well, you can solve this when you build with xcode but with Intel XDK you can't change Intel build settings. And Xcode stuff gets done on server side and is not in your control.

arjankuijpers commented 8 years ago

I don't know if the plugin creator can do anything to fix this without Intels support to change the build archs

arjankuijpers commented 8 years ago

I the commit history there is a commit named add support for armv7s #beaa0c so isnt it already supported ?

I cant fully understand if the plugin needs to support armv7s or needs to drop this (if it already supports)

ugierth commented 8 years ago

From your debug post I can see: Build settings from command line: ARCHS = armv7 armv7s arm64 This will be used for the "ld" command and its failing because of missing armv7s. So my last comment was wrong. The armv7s slice is missing at AdmobAPI file. maybe the maintainer can recompile with added arch flag armv7s

arjankuijpers commented 8 years ago

is it maintained by google or is it a creation of this Admob project.. otherwise I could change and compile it myself right ? framework name used by this project: AdmobAPI.framework framework name downloaded from Google Admob download pages: GoogleMobileAds.framework

if so it would be great to include this framework as well in this project, so people can also contribute.

ugierth commented 8 years ago

Sorry I don't know if this is maintaned by google. The file included at this plugin is a compiled binary.

arjankuijpers commented 8 years ago

thats why I ask :) but its a question for the maintainer I guess. @admob-google

admob-google commented 8 years ago

wait

admob-google commented 8 years ago

upgraded

ugierth commented 8 years ago

sorry to say but the build is also failing with upgraded plugin:

ld: file is universal (4 slices) but does not contain a(n) armv7s slice: SuggestMe/Plugins/admob/AdmobAPI.framework/AdmobAPI file 'SuggestMe/Plugins/admob/AdmobAPI.framework/AdmobAPI' for architecture armv7s

if you need the complete build log I can send it.

admob-google commented 8 years ago

upgraded ,5.4.1