On Sat, Dec 14, 2013 at 06:33:43AM -0800, Dylan Baker wrote:
> On Friday, December 13, 2013 07:07:13 PM Brian Dolbec wrote:
> > - confdefaults={ "storedir":"/var/tmp/catalyst",\
> > -
> > "sharedir":"/usr/share/catalyst","distdir":"/usr/portage/distfiles",\
> > - "portdir":"/usr/portage","options":"",\
> > - "snapshot_cache":"/var/tmp/catalyst/snapshot_cache",\
> > - "hash_function":"crc32"}
> > + confdefaults={
> > + "hash_function": "crc32",
> > + "portdir": "/usr/portage",
> > + "repo_name": "portage",
> > + "sharedir": "/usr/lib/catalyst",
> > + "snapshot_name": "portage-",
> > + "snapshot_cache": "/var/tmp/catalyst/snapshot_cache",
> > + "storedir": "/var/tmp/catalyst",
> > + }
>
> pleas keep the formatting style; no comma on the last element and
> closing brace on the same lineThe “one entry per line, comma after last element, separate brace lines” format is both easier to read and easier to diff. All the keys are lined up in an alphabetized column for you, and if you add something after storedir entry, you don't have to touch the storedir line to remove a closing brace and add a comma. Cheers, Trevor -- This email may be signed or encrypted with GnuPG (http://www.gnupg.org). For more information, see http://en.wikipedia.org/wiki/Pretty_Good_Privacy
signature.asc
Description: OpenPGP digital signature
