Project

General

Profile

Actions

Task #4661

closed

Upgrade to NFD 0.6.2

Added by Junxiao Shi over 5 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Normal
Category:
jni
Start date:
Due date:
% Done:

100%

Estimated time:

Files

4841-6_armv7.log (779 KB) 4841-6_armv7.log Junxiao Shi, 07/24/2018 10:21 AM
4841-6_arm64.log (790 KB) 4841-6_arm64.log Junxiao Shi, 07/24/2018 10:28 AM
app-armeabi-v7a-debug.7z.001 (4.69 MB) app-armeabi-v7a-debug.7z.001 4841,6 app-armeabi-v7a-debug.apk Junxiao Shi, 07/24/2018 11:17 AM
app-armeabi-v7a-debug.7z.002 (4.23 MB) app-armeabi-v7a-debug.7z.002 4841,6 app-armeabi-v7a-debug.apk Junxiao Shi, 07/24/2018 11:17 AM
Actions #1

Updated by Junxiao Shi over 5 years ago

I'm verifying Change 4841,5 with the steps given in README.md, but an installation step fails:

jns23@P862384:~/android-sdk-linux/ndk-bundle$ CREW_OWNER=cawka crew.dir/crew install target/ndn_cxx:0.6.2-19-g317e61ec t
arget/nfd:0.6.2-11-gd657d53
Warning: using system ruby
warning: target/ndn_cxx:0.6.2-19-g317e61ec_1 already installed

calculating dependencies for target/nfd:
  dependencies to install:
downloading https://github.com/cawka/android-crew-staging/releases/download/staging/packages.nfd-0.6.2-11-gd657d53_2.tar
.xz
warning: not found 'curl' program at the expected path: /home/jns23/android-sdk-linux/ndk-bundle/prebuilt/linux-x86_64/b
in/curl
error: failed to download https://github.com/cawka/android-crew-staging/releases/download/staging/packages.nfd-0.6.2-11-
gd657d53_2.tar.xz: code: 22; text: HTTP page not retrieved
Actions #2

Updated by Alex Afanasyev over 5 years ago

The code under review is not yet working (compiles, but crashes with some strange errors)

Actions #3

Updated by Junxiao Shi over 5 years ago

compiles, but crashes with some strange errors

It does NOT compile because the dependent repository "android-crew-staging" has been updated since Jenkins ran. Why not put "android-crew-staging" on Gerrit as well?

Updated by Junxiao Shi over 5 years ago

I'm verifying Change 4841,6 with the steps given in README.md, using a Sonim XP8800 running Android 7.1.1.

./gradlew assembleRelease builds ./app/build/outputs/apk/release/app-armeabi-v7a-release-unsigned.apk (and three other files) but this APK failed to install:

$ ./adb install ../code/NFD-android/app/build/outputs/apk/release/app-armeabi-v7a-release-unsigned.apk
adb: failed to install ../code/NFD-android/app/build/outputs/apk/release/app-armeabi-v7a-release-unsigned.apk: Failure [INSTALL_PARSE_FAILED_NO_CERTIFICATES: Package /data/app/vmdl925776501.tmp/base.apk has no certificates at entry AndroidManifest.xml]

If an additional setup step is necessary, it should be specified in README.md.

./gradlew assembleDebug builds ./app/build/outputs/apk/debug/app-armeabi-v7a-debug.apk and I'm able to install the APK.
When I start the app, the GUI loads, but it immediately says "NFD keeps stopping". logcat is attached as 4841-6_armv7.log.

./gradlew assembleDebug also builds ./app/build/outputs/apk/debug/app-arm64-v8a-debug.apk and I'm able to install the APK.
NFD service starts successfully.
"General", "Routes", and "Ping" tabs work as expected.
"Faces" tab does not work: it displays "Oops! Not available, check that NFd is started" and "Error communicating with NFD (Failed to read status dataset.)". logcat is attached at 4841-6_arm64.log.
"Logcat" tab does not work: it only shows "Loading logger ..." and "---- beginning of main" but nothing else.
I'm uncertain about "Connect to the nearest hub automatically" function because "Faces" tab does not work.

Actions #5

Updated by Alex Afanasyev over 5 years ago

For assembleRelease you don't have the same private key that I have been publishing the package and if you didn't configure environment with some keys, it would fail.

Debug works. I am a bit puzzled now with armeabi, as I was fighting this specific error and it seems to be working with my compile. Do you mind sending me app-armeabi-v7a-debug.apk, I want to double check symbol tables in the included shared libs.

Actions #7

Updated by Junxiao Shi over 5 years ago

if you didn't configure environment with some keys, it would fail.

These steps should be given in README.md. Alternatively, ask the user to run ./gradlew assembleDebug instead.

Actions #8

Updated by Alex Afanasyev over 5 years ago

Something wrong with your compiled version of precomplied libraries (the one you attached incorrectly linked against libstdc++.so, which I have fixed). Can you do a full cleanup:

  • ./gradlew clean
  • rm -Rf ~/.caches/crew

and then try to build the debug version again.

Actions #9

Updated by Alex Afanasyev over 5 years ago

One more cleanup, remove $NDK_DIR/packages/* if you are reusing previous versions, as packages they need to be redownloaded or rebuilt.

Actions #10

Updated by Junxiao Shi over 5 years ago

I did the cleanups in note-8 and note-9. "Faces" and "Logcat" tabs still do not work, same behavior as note-4.

Actions #11

Updated by Alex Afanasyev over 5 years ago

Just pushed patch set 7 that removes logcat and fixes face (= update of the NFD management library). Worked for my quick test with armeabi-based phone.

Actions #12

Updated by Davide Pesavento over 5 years ago

  • Status changed from New to Closed
  • % Done changed from 0 to 100

The new NFD has been published on the Google Play store.

Actions

Also available in: Atom PDF