diff --git a/.github/workflows/workflow.yml b/.github/workflows/workflow.yml index d0200de66..ff1e49e19 100644 --- a/.github/workflows/workflow.yml +++ b/.github/workflows/workflow.yml @@ -91,7 +91,7 @@ jobs: - run: opam install lwt_react lwt --deps-only --with-test - - run: opam install lwt_ppx lwt_ppx_let --deps-only --with-test + - run: opam install lwt_ppx --deps-only --with-test if: ${{ matrix.ppx == true }} - run: opam exec -- dune build --only-packages lwt_react,lwt @@ -104,9 +104,6 @@ jobs: - run: opam exec -- dune runtest --only-packages lwt_ppx if: ${{ matrix.ppx == true }} - - run: opam exec -- make ppx_let-test - if: ${{ matrix.ppx == true }} - lint-opam: runs-on: ubuntu-latest diff --git a/dune-project b/dune-project index 41f55cfaa..c7f9767e8 100644 --- a/dune-project +++ b/dune-project @@ -12,21 +12,13 @@ (source (github ocsigen/lwt)) (documentation "https://ocsigen.org/lwt") -(package - (name lwt_ppx_let) - (synopsis "Dummy package context for ppx_let tests") - (description "Internal package used to partition ppx_let tests.") - (depends - (ocaml (>= 4.08)) - (ppx_let :with-test) - lwt)) - (package (name lwt_ppx) (synopsis "PPX syntax for Lwt, providing something similar to async/await from JavaScript") (depends (ocaml (>= 4.08)) (ppxlib (>= 0.16.0)) + (ppx_let :with-test) lwt)) (package diff --git a/lwt.opam.template b/lwt.opam.template index dc64b241a..8eeecc475 100644 --- a/lwt.opam.template +++ b/lwt.opam.template @@ -22,3 +22,4 @@ build: [ "@doc" {with-doc} ] ] +available: os-family != "windows" | "ocaml-version" < "5.2" diff --git a/lwt_ppx.opam b/lwt_ppx.opam index e12253630..4bdd4df72 100644 --- a/lwt_ppx.opam +++ b/lwt_ppx.opam @@ -14,8 +14,10 @@ depends: [ "dune" {>= "2.0"} "ocaml" {>= "4.08"} "ppxlib" {>= "0.16.0"} + "ppx_let" {with-test} "lwt" ] +dev-repo: "git+https://github.com/ocsigen/lwt.git" build: [ ["dune" "subst"] {dev} [ @@ -30,4 +32,3 @@ build: [ "@doc" {with-doc} ] ] -dev-repo: "git+https://github.com/ocsigen/lwt.git" diff --git a/lwt_ppx.opam.template b/lwt_ppx.opam.template new file mode 100644 index 000000000..87c8e5dcd --- /dev/null +++ b/lwt_ppx.opam.template @@ -0,0 +1,14 @@ +build: [ + ["dune" "subst"] {dev} + [ + "dune" + "build" + "-p" + name + "-j" + jobs + "@install" + "@runtest" {with-test} + "@doc" {with-doc} + ] +] diff --git a/lwt_ppx_let.opam b/lwt_ppx_let.opam deleted file mode 100644 index 4d5e92375..000000000 --- a/lwt_ppx_let.opam +++ /dev/null @@ -1,33 +0,0 @@ -# This file is generated by dune, edit dune-project instead -opam-version: "2.0" -synopsis: "Dummy package context for ppx_let tests" -description: "Internal package used to partition ppx_let tests." -maintainer: [ - "Raphaël Proust " "Anton Bachin " -] -authors: ["Jérôme Vouillon" "Jérémie Dimino"] -license: "MIT" -homepage: "https://github.com/ocsigen/lwt" -doc: "https://ocsigen.org/lwt" -bug-reports: "https://github.com/ocsigen/lwt/issues" -depends: [ - "dune" {>= "2.0"} - "ocaml" {>= "4.08"} - "ppx_let" {with-test} - "lwt" -] -build: [ - ["dune" "subst"] {dev} - [ - "dune" - "build" - "-p" - name - "-j" - jobs - "@install" - "@runtest" {with-test} - "@doc" {with-doc} - ] -] -dev-repo: "git+https://github.com/ocsigen/lwt.git" diff --git a/lwt_react.opam b/lwt_react.opam index bb73d0460..39a9f58d9 100644 --- a/lwt_react.opam +++ b/lwt_react.opam @@ -16,6 +16,7 @@ depends: [ "lwt" {>= "3.0.0"} "react" {>= "1.0.0"} ] +dev-repo: "git+https://github.com/ocsigen/lwt.git" build: [ ["dune" "subst"] {dev} [ @@ -30,4 +31,3 @@ build: [ "@doc" {with-doc} ] ] -dev-repo: "git+https://github.com/ocsigen/lwt.git" diff --git a/lwt_react.opam.template b/lwt_react.opam.template new file mode 100644 index 000000000..87c8e5dcd --- /dev/null +++ b/lwt_react.opam.template @@ -0,0 +1,14 @@ +build: [ + ["dune" "subst"] {dev} + [ + "dune" + "build" + "-p" + name + "-j" + jobs + "@install" + "@runtest" {with-test} + "@doc" {with-doc} + ] +] diff --git a/test/ppx_let/dune b/test/ppx_let/dune index b4147a508..fb4f0e71d 100644 --- a/test/ppx_let/dune +++ b/test/ppx_let/dune @@ -5,5 +5,5 @@ (rule (alias runtest) - (package lwt_ppx_let) + (package lwt_ppx) ;; technically not part of lwt_ppx, but we want it tested and the dependency to ppxlib is already there (action (run %{exe:test.exe})))