Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merging type definitions with rpm-rs #5

Open
15 tasks
madonuko opened this issue Jun 27, 2024 · 1 comment
Open
15 tasks

Merging type definitions with rpm-rs #5

madonuko opened this issue Jun 27, 2024 · 1 comment
Labels
enhancement New feature or request

Comments

@madonuko
Copy link
Collaborator

The current state of rpmspec-rs is "good enough" as in it can be used in production to read preambles. With that in mind, we probably might want to start merging code between rpmspec-rs and rpm-rs.

Personally I think it makes more sense for rpmspec-rs to use the types defined in rpm-rs as obviously they have a larger userbase and less cargo dependencies. In that case, the following entries need to be merged:

@madonuko madonuko added the enhancement New feature or request label Jun 27, 2024
@dralley
Copy link
Collaborator

dralley commented Jun 27, 2024

The File* APIs probably need some adjustment, since as it stands they are barely usable IMO. But hooking it up to specfile parsing is probably a great way to sort out the issues and figure out what needs to be done about them.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants