[ITK] [ITK-dev] Pre-commit Failure uncrustify

Somesh someshsandbox at gmail.com
Wed Sep 14 19:05:57 EDT 2016


Hi Dženan,
You are right.  I accidentally sent you the wrong commit info. I use
homebrew package manager for mac and I accidentally sent you the commit ID
of homebrew commit instead of uncrustify. See
https://github.com/Homebrew/brew/commit/5a264cf59733fa8e0f69f8e6567e069a8cfd8bde
. My apologies :)
However it seems I have correct version of uncrustify. Looking into the
uncrustify remotes that I have installed, this is what I am using:

origin https://github.com/uncrustify/uncrustify.git (fetch)
origin https://github.com/uncrustify/uncrustify.git (push)

With the last 5 commits as:


   - 5d703fa01ff67afc029f31a5ac36bd2fbafb6aea Merge pull request #756 from
   Unity-Technologies/fixes/make_version_with_cmake
   - bc2b0c3e0c5b485cabbbcefcfb1eee76dbd62356 removed cmake dependency on
   autotools and directly use make_version.py
   - ac79e4081241359bdf3eec42baca690776ab565f make_version.py will be used
   only to retrieve the version from the VCS.
   - 3ef03cd6ac3fe218e77f14336e7c17daf75ba551 fixed instability retrieving
   the version when under hg-git
   - d9b0253f2d89c3556704bd45145d5bd4108f1043 Merge pull request #661 from
   uncrustify/NewLineProt


I can take a look into why its not working for me in the weekend.

Thanks,
Somesh



On Wed, Sep 14, 2016 at 4:04 PM, Dženan Zukić <dzenanz at gmail.com> wrote:

> Hi Somesh,
>
> I just pulled master version of uncrustify, the last commit is on 10th
> (you have a commit on 12th!):
> Revision: 5d703fa01ff67afc029f31a5ac36bd2fbafb6aea
> Author: gmaurel <guy.j at maurel.de>
> Date: 2016-09-10 15:06:05
> Message: Merge pull request #756 from Unity-Technologies/fixes/make_
> version_with_cmake
>
> I must conclude that you are using somebody's fork of uncrustify, not HEAD
> from the official repository. Can you verify this?
>
> Regards,
> Dženan
>
> On Mon, Sep 12, 2016 at 10:17 PM, Dženan Zukić <dzenanz at gmail.com> wrote:
>
>> It is not required.
>>
>>enan at phone
>>
>> On Sep 12, 2016 22:15, "Somesh" <someshsandbox at gmail.com> wrote:
>>
>>> Is running uncrustify hooks a required process in the ITK development
>>> workflow or can I disable it if I have to submit a patch ?
>>>
>>> Thanks,
>>> Somesh
>>>
>>> On Mon, Sep 12, 2016 at 9:30 PM, Somesh <someshsandbox at gmail.com> wrote:
>>>
>>>> Hi Dženan,
>>>> I am using the HEAD of master uncrustify. The last commit to this repo
>>>> was made today.
>>>>
>>>> commit 5a264cf59733fa8e0f69f8e6567e069a8cfd8bde
>>>> Merge: 553ef05 8ce4ca1
>>>> Author: Zhiming Wang <zmwangx at gmail.com>
>>>> Date:   Mon Sep 12 10:22:27 2016 -0400
>>>>
>>>> I have build from source using brew for mac.
>>>>
>>>> Also my mergetool/difftool are all set. I am using OS X El
>>>> Captain 10.11.6 and kaleidoscope tool for mac
>>>> http://www.kaleidoscopeapp.com/.
>>>>
>>>> Thanks,
>>>> Somesh
>>>>
>>>> On Mon, Sep 12, 2016 at 9:11 PM, Dzenan Zukic <dzenan.zukic at kitware.com
>>>> > wrote:
>>>>
>>>>> Hi Somesh,
>>>>>
>>>>> You need a recent master version (2016-08-26 or newer) which you can
>>>>> build yourself from source available on GitHub
>>>>> <https://github.com/uncrustify/uncrustify>.
>>>>>
>>>>> Also, you need to have merge tool set. On Windows kdiff3 works well
>>>>> but TortoiseMerge doesn't.
>>>>>
>>>>> Regards,
>>>>> Dženan Zukić, PhD, Senior R&D Engineer, Kitware (Carrboro, N.C.)
>>>>>
>>>>> On Mon, Sep 12, 2016 at 9:03 PM, Somesh <someshsandbox at gmail.com>
>>>>> wrote:
>>>>>
>>>>>> Hi Matt,
>>>>>> Do KWStyle and uncrustify play well with both enabled ?
>>>>>> I have  uncrustify  version 0.63 and the prehooks still fail. I
>>>>>> believe 0.63 is the latest uncrustify version: https://github.com/un
>>>>>> crustify/uncrustify/blob/master/CMakeLists.txt#L124
>>>>>>
>>>>>> Thanks,
>>>>>> Somesh
>>>>>>
>>>>>> On Mon, Sep 12, 2016 at 5:12 PM, Matt McCormick <
>>>>>> matt.mccormick at kitware.com> wrote:
>>>>>>
>>>>>>> Hi Somesh,
>>>>>>>
>>>>>>> Yes, Dzenan recently improved upstream uncrustify and its
>>>>>>> configuration in ITK so it works much better. A newer version of
>>>>>>> uncrustify is required as described here:
>>>>>>>
>>>>>>>   https://github.com/InsightSoftwareConsortium/ITK/blob/6ee105
>>>>>>> 75445aa1c878797db4febf7d775a38f1f3/Utilities/DevelopmentSetu
>>>>>>> pScripts/GitTips.sh#L66-L74
>>>>>>>
>>>>>>> Thanks,
>>>>>>> Matt
>>>>>>>
>>>>>>> On Mon, Sep 12, 2016 at 2:49 PM, Somesh <someshsandbox at gmail.com>
>>>>>>> wrote:
>>>>>>> > Hi,
>>>>>>> > I recently set up a ITK dev on my mac. While commiting a patch
>>>>>>> locally, I am
>>>>>>> > getting failure in precommit hooks. Specifically uncrustify is
>>>>>>> giving the
>>>>>>> > following errors. Do I have to install a specific version of
>>>>>>> > uncrustify/python ? I installed uncrustify using brew.
>>>>>>> >
>>>>>>> > ./Utilities/Maintenance/uncrustify_itk.cfg:99 Unknown symbol
>>>>>>> > 'indent_cs_delegate_brace'
>>>>>>> >
>>>>>>> > ./Utilities/Maintenance/uncrustify_itk.cfg:287 Unknown symbol
>>>>>>> > 'indent_token_after_brace'
>>>>>>> >
>>>>>>> > ./Utilities/Maintenance/uncrustify_itk.cfg:290 Unknown symbol
>>>>>>> > 'indent_cpp_lambda_body'
>>>>>>> >
>>>>>>> > ./Utilities/Maintenance/uncrustify_itk.cfg:424 Unknown symbol
>>>>>>> > 'sp_angle_paren_empty'
>>>>>>> >
>>>>>>> > ./Utilities/Maintenance/uncrustify_itk.cfg:533 Unknown symbol
>>>>>>> > 'sp_after_operator_sym_empty'
>>>>>>> >
>>>>>>> > ./Utilities/Maintenance/uncrustify_itk.cfg:570 Unknown symbol
>>>>>>> > 'sp_func_proto_paren_empty'
>>>>>>> >
>>>>>>> > ./Utilities/Maintenance/uncrustify_itk.cfg:576 Unknown symbol
>>>>>>> > 'sp_func_def_paren_empty'
>>>>>>> >
>>>>>>> > ./Utilities/Maintenance/uncrustify_itk.cfg:614 Unknown symbol
>>>>>>> > 'sp_func_class_paren_empty'
>>>>>>> >
>>>>>>> > ./Utilities/Maintenance/uncrustify_itk.cfg:644 Unknown symbol
>>>>>>> > 'sp_super_paren'
>>>>>>> >
>>>>>>> > ./Utilities/Maintenance/uncrustify_itk.cfg:647 Unknown symbol
>>>>>>> > 'sp_this_paren'
>>>>>>> >
>>>>>>> > ./Utilities/Maintenance/uncrustify_itk.cfg:851 Unknown symbol
>>>>>>> > 'sp_skip_vbrace_tokens'
>>>>>>> >
>>>>>>> > ./Utilities/Maintenance/uncrustify_itk.cfg:922 Unknown symbol
>>>>>>> > 'align_var_class_span'
>>>>>>> >
>>>>>>> > ./Utilities/Maintenance/uncrustify_itk.cfg:925 Unknown symbol
>>>>>>> > 'align_var_class_thresh'
>>>>>>> >
>>>>>>> > ./Utilities/Maintenance/uncrustify_itk.cfg:928 Unknown symbol
>>>>>>> > 'align_var_class_gap'
>>>>>>> >
>>>>>>> > ./Utilities/Maintenance/uncrustify_itk.cfg:1068 Unknown symbol
>>>>>>> > 'nl_oc_block_brace'
>>>>>>> >
>>>>>>> > ./Utilities/Maintenance/uncrustify_itk.cfg:1245 Unknown symbol
>>>>>>> > 'nl_enum_own_lines'
>>>>>>> >
>>>>>>> > ./Utilities/Maintenance/uncrustify_itk.cfg:1256 Unknown symbol
>>>>>>> > 'nl_func_class_scope'
>>>>>>> >
>>>>>>> > ./Utilities/Maintenance/uncrustify_itk.cfg:1284 Unknown symbol
>>>>>>> > 'nl_func_decl_start_multi_line'
>>>>>>> >
>>>>>>> > ./Utilities/Maintenance/uncrustify_itk.cfg:1287 Unknown symbol
>>>>>>> > 'nl_func_def_start_multi_line'
>>>>>>> >
>>>>>>> > ./Utilities/Maintenance/uncrustify_itk.cfg:1296 Unknown symbol
>>>>>>> > 'nl_func_decl_args_multi_line'
>>>>>>> >
>>>>>>> > ./Utilities/Maintenance/uncrustify_itk.cfg:1299 Unknown symbol
>>>>>>> > 'nl_func_def_args_multi_line'
>>>>>>> >
>>>>>>> > ./Utilities/Maintenance/uncrustify_itk.cfg:1314 Unknown symbol
>>>>>>> > 'nl_func_decl_end_multi_line'
>>>>>>> >
>>>>>>> > ./Utilities/Maintenance/uncrustify_itk.cfg:1317 Unknown symbol
>>>>>>> > 'nl_func_def_end_multi_line'
>>>>>>> >
>>>>>>> > ./Utilities/Maintenance/uncrustify_itk.cfg:1326 Unknown symbol
>>>>>>> > 'nl_func_call_start_multi_line'
>>>>>>> >
>>>>>>> > ./Utilities/Maintenance/uncrustify_itk.cfg:1329 Unknown symbol
>>>>>>> > 'nl_func_call_args_multi_line'
>>>>>>> >
>>>>>>> > ./Utilities/Maintenance/uncrustify_itk.cfg:1332 Unknown symbol
>>>>>>> > 'nl_func_call_end_multi_line'
>>>>>>> >
>>>>>>> > ./Utilities/Maintenance/uncrustify_itk.cfg:1388 Unknown symbol
>>>>>>> > 'nl_squeeze_ifdef_top_level'
>>>>>>> >
>>>>>>> > ./Utilities/Maintenance/uncrustify_itk.cfg:1461 Unknown symbol
>>>>>>> > 'nl_split_if_one_liner'
>>>>>>> >
>>>>>>> > ./Utilities/Maintenance/uncrustify_itk.cfg:1465 Unknown symbol
>>>>>>> > 'nl_split_for_one_liner'
>>>>>>> >
>>>>>>> > ./Utilities/Maintenance/uncrustify_itk.cfg:1469 Unknown symbol
>>>>>>> > 'nl_split_while_one_liner'
>>>>>>> >
>>>>>>> > ./Utilities/Maintenance/uncrustify_itk.cfg:1541 Unknown symbol
>>>>>>> > 'nl_after_func_class_proto'
>>>>>>> >
>>>>>>> > ./Utilities/Maintenance/uncrustify_itk.cfg:1544 Unknown symbol
>>>>>>> > 'nl_after_func_class_proto_group'
>>>>>>> >
>>>>>>> > ./Utilities/Maintenance/uncrustify_itk.cfg:1662 Unknown symbol
>>>>>>> > 'mod_full_brace_if_chain_only'
>>>>>>> >
>>>>>>> > ./Utilities/Maintenance/uncrustify_itk.cfg:1781 Unknown symbol
>>>>>>> > 'cmt_multi_first_len_minimum'
>>>>>>> >
>>>>>>> > ./Utilities/Maintenance/uncrustify_itk.cfg:1812 Unknown symbol
>>>>>>> > 'cmt_insert_before_inlines'
>>>>>>> >
>>>>>>> > ./Utilities/Maintenance/uncrustify_itk.cfg:1817 Unknown symbol
>>>>>>> > 'cmt_insert_before_ctor_dtor'
>>>>>>> >
>>>>>>> > ./Utilities/Maintenance/uncrustify_itk.cfg:1878 Unknown symbol
>>>>>>> > 'use_options_overriding_for_qt_macros'
>>>>>>> >
>>>>>>> > ./Utilities/Maintenance/uncrustify_itk.cfg:1885 Unknown symbol
>>>>>>> > 'warn_level_tabs_found_in_verbatim_string_literals'
>>>>>>> >
>>>>>>> >
>>>>>>> > Thanks,
>>>>>>> > Somesh
>>>>>>> >
>>>>>>> >
>>>>>>> > _______________________________________________
>>>>>>> > Powered by www.kitware.com
>>>>>>> >
>>>>>>> > Visit other Kitware open-source projects at
>>>>>>> > http://www.kitware.com/opensource/opensource.html
>>>>>>> >
>>>>>>> > Kitware offers ITK Training Courses, for more information visit:
>>>>>>> > http://kitware.com/products/protraining.php
>>>>>>> >
>>>>>>> > Please keep messages on-topic and check the ITK FAQ at:
>>>>>>> > http://www.itk.org/Wiki/ITK_FAQ
>>>>>>> >
>>>>>>> > Follow this link to subscribe/unsubscribe:
>>>>>>> > http://public.kitware.com/mailman/listinfo/insight-developers
>>>>>>> >
>>>>>>>
>>>>>>
>>>>>>
>>>>>
>>>>
>>>
>>> _______________________________________________
>>> Powered by www.kitware.com
>>>
>>> Visit other Kitware open-source projects at
>>> http://www.kitware.com/opensource/opensource.html
>>>
>>> Kitware offers ITK Training Courses, for more information visit:
>>> http://kitware.com/products/protraining.php
>>>
>>> Please keep messages on-topic and check the ITK FAQ at:
>>> http://www.itk.org/Wiki/ITK_FAQ
>>>
>>> Follow this link to subscribe/unsubscribe:
>>> http://public.kitware.com/mailman/listinfo/insight-developers
>>>
>>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://public.kitware.com/pipermail/community/attachments/20160914/bca1aa9f/attachment-0001.html>
-------------- next part --------------
_______________________________________________
Powered by www.kitware.com

Visit other Kitware open-source projects at
http://www.kitware.com/opensource/opensource.html

Kitware offers ITK Training Courses, for more information visit:
http://kitware.com/products/protraining.php

Please keep messages on-topic and check the ITK FAQ at:
http://www.itk.org/Wiki/ITK_FAQ

Follow this link to subscribe/unsubscribe:
http://public.kitware.com/mailman/listinfo/insight-developers


More information about the Community mailing list