Rename source to url in repository stanza (#11183)

This aligns the naming of `repository` and `pin`, where pin uses `url`
and repository used `source` for a field with the same semantics.

Signed-off-by: Marek Kubica <[email protected]>

Author: Leonidas-from-XIV

doc/tutorials/dune-package-management/repos/dune-workspace

@@ -5,4 +5,4 @@
 
 (repository
  (name specific-upstream)
- (source "git+https://github.com/ocaml/opam-repository.git#00ac3727bc4ac0eabd3c89e69c1660d6b63a3d48"))
+ (url "git+https://github.com/ocaml/opam-repository.git#00ac3727bc4ac0eabd3c89e69c1660d6b63a3d48"))

src/dune_pkg/workspace.ml

@@ -22,39 +22,38 @@ module Repository = struct
 
   type t =
     { name : Name.t
-    ; source : Loc.t * OpamUrl.t
+    ; url : Loc.t * OpamUrl.t
     }
 
   let name { name; _ } = name
 
-  let to_dyn { name; source = _, source } =
+  let to_dyn { name; url = _, url } =
     let open Dyn in
-    variant "repository" [ Name.to_dyn name; string (OpamUrl.to_string source) ]
+    variant "repository" [ Name.to_dyn name; string (OpamUrl.to_string url) ]
   ;;
 
-  let equal { name; source } t =
-    Name.equal name t.name && Tuple.T2.equal Loc.equal OpamUrl.equal source t.source
+  let equal { name; url } t =
+    Name.equal name t.name && Tuple.T2.equal Loc.equal OpamUrl.equal url t.url
   ;;
 
-  let hash { name; source } = Tuple.T2.hash Name.hash Poly.hash (name, source)
+  let hash { name; url } = Tuple.T2.hash Name.hash Poly.hash (name, url)
 
   let upstream =
     { name = "upstream"
-    ; source =
-        Loc.none, OpamUrl.of_string "git+https://github.com/ocaml/opam-repository.git"
+    ; url = Loc.none, OpamUrl.of_string "git+https://github.com/ocaml/opam-repository.git"
     }
   ;;
 
   let overlay =
     { name = "overlay"
-    ; source =
+    ; url =
         Loc.none, OpamUrl.of_string "git+https://github.com/ocaml-dune/opam-overlays.git"
     }
   ;;
 
   let binary_packages =
     { name = "binary-packages"
-    ; source =
+    ; url =
         ( Loc.none
         , OpamUrl.of_string "git+https://github.com/ocaml-dune/ocaml-binary-packages.git"
         )
@@ -65,9 +64,9 @@ module Repository = struct
     let open Decoder in
     fields
       (let+ name = field "name" Name.decode
-       and+ source = field "source" OpamUrl.decode_loc in
-       { name; source })
+       and+ url = field "url" OpamUrl.decode_loc in
+       { name; url })
   ;;
 
-  let opam_url { source; _ } = source
+  let opam_url { name = _; url } = url
 end

test/blackbox-tests/test-cases/pkg/additional-constraints-ocaml-system.t

@@ -47,7 +47,7 @@ Now make a workspace file adding the constarint on ocaml-system:
   >  (repositories mock))
   > (repository
   >  (name mock)
-  >  (source "file://$(pwd)/mock-opam-repository"))
+  >  (url "file://$(pwd)/mock-opam-repository"))
   > EOF
 
 Solve again. This time ocaml-system is chosen.

test/blackbox-tests/test-cases/pkg/additional-constraints.t

@@ -9,7 +9,7 @@ It's possible to include additional packages or constraints in workspace files:
   >  (repositories mock))
   > (repository
   >  (name mock)
-  >  (source "file://$(pwd)/mock-opam-repository"))
+  >  (url "file://$(pwd)/mock-opam-repository"))
   > EOF
 
   $ mkrepo

test/blackbox-tests/test-cases/pkg/commit-hash-references.t

@@ -23,7 +23,7 @@ Use this ref in a project
   >  (repositories mock))
   > (repository
   >  (name mock)
-  >  (source "git+file://$PWD/mock-opam-repository#$AMBIGUOUS_REF"))
+  >  (url "git+file://$PWD/mock-opam-repository#$AMBIGUOUS_REF"))
   > (context
   >  (default
   >   (name default)))

test/blackbox-tests/test-cases/pkg/describe-pkg-lock.t

@@ -27,7 +27,7 @@ First we setup a repo.
   >  (repositories mock))
   > (repository
   >  (name mock)
-  >  (source "file://$(pwd)/mock-opam-repository"))
+  >  (url "file://$(pwd)/mock-opam-repository"))
   > EOF
 
 Here is the output of solving for multiple contexts:

test/blackbox-tests/test-cases/pkg/duplicate-repositories.t

@@ -4,10 +4,10 @@ Duplicate repository definition in the same workspace file:
   > (lang dune 3.11)
   > (repository
   >  (name foo)
-  >  (source "git+file//$PWD/foo"))
+  >  (url "git+file//$PWD/foo"))
   > (repository
   >  (name foo)
-  >  (source "git+file//$PWD/foo"))
+  >  (url "git+file//$PWD/foo"))
   > EOF
 
   $ dune pkg outdated 2>&1 | awk '/Internal error/,/Raised/'

test/blackbox-tests/test-cases/pkg/env-conditional-dependencies.t

@@ -51,7 +51,7 @@ Create a workspace config that defines separate build contexts for macos and lin
   >   (lock_dir dune.macos.lock)))
   > (repository
   >  (name mock)
-  >  (source "file://$(pwd)/mock-opam-repository"))
+  >  (url "file://$(pwd)/mock-opam-repository"))
   > EOF
 
 Now the os-specific dependencies are included on their respective systems.

test/blackbox-tests/test-cases/pkg/git-repo.t

@@ -19,7 +19,7 @@ We'll set up a project that uses (only this) this repository, so doesn't use
   >  (repositories mock))
   > (repository
   >  (name mock)
-  >  (source "git+file://$PWD/mock-opam-repository"))
+  >  (url "git+file://$PWD/mock-opam-repository"))
   > (context
   >  (default
   >   (name default)))

test/blackbox-tests/test-cases/pkg/helpers.sh

@@ -52,7 +52,7 @@ add_mock_repo_if_needed() {
  (repositories mock))
 (repository
  (name mock)
- (source "file://$(pwd)/mock-opam-repository"))
+ (url "file://$(pwd)/mock-opam-repository"))
 EOF
   else
     if ! grep '(name mock)' > /dev/null dune-workspace
@@ -61,7 +61,7 @@ EOF
       cat >>dune-workspace <<EOF
 (repository
  (name mock)
- (source "file://$(pwd)/mock-opam-repository"))
+ (url "file://$(pwd)/mock-opam-repository"))
 EOF
 
       # reference the repo