Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update last Flurry SDK #24

Open
wants to merge 13 commits into
base: master
Choose a base branch
from
Open

Update last Flurry SDK #24

wants to merge 13 commits into from

Conversation

xviricel
Copy link

Update Flurry SDK:

  • android SDK 6.1.0
  • iOS SDK 7.2.1

@skolesnyk
Copy link

Oh that'd be awesome.

@skolesnyk
Copy link

@xviricel could you add Android x86 support? to platform.xml ?

@xviricel
Copy link
Author

xviricel commented Apr 20, 2016

With the lasts Air SDK (from 15 I think), doesn't necessary define Android x86 support ;)
For us development on iOS and Android, we don't define this propertie.

@skolesnyk
Copy link

Are you saying x86 support is embedded into every ANE compiled with Air >= 15? I don't think it is. You still have to include x86 support into ant build file. Really, it takes just several lines of copy-paste code.

Just open any other commercial ane and you'll see that they have a separate binary specifically for Android x86.

@xviricel
Copy link
Author

AirFlurry.ane.zip

@xviricel
Copy link
Author

This part for apk (all platform) with Air SDK 19, it's since Air SDK 16 (not 15), I don't use "-arch" today in adt command line:

<!-- MobileAppAS APK -->

        <echo>:: [ PACKAGE APK ] :: ${app.name}.apk</echo>
        <echo>:: [ PACKAGE APK ] :: type of apk ${apk.target}</echo>

        <copy file="${src.dir}/main/resources/Default-Landscape.png" todir="../temp" />
        <copy file="${src.dir}/main/resources/[email protected]" todir="../temp" />

        <exec executable="${sdk.air.adt}" failonerror="true" dir="../temp">
            <arg value="-package" />
            <arg value="-target" />
            <arg value="apk-captive-runtime" />

            <arg value="-storetype" />
            <arg value="pkcs12" />

            <arg value="-keystore" />
            <arg value="${src.dir}/main/resources/certificats/${apk.p12}" />

            <arg value="-storepass" />
            <arg value="${apk.p12.storepass}" />

            <arg value="${temp.dir}/${app.name}.apk" />
            <arg value="${temp.dir}/myapp.xml" />

            <arg value="Default-Landscape.png" />
            <arg value="[email protected]" />

            <arg line="-C" />
            <arg line="./" />
            <arg line="${app.name}.swf" />
            <arg line="-C" />
            <arg line="${src.dir}/main/resources/include" />
            <arg line="." />

            <arg line="-extdir" />
            <arg line="${ane.dir}" />
        </exec>

@skolesnyk
Copy link

skolesnyk commented Apr 20, 2016

Take a look at your build.xml — that's where extension is packaged. It should contain extra line

< arg line="-platform Android-x86 -platformoptions ../build/platformandroid.xml -C Android-x86/ ."/ >

<target name="package" description="Create the extension package">
    <condition property="OUTPUT_FILENAME" value="${name}_multiMsg" else="${name}">
        <istrue value="${useMultiMsg}"/>
    </condition>

    <exec executable="${flex.sdk}/bin/adt${bin.ext}" failonerror="true" dir="../temp">
        <arg value="-package"/>
        <arg value="-target"/>
        <arg value="ane"/>
        <arg value="${OUTPUT_FILENAME}.ane"/>
        <arg value="../build/extension.xml"/>
        <arg line="-swc swc/${name}.swc"/>
        <arg line="-platform iPhone-ARM -platformoptions ../build/platform.xml -C ios/ ."/>
        <arg line="-platform Android-ARM  -platformoptions ../build/platformandroid.xml  -C Android-ARM/ ."/>
        <arg line="-platform default -C default/ ."/>
    </exec>

    <move file="../temp/${OUTPUT_FILENAME}.ane" todir="../bin"/>
    <delete dir="../temp"/>
</target>

@xviricel
Copy link
Author

I'm ok with you but today it's not necessary, tested with SAMSUNG GT-P5210 (Galaxy Tab 3)

@skolesnyk
Copy link

And analytics works fine? Good to hear!

@xviricel
Copy link
Author

For us yes 💃

@xviricel
Copy link
Author

You can active a developer key in Flurry and test...
Choose apk-captive-runtime in adt compilation and normally...

@ElmDev
Copy link

ElmDev commented Feb 25, 2017

Hello, i included your ANE, but there are no events on Flurry so far....

I am doing:

Flurry.setAndroidAPIKey(ANDROID_KEY);
Flurry.setAppVersion("1.0.0_test");
Flurry.startSession();

Flurry.logEvent(FlurryEvent.EVENT_LOGIN, {userEmail:email}); 
// FlurryEvent.EVENT_LOGIN and email are String

Is there something wrong in here ?

nb: and i call Flurry.stopSession(); when app (Starling) is paused or stopped, and Flurry.startSession(); if the app resumes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants