On Tue, 5 Feb 2008, Johannes Schindelin wrote:
> Hi,
>
> On Mon, 4 Feb 2008, Daniel Barkalow wrote:
>
> > You can also create branches, in exactly the same way, with checkout -b.
> >
> > Signed-off-by: Daniel Barkalow <barkalow@xxxxxxxxxxxx>
> > ---
> > builtin-branch.c | 4 ++--
> > cache.h | 4 ++++
>
> Note that builtin-branch.c is not part of libgit.a, so you are only moving
> the declaration into the "libgit" part.
>
> Maybe it would be better to move this function, say, into refs.c?
I think it's actually time to start a branch.{c,h} library file for
operations that deal with the porcelain-ish information about branches.
This would go there, and a function to discard MERGE_HEAD and similar
files (whatever they are, which isn't currently collected anywhere
authoritative). Probably also most of the other stuff that should be
shared between reset and checkout, too, and functions should probably
migrate there from builtin-branch and builtin-checkout as they become
needed, leaving the builtin-* code as command-line parsing and calls to
these functions. In any case, I'll start with this.
-Daniel
*This .sig left intentionally blank*
-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
