[Users] Checking out documentation and parameter pseudo-thorns

[Frank Loeffler]

On Tue, Jun 15, 2010 at 11:24:08AM -0500, Erik Schnetter wrote:
> Please review this patch.
> > +!TARGET   = $ARR
> > +!TYPE     = git
> > +!URL      = git://carpetcode.dyndns.org/McLachlan
> > +!AUTH_URL = carpetgit at carpetcode.dyndns.org:McLachlan
> > +!REPO_PATH= $2
> > +!CHECKOUT = McLachlan/doc
> > +!TARGET   = $ARR
> > +!TYPE     = git
> > +!URL      = git://carpetcode.dyndns.org/McLachlan
> > +!AUTH_URL = carpetgit at carpetcode.dyndns.org:McLachlan
> > +!REPO_PATH= $2
> > +!CHECKOUT = McLachlan/m
> > +!TARGET   = $ARR
> > +!TYPE     = git
> > +!URL      = git://carpetcode.dyndns.org/McLachlan
> > +!AUTH_URL = carpetgit at carpetcode.dyndns.org:McLachlan
> > +!REPO_PATH= $2
> > +!CHECKOUT = McLachlan/par

Is there a reason they are not combined into the CHECKOUT block of
McLachlan which is already inside?

> > +!CHECKOUT = Carpet/doc

Same here.

If there is no reason why they cannot be combined with the existing
blocks I would combine them and commit this.

Frank

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 835 bytes
Desc: not available
Url : http://lists.einsteintoolkit.org/pipermail/users/attachments/20100615/c261b34a/attachment.bin