Skip to content
This repository has been archived by the owner on May 6, 2020. It is now read-only.

implement serialization #3

Open
oli-obk opened this issue May 26, 2015 · 11 comments
Open

implement serialization #3

oli-obk opened this issue May 26, 2015 · 11 comments

Comments

@oli-obk
Copy link

oli-obk commented May 26, 2015

deserializing xml is great, but we also need serialization.

@johansja
Copy link

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

@oli-obk
Copy link
Author

oli-obk commented Sep 5, 2016

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

@johansja
Copy link

johansja commented Sep 6, 2016

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

Do you know anyone who might be knowledgeable in this?

@oli-obk
Copy link
Author

oli-obk commented Sep 6, 2016

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

@johansja
Copy link

johansja commented Sep 6, 2016

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

@oli-obk
Copy link
Author

oli-obk commented Sep 6, 2016

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

@softprops
Copy link

No progress on this yet?

@oli-obk
Copy link
Author

oli-obk commented Feb 6, 2017

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

@blanham
Copy link

blanham commented May 23, 2017

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

@oli-obk
Copy link
Author

oli-obk commented May 23, 2017

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

@RReverser
Copy link

@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.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Development

No branches or pull requests

6 participants