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

Improve config examples for contrib modules #39

Open
KoraggKnightWolf opened this issue Jun 4, 2019 · 3 comments
Open

Improve config examples for contrib modules #39

KoraggKnightWolf opened this issue Jun 4, 2019 · 3 comments
Assignees

Comments

@KoraggKnightWolf
Copy link

Often times contrib modules have vague examples on how to include them in the config file. It is at times a guessing work, ideally comments would give a clear "this is a working example, add more to it" format (e.g. I personally found mlocktweaker very tricky to customize and even had a closed issue about it on the main repo). Also those that spawn more services could do with this as well, as "add a *Serv block to your config" may not be elaborate enough for everyone.

Regards,

Koragg

@alyx alyx self-assigned this Jun 5, 2019
@alyx
Copy link
Member

alyx commented Jun 5, 2019

My immediate thought here is to include a contrib.conf.example which gets installed to $ATHEMEDIR/etc next to atheme.conf.example when --enable-contrib is enabled. This would keep atheme.conf clean from the misc. contrib clutter while also adding config support for these modules when requested.

Thoughts?

@KoraggKnightWolf
Copy link
Author

Sorry for the late response, exam stuff and all <.<

Definitely a great idea I would say, and also with elaborate comments (where needed of course, I do find the ones in atheme.conf to suffice in most cases).

Also I believe it would have to be included somehow in the main atheme.conf ? If yes, that must also be added (what if it includes contrib.conf.example even if it does not exist? If that does not break stuff, might be the easiest way to implement that).

Regards

Koragg

@alyx
Copy link
Member

alyx commented Jun 17, 2019

That's what I was thinking; have a commented include in atheme.conf or something along those lines. @aaronmdjones brought up a good point on IRC as to how flexible Atheme's config handler is which I still need to evaluate but I believe this should be the best direction.

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

No branches or pull requests

2 participants