Update entrypoint.sh

[jplitza]

Set usual shell script fail-safe options

[trois-six]

Please look at b5ecf648f8f6af3f14096fb0602aa20ed3736258.

[jplitza]

That's not really the same thing. This is primarily intended to stop the whole script if any of the commands is failing (-e). The option regarding variables (-u) is more like a habit.

Also, I don't actually use Docker. I just found the bash script useful to build the image.

[trois-six]

Did you succeed ? It's failing here with

============================================
Running kati to generate build-lineage_a5ultexx.ninja...
hardware/samsung/ril/libsecril-client/Android.mk was modified, regenerating...
============================================
PLATFORM_VERSION_CODENAME=REL
PLATFORM_VERSION=7.1.2
LINEAGE_VERSION=14.1-20171006-UNOFFICIAL-a5ultexx
TARGET_PRODUCT=lineage_a5ultexx
TARGET_BUILD_VARIANT=userdebug
TARGET_BUILD_TYPE=release
TARGET_BUILD_APPS=
TARGET_ARCH=arm
TARGET_ARCH_VARIANT=armv7-a-neon
TARGET_CPU_VARIANT=cortex-a53
TARGET_2ND_ARCH=
TARGET_2ND_ARCH_VARIANT=
TARGET_2ND_CPU_VARIANT=
HOST_ARCH=x86_64
HOST_2ND_ARCH=x86
HOST_OS=linux
HOST_OS_EXTRA=Linux-4.9.0-0.bpo.3-amd64-x86_64-with-debian-9.1
HOST_CROSS_OS=windows
HOST_CROSS_ARCH=x86
HOST_CROSS_2ND_ARCH=x86_64
HOST_BUILD_TYPE=release
BUILD_ID=NJH47F
OUT_DIR=/home/builder/android/lineage/out
============================================
frameworks/av/camera/cameraserver/Android.mk:18: Target has integrated cameraserver into mediaserver. This is weakening security measures introduced in 7.0
find: '/home/builder/android/lineage/out/target/common/obj/SHARED_LIBRARIES/libwifi-hal-mock_intermediates': No such file or directory
build/core/dex_preopt.mk:74: warning: overriding commands for target `/home/builder/android/lineage/out/target/product/a5ultexx/dex_bootjars/system/framework/telephony-ext_nodex.jar'
build/core/dex_preopt.mk:74: warning: ignoring old commands for target `/home/builder/android/lineage/out/target/product/a5ultexx/dex_bootjars/system/framework/telephony-ext_nodex.jar'
build/core/Makefile:34: warning: overriding commands for target `/home/builder/android/lineage/out/target/product/a5ultexx/system/bin/rild'
build/core/base_rules.mk:320: warning: ignoring old commands for target `/home/builder/android/lineage/out/target/product/a5ultexx/system/bin/rild'
build/core/Makefile:34: warning: overriding commands for target `/home/builder/android/lineage/out/target/product/a5ultexx/system/lib/libreference-ril.so'
build/core/base_rules.mk:320: warning: ignoring old commands for target `/home/builder/android/lineage/out/target/product/a5ultexx/system/lib/libreference-ril.so'
build/core/Makefile:34: warning: overriding commands for target `/home/builder/android/lineage/out/target/product/a5ultexx/system/lib/libsecril-client.so'
build/core/base_rules.mk:320: warning: ignoring old commands for target `/home/builder/android/lineage/out/target/product/a5ultexx/system/lib/libsecril-client.so'
build/core/Makefile:34: warning: overriding commands for target `/home/builder/android/lineage/out/target/product/a5ultexx/system/lib/librmnetctl.so'
build/core/base_rules.mk:320: warning: ignoring old commands for target `/home/builder/android/lineage/out/target/product/a5ultexx/system/lib/librmnetctl.so'
build/core/Makefile:34: warning: overriding commands for target `/home/builder/android/lineage/out/target/product/a5ultexx/system/lib/libcommon_time_client.so'
build/core/base_rules.mk:320: warning: ignoring old commands for target `/home/builder/android/lineage/out/target/product/a5ultexx/system/lib/libcommon_time_client.so'
build/core/Makefile:34: warning: overriding commands for target `/home/builder/android/lineage/out/target/product/a5ultexx/system/lib/hw/audio.primary.msm8916.so'
build/core/base_rules.mk:320: warning: ignoring old commands for target `/home/builder/android/lineage/out/target/product/a5ultexx/system/lib/hw/audio.primary.msm8916.so'
build/core/Makefile:34: warning: overriding commands for target `/home/builder/android/lineage/out/target/product/a5ultexx/system/lib/libaudioroute.so'
build/core/base_rules.mk:320: warning: ignoring old commands for target `/home/builder/android/lineage/out/target/product/a5ultexx/system/lib/libaudioroute.so'
build/core/Makefile:34: warning: overriding commands for target `/home/builder/android/lineage/out/target/product/a5ultexx/system/lib/libtinyalsa.so'
build/core/base_rules.mk:320: warning: ignoring old commands for target `/home/builder/android/lineage/out/target/product/a5ultexx/system/lib/libtinyalsa.so'
build/core/Makefile:34: warning: overriding commands for target `/home/builder/android/lineage/out/target/product/a5ultexx/system/etc/permissions/android.software.live_wallpaper.xml'
build/core/base_rules.mk:320: warning: ignoring old commands for target `/home/builder/android/lineage/out/target/product/a5ultexx/system/etc/permissions/android.software.live_wallpaper.xml'
Starting build with ninja
ninja: Entering directory `.'
ninja: error: '/home/builder/android/lineage/out/target/product/a5ultexx/obj/SHARED_LIBRARIES/libril_intermediates/export_includes', needed by '/home/builder/android/lineage/out/target/product/a5ultexx/obj/EXECUTABLES/rild_intermediates/import_includes', missing and no know
n rule to make it
build/core/ninja.mk:151: recipe for target 'ninja_wrapper' failed
make: Leaving directory '/home/builder/android/lineage'
make: *** [ninja_wrapper] Error 1

#### make failed to build some targets (50 seconds) ####

Le 2017-10-23 22:04, Jan-Philipp Litzanotifications@github.coma écrit :

That's not really the same thing. This is primarily intended to stop the whole script if any of the commands is failing (-e). The option regarding variables (-u) is more like a habit.

Also, I don't actually use Docker. I just found the bash script useful to build the image.

— You are receiving this because you commented. Reply to this email directly,view it on GitHub(https://github.com/Trois-Six/docker-build-lineage-a5ultexx/pull/1#issuecomment-338779900), ormute the thread(https://github.com/notifications/unsubscribe-auth/ABPmrcKfpVcSOabzbCTifOJYt3tsZs24ks5svPFDgaJpZM4QDRd0).