xml icon indicating copy to clipboard operation
xml copied to clipboard

implement serialization

Open oli-obk opened this issue 10 years ago • 11 comments

deserializing xml is great, but we also need serialization.

oli-obk avatar May 26 '15 19:05 oli-obk

Hi. This issue has been opened for more than a year. What is the progress on this? What help might be needed?

johansja avatar Aug 29 '16 14:08 johansja

Unfortunately I have no need for serialization. Therefor I don't think I'm very qualified to make decisions on how to serialize certain things.

I think we could just copy the json Serializers so we get the pretty printing for free and then adjust it to produce xml tags.

There is the big question on how to decide which things go into attributes and which go into inner tags. I don't have a good solution to this problem

oli-obk avatar Sep 05 '16 09:09 oli-obk

Interesting on the last point. Can we have a custom tag for that?

Do you know anyone who might be knowledgeable in this?

johansja avatar Sep 06 '16 03:09 johansja

we could use the #[serde(rename="@foo")] method that I'm using for inner text.

oli-obk avatar Sep 06 '16 08:09 oli-obk

Good. I am just starting in Rust and maybe this will be a good starting point for me too. What do you think?

johansja avatar Sep 06 '16 08:09 johansja

I'm happy for any contributions. If you have any questions, just ask. I'm also available on irc.mozilla.org on #serde

oli-obk avatar Sep 06 '16 09:09 oli-obk

No progress on this yet?

softprops avatar Feb 05 '17 04:02 softprops

Nope, gonna do some serde 0.9 changes first + probably break a lot of code, because the current design is somewhat unfortunate.

oli-obk avatar Feb 06 '17 08:02 oli-obk

@oli-obk What's the current status of this?

blanham avatar May 23 '17 01:05 blanham

This entire repository is deprecated in favour of serde-xml-rs which does serialization

oli-obk avatar May 23 '17 05:05 oli-obk

@oli-obk I'm afraid that's not correct, serde-xml-rs currently implements only deserialization. There was some unfinished attempt in a PR, but that's it for now.

RReverser avatar Jun 05 '17 17:06 RReverser