On Sun, 2008-12-21 at 20:10 +0000, [email protected] wrote: > The following page has been changed by AdrianChadd: > http://wiki.squid-cache.org/Features/SourceLayout?action=diff&rev1=38&rev2=39 > > ------------------------------------------------------------------------------ > || || '''Group''' || '''Definition''' || '''Files''' || > || || ''acl/'' || Access Controls || ACL* external_acl.* || > || {2} || ''auth/'' || Authentication support || auth/* || > + || {3} || ''core/'' || Core stuff which hasn't been further broken apart || > ? || > || || ''comm/'' || I/O subsystem || Comm*, comm* || > || || ''store/'' || generic (fs-agnostic) disk and memory cache support? > || Store* store* || > || || ''http/'' || HTTP primitives shared by client, server, and ICAP > sides || Http* ||
There are two problems with this addition: 1) The directory name should not be "core". That name is treated specially in some environments. I have tried that directory name in the past and ran into problems with makefiles, archival tools, etc. I ended up using "base" in other projects, but there may be better alternatives. 2) The definition may imply that this directory will accumulate stuff that has not been broken apart _yet_. That implication would be wrong. The base directory should only contain code that we do not expect to isolate into a more specific directory. Very few and very basic things should go there. Stuff that we have not isolated yet should remain where it is, IMO. Thank you, Alex.
