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

Potentially switch from remembering arguments to remembering configuration #3005

Open
2 tasks done
TheCakeIsNaOH opened this issue Jan 28, 2023 · 0 comments
Open
2 tasks done

Comments

@TheCakeIsNaOH
Copy link
Member

Checklist

  • I have verified this is the correct repository for opening this issue.
  • I have verified no other issues exist related to my request.

Is Your Feature Request Related To A Problem? Please describe.

There have been a number of issues around remembered arguments (#1443, #2111, #2752, #2886, #2761), which largely have been around how the remembered argument are saved, specifically with how they have to be parsed back into the configuration.

Describe The Solution. Why is it needed?

So it might be a good idea to not save command line arguments, but instead serialize an object to a file. So for example an xml file with the package parameters, install arguments, etc as elements. This allows for more flexibility in parsing the file later, and more options for mapping the resulting object to the current configuration. It also avoids having to go through a command line argument parser.

Additional Context

I have a prototype of how this might work here: https://github.com/TheCakeIsNaOH/choco/tree/remembered-args-xml
The implementation has similarities to the changes in #3003

Related Issues

No response

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant