AmateurECE / redfish-codegen

Implementation of DMTF's Redfish data models, generated from their OpenAPI documents.
Apache License 2.0
4 stars 2 forks source link

Rewrite generator not to use `swagger-parser` #4

Open AmateurECE opened 3 months ago

AmateurECE commented 3 months ago

This dependency has proved itself to be problematic, so the generator implementation should be relieved of it. I'm thinking that it would be easiest for the consumers of this project if this rewrite took the form of a pure-Rust solution, however, I'm not yet convinced either way.

This could also open the door to improving the build system in this repository--currently, the generator is built using maven, and the whole thing is glued together with a Makefile. Perhaps there's an opportunity to generate the redfish-codegen crate using something like Bazel.

In the coming months, I'll try out a couple of solutions and begin work on this, and I'll try to keep this issue updated with progress.

A full Rust rewrite needs the following:

AmateurECE commented 3 months ago

Building with Bazel might be possible leveraging the work described in bazelbuild/rules_rust#458.

AmateurECE commented 3 months ago

The text package has been rewritten on the rust-generator branch.

AmateurECE commented 2 months ago

The resolution for this issue should also include a resolution for not just #3, but also #5, #6, #7, & #8, or at least lend itself easily to a resolution for these issues in any case.