Closed
Description
Currently we use an ad hoc csv format for dumping analysis data (see rustc::middle::save::recorder). It would be better to use libserialize (or serde) for the dumping step so we can easily provide data in different formats.
By default, I think save_analysis should output JSON, since that seems to be the current hotness for this sort of thing.
I expect this to be not too hard, but non-trivial, and possibly a little bit tedious.
Metadata
Metadata
Assignees
Labels
Type
Projects
Milestone
Relationships
Development
No branches or pull requests
Participants
Activity
sinistersnare commentedon Nov 3, 2014
cc me
gamazeps commentedon Nov 4, 2014
Willing to work on this too :)
gamazeps commentedon Nov 5, 2014
@sinistersnare Wanna split the work ? Or did you cc just in order to see the progress ?
sinistersnare commentedon Nov 6, 2014
Sure. this could be interesting.
rolandshoemaker commentedon Jan 6, 2015
Is this still being worked on or abandoned?
nrc commentedon Jan 6, 2015
@sinistersnare @gamazeps ? (See previous comment)
sinistersnare commentedon Jan 7, 2015
Oh goodness, I forgot about this :)
I can start work on this next week, but if you want to take this @rolandshoemaker, than feel free. Sorry!
Potpourri commentedon Feb 3, 2015
I would like to work on this.
sinistersnare commentedon Feb 3, 2015
Please do, I have no more plans of contributing.
Potpourri commentedon Feb 5, 2015
@nick29581 Do I need to keep csv support?
nrc commentedon Feb 5, 2015
@Potpourri tl;dr: no.
What would be ideal would be to have a generic serialisation API which could serialise to JSON, CSV or whatever else. I'm not sure if that is possible or how easy it is (@erickt might be able to advise). I'd be pretty happy with just JSON, as long as it wasn't too hard-wired in. (You could also earn mega-love from me by converting DXR to understand JSON rather than CSV as a follow up, but I'm also happy to do that).
steveklabnik commentedon Mar 4, 2016
Triage: looks like we're still using the CSV format.
17 remaining items