diff options
| author | Profpatsch <mail@profpatsch.de> | 2024-05-20T13·50+0200 |
|---|---|---|
| committer | clbot <clbot@tvl.fyi> | 2024-05-22T10·32+0000 |
| commit | 5b2ba0efa1b87aa514d665a7f64ada36617c720e (patch) | |
| tree | 27a4bb08f20e997377cc1bc884ee16c38e203f86 /tvix/Cargo.lock | |
| parent | e7be3422566b36e5bd3aeaaf7d47537dfd050a5c (diff) | |
refactor(tvix/eval): rewrite xml emitter to be simple-stupid r/8160
In order to be compatible with the nix XML generator, it’s easier to generate the XML directly, instead of going through a library which we have to bend to do what we need. Removes dependency on `xml-rs`, which came with a full XML parser that we didn’t use. Only takes a tiny bit of code for the XML escaping, somewhat simplified. I add a little escaping value, to make sure we have the same behaviour as nix proper. Interestingly enough, we never need to escape XML attribute names, because the `builtins.toXML` format encodes user-defined values as attribute keys only. So we only escape attribute values. Fixes: https://b.tvl.fyi/issues/399 Change-Id: If4d407d324864b3bb9aa3160e2ec6889f7727127 Reviewed-on: https://cl.tvl.fyi/c/depot/+/11697 Tested-by: BuildkiteCI Reviewed-by: flokli <flokli@flokli.de> Autosubmit: Profpatsch <mail@profpatsch.de>
Diffstat (limited to 'tvix/Cargo.lock')
| -rw-r--r-- | tvix/Cargo.lock | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/tvix/Cargo.lock b/tvix/Cargo.lock index dc5298c45b5b..9ad121371264 100644 --- a/tvix/Cargo.lock +++ b/tvix/Cargo.lock @@ -4167,7 +4167,6 @@ dependencies = [ "test-strategy", "toml", "tvix-eval-builtin-macros", - "xml-rs", ] [[package]] @@ -4845,12 +4844,6 @@ dependencies = [ ] [[package]] -name = "xml-rs" -version = "0.8.19" -source = "registry+https://github.com/rust-lang/crates.io-index" -checksum = "0fcb9cbac069e033553e8bb871be2fbdffcab578eb25bd0f7c508cedc6dcd75a" - -[[package]] name = "xz2" version = "0.1.7" source = "registry+https://github.com/rust-lang/crates.io-index" |