Yet more issues surfacing with CMake 3.20.  Are these regressions being
reported back to Kitware somewhere?

On Fri, 28 May 2021 at 02:03, chuck c <bubbas...@gmail.com> wrote:

> My bad. Upgraded Visual Studio and it now ships with cmake 3.20.
> choco installing cmake 3.19 fixed the errors.
>
> Original working system:
> C:\Program Files (x86)\Microsoft Visual Studio\2019\Community>cl
> Microsoft (R) C/C++ Optimizing Compiler Version 19.28.29912 for x64
> Copyright (C) Microsoft Corporation.  All rights reserved.
>
> C:\Program Files (x86)\Microsoft Visual Studio\2019\Community>cmake
> --version
> cmake version 3.19.20122902-MSVC_2
>
> --------------------
>
> New system (bad cmake):
> W:\Development\wsbuild>cl
> Microsoft (R) C/C++ Optimizing Compiler Version 19.29.30037 for x64
> Copyright (C) Microsoft Corporation.  All rights reserved.
>
> W:\Development\wsbuild>cmake --version
> cmake version 3.20.21032501-MSVC_2
>
> Fixed with:
> choco install -y cmake --version 3.19.8
> update PATH to reach this cmake first
>
> W:\Development\wsbuild64>where cmake
> C:\Program Files\CMake\bin\cmake.exe
> C:\Program Files (x86)\Microsoft Visual
> Studio\2019\Community\Common7\IDE\CommonExtensions\Microsoft\CMake\CMake\bin\cmake.exe
>
> W:\Development\wsbuild64>
> W:\Development\wsbuild64>cmake --version
> cmake version 3.19.8
>
> CMake suite maintained and supported by Kitware (kitware.com/cmake).
>
> W:\Development\wsbuild64>
>
>
> On Thu, May 27, 2021 at 6:57 PM chuck c <bubbas...@gmail.com> wrote:
>
>> Is it a matter of tweaking the build to allow the warning to pass or does
>> this require a code change?
>>
>> W:\Development\wsbuild>cl
>> Microsoft (R) C/C++ Optimizing Compiler Version 19.29.30037 for x64
>> Copyright (C) Microsoft Corporation.  All rights reserved.
>>
>> usage: cl [ option... ] filename... [ /link linkoption... ]
>>
>> Build FAILED.
>>
>>        "W:\Development\wsbuild\Wireshark.sln" (default target) (1) ->
>>        "W:\Development\wsbuild\epan\wslua\wslua.vcxproj.metaproj"
>> (default target) (73) ->
>>        "W:\Development\wsbuild\epan\wslua\wslua.vcxproj" (default target)
>> (79) ->
>>        (ClCompile target) ->
>>          W:\Development\wireshark\epan\wslua\wslua_tvb.c(897,19): warning
>> C4013: 'pushNSTime' undefined; assuming extern returning int
>> [W:\Development\wsbuild\epan\wslua\wslua.vcxproj]
>>        W:\Development\wireshark\epan\wslua\wslua_tvb.c(897,19): warning
>> C4013:         pushNSTime(L, nstime);
>> [W:\Development\wsbuild\epan\wslua\wslua.vcxproj]
>>        W:\Development\wireshark\epan\wslua\wslua_tvb.c(897,19): warning
>> C4013:                   ^ [W:\Development\wsbuild\epan\wslua\wslua.vcxproj]
>>          W:\Development\wireshark\epan\wslua\wslua_field.c(148,27):
>> warning C4013: 'pushNSTime' undefined; assuming extern returning int
>> [W:\Development\wsbuild\epan\wslua\wslua.vcxproj]
>>        W:\Development\wireshark\epan\wslua\wslua_field.c(148,27): warning
>> C4013:                 pushNSTime(L,nstime);
>> [W:\Development\wsbuild\epan\wslua\wslua.vcxproj]
>>        W:\Development\wireshark\epan\wslua\wslua_field.c(148,27): warning
>> C4013:                           ^
>> [W:\Development\wsbuild\epan\wslua\wslua.vcxproj]
>>          W:\Development\wireshark\epan\wslua\wslua_tree.c(111,31):
>> warning C4013: 'pushNSTime' undefined; assuming extern returning int
>> [W:\Development\wsbuild\epan\wslua\wslua.vcxproj]
>>        W:\Development\wireshark\epan\wslua\wslua_tree.c(111,31): warning
>> C4013:                     pushNSTime(L,nstime);
>> [W:\Development\wsbuild\epan\wslua\wslua.vcxproj]
>>        W:\Development\wireshark\epan\wslua\wslua_tree.c(111,31): warning
>> C4013:                               ^
>> [W:\Development\wsbuild\epan\wslua\wslua.vcxproj]
>>          W:\Development\wireshark\epan\wslua\wslua_tree.c(383,121):
>> warning C4013: 'checkNSTime' undefined; assuming extern returning int
>> [W:\Development\wsbuild\epan\wslua\wslua.vcxproj]
>>        W:\Development\wireshark\epan\wslua\wslua_tree.c(383,121): warning
>> C4013:                     item =
>> proto_tree_add_time(tree_item->tree,hfid,tvbr->tvb->ws_tvb,tvbr->offset,tvbr->len,checkNSTime(L,1));
>> [W:\Development\wsbuild\epan\wslua\wslua.vcxproj]
>>        W:\Development\wireshark\epan\wslua\wslua_tree.c(383,121): warning
>> C4013:
>>                                                     ^
>> [W:\Development\wsbuild\epan\wslua\wslua.vcxproj]
>>          W:\Development\wireshark\epan\wslua\wslua_tree.c(383,126):
>> warning C4047: 'function': 'const nstime_t *' differs in levels of
>> indirection from 'int' [W:\Development\wsbuild\epan\wslua\wslua.vcxproj]
>>        W:\Development\wireshark\epan\wslua\wslua_tree.c(383,126): warning
>> C4047:                     item =
>> proto_tree_add_time(tree_item->tree,hfid,tvbr->tvb->ws_tvb,tvbr->offset,tvbr->len,checkNSTime(L,1));
>> [W:\Development\wsbuild\epan\wslua\wslua.vcxproj]
>>        W:\Development\wireshark\epan\wslua\wslua_tree.c(383,126): warning
>> C4047:
>>                                                          ^
>> [W:\Development\wsbuild\epan\wslua\wslua.vcxproj]
>>          W:\Development\wireshark\epan\wslua\wslua_tree.c(383,121):
>> warning C4024: 'proto_tree_add_time': different types for formal and actual
>> parameter 6 [W:\Development\wsbuild\epan\wslua\wslua.vcxproj]
>>        W:\Development\wireshark\epan\wslua\wslua_tree.c(383,121): warning
>> C4024:                     item =
>> proto_tree_add_time(tree_item->tree,hfid,tvbr->tvb->ws_tvb,tvbr->offset,tvbr->len,checkNSTime(L,1));
>> [W:\Development\wsbuild\epan\wslua\wslua.vcxproj]
>>        W:\Development\wireshark\epan\wslua\wslua_tree.c(383,121): warning
>> C4024:
>>                                                     ^
>> [W:\Development\wsbuild\epan\wslua\wslua.vcxproj]
>>          W:\Development\wireshark\epan\wslua\wslua_frame_info.c(112,32):
>> warning C4013: 'checkNSTime' undefined; assuming extern returning int
>> [W:\Development\wsbuild\epan\wslua\wslua.vcxproj]
>>        W:\Development\wireshark\epan\wslua\wslua_frame_info.c(112,32):
>> warning C4013:     NSTime nstime = checkNSTime(L,2);
>> [W:\Development\wsbuild\epan\wslua\wslua.vcxproj]
>>        W:\Development\wireshark\epan\wslua\wslua_frame_info.c(112,32):
>> warning C4013:                                ^
>> [W:\Development\wsbuild\epan\wslua\wslua.vcxproj]
>>          W:\Development\wireshark\epan\wslua\wslua_frame_info.c(112,1):
>> warning C4047: 'initializing': 'NSTime' differs in levels of indirection
>> from 'int' [W:\Development\wsbuild\epan\wslua\wslua.vcxproj]
>>        W:\Development\wireshark\epan\wslua\wslua_frame_info.c(112,1):
>> warning C4047:     NSTime nstime = checkNSTime(L,2);
>> [W:\Development\wsbuild\epan\wslua\wslua.vcxproj]
>>        W:\Development\wireshark\epan\wslua\wslua_frame_info.c(112,1):
>> warning C4047: ^ [W:\Development\wsbuild\epan\wslua\wslua.vcxproj]
>>          W:\Development\wireshark\epan\wslua\wslua_frame_info.c(131,15):
>> warning C4013: 'pushNSTime' undefined; assuming extern returning int
>> [W:\Development\wsbuild\epan\wslua\wslua.vcxproj]
>>        W:\Development\wireshark\epan\wslua\wslua_frame_info.c(131,15):
>> warning C4013:     pushNSTime(L,nstime);
>> [W:\Development\wsbuild\epan\wslua\wslua.vcxproj]
>>        W:\Development\wireshark\epan\wslua\wslua_frame_info.c(131,15):
>> warning C4013:               ^
>> [W:\Development\wsbuild\epan\wslua\wslua.vcxproj]
>>
>>
>>        "W:\Development\wsbuild\Wireshark.sln" (default target) (1) ->
>>        "W:\Development\wsbuild\epan\wslua\wslua.vcxproj.metaproj"
>> (default target) (73) ->
>>        "W:\Development\wsbuild\epan\wslua\wslua.vcxproj" (default target)
>> (79) ->
>>        (ClCompile target) ->
>>          W:\Development\wireshark\epan\wslua\wslua_tvb.c(897,19): error
>> C2220: the following warning is treated as an error
>> [W:\Development\wsbuild\epan\wslua\wslua.vcxproj]
>>        W:\Development\wireshark\epan\wslua\wslua_tvb.c(897,19): error
>> C2220:         pushNSTime(L, nstime);
>> [W:\Development\wsbuild\epan\wslua\wslua.vcxproj]
>>        W:\Development\wireshark\epan\wslua\wslua_tvb.c(897,19): error
>> C2220:                   ^ [W:\Development\wsbuild\epan\wslua\wslua.vcxproj]
>>          W:\Development\wireshark\epan\wslua\wslua_field.c(148,27): error
>> C2220: the following warning is treated as an error
>> [W:\Development\wsbuild\epan\wslua\wslua.vcxproj]
>>        W:\Development\wireshark\epan\wslua\wslua_field.c(148,27): error
>> C2220:                 pushNSTime(L,nstime);
>> [W:\Development\wsbuild\epan\wslua\wslua.vcxproj]
>>        W:\Development\wireshark\epan\wslua\wslua_field.c(148,27): error
>> C2220:                           ^
>> [W:\Development\wsbuild\epan\wslua\wslua.vcxproj]
>>          W:\Development\wireshark\epan\wslua\wslua_tree.c(111,31): error
>> C2220: the following warning is treated as an error
>> [W:\Development\wsbuild\epan\wslua\wslua.vcxproj]
>>        W:\Development\wireshark\epan\wslua\wslua_tree.c(111,31): error
>> C2220:                     pushNSTime(L,nstime);
>> [W:\Development\wsbuild\epan\wslua\wslua.vcxproj]
>>        W:\Development\wireshark\epan\wslua\wslua_tree.c(111,31): error
>> C2220:                               ^
>> [W:\Development\wsbuild\epan\wslua\wslua.vcxproj]
>>          W:\Development\wireshark\epan\wslua\wslua_frame_info.c(112,32):
>> error C2220: the following warning is treated as an error
>> [W:\Development\wsbuild\epan\wslua\wslua.vcxproj]
>>        W:\Development\wireshark\epan\wslua\wslua_frame_info.c(112,32):
>> error C2220:     NSTime nstime = checkNSTime(L,2);
>> [W:\Development\wsbuild\epan\wslua\wslua.vcxproj]
>>        W:\Development\wireshark\epan\wslua\wslua_frame_info.c(112,32):
>> error C2220:                                ^
>> [W:\Development\wsbuild\epan\wslua\wslua.vcxproj]
>>
>>     9 Warning(s)
>>     4 Error(s)
>>
> ___________________________________________________________________________
> Sent via:    Wireshark-dev mailing list <wireshark-dev@wireshark.org>
> Archives:    https://www.wireshark.org/lists/wireshark-dev
> Unsubscribe: https://www.wireshark.org/mailman/options/wireshark-dev
>              mailto:wireshark-dev-requ...@wireshark.org
> ?subject=unsubscribe
>


-- 
Graham Bloice
___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <wireshark-dev@wireshark.org>
Archives:    https://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://www.wireshark.org/mailman/options/wireshark-dev
             mailto:wireshark-dev-requ...@wireshark.org?subject=unsubscribe

Reply via email to