On Fri, Oct 19, 2007 at 03:39:39AM -0400, Shawn O. Pearce wrote:
> What about this on top of Jeff's patch?
>
> $ git fetch jc
> ...
> ==> git://repo.or.cz/alt-git.git
> * tag junio-gpg-pub ......................... (new)
> * tag v1.5.0 .......................... (tag moved)
Honestly, I find it a bit ugly with the dots.
> $ git fetch me
> ...
> ==> git://repo.or.cz/git/spearce.git
> * branch gitk -> spearce/gitk ............... (new)
> * branch maint -> spearce/maint
> * branch master -> spearce/master
> * branch next -> spearce/next
> * branch pu -> spearce/pu ......... (forced update)
> * branch todo -> spearce/todo ............... (new)
More so with the ragged right of the branch names. I think it would
probably look better to line up the columns, but that will eventually
look ugly when somebody tries to fetch sp/totally-annoying-branchname.
I also think having the dots for some lines and others looks awkward.
> The width of the terminal is computed to produce the ... padding.
> I used a very narrow terminal to produce the above so it doesn't
> linewrap badly in email. If we cannot get the terminal width then
> we just don't produce the padding.
Ugh. I strongly suspect that it would look ugly on anything bigger than
about 80 columns, anyway. You are probably better off just not worrying
about the terminal width, and always using an 80-ish column total. And
then you don't have to worry about the ugly ioctl call.
> We also only show the URL once now, and only if at least one ref
> was somehow changed. This way we avoid showing the URL on a no-op
> or twice when we are fetching tags too.
Much nicer, and I like the refactoring into a separate show_update
function (especially if somebody ends up adding color later).
> + show_update("* branch", note, "->", "FETCH_HEAD", NULL);
Hrm, btw, I can't seem to get this one to show (I was curious how ugly
the FETCH_HEAD would look).
> if (verbose)
> - fprintf(stderr, " - %s == %s\n",
> - note, pretty_ref);
> + show_update("-", note, "==", pretty_ref, "unchanged");
> return 0;
Also, I was unable to generate a test case that showed this one. Did
you?
> - msg = "storing tag";
> [...]
> + msg = "storing new tag";
Nice.
> + show_update("- branch", note, "->", pretty_ref, "non-fast forward, refused");
Line wrap?
> +static void determine_window_size(void)
> +{
> + struct winsize ws;
> + if (!ioctl(2, TIOCGWINSZ, &ws))
> + ws_cols = ws.ws_col;
> +}
> +
Ugh. How portable is this?
-Peff
-
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
