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

replace libpopt with cxxopts #82

Open
2 of 4 tasks
ggutierrez-sunbright opened this issue Feb 6, 2024 · 0 comments
Open
2 of 4 tasks

replace libpopt with cxxopts #82

ggutierrez-sunbright opened this issue Feb 6, 2024 · 0 comments
Assignees
Labels
upgrade a design modification, refactor, or component replacement

Comments

@ggutierrez-sunbright
Copy link
Contributor

ggutierrez-sunbright commented Feb 6, 2024

cxxopts is a modern, C++, header-only library for parsing arguments from the command line.

this would mean removing an external dependency, simplifying the build process, and a good chance to make a cleaner version of the binaries, with a clear separation between the config parsing into a dedicated structure, and the execution based on this configuration.

* they don't use popt at all, maybe we should consider using cxxopts to parse the configuration, but I'd suggest to do it in a different issue

@ggutierrez-sunbright ggutierrez-sunbright added the upgrade a design modification, refactor, or component replacement label Feb 6, 2024
@ggutierrez-sunbright ggutierrez-sunbright added this to the libpopt removal milestone Feb 6, 2024
@ggutierrez-sunbright ggutierrez-sunbright moved this from Todo to In Progress in rDock general development board Apr 14, 2024
@ggutierrez-sunbright ggutierrez-sunbright moved this from In Progress to Todo in rDock general development board Apr 14, 2024
@ggutierrez-sunbright ggutierrez-sunbright moved this from Todo to In Progress in rDock general development board Apr 14, 2024
@ggutierrez-sunbright ggutierrez-sunbright moved this from In Progress to Todo in rDock general development board Apr 14, 2024
ggutierrez-sunbright added a commit that referenced this issue May 3, 2024
…DD/rDock into issue-#82/replace-libpopt-in-rbcavity
ggutierrez-sunbright added a commit that referenced this issue May 3, 2024
* cxxopts proof of concept with rbcavity

* refactor rbcavity for readability and common code

* decouple main from rbcavity procedure

* change add option to templated version

* change cxxopts to original fork latest version

* improve includes

* remove legacy options

* upgrade gcc version for centos7

* simplify argument extraction api
@ggutierrez-sunbright ggutierrez-sunbright self-assigned this Jun 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
upgrade a design modification, refactor, or component replacement
Projects
Development

No branches or pull requests

1 participant