BeID-lab / eIDClientCore

Präambel: Die Humboldt-Universität zu Berlin, Unter den Linden 6, 10099 Berlin, im Folgenden als Humboldt-Universität bezeichnet, stellt für die Anwendung im Zusammenhang mit deutschen hoheitlichen Dokumenten, wie beispielsweise den neuen Personalausweis und den elektronischen Aufenthaltstitel, eine clientseitige eID-Basis-Software zum Bereitstellen der sogenannten eID-Funktionalität, d.h. Ermöglichen eines elektronischen Identitätsnachweises, die sich an den Richtlinien des BSI TR-03110, TR-03112, TR-03130 orientiert, bereit, und zwar in dem Entwicklungszustand, in dem sie sich zum Zeitpunkt der Bereitstellung befindet. Diese Software, die im Folgenden als eIDClientCore bezeichnet wird, hat die Humboldt-Universität von der Bundesdruckerei GmbH, Oranienstraße 91, 10969 Berlin zur Verwendung in Lehre und Forschung sowie zur Bereitstellung an die Öffentlichkeit („Hosten“) für die allgemeine Prüfung und Verwendung durch jedermann erhalten.
15 stars 16 forks source link

Some updates #74

Closed frankmorgner closed 8 years ago

frankmorgner commented 8 years ago
eriknellessen commented 8 years ago

In the file eIDClientCore/android/Makefile, line 93 (https://github.com/BeID-lab/eIDClientCore/pull/74/files#diff-be8e95fe3c5c5544a6e911928865eb75R93): sed -i '' 's/^_ANDROID_API=\"android-.*/_ANDROID_API=\"'$(ANDROID_API)'\"/' setenv-android.sh &&\

Please remove the '' after -i, i.e.

sed -i 's/^_ANDROID_API=\"android-.*/_ANDROID_API=\"'$(ANDROID_API)'\"/' setenv-android.sh &&\

instead of

sed -i '' 's/^_ANDROID_API=\"android-.*/_ANDROID_API=\"'$(ANDROID_API)'\"/' setenv-android.sh &&\

It is the reason, why Travis CI build is failing:

Archive:  Setenv-android.sh.zip
  inflating: setenv-android.sh       
sed: can't read s/^_ANDROID_API=\"android-.*/_ANDROID_API=\"android-19\"/: No such file or directory
make[1]: *** [/home/travis/build/BeID-lab/eIDClientCore/eIDClientCore/android/eIDClientCore-0.2_arm/lib/libcryptopp.so] Error 2

After that, the other make command is executed (I wrote you an e-mail about this on 11th of November), which fails, as it tries to unpack Setenv-android.sh.zip again:

Archive:  Setenv-android.sh.zip
replace setenv-android.sh? [y]es, [n]o, [A]ll, [N]one, [r]ename: 

No output has been received in the last 10 minutes, this potentially indicates a stalled build or something wrong with the build itself.
frankmorgner commented 8 years ago

hmm, in my case sed was used on OS X, which requires the additional argument, see http://stackoverflow.com/questions/4247068/sed-command-failing-on-mac-but-works-on-linux. It's fixed now.

eriknellessen commented 8 years ago

We fixed the CryptoPP cross-compiling for android problem in dc31b2075e67f2ab83e600e5d0786cd3d166d692. We also do the new procedure with the GNUmakefile-cross now. So the changes of 7da8d9a87f7fd313f7242bd8e8bb40c5343f42f7 are no longer useful.

Please resolve the merge conflict and merge your pull request.

frankmorgner commented 8 years ago

done