Skip to content

Commit

Permalink
xs: do not package ezxenstore, it's a part of xen-api
Browse files Browse the repository at this point in the history
This allows removing xenctrl.dummy and all the special code needed to
handle the duplicate package

Signed-off-by: Pau Ruiz Safont <[email protected]>
  • Loading branch information
psafont committed Apr 5, 2023
1 parent bfa47f8 commit 0f766d3
Show file tree
Hide file tree
Showing 7 changed files with 2 additions and 27 deletions.
1 change: 0 additions & 1 deletion .github/duplicates-upstream-extra.txt
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
xenctrl
ocaml
ocaml-base-compiler
ocaml-system
1 change: 0 additions & 1 deletion .github/duplicates-xs-toolstack.txt
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
xenctrl
ocaml
ocaml-base-compiler
ocaml-system
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,5 @@ An easy-to-use xenstore library with a simplified interface geared
towards use within a daemon that maintains a single connection to
xenstored."""
url {
src:
"https://github.com/xapi-project/ezxenstore/archive/v0.4.2.tar.gz"
checksum: "sha256=709ba936f8b1a3c76e31c655752294287f215601b3238204688818676c6e7d97"
src: "https://github.com/xapi-project/xen-api/archive/master.tar.gz"
}
18 changes: 0 additions & 18 deletions packages/xs/xenctrl.dummy/opam

This file was deleted.

1 change: 0 additions & 1 deletion packages/xs/xenctrl.dummy/url.local

This file was deleted.

2 changes: 1 addition & 1 deletion tools/cleanup.sh
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,6 @@ set -euo pipefail
opam admin check --ignore-test-doc
# there is a conflict in ppxlib and sexplib0 version in with-test, ignore its tests for now
RESOLVE=$(opam admin filter --verbose --dry-run --resolve=xs-toolstack | sed -n 2p | grep -v ppxlib)
RESOLVE+=" upstream-extra-dummy xenctrl.dummy opam-depext opam-ed.0.3"
RESOLVE+=" upstream-extra-dummy opam-depext opam-ed.0.3"
RESOLVE+=" ocaml-base-compiler.${OCAML_VERSION_FULL} ocaml-system.${OCAML_VERSION_FULL}"
exec opam admin filter --dry-run "--resolve=$(echo ${RESOLVE} | tr ' ' ',')" --or --with-test
2 changes: 0 additions & 2 deletions tools/no-duplicates-check.bash
Original file line number Diff line number Diff line change
Expand Up @@ -16,11 +16,9 @@ function check_no_duplicates {
}

# makes sure no ordinary opam packages have more than a single version
# the only exception is xenctrl.
# This is required because of how the rpm distributions are built: they
# try to install all versions of all ordinary packages, which fails because
# the versions of a single package are never compatible.
# xenctrl has 2 versions: dummy and master
# several versions of the compiler might be wanted for testing:
# ocaml, ocaml-system, ocaml-base-compiler
toolstack_packages=$(opam list --required-by "xs-toolstack" --recursive --short)
Expand Down

0 comments on commit 0f766d3

Please sign in to comment.