Skip to content

Commit 172dfbb

Browse files
authored
Merge pull request #12652 from Alizter/push-sunuzwqktqqn
test(pkg): don't use (pkg enabled) by default
2 parents 821ae4f + bce0825 commit 172dfbb

File tree

4 files changed

+8
-12
lines changed

4 files changed

+8
-12
lines changed

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

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -105,7 +105,6 @@ add_mock_repo_if_needed() {
105105
then
106106
cat >dune-workspace <<EOF
107107
(lang dune 3.20)
108-
(pkg enabled)
109108
(lock_dir
110109
(repositories mock))
111110
(repository
@@ -140,7 +139,6 @@ create_mock_repo() {
140139
repo="${1:-file://$(pwd)/mock-opam-repository}"
141140
cat >dune-workspace <<EOF
142141
(lang dune 3.20)
143-
(pkg enabled)
144142
(lock_dir
145143
(repositories mock))
146144
(repository

test/blackbox-tests/test-cases/pkg/invalid-opam-repo-errors.t/run.t

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -19,8 +19,8 @@ Test the error cases for invalid opam repositories
1919
> }
2020

2121
$ lock
22-
File "dune-workspace", line 7, characters X-X:
23-
7 | (url ..))
22+
File "dune-workspace", line 6, characters X-X:
23+
6 | (url ..))
2424
Error:
2525
$TESTCASE_ROOT/directory-that-does-not-exist
2626
does not exist
@@ -30,8 +30,8 @@ Test the error cases for invalid opam repositories
3030
$ rm dune-workspace
3131
$ add_mock_repo_if_needed "file://$(pwd)/empty"
3232
$ lock
33-
File "dune-workspace", line 7, characters X-X:
34-
7 | (url ..))
33+
File "dune-workspace", line 6, characters X-X:
34+
6 | (url ..))
3535
Error:
3636
$TESTCASE_ROOT/empty
3737
is not a directory
@@ -40,8 +40,8 @@ Test the error cases for invalid opam repositories
4040
$ rm dune-workspace
4141
$ add_mock_repo_if_needed "file://$(pwd)/no-packages-dir"
4242
$ lock
43-
File "dune-workspace", line 7, characters X-X:
44-
7 | (url ..))
43+
File "dune-workspace", line 6, characters X-X:
44+
6 | (url ..))
4545
Error:
4646
$TESTCASE_ROOT/no-packages-dir
4747
doesn't look like a path to an opam repository as it lacks a subdirectory

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

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -106,8 +106,6 @@ EOF
106106
(repository
107107
(name mock)
108108
(url "file://$(pwd)/mock-opam-repository"))
109-
110-
(pkg enabled)
111109
EOF
112110
}
113111

test/blackbox-tests/test-cases/pkg/pkg-disabled-workflow.t

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -75,8 +75,8 @@ Test that dune pkg enabled reports disabled:
7575
Test that pkg commands fail when disabled:
7676

7777
$ dune pkg lock
78-
File "dune-workspace", line 2, characters 5-13:
79-
2 | (pkg disabled)
78+
File "dune-workspace", line 7, characters 5-13:
79+
7 | (pkg disabled)
8080
^^^^^^^^
8181
Error: Package management is disabled in workspace configuration.
8282
Hint: To enable package management, remove the explicit (pkg disabled)

0 commit comments

Comments
 (0)