From 701dbedf76d8f1b317728687d9bac5a49b696b7f Mon Sep 17 00:00:00 2001 From: Matteo Nastasi Date: Sun, 13 Sep 2026 15:16:42 +0000 Subject: [PATCH] say so when installing a site without its images brisk-img carries every image of the site: the cards, the table, the icons. INSTALL.sh picks it up from beside this repository, and when it is not there it used to skip it without a word - so a run that had quietly dropped every image looked exactly like a good one, and the site came up with each img answering 404. That is how this went unnoticed: the bench was moved into the container with only the brisk repository mounted, brisk-img was left outside, and several runs of INSTALL.sh rebuilt the tree without images while reporting success. It now warns, and says what to do about it. curl-de-sac gets a milder note on the same spot: the code that uses it is guarded by defined('CURL_DE_SAC_VERS'), so without it the feature is simply absent rather than broken. Co-Authored-By: Claude Opus 5 Claude-Session: https://claude.ai/code/session_01Fseposbqj9mBbQXoT8KnBj --- INSTALL.sh | 15 +++++++++++++++ 1 file changed, 15 insertions(+) diff --git a/INSTALL.sh b/INSTALL.sh index 4095c73..d1aa09a 100755 --- a/INSTALL.sh +++ b/INSTALL.sh @@ -514,11 +514,24 @@ if [ -d docroot ] && [ ! -z "$document_root" ]; then done fi +# brisk-img carries every image of the site: the cards, the table, the icons. +# Without it the tree installed here is complete as far as the code goes and +# has no image at all, so the pages come up and every img answers 404. It used +# to be skipped without a word, and a run that had quietly dropped the images +# looked exactly like a good one. if [ -d ../brisk-img ]; then cd ../brisk-img ./INSTALL.sh -w ${web_path}__ cd - >/dev/null 2>&1 +else + echo + echo "WARNING: ../brisk-img not found: INSTALLING A SITE WITHOUT IMAGES." + echo " The cards and every other image will answer 404. Put the" + echo " brisk-img repository beside this one and run again." + echo fi +# curl-de-sac is optional: the code that uses it is guarded by +# defined('CURL_DE_SAC_VERS'), so without it the feature is simply absent. if [ -d ../curl-de-sac ]; then cd ../curl-de-sac if [ ! -z "$conffile_in" ]; then @@ -527,6 +540,8 @@ if [ -d ../curl-de-sac ]; then ./INSTALL.sh -w ${web_path}__ fi cd - >/dev/null 2>&1 +else + echo "note: ../curl-de-sac not found, the site is installed without it" fi # config file installation or diff -- 2.47.3