[Opam] Don't generate the dune-site build part when not needed

       Fixes #4849

Signed-off-by: François Bobot <[email protected]>

Author: bobot

CHANGES.md

@@ -1,6 +1,9 @@
 Unreleased
 ----------
 
+- Don't generate the dune-site build part when not needed (#4861, @bobot,
+  review @kit-ty-kate)
+
 - Fixes `opam` META file requires entry for private libs (#4841, fixes #4839, @toots)
 
 - Fixes `dune exec` not adding .exe on Windows (#4371, fixes #3322, @MisterDA)

otherlibs/site/test/run.t

@@ -3,26 +3,37 @@ Test embedding of sites locations information
 
   $ mkdir -p a b c
 
-  $ for i in a b d; do
+  $ mkdir -p a
+
+  $ cat >a/dune-project <<EOF
+  > (lang dune 3.0)
+  > (generate_opam_files true)
+  > (using dune_site 0.1)
+  > (name a)
+  > (version 0.a)
+  > (package (name a) (sites (share data)))
+  > EOF
+
+  $ for i in b d; do
   > mkdir -p $i
   > cat >$i/dune-project <<EOF
-  > (lang dune 2.9)
+  > (lang dune 3.0)
   > (generate_opam_files true)
   > (using dune_site 0.1)
   > (name $i)
   > (version 0.$i)
-  > (package (name $i) (sites (share data)))
+  > (package (name $i) (sites (share data)) (depends c))
   > EOF
   > done
 
   $ for i in c; do
   >   mkdir -p $i
   >   cat >$i/dune-project <<EOF
-  > (lang dune 2.9)
+  > (lang dune 3.0)
   > (generate_opam_files true)
   > (using dune_site 0.1)
   > (name $i)
-  > (package (name $i) (sites (share data) (lib plugins)))
+  > (package (name $i) (sites (share data) (lib plugins)) (depends a))
   > EOF
   > done
 
@@ -138,7 +149,7 @@ Test with an opam like installation
   opam-version: "2.0"
   version: "0.a"
   depends: [
-    "dune" {>= "2.9"}
+    "dune" {>= "3.0"}
     "odoc" {with-doc}
   ]
   build: [
@@ -150,16 +161,15 @@ Test with an opam like installation
       name
       "-j"
       jobs
-      "--promote-install-files"
-      "false"
       "@install"
       "@runtest" {with-test}
       "@doc" {with-doc}
+      "--promote-install-files=false"
     ]
     ["dune" "install" "-p" name "--create-install-files" name]
   ]
 
-  $ dune build -p a --promote-install-files "false" @install
+  $ dune build -p a --promote-install-files=false @install
 
   $ test -e a/a.install
   [1]
@@ -377,7 +387,7 @@ Test %{version:installed-pkg}
   $ for i in f; do
   >   mkdir -p $i
   >   cat >$i/dune-project <<EOF
-  > (lang dune 2.9)
+  > (lang dune 3.0)
   > (using dune_site 0.1)
   > (name $i)
   > (version 0.$i)