[Koha-devel] Koha 3.8 release schedule & perltidy process

Chris Cormack chrisc at catalyst.net.nz
Thu Mar 8 19:50:23 CET 2012


* Ian Walls (koha.sekjal at gmail.com) wrote:
>    Doing a large updating commit does not cost us any history.A  It just
>    counts as an "update" to the code, even though none of the logic has
>    changed.A A  This change would alter SLOC counts and such, messing with
>    our statistics, since we only want to measure intellectually significant
>    contributions (tidying someone else's work doesn't make it yours).A  There
>    is no way for Git to know if a change to a line of text is a logical
>    change or just a formatting change (aside from whitespace), because Git
>    doesn't understand Perl.A  There isn't too much we can do about this.

It's not so much statistics I care about, although I do. But also that
it makes it hard to to do a git blame to find which commit actually
changed the line. Since now every line is changed by the same commit.

Chris
> 
>    So, best to keep cleaning up incrementally, I think.A  As we move from C4
>    to Koha, that'll be an opportunity to clean up all the modules
> 
>    -Ian
> 
>    2012/3/8 Jared Camins-Esakov <jcamins at cpbibliography.com>
> 
>      +1 to a gradual perltidy
> 
>      2012/3/8 Fischer, Katrin <Katrin.Fischer at bsz-bw.de>
> 
>        I agree with Chris C. and Chris N. - I think what we would win does
>        not outweigh the loss of history.
> 
>        Katrin
> 
>        -----UrsprA 1/4ngliche Nachricht-----
>        Von: koha-devel-bounces at lists.koha-community.org im Auftrag von Chris
>        Nighswonger
>        Gesendet: Do 08.03.2012 19:26
>        An: Chris Cormack
>        Cc: koha-devel at lists.koha-community.org
>        Betreff: Re: [Koha-devel] Koha 3.8 release schedule & perltidy process
> 
>        2012/3/8 Chris Cormack <chris at bigballofwax.co.nz>
> 
>        > My counter proposal is tidy as you go. Fix code as you touch it.
>        >
>        > With vim (and other editors) you can easily tidy a block, doing that
>        as
>        > code is changed would be my preference.
>        >
> 
>        I'd prefer a "pay-as-you-go" approach as well. We could simply require
>        all
>        work to be tidied before submitting.
> 
>        Slow? Yes, but not nearly as messy and preserves the history.
> 
>        Kind Regards,
>        Chris
> 
>        _______________________________________________
>        Koha-devel mailing list
>        Koha-devel at lists.koha-community.org
>        http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-devel
>        website : http://www.koha-community.org/
>        git : http://git.koha-community.org/
>        bugs : http://bugs.koha-community.org/
> 
>      --
>      Jared Camins-Esakov
>      Bibliographer, C & P Bibliography Services, LLC
>      (phone) +1 (917) 727-3445
>      (e-mail) jcamins at cpbibliography.com
>      (web) http://www.cpbibliography.com/
>      _______________________________________________
>      Koha-devel mailing list
>      Koha-devel at lists.koha-community.org
>      http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-devel
>      website : http://www.koha-community.org/
>      git : http://git.koha-community.org/
>      bugs : http://bugs.koha-community.org/

> _______________________________________________
> Koha-devel mailing list
> Koha-devel at lists.koha-community.org
> http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-devel
> website : http://www.koha-community.org/
> git : http://git.koha-community.org/
> bugs : http://bugs.koha-community.org/


-- 
Chris Cormack
Catalyst IT Ltd.
+64 4 803 2238
PO Box 11-053, Manners St, Wellington 6142, New Zealand
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: </pipermail/koha-devel/attachments/20120309/3b57e443/attachment.pgp>


More information about the Koha-devel mailing list