On 03/17/2017 02:05 AM, Gary Wang wrote: > I made another change to keep new DEFAULT_IPC_PREFIX > consistent in src/ipc/Port.cc file. > Could you guys help review the MP? > https://code.launchpad.net/~gary-wzl77/squid/ipc_prefix/+merge/318714
I can only repeat what I have already said: > Why not make DEFAULT_STATEDIR configurable instead? (If we do make > it configurable, we may need to renamed it to STATEDIR, IPCDIR, or > something like that but that is a separate and minor issue.) and, later, a design sketch detailing the above suggestion: > Can the updated/renamed DEFAULT_STATEDIR be interpreted as > > <path> / <filename prefix> > > where the first component is used if and only when nameIsPath is true > and the second component is optional? AFAICT, such an implementation > would cover the already supported use cases and your new use case. > > For snap environments, the option could be set to something like: > > $(localstatedir)/run/squid/sem.snap.squid. I do not know whether you consider the above design broken or inferior to yours -- AFAIK, you have not addressed these comments/concerns. Alex. _______________________________________________ squid-dev mailing list squid-dev@lists.squid-cache.org http://lists.squid-cache.org/listinfo/squid-dev