score-p/otf2_cli_profile

'struct meta_data' has no member named 'Buffer'

Closed this issue · 3 comments

Hi,

I need to apply this change (patch) to the src to compile otf2_cli_profile with gcc/7.5.0

Could you have a quick look and tell me if that's acceptable ?

Thanks,

[ 52%] Building CXX object CMakeFiles/otf-profiler-mpi.dir/src/output/data_out.cpp.o

/opt/cray/pe/craype/2.6.4/bin/CC  -DOTFPROFILER_MPI -I/apps/common/UES/sandbox/jgp/production.git/easybuild/easyconfigs/o/otf2_cli_profile/otf2_cli_profile.git/JG -I/apps/common/UES/sandbox/jgp/production.git/easybuild/easyconfigs/o/otf2_cli_profile/otf2_cli_profile.git/include -I/apps/common/UES/sandbox/jgp/production.git/easybuild/easyconfigs/o/otf2_cli_profile/otf2_cli_profile.git/include/reader -I/apps/common/UES/sandbox/jgp/production.git/easybuild/easyconfigs/o/otf2_cli_profile/otf2_cli_profile.git/include/output -I/apps/common/UES/sandbox/jgp/production.git/easybuild/easyconfigs/o/otf2_cli_profile/otf2_cli_profile.git -I/apps/dom/UES/jenkins/7.0.UP01/gpu/easybuild/tools/software/OTF2/2.2/include -I/apps/dom/UES/jenkins/7.0.UP01/gpu/easybuild/tools/software/CubeLib/4.4.4/include/cubelib -I/users/piccinal/easybuild/dom/haswell/software/rapidjson/a895ce1/include  -fpermissive -O3 -DNDEBUG   -std=gnu++11 -o CMakeFiles/otf-profiler-mpi.dir/src/output/data_out.cpp.o -c /apps/common/UES/sandbox/jgp/production.git/easybuild/easyconfigs/o/otf2_cli_profile/otf2_cli_profile.git/src/output/data_out.cpp
/apps/common/UES/sandbox/jgp/production.git/easybuild/easyconfigs/o/otf2_cli_profile/otf2_cli_profile.git/src/output/data_out.cpp: In function 'void display_meta_data(AllData, Writer&)':
/apps/common/UES/sandbox/jgp/production.git/easybuild/easyconfigs/o/otf2_cli_profile/otf2_cli_profile.git/src/output/data_out.cpp:363:44: error: 'struct meta_data' has no member named 'Buffer'; did you mean 'packBuffer'?
             writer.String(alldata.metaData.Buffer);
                                            ^~~~~~
                                            packBuffer

Looks reasonable to me. @LocNgu @dhinf can one of you confirm?

dhinf commented

fixed in 9be64de

@jgphpc thanks for reporting the bug

thank you !