You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I implemented the following hack – adding a dummy field to the app’s options only to overwrite application description that has been overwritten by Opt above:
/// My description#[derive(Debug,StructOpt)]structMyOpt{
…
#[structopt(flatten)]server:Opt,#[allow(dead_code)]#[structopt(flatten)]dummy:StructOptDummy,}#[derive(Debug,StructOpt)]#[structopt(about = "My description", long_about = "My description")]structStructOptDummy{}
The text was updated successfully, but these errors were encountered:
It isn’t complicated, structopt to clap is almost search&replace – I could do it if I knew that the pull request will be accepted rather than stuck because nobody has time to review it.
Yeah, the only hurdle is to get buy in from the team since it's a change a pretty central place. If you're willing to put in the work/pr. That will make it easier for me to champion 💪
palant
added a commit
to palant/pingora
that referenced
this issue
May 10, 2024
What is the problem your feature solves, or the need it fulfills?
structopt crate is unmaintained and superseded by clap v3 (Maintenance note). It has bugs that clap fixed three years ago before even releasing them (TeXitoi/structopt#539, clap-rs/clap#2527). This creates issues when expanding Pingora’s Opt structure.
Describe the solution you'd like
Pingora should use a current clap version.
Describe alternatives you've considered
I implemented the following hack – adding a dummy field to the app’s options only to overwrite application description that has been overwritten by
Opt
above:The text was updated successfully, but these errors were encountered: