Created attachment 195664 [details] Patch from upstream After update GCC from 6.4 to 7.3 multimedia/avidemux-plugins fails to build with this error: > /tmp/work/usr/ports/multimedia/avidemux-plugins/work/avidemux_2.6.11/avidemux_plugins/ADM_muxers/muxerMp4v2/libmp4v2/src/rtphint.cpp: In member function 'void mp4v2::impl::MP4RtpHintTrack::GetPayload(char**, uint8_t*, uint16_t*, char**)': > /tmp/work/usr/ports/multimedia/avidemux-plugins/work/avidemux_2.6.11/avidemux_plugins/ADM_muxers/muxerMp4v2/libmp4v2/src/rtphint.cpp:342:35: error: ISO C++ forbids comparison between pointer and integer [-fpermissive] > if (pSlash != '\0') { Look at code: > const char* pSlash; … > if (pSlash && ppEncodingParams) { > pSlash++; > pSlash = strchr(pSlash, '/'); > if (pSlash != NULL) { > pSlash++; > if (pSlash != '\0') { Error on this line ^^^^^^^^^^^^^^^^^^^^^^^^. Obvious it's incorrect code and most probably it have to be "if (*pSlash != '\0')". > length = (uint32_t)strlen(pRtpMap) - (pSlash - pRtpMap); > *ppEncodingParams = (char *)MP4Calloc(length + 1); > strncpy(*ppEncodingParams, pSlash, length); > } > } > } In upstream this line was replaced on "if (*pSlash)", as expected. Just put file patch-avidemux__plugins_ADM__muxers_muxerMp4v2_libmp4v2_src_rtphint.cpp in multimedia/avidemux/files/ - work for me on 11.2 amd64 and i386.
Gerald, you just fixed this issue by commit: https://svnweb.freebsd.org/ports/head/multimedia/avidemux/files/patch-avidemux__plugins_ADM__muxers_muxerMp4v2_libmp4v2_src_rtphint.cpp?revision=476028&view=markup But my patch is same as in upstream.
A commit references this bug: Author: bar Date: Wed Aug 1 07:55:20 UTC 2018 New revision: 476163 URL: https://svnweb.freebsd.org/changeset/ports/476163 Log: Re-create patch for avidemux-plugins after r464084 as in upstream. PR: 230211 Changes: head/multimedia/avidemux/files/patch-avidemux__plugins_ADM__muxers_muxerMp4v2_libmp4v2_src_rtphint.cpp head/multimedia/avidemux-plugins/Makefile
Patch modified as in upstream. Thanks.