Re: git diff <-> diffstat

[Adrian Bunk]

On Sun, Sep 24, 2006 at 07:05:39PM -0700, Linus Torvalds wrote:
> 
> 
> On Mon, 25 Sep 2006, Adrian Bunk wrote:
> > 
> > Is there any way for "git diff" to handle additional options diffstat 
> > handles? I'm a big fan of the -w72 diffstat option.
> 
> No, I think we've got the width fixed at 80 columns.
> 
> > Oh, and with git 1.4.2.1,
> >   git diff -M --stat --summary v2.6.18..master
> > in your tree gives me some funny lines like:
> > 
> >  .../netlabel/draft-ietf-cipso-ipsecurity-01.txt    |  791 +
> >  .../{cpu_setup_power4.S => cpu_setup_ppc970.S}     |  103 
> >  .../powerpc/platforms}/iseries/it_exp_vpd_panel.h  |    6 
> >  .../powerpc/platforms}/iseries/it_lp_naca.h        |    6 
> > 
> > I don't know what's going wrong here, but diffstat doesn't produce this.
> 
> Nothing is going wrong, and diffstat doesn't produce it, exactly because 
> diffstat cannot understand renames.
>...
> With long path-names, it can get a bit confusing, since we then truncate 
> the end result and just show the last parts to make it fit, of course.
>...

Ah, OK. The truncates are something I wasn't used from diffstat 
(diffstat always prints the complete name).



> 		Linus

cu
Adrian

-- 

       "Is there not promise of rain?" Ling Tan asked suddenly out
        of the darkness. There had been need of rain for many days.
       "Only a promise," Lao Er said.
                                       Pearl S. Buck - Dragon Seed

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/