[optionParser] mv to builtins dir - #23037
Conversation
to show that it's an external source and to potentially later resync with upstream 1.7 rather than current 1.3
Should we keep this dependency at all, since it's only used in rootcling? Perhaps we can instead replace it with Edit: I guess the advantage of this is that it handles subcommands (which rootcling needs), so perhaps the better path would be to remove Edit2: Ah, but subcommands are not actually handled directly by OptionParser.h, it's done via |
Test Results 23 files 23 suites 3d 13h 29m 28s ⏱️ For more details on these failures, see this check. Results for commit f7d3cf3. |
|
Sure, I would vote for dropping this if CLI11 or optparse.h can be used instead |
This Pull request:
Changes or fixes:
Move this to hard-coded builtin since it's forked from 1.3 and at some point we might resync with upstream 1.7
so having it in builtins is more consistent with other builtins such as mathtext, etc. rather than having it lost somewhere inside.