[Graham Bloice <graham.bloice@xxxxxxxxxxxxx>]

On Fri, 28 Sep 2018 at 16:45, Bruno Rijsman <brunorijsman@xxxxxxxxx> wrote:

I found the solution already:

brew install nghttp2

fixes it

(It causes HAVE_NGHTTP2 to be set, which adds current_stream_id to http2_session_t)

The code should probably still be fixed to avoid a compile error when NGHTTP2 is not installed

See change 29896 (https://code.wireshark.org/review/29896) which fixes this issue.

On Sep 27, 2018, at 10:21 AM, Bruno Rijsman <brunorijsman@xxxxxxxxx> wrote:

Any suggestions on how to avoid the following compiler error?

[ 30%] Building C object epan/dissectors/CMakeFiles/dissectors.dir/packet-hsrp.c.o

[ 30%] Building C object epan/dissectors/CMakeFiles/dissectors.dir/packet-http.c.o

[ 30%] Building C object epan/dissectors/CMakeFiles/dissectors.dir/packet-http2.c.o

/Users/brunorijsman/wireshark/epan/dissectors/packet-http2.c:1124:23: error: no member named 'current_stream_id' in 'http2_session_t'

    return h2session->current_stream_id;

           ~~~~~~~~~  ^

1 error generated.

Info:

macOS High Sierra 10.13.6

git clone https://code.wireshark.org/review/wireshark

cd wireshark/

mkdir build

cd build

brew install doxygen

brew install libgcrypt

brew install glib

brew install qt5

cmake ..

make

--

Graham Bloice