Skip to content

RFC: Flag parsing#732

Draft
BogdanTheGeek wants to merge 1 commit intocnlohr:masterfrom
BogdanTheGeek:newflags
Draft

RFC: Flag parsing#732
BogdanTheGeek wants to merge 1 commit intocnlohr:masterfrom
BogdanTheGeek:newflags

Conversation

@BogdanTheGeek
Copy link
Copy Markdown
Contributor

DO NOT MERGE!!!!!
Draft attempt at a new flag parsing methodology. Currently it has the following known limitations:

  • no combined args: -3tb
  • order is not retained (-a -b == -b -a)
  • not all flags have been implemented with new method, fallbacks to old method

Things gained:

  • aliases: -help | -h
  • usage() is updated automatically (there were quite a few flags missing)
  • variables as flags
  • defaults

@cnlohr
Copy link
Copy Markdown
Owner

cnlohr commented Nov 3, 2025

I do have a strong preference for maintaining order, because sometimes I do personally sequence things, especially surrounding unbricking, etc. I.e. applying power doing X removing power.

Also, I really wish we had long-arguments.

@BogdanTheGeek
Copy link
Copy Markdown
Contributor Author

BogdanTheGeek commented Nov 3, 2025 via email

@cnlohr
Copy link
Copy Markdown
Owner

cnlohr commented Nov 15, 2025

@monte-monte @BogdanTheGeek @dwillmore is there any hope for us to fix flags in minichlink?

I think we have to have --long-parameters

@cnlohr
Copy link
Copy Markdown
Owner

cnlohr commented Nov 15, 2025

This branch is no longer compatible btw

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants