Skip to content
New issue

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

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

Already on GitHub? # to your account

xapi-stdext: update to 4.21.0 #617

Merged
merged 1 commit into from
Dec 1, 2022
Merged

Conversation

psafont
Copy link
Member

@psafont psafont commented Nov 30, 2022

  • date: consolidate the types into a single t
  • date: add conversion functions that have semantic meaning, the previous functions containing 'float' and 'string' will be deprecated in a future release.
  • unix: add permissions to write_{bytes,string}_to_file
  • threads, unix: avoid using C functions deprecated in OCaml 5
  • zerocheck: remove wrong, unused code. It was dangerous to leave it available

This is a needs to be merged along with xapi-project/xen-api#4857

- date: consolidate the types into a single t
- date: add conversion functions that have semantic meaning, the
  previous functions containing 'float' and 'string' will be
  deprecated in a future release.
- unix: add permissions to write_{bytes,string}_to_file
- threads, unix: avoid using C functions deprecated in OCaml 5
- zerocheck: remove wrong, unused code. It was dangerous to leave it available

Signed-off-by: Pau Ruiz Safont <pau.safont@citrix.com>
@lindig lindig self-requested a review November 30, 2022 11:24
@psafont
Copy link
Member Author

psafont commented Dec 1, 2022

@snwoods has tested this locally and it works together with the xapi PR.

I'll merge this, kick xapi's CI again. when that's passing I'll merge it and kick this repo's CI to make all the CI green

@psafont psafont merged commit 520ab35 into xapi-project:master Dec 1, 2022
@psafont psafont deleted the stdext421 branch December 1, 2022 15:53
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants