[Insight-developers] KWStyle not run when adding a file in git?

Gaëtan Lehmann gaetan.lehmann at jouy.inra.fr
Tue Oct 26 13:41:49 EDT 2010


Le 26 oct. 10 à 19:37, Matthew McCormick (thewtex) a écrit :

> 2010/10/25 Gaëtan Lehmann <gaetan.lehmann at jouy.inra.fr>:
>>
>> Le 25 oct. 10 à 04:20, Matthew McCormick (thewtex) a écrit :
>>
>>> Hi Gaetan,
>>>
>>>>> Hi,
>>>>>
>>>>> I had to fix a set of new files because of missing copyright  
>>>>> headers
>>>>> today.
>>>>> When I tried to commit my changes, KWStyle blocked me because of  
>>>>> some
>>>>> non
>>>>> aligned typedefs and things of that kind.
>>>>> Shouldn't they have been detected first when I have added them  
>>>>> in the
>>>>> repository?
>>>>>
>>>
>>> Thanks for reporting this and your supposition is correct.   
>>> However, I
>>> cannot reproduce this behavior.  Can you please try again and send a
>>> problem file along with its path in its tree.
>>>
>>
>> If you look in my patch "pad by 1 by default in
>> ObjectByObjectLabelMapFilter",  
>> 619bf1ae54af37cdf314884bce46bbd0477ef6a4, you
>> can see that the Self typedef was misaligned in
>> Code/Review/itkObjectByObjectLabelMapFilter.h. The alignment has  
>> been fixed
>> in that patch, but should have been detected earlier, during the  
>> addition of
>> the file in git.
>>
>
> Here is what I did, but it detects the bug in the first commit.  Maybe
> you had the hook temporarily turned off?


maybe... but I thought this kind of hook are also applied on the itk  
repository during the push.
If that's the case, the commits should have been rejected on itk.org,  
right?

-- 
Gaëtan Lehmann
Biologie du Développement et de la Reproduction
INRA de Jouy-en-Josas (France)
tel: +33 1 34 65 29 66    fax: 01 34 65 29 09
http://voxel.jouy.inra.fr  http://www.itk.org
http://www.mandriva.org  http://www.bepo.fr

-------------- next part --------------
A non-text attachment was scrubbed...
Name: PGP.sig
Type: application/pgp-signature
Size: 203 bytes
Desc: Ceci est une signature ?lectronique PGP
URL: <http://www.itk.org/mailman/private/insight-developers/attachments/20101026/529e053f/attachment.pgp>


More information about the Insight-developers mailing list