-
Notifications
You must be signed in to change notification settings - Fork 27
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
Convert runtime files into YAML #68
Comments
As to the registry.xml the new ldmadmin (Python) already parses it seamlessly. Should it be included in the conversion? |
What's crucial is whether or not a programmatic library exists for parsing the particular configuration-file. One exists for XML, so "registry.xml" could be left alone. Parsers for the other configuration-files, however, are hand-coded. |
Indeed, I had once a syntactic error in the ldmd.conf file that the current yy parser could not point me to. |
Oh, and on the user... (the mandatory learning curve and the subsequent ease of use and robustness thereafter) |
This issue is very low priority because 1) the amount of effort is
significant; 2) user's are used to the current solution; 3) providing
forward and rollback compatibility is very non-trivial (the
configuration-files would have to be automatically converted to the new
format and changes to the new configuration-files would have to be rolled
back into the old configuration-files); and 4) there are, doubtless, more
important issues.
So, don't work on it if you were planning too.
This issue was more aspirational (i.e., wishful thinking) on my part.
…On Tue, Apr 6, 2021 at 10:59 AM mustbei ***@***.***> wrote:
Indeed, I had once a syntactic error in the ldmd.conf file that the
current yy parser could not point me to.
Will do for the all .conf files, acknowledging however the impact on their
current usage in both the new ldmadmin and the underlying C code (pqact,
etc.), as well as their backward incompatibility.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#68 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAEVZ7PHKE27RYYWC2LGHEDTHM4YLANCNFSM4GWKUNZA>
.
|
Makes sense. Should it be closed then? |
Leave it open. It's aspirational.
Hope is good.
|
Absolutely! |
Files ldmd.conf, pqact.conf, pqsurf.conf, registry.xml, scour.conf
The text was updated successfully, but these errors were encountered: