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

CONFIGATOR_MAX_LINE is non-overridable and has a bad default #2

Open
3442 opened this issue Jan 8, 2022 · 0 comments · May be fixed by #3
Open

CONFIGATOR_MAX_LINE is non-overridable and has a bad default #2

3442 opened this issue Jan 8, 2022 · 0 comments · May be fixed by #3

Comments

@3442
Copy link

3442 commented Jan 8, 2022

configator.h reads:

#define CONFIGATOR_MAX_LINE 80

Just 80 bytes is pretty bad and not justified at all. This is currently blocking ly support in NixOS, since /nix/store path are usually long, specially when multiple store objects are involved. The default should be much larger at the very. More over, maybe we can require configator users to provide their own line buffer so as to prevent an arbitrary limit?

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 a pull request may close this issue.

1 participant