Skip to content

Commit

Permalink
Several improvements:
Browse files Browse the repository at this point in the history 
- Use command yarnpkg on Debian 12+
- Add --skip-yarn-install option to use-git subcommand to skip installing NPM deps in the container
- Add use -y on apt command to avoid asking confirmation when installing yarn
- Always 'systemctl start yunohost-api' on 'use-git yunohost-admin'
  • Loading branch information
@christophehenry
christophehenry committed Feb 16, 2025
1 parent 252fc44 commit 9f66625
Showing 1 changed file with 51 additions and 30 deletions.
81 changes: 51 additions & 30 deletions ynh-dev
View file
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,12 @@

# shellcheck disable=SC2155,SC2034,SC2164

if (( $(lsb_release -rs) >= 12 )); then
YARN_CMD=yarnpkg
else
YARN_CMD=yarn
fi

function show_usage() {
cat <<EOF
Expand All @@ -17,7 +23,9 @@ function show_usage() {
${BLUE}=======================${NORMAL}
ip Give the ip of the guest container
use-git [PKG] Use Git repositories from dev environment path
use-git [--skip-yarn-install,-s] [PKG]
Use Git repositories from dev environment path.
-s option skips yarn install in case you want to install NPM packages outside the container yourself.
lint [PKG] Lint source code from core or app packages.
e.g. ./ynh-dev lint yunohost
e.g. ./ynh-dev lint nextcloud_ynh
Expand Down Expand Up @@ -132,29 +140,33 @@ function create_sym_link() {
function prepare_cache_and_deps() {
local DEV_PATH="$1"
local CACHE_PATH="$2"
local SKIP_YARN_INSTALL=${3:-false}

mkdir -p "$CACHE_PATH"
# create_sym_link "$DEV_PATH/.env" "$CACHE_PATH/.env"
create_sym_link "$CACHE_PATH/node_modules" "$DEV_PATH/node_modules"

create_sym_link "$DEV_PATH/package.json" "$CACHE_PATH/package.json"
create_sym_link "$DEV_PATH/yarn.lock" "$CACHE_PATH/yarn.lock"


# Vite require node v14 to parse modern syntax
local DISTRO="$(lsb_release -s -c)"

# install yarn if not already
if [[ $(dpkg-query -W -f='${Status}' yarn 2>/dev/null | grep -c "ok installed") -eq 0 ]];
if [[ $(dpkg-query -W -f='${Status}' $YARN_CMD 2>/dev/null | grep -c "ok installed") -eq 0 ]];
then
info "Installing yarn…"
apt update
apt install yarn
info "Installing $YARN_CMD…"
apt update -y
apt install -y $YARN_CMD
fi

if [[ $SKIP_YARN_INSTALL != true ]]; then
create_sym_link "$CACHE_PATH/node_modules" "$DEV_PATH/node_modules"
pushd "$CACHE_PATH"
# Install dependencies with yarn forced to lock file versions (equivalent to `npm ci`)
info "Installing dependencies ... (this may take a while)"
$YARN_CMD install --frozen-lockfile
popd
fi

pushd "$CACHE_PATH"
# Install dependencies with yarn forced to lock file versions (equivalent to `npm ci`)
info "Installing dependencies ... (this may take a while)"
yarn install --frozen-lockfile
popd
}

##################################################################
Expand Down Expand Up @@ -272,7 +284,20 @@ function show_vm_ip()
function use_git()
{
assert_inside_vm
local PACKAGES=("$@")
local ARGS=("$@")
local SKIP_YARN_INSTALL=false
declare -a PACKAGES

for i in "${!ARGS[@]}"; do
case ${ARGS[i]} in
--skip-yarn-install|-s)
SKIP_YARN_INSTALL=true
;;
*)
PACKAGES+=(${ARGS[i]})
;;
esac
done

if [ "${#PACKAGES[@]}" -eq 0 ]; then
PACKAGES=('moulinette' 'ssowat' 'yunohost' 'yunohost-admin')
Expand Down Expand Up @@ -338,17 +363,10 @@ function use_git()

local DEV_PATH="/ynh-dev/yunohost-admin/app"
local CACHE_PATH="/var/cache/ynh-dev/yunohost-admin"

create_sym_link "/ynh-dev/yunohost-admin/app/.env" "/var/cache/ynh-dev/yunohost-admin/.env"
prepare_cache_and_deps "$DEV_PATH" "$CACHE_PATH"
prepare_cache_and_deps "$DEV_PATH" "$CACHE_PATH" $SKIP_YARN_INSTALL

cd "$CACHE_PATH"

# Inject container ip in .env file
# Used by vite to expose itself on network and proxy api requests.
IP=$(hostname -I | tr ' ' '\n' | grep "\.")
echo "VITE_IP=$IP" > .env

# Allow port 8080 in config file or else the dev server will stop working after postinstall
if [[ ! -e /etc/yunohost/installed ]]
then
Expand All @@ -365,11 +383,14 @@ with open(setting_file) as f:
EOF
fi

local DISTRO="$(lsb_release -s -c)"

cd "$DEV_PATH"
systemctl start yunohost-api
# Inject container ip in .env file
# Used by vite to expose itself on network and proxy api requests.
echo "VITE_IP=$(hostname -I | tr ' ' '\n' | grep "\.")" > "$DEV_PATH/.env"
create_sym_link "$DEV_PATH/.env" "$CACHE_PATH/.env"
info "Now running dev server"
yarn dev --host
$YARN_CMD dev --host &
;;
yunohost-admin-build)
if [[ ! -e "/usr/share/yunohost/admin-bkp" ]]
Expand All @@ -381,7 +402,7 @@ EOF
local DISTRO="$(lsb_release -s -c)"

cd /ynh-dev/yunohost-admin/app
yarn build
$YARN_CMD build

create_sym_link "/ynh-dev/yunohost-admin/app/dist" "/usr/share/yunohost/admin"

Expand Down Expand Up @@ -411,11 +432,11 @@ If not already, add your instance's IP into '/etc/yunohost/.portal-api-allowed-c
$IP $MAIN_DOMAIN"
fi

prepare_cache_and_deps "$DEV_PATH" "$CACHE_PATH"
prepare_cache_and_deps "$DEV_PATH" "$CACHE_PATH" $SKIP_YARN_INSTALL

cd "$DEV_PATH"
info "Now running dev server"
yarn dev --host
$YARN_CMD dev --host
;;
yunohost-portal-build)
assert_yunohost_is_installed
Expand All @@ -430,10 +451,10 @@ $IP $MAIN_DOMAIN"
mv "$SOURCE_PATH" "$SOURCE_PATH-bkp"
fi

prepare_cache_and_deps "$DEV_PATH" "$CACHE_PATH"
prepare_cache_and_deps "$DEV_PATH" "$CACHE_PATH" $SKIP_YARN_INSTALL

cd "$DEV_PATH"
yarn generate
$YARN_CMD generate

create_sym_link "$DEV_PATH/.output/public" "$SOURCE_PATH"

Expand Down

0 comments on commit 9f66625

Please sign in to comment.