search

alsuren / mijia-homie

A Homie MQTT bridge for the Xiaomi Mijia 2 hygrometer-thermometer. This repo also serves as the monorepo for a handful of helper crates related to Homie and Bluetooth.
Other
65 stars 14 forks source link

Fix ldd auto-dependency warning for armv7 Debian package #56

Open qwandor opened 4 years ago

qwandor commented 4 years ago

As mentioned on #55, building a Debian package for armv7 gives a warning that ldd doesn't recognise the binary format. This means that cargo-deb's automatic dependency feature doesn't work. We need to figure out how do get an ldd which supports other architectures, or patch cargo-deb to use something else.

TARGET=armv7-unknown-linux-gnueabihf ./package.sh
...
warning: ldd (/home/qwandor/projects/reading-xiaomi-temp/target/armv7-unknown-linux-gnueabihf/release/mijia-homie):  (no auto deps for /home/qwandor/projects/reading-xiaomi-temp/target/armv7-unknown-linux-gnueabihf/release/mijia-homie)
qwandor commented 3 years ago

Relatedly, homie-influx ends up depending on the wrong version of libssl1.1 (>= 1.1.1g, while Debian stable only has 1.1.1d), which makes it fail to install. The problem is that we run cargo deb in the GitHub actions host rather than in Docker, so ldd does the wrong thing.

Maybe we could patch cargo deb to use something different? Or as a temporary fix, hard-code the dependencies rather than using the automatic dependency feature.