Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

xs-extra: update metadata from xen-api.git #703

Merged
merged 1 commit into from
Oct 2, 2024

Conversation

psafont
Copy link
Member

@psafont psafont commented Oct 1, 2024

This updates the metadata to be up-to-date with xapi's

@psafont psafont force-pushed the remove-packages branch 2 times, most recently from a0dc667 to 4ff2b4a Compare October 1, 2024 14:45
@psafont psafont force-pushed the remove-packages branch 2 times, most recently from 386c48f to 724625d Compare October 1, 2024 15:06
@psafont psafont changed the title xs-extra: remove disappearing packages xs-extra: update metadata from xen-api.git Oct 1, 2024
@psafont
Copy link
Member Author

psafont commented Oct 1, 2024

xapi-tools fails to build because of

File "ocaml/xenopsd/scripts/dune", line 5, characters 3-6:
5 | 		(vif as vif)
       ^^^

Not sure why, the file is present in that directory

@psafont psafont force-pushed the remove-packages branch 2 times, most recently from 945750a to 7e3c1ae Compare October 2, 2024 09:42
@psafont
Copy link
Member Author

psafont commented Oct 2, 2024

The failure is very annoying, I can reproduce it when executing the installation of the package with opam, but now when executing the dune command that opam supposedly executes. This seems to be a problem with the environment of some sort that fails to find the vif file

It's not present in cloned repository that opam makes, this makes sense because it's ignored by git, so it's not present in the tarballs made by github:

https://github.com/xapi-project/xen-api/blob/master/.gitignore#L25

Quite a few packages were removed, and a couple of ones to replace them
were added.

Signed-off-by: Pau Ruiz Safont <[email protected]>
@psafont psafont marked this pull request as ready for review October 2, 2024 13:37
@psafont psafont merged commit 00e4051 into xapi-project:master Oct 2, 2024
3 checks passed
@psafont psafont deleted the remove-packages branch October 2, 2024 13:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants