Merge pull request #82 from selfhoster1312/no-dep-which

More portable dependency checks
This commit is contained in:
Alexandre Aubin 2024-05-08 18:06:07 +02:00 committed by GitHub
commit 021514666f
No known key found for this signature in database
GPG key ID: B5690EEEBB952194

11
ynh-dev
View file

@ -180,10 +180,10 @@ function prepare_cache_and_deps() {
function check_incus_setup()
{
# Check incus is installed somehow
if ! which incus &>/dev/null; then
if ! command -v incus &>/dev/null; then
critical "You need to have Incus installed for ynh-dev to be usable from the host machine. Refer to the README to know how to install it."
fi
if ! id -nG "$(whoami)" | grep -qw "incus-admin"; then
if ! id -nG "$(whoami)" | grep -qw "incus-admin" && [ ! $(id -u) -eq 0 ]; then
critical "You need to be in the incus-admin group!"
fi
@ -195,6 +195,11 @@ function check_incus_setup()
function set_incus_remote()
{
# Check jq is installed somehow
if ! command -v jq &>/dev/null; then
critical "You need jq installed for ynh-dev"
fi
configured=$(incus remote list -f json | jq 'has("yunohost")')
if [[ "$configured" != "true" ]]; then
incus remote add yunohost https://devbaseimgs.yunohost.org --public
@ -455,7 +460,7 @@ function dev()
{
assert_inside_vm
which inotifywait &>/dev/null || critical "You should first run: apt install inotify-tools"
command -v inotifywait &>/dev/null || critical "You should first run: apt install inotify-tools"
info "Now monitoring for changes in python files, restarting yunohost-api and yunohost-portal-api when changes occur!"