]> mop.ddnsfree.com - git repositories - brisk.git/commitdiff
say so when installing a site without its images
authorMatteo Nastasi <nastasi@alternativeoutput.it>
Sun, 13 Sep 2026 15:16:42 +0000 (15:16 +0000)
committerMatteo Nastasi <nastasi@alternativeoutput.it>
Sun, 13 Sep 2026 15:16:42 +0000 (15:16 +0000)
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 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Fseposbqj9mBbQXoT8KnBj

INSTALL.sh

index 4095c739db0cdda41c847cd8c66b0863fb25c3d3..d1aa09a20820529d756b1d7beb457082b0087249 100755 (executable)
@@ -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