search

example42 / tinydata

Tiny Data for Tiny Puppet & C.
7 stars 12 forks source link

squid with debian 12 has now a conf.d dir #77

Closed dagobert closed 4 months ago

dagobert commented 6 months ago

squid in Debian Bookworm (12) has now a conf.d directory.

alvagante commented 6 months ago

@dagobert if it's only for Debian 12, it' better to add the line in a new file: squid/osfamily/Debian12.yaml

dagobert commented 6 months ago

I am only using Debian, hence I don't know about other distributions and their releases.

The descision is yours to make. I think it belongs in the default file since it is a directory you implemented in tp::conf as a short - as also described in the readme. I mean that it is in general not an unusula directory for many services. I was thinking ahead imagining what happens other dirstibutions also add this conf.d directory. Than you would need to decide again to add an exception and on some point there are more exception than the default. Hence, I would, since this exact directory is part of your standard implementation, add it to default. Also, no one needs to use it and will use it on accident. It is your Project in the end.