Wireshark-dev: Re: [Wireshark-dev] wslua: checkNSTime / pushNSTime undefined
From: chuck c <bubbasnmp@xxxxxxxxx>
Date: Thu, 27 May 2021 20:03:07 -0500
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.
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
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
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>
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 <bubbasnmp@xxxxxxxxx> 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)
- Follow-Ups:
- Re: [Wireshark-dev] wslua: checkNSTime / pushNSTime undefined
- From: Graham Bloice
- Re: [Wireshark-dev] wslua: checkNSTime / pushNSTime undefined
- References:
- Prev by Date: [Wireshark-dev] wslua: checkNSTime / pushNSTime undefined
- Next by Date: Re: [Wireshark-dev] wslua: checkNSTime / pushNSTime undefined
- Previous by thread: [Wireshark-dev] wslua: checkNSTime / pushNSTime undefined
- Next by thread: Re: [Wireshark-dev] wslua: checkNSTime / pushNSTime undefined
- Index(es):