TDE
/
gwenview
6
0
Fork 0
Code Issues 2 Pull Requests Releases Wiki Activity
Labels Milestones
New Issue

Build issue: FTBFS on OMV cooker #38

Closed
opened 4 weeks ago by Francois · 11 comments
Francois commented 4 weeks ago
Collaborator

Basic information

  • TDE version: 14.1.2
  • Distribution: OpenMandriva cooker
  • Hardware: amd64

Description

FTBFS

OpenMandriva 18.1.5-1 clang version 18.1.5 (/builddir/build/BUILD/llvm-18.1.5-x86_64/llvm-project-18.1.5.src/clang bce1a7458abcaba7efee8306802245abfe928979)

Steps to reproduce

Build

Screenshots

Backtrace 
make[4]: Entering directory '/tmp/BUILD.omv2490.x86_64/trinity-gwenview-1.4.2-x86_64/trinity-gwenview-14.1.2/src/imageutils'
/bin/sh ../../libtool  --tag=CC   --mode=compile /usr/bin/cc -DHAVE_CONFIG_H -I. -I../..  -I. -I./..  -I/opt/trinity/include/tde -I/usr/include/tqt3 -I/usr/include -include tqt.h  -DTQT_THREAD_SUPPORT  -D_REENTRANT  -W -Wall -Wchar-subscripts -Wshadow -Wpointer-arith -Wmissing-prototypes -Wwrite-strings -D_DEFAULT_SOURCE -DNDEBUG -O2  -Os -fomit-frame-pointer -g3 -gdwarf-4 -Wstrict-aliasing=2 -pipe -Wformat -Werror=format-security -D_FORTIFY_SOURCE=2 -fstack-protector-all --param=ssp-buffer-size=4 -m64 -mtune=generic -flto -Wformat-security -Wmissing-format-attribute -I/usr/include/tqt -c -o transupp.lo transupp.c
/bin/sh ../../libtool  --tag=CC   --mode=compile /usr/bin/cc -DHAVE_CONFIG_H -I. -I../..  -I. -I./..  -I/opt/trinity/include/tde -I/usr/include/tqt3 -I/usr/include -include tqt.h  -DTQT_THREAD_SUPPORT  -D_REENTRANT -I.  -W -Wall -Wchar-subscripts -Wshadow -Wpointer-arith -Wmissing-prototypes -Wwrite-strings -D_DEFAULT_SOURCE -DNDEBUG -O2  -Os -fomit-frame-pointer -g3 -gdwarf-4 -Wstrict-aliasing=2 -pipe -Wformat -Werror=format-security -D_FORTIFY_SOURCE=2 -fstack-protector-all --param=ssp-buffer-size=4 -m64 -mtune=generic -flto -Wformat-security -Wmissing-format-attribute -I/usr/include/tqt -c -o asm_scale.lo asm_scale.S
creating libgvimageutils_la.all_cpp.cpp ...
egrep: warning: egrep is obsolescent; using grep -E
egrep: warning: egrep is obsolescent; using grep -E
egrep: warning: egrep is obsolescent; using grep -E
egrep: warning: egrep is obsolescent; using grep -E
/bin/sh ../../libtool  --tag=CXX   --mode=compile /usr/bin/c++ -DHAVE_CONFIG_H -I. -I../..  -I. -I./..  -I/opt/trinity/include/tde -I/usr/include/tqt3 -I/usr/include -include tqt.h  -DTQT_THREAD_SUPPORT  -D_REENTRANT  -Wno-long-long -Wundef -D_DEFAULT_SOURCE -Wcast-align -Wchar-subscripts -Wall -W -Wpointer-arith -DNDEBUG -DNO_DEBUG -O2 -Os -fomit-frame-pointer -g3 -gdwarf-4 -Wstrict-aliasing=2 -pipe -Wformat -Werror=format-security -D_FORTIFY_SOURCE=2 -fstack-protector-all --param=ssp-buffer-size=4 -m64 -mtune=generic -flto -Wformat-security -Wmissing-format-attribute -Wno-non-virtual-dtor -fno-exceptions -fno-check-new -fno-common -I/usr/include/tqt -fexceptions -c -o libgvimageutils_la.all_cpp.lo libgvimageutils_la.all_cpp.cpp
libtool: compile:  /usr/bin/cc -DHAVE_CONFIG_H -I. -I../.. -I. -I./.. -I/opt/trinity/include/tde -I/usr/include/tqt3 -I/usr/include -include tqt.h -DTQT_THREAD_SUPPORT -D_REENTRANT -W -Wall -Wchar-subscripts -Wshadow -Wpointer-arith -Wmissing-prototypes -Wwrite-strings -D_DEFAULT_SOURCE -DNDEBUG -O2 -Os -fomit-frame-pointer -g3 -gdwarf-4 -Wstrict-aliasing=2 -pipe -Wformat -Werror=format-security -D_FORTIFY_SOURCE=2 -fstack-protector-all --param=ssp-buffer-size=4 -m64 -mtune=generic -flto -Wformat-security -Wmissing-format-attribute -I/usr/include/tqt -c transupp.c  -fPIC -DPIC -o .libs/transupp.o
libtool: compile:  /usr/bin/cc -DHAVE_CONFIG_H -I. -I../.. -I. -I./.. -I/opt/trinity/include/tde -I/usr/include/tqt3 -I/usr/include -include tqt.h -DTQT_THREAD_SUPPORT -D_REENTRANT -I. -W -Wall -Wchar-subscripts -Wshadow -Wpointer-arith -Wmissing-prototypes -Wwrite-strings -D_DEFAULT_SOURCE -DNDEBUG -O2 -Os -fomit-frame-pointer -g3 -gdwarf-4 -Wstrict-aliasing=2 -pipe -Wformat -Werror=format-security -D_FORTIFY_SOURCE=2 -fstack-protector-all --param=ssp-buffer-size=4 -m64 -mtune=generic -flto -Wformat-security -Wmissing-format-attribute -I/usr/include/tqt -c asm_scale.S  -fPIC -DPIC -o .libs/asm_scale.o
libtool: compile:  /usr/bin/c++ -DHAVE_CONFIG_H -I. -I../.. -I. -I./.. -I/opt/trinity/include/tde -I/usr/include/tqt3 -I/usr/include -include tqt.h -DTQT_THREAD_SUPPORT -D_REENTRANT -Wno-long-long -Wundef -D_DEFAULT_SOURCE -Wcast-align -Wchar-subscripts -Wall -W -Wpointer-arith -DNDEBUG -DNO_DEBUG -O2 -Os -fomit-frame-pointer -g3 -gdwarf-4 -Wstrict-aliasing=2 -pipe -Wformat -Werror=format-security -D_FORTIFY_SOURCE=2 -fstack-protector-all --param=ssp-buffer-size=4 -m64 -mtune=generic -flto -Wformat-security -Wmissing-format-attribute -Wno-non-virtual-dtor -fno-exceptions -fno-check-new -fno-common -I/usr/include/tqt -fexceptions -c libgvimageutils_la.all_cpp.cpp  -fPIC -DPIC -o .libs/libgvimageutils_la.all_cpp.o
transupp.c:114:7: error: call to undeclared function 'jcopy_block_row'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
  114 |       jcopy_block_row(src_buffer[offset_y] + x_crop_blocks,
      |       ^
transupp.c:179:11: error: call to undeclared function 'jcopy_block_row'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
  179 |           jcopy_block_row(buffer[offset_y] + blk_x + x_crop_blocks,
      |           ^
transupp.c:241:11: error: call to undeclared function 'jcopy_block_row'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
  241 |           jcopy_block_row(src_row_ptr + dst_blk_x + x_crop_blocks,
      |           ^
transupp.c:322:9: error: call to undeclared function 'jcopy_block_row'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
  322 |         jcopy_block_row(src_buffer[offset_y] + x_crop_blocks,
      |         ^
transupp.c:628:13: error: call to undeclared function 'jcopy_block_row'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
  628 |             jcopy_block_row(src_row_ptr + dst_blk_x + x_crop_blocks,
      |             ^
transupp.c:1074:7: error: call to undeclared function 'jdiv_round_up'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
 1074 |       jdiv_round_up((long) info->output_width,
      |       ^
transupp.c:1523:39: warning: unused parameter 'srcinfo' [-Wunused-parameter]
 1523 | jcopy_markers_setup (j_decompress_ptr srcinfo, JCOPY_OPTION option)
      |                                       ^
transupp.c:1523:61: warning: unused parameter 'option' [-Wunused-parameter]
 1523 | jcopy_markers_setup (j_decompress_ptr srcinfo, JCOPY_OPTION option)
      |                                                             ^
transupp.c:1549:33: warning: unused parameter 'option' [-Wunused-parameter]
 1549 |                    JCOPY_OPTION option)
      |                                 ^
3 warnings and 6 errors generated.
make[4]: *** [Makefile:691: transupp.lo] Error 1
make[4]: *** Waiting for unfinished jobs....
In file included from libgvimageutils_la.all_cpp.cpp:4:
./scale.cpp:811:10: warning: cast from 'uchar *' (aka 'unsigned char *') to 'TQRgb *' (aka 'unsigned int *') increases required alignment from 1 to 4 [-Wcast-align]
  811 |         *(TQRgb*)q=((TQRgb*)pixels)[ x_offset[x] ];
      |          ^~~~~~~~~
./scale.cpp:811:21: warning: cast from 'uchar *' (aka 'unsigned char *') to 'TQRgb *' (aka 'unsigned int *') increases required alignment from 1 to 4 [-Wcast-align]
  811 |         *(TQRgb*)q=((TQRgb*)pixels)[ x_offset[x] ];
      |                     ^~~~~~~~~~~~~~
./scale.cpp:992:42: warning: cast from 'uchar *' (aka 'unsigned char *') to 'unsigned int *' increases required alignment from 1 to 4 [-Wcast-align]
  992 |             mimageScaleAARGBA(scaleinfo, (unsigned int *)buffer.scanLine(0), 0, 0,
      |                                          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
./scale.cpp:995:41: warning: cast from 'uchar *' (aka 'unsigned char *') to 'unsigned int *' increases required alignment from 1 to 4 [-Wcast-align]
  995 |             mimageScaleAARGB(scaleinfo, (unsigned int *)buffer.scanLine(0), 0, 0,
      |                                         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
./scale.cpp:1165:38: warning: cast from 'uchar *' (aka 'unsigned char *') to 'unsigned int *' increases required alignment from 1 to 4 [-Wcast-align]
 1165 |     isi->ypoints = mimageCalcYPoints((unsigned int *)img.scanLine(0),
      |                                      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
5 warnings generated.
make[4]: Leaving directory '/tmp/BUILD.omv2490.x86_64/trinity-gwenview-1.4.2-x86_64/trinity-gwenview-14.1.2/src/imageutils'
make[3]: *** [Makefile:538: all-recursive] Error 1
make[3]: Leaving directory '/tmp/BUILD.omv2490.x86_64/trinity-gwenview-1.4.2-x86_64/trinity-gwenview-14.1.2/src'
make[2]: *** [Makefile:593: all-recursive] Error 1
make[2]: Leaving directory '/tmp/BUILD.omv2490.x86_64/trinity-gwenview-1.4.2-x86_64/trinity-gwenview-14.1.2'
make[1]: *** [Makefile:510: all] Error 2
make[1]: Leaving directory '/tmp/BUILD.omv2490.x86_64/trinity-gwenview-1.4.2-x86_64/trinity-gwenview-14.1.2'
<!-- This is a comment. Please fill in the required fields below. The comments provide instructions on how to do so. Note: You do not need to remove comments. --> ## Basic information - TDE version: 14.1.2 - Distribution: OpenMandriva cooker - Hardware: amd64 <!-- Use SL/* labels to set the severity level. Please do not set a milestone. --> ## Description FTBFS OpenMandriva 18.1.5-1 clang version 18.1.5 (/builddir/build/BUILD/llvm-18.1.5-x86_64/llvm-project-18.1.5.src/clang bce1a7458abcaba7efee8306802245abfe928979) ## Steps to reproduce Build ## Screenshots <details> <summary>Backtrace</summary> ``` make[4]: Entering directory '/tmp/BUILD.omv2490.x86_64/trinity-gwenview-1.4.2-x86_64/trinity-gwenview-14.1.2/src/imageutils' /bin/sh ../../libtool --tag=CC --mode=compile /usr/bin/cc -DHAVE_CONFIG_H -I. -I../.. -I. -I./.. -I/opt/trinity/include/tde -I/usr/include/tqt3 -I/usr/include -include tqt.h -DTQT_THREAD_SUPPORT -D_REENTRANT -W -Wall -Wchar-subscripts -Wshadow -Wpointer-arith -Wmissing-prototypes -Wwrite-strings -D_DEFAULT_SOURCE -DNDEBUG -O2 -Os -fomit-frame-pointer -g3 -gdwarf-4 -Wstrict-aliasing=2 -pipe -Wformat -Werror=format-security -D_FORTIFY_SOURCE=2 -fstack-protector-all --param=ssp-buffer-size=4 -m64 -mtune=generic -flto -Wformat-security -Wmissing-format-attribute -I/usr/include/tqt -c -o transupp.lo transupp.c /bin/sh ../../libtool --tag=CC --mode=compile /usr/bin/cc -DHAVE_CONFIG_H -I. -I../.. -I. -I./.. -I/opt/trinity/include/tde -I/usr/include/tqt3 -I/usr/include -include tqt.h -DTQT_THREAD_SUPPORT -D_REENTRANT -I. -W -Wall -Wchar-subscripts -Wshadow -Wpointer-arith -Wmissing-prototypes -Wwrite-strings -D_DEFAULT_SOURCE -DNDEBUG -O2 -Os -fomit-frame-pointer -g3 -gdwarf-4 -Wstrict-aliasing=2 -pipe -Wformat -Werror=format-security -D_FORTIFY_SOURCE=2 -fstack-protector-all --param=ssp-buffer-size=4 -m64 -mtune=generic -flto -Wformat-security -Wmissing-format-attribute -I/usr/include/tqt -c -o asm_scale.lo asm_scale.S creating libgvimageutils_la.all_cpp.cpp ... egrep: warning: egrep is obsolescent; using grep -E egrep: warning: egrep is obsolescent; using grep -E egrep: warning: egrep is obsolescent; using grep -E egrep: warning: egrep is obsolescent; using grep -E /bin/sh ../../libtool --tag=CXX --mode=compile /usr/bin/c++ -DHAVE_CONFIG_H -I. -I../.. -I. -I./.. -I/opt/trinity/include/tde -I/usr/include/tqt3 -I/usr/include -include tqt.h -DTQT_THREAD_SUPPORT -D_REENTRANT -Wno-long-long -Wundef -D_DEFAULT_SOURCE -Wcast-align -Wchar-subscripts -Wall -W -Wpointer-arith -DNDEBUG -DNO_DEBUG -O2 -Os -fomit-frame-pointer -g3 -gdwarf-4 -Wstrict-aliasing=2 -pipe -Wformat -Werror=format-security -D_FORTIFY_SOURCE=2 -fstack-protector-all --param=ssp-buffer-size=4 -m64 -mtune=generic -flto -Wformat-security -Wmissing-format-attribute -Wno-non-virtual-dtor -fno-exceptions -fno-check-new -fno-common -I/usr/include/tqt -fexceptions -c -o libgvimageutils_la.all_cpp.lo libgvimageutils_la.all_cpp.cpp libtool: compile: /usr/bin/cc -DHAVE_CONFIG_H -I. -I../.. -I. -I./.. -I/opt/trinity/include/tde -I/usr/include/tqt3 -I/usr/include -include tqt.h -DTQT_THREAD_SUPPORT -D_REENTRANT -W -Wall -Wchar-subscripts -Wshadow -Wpointer-arith -Wmissing-prototypes -Wwrite-strings -D_DEFAULT_SOURCE -DNDEBUG -O2 -Os -fomit-frame-pointer -g3 -gdwarf-4 -Wstrict-aliasing=2 -pipe -Wformat -Werror=format-security -D_FORTIFY_SOURCE=2 -fstack-protector-all --param=ssp-buffer-size=4 -m64 -mtune=generic -flto -Wformat-security -Wmissing-format-attribute -I/usr/include/tqt -c transupp.c -fPIC -DPIC -o .libs/transupp.o libtool: compile: /usr/bin/cc -DHAVE_CONFIG_H -I. -I../.. -I. -I./.. -I/opt/trinity/include/tde -I/usr/include/tqt3 -I/usr/include -include tqt.h -DTQT_THREAD_SUPPORT -D_REENTRANT -I. -W -Wall -Wchar-subscripts -Wshadow -Wpointer-arith -Wmissing-prototypes -Wwrite-strings -D_DEFAULT_SOURCE -DNDEBUG -O2 -Os -fomit-frame-pointer -g3 -gdwarf-4 -Wstrict-aliasing=2 -pipe -Wformat -Werror=format-security -D_FORTIFY_SOURCE=2 -fstack-protector-all --param=ssp-buffer-size=4 -m64 -mtune=generic -flto -Wformat-security -Wmissing-format-attribute -I/usr/include/tqt -c asm_scale.S -fPIC -DPIC -o .libs/asm_scale.o libtool: compile: /usr/bin/c++ -DHAVE_CONFIG_H -I. -I../.. -I. -I./.. -I/opt/trinity/include/tde -I/usr/include/tqt3 -I/usr/include -include tqt.h -DTQT_THREAD_SUPPORT -D_REENTRANT -Wno-long-long -Wundef -D_DEFAULT_SOURCE -Wcast-align -Wchar-subscripts -Wall -W -Wpointer-arith -DNDEBUG -DNO_DEBUG -O2 -Os -fomit-frame-pointer -g3 -gdwarf-4 -Wstrict-aliasing=2 -pipe -Wformat -Werror=format-security -D_FORTIFY_SOURCE=2 -fstack-protector-all --param=ssp-buffer-size=4 -m64 -mtune=generic -flto -Wformat-security -Wmissing-format-attribute -Wno-non-virtual-dtor -fno-exceptions -fno-check-new -fno-common -I/usr/include/tqt -fexceptions -c libgvimageutils_la.all_cpp.cpp -fPIC -DPIC -o .libs/libgvimageutils_la.all_cpp.o transupp.c:114:7: error: call to undeclared function 'jcopy_block_row'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration] 114 | jcopy_block_row(src_buffer[offset_y] + x_crop_blocks, | ^ transupp.c:179:11: error: call to undeclared function 'jcopy_block_row'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration] 179 | jcopy_block_row(buffer[offset_y] + blk_x + x_crop_blocks, | ^ transupp.c:241:11: error: call to undeclared function 'jcopy_block_row'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration] 241 | jcopy_block_row(src_row_ptr + dst_blk_x + x_crop_blocks, | ^ transupp.c:322:9: error: call to undeclared function 'jcopy_block_row'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration] 322 | jcopy_block_row(src_buffer[offset_y] + x_crop_blocks, | ^ transupp.c:628:13: error: call to undeclared function 'jcopy_block_row'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration] 628 | jcopy_block_row(src_row_ptr + dst_blk_x + x_crop_blocks, | ^ transupp.c:1074:7: error: call to undeclared function 'jdiv_round_up'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration] 1074 | jdiv_round_up((long) info->output_width, | ^ transupp.c:1523:39: warning: unused parameter 'srcinfo' [-Wunused-parameter] 1523 | jcopy_markers_setup (j_decompress_ptr srcinfo, JCOPY_OPTION option) | ^ transupp.c:1523:61: warning: unused parameter 'option' [-Wunused-parameter] 1523 | jcopy_markers_setup (j_decompress_ptr srcinfo, JCOPY_OPTION option) | ^ transupp.c:1549:33: warning: unused parameter 'option' [-Wunused-parameter] 1549 | JCOPY_OPTION option) | ^ 3 warnings and 6 errors generated. make[4]: *** [Makefile:691: transupp.lo] Error 1 make[4]: *** Waiting for unfinished jobs.... In file included from libgvimageutils_la.all_cpp.cpp:4: ./scale.cpp:811:10: warning: cast from 'uchar *' (aka 'unsigned char *') to 'TQRgb *' (aka 'unsigned int *') increases required alignment from 1 to 4 [-Wcast-align] 811 | *(TQRgb*)q=((TQRgb*)pixels)[ x_offset[x] ]; | ^~~~~~~~~ ./scale.cpp:811:21: warning: cast from 'uchar *' (aka 'unsigned char *') to 'TQRgb *' (aka 'unsigned int *') increases required alignment from 1 to 4 [-Wcast-align] 811 | *(TQRgb*)q=((TQRgb*)pixels)[ x_offset[x] ]; | ^~~~~~~~~~~~~~ ./scale.cpp:992:42: warning: cast from 'uchar *' (aka 'unsigned char *') to 'unsigned int *' increases required alignment from 1 to 4 [-Wcast-align] 992 | mimageScaleAARGBA(scaleinfo, (unsigned int *)buffer.scanLine(0), 0, 0, | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ./scale.cpp:995:41: warning: cast from 'uchar *' (aka 'unsigned char *') to 'unsigned int *' increases required alignment from 1 to 4 [-Wcast-align] 995 | mimageScaleAARGB(scaleinfo, (unsigned int *)buffer.scanLine(0), 0, 0, | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ./scale.cpp:1165:38: warning: cast from 'uchar *' (aka 'unsigned char *') to 'unsigned int *' increases required alignment from 1 to 4 [-Wcast-align] 1165 | isi->ypoints = mimageCalcYPoints((unsigned int *)img.scanLine(0), | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 5 warnings generated. make[4]: Leaving directory '/tmp/BUILD.omv2490.x86_64/trinity-gwenview-1.4.2-x86_64/trinity-gwenview-14.1.2/src/imageutils' make[3]: *** [Makefile:538: all-recursive] Error 1 make[3]: Leaving directory '/tmp/BUILD.omv2490.x86_64/trinity-gwenview-1.4.2-x86_64/trinity-gwenview-14.1.2/src' make[2]: *** [Makefile:593: all-recursive] Error 1 make[2]: Leaving directory '/tmp/BUILD.omv2490.x86_64/trinity-gwenview-1.4.2-x86_64/trinity-gwenview-14.1.2' make[1]: *** [Makefile:510: all] Error 2 make[1]: Leaving directory '/tmp/BUILD.omv2490.x86_64/trinity-gwenview-1.4.2-x86_64/trinity-gwenview-14.1.2' ``` </details>
Francois added the SL/critical label 4 weeks ago
zeroability commented 4 weeks ago
Collaborator

The undeclared seems to be part of lib64turbo-jpeg0.

The undeclared seems to be part of `lib64turbo-jpeg0`.
Francois commented 4 weeks ago
Poster
Collaborator

Here is the build log file for gwenview.
It builds correctly with gcc, buts fails with clang.

OpenMandriva 18.1.5-1 clang version 18.1.5 (/builddir/build/BUILD/llvm-18.1.5-x86_64/llvm-project-18.1.5.src/clang bce1a7458abcaba7efee8306802245abfe928979)

gcc version 13.2.1 20240330 (OpenMandriva) (GCC)

Here is the build log file for gwenview. It builds correctly with gcc, buts fails with clang. OpenMandriva 18.1.5-1 clang version 18.1.5 (/builddir/build/BUILD/llvm-18.1.5-x86_64/llvm-project-18.1.5.src/clang bce1a7458abcaba7efee8306802245abfe928979) gcc version 13.2.1 20240330 (OpenMandriva) (GCC)
gwenview-ftbfs-clang.log.gz
10 KiB
bero commented 4 weeks ago

Clang is less forgiving about some errors/warnings.

transupp.c:114:7: error: call to undeclared function 'jcopy_block_row'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]

The problem is that jcopy_block_row isn't declared anywhere, which is always a problem (because the compiler has to guess how to pass the parameters, given it doesn't know for sure what size they are).

These days jcopy_block_row is declared in jpegint.h, so #include <jpegint.h> should fix it. (Of course it may be even better to avoid using jcopy_block_row given it may go away in future libjpeg versions - the int in jpegint.h is for internal).

Clang is less forgiving about some errors/warnings. ``` transupp.c:114:7: error: call to undeclared function 'jcopy_block_row'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration] ``` The problem is that jcopy_block_row isn't declared anywhere, which is always a problem (because the compiler has to guess how to pass the parameters, given it doesn't know for sure what size they are). These days `jcopy_block_row` is declared in `jpegint.h`, so `#include <jpegint.h>` should fix it. (Of course it may be even better to avoid using jcopy_block_row given it may go away in future libjpeg versions - the int in jpegint.h is for internal).
Francois commented 4 weeks ago
Poster
Collaborator

Hello, thanks for the advice.
You are correct, adding #include <jpegint.h> solves the issue.

Hello, thanks for the advice. You are correct, adding #include <jpegint.h> solves the issue.
MicheleC commented 4 weeks ago
Owner

Ok, another patch to prepare. Will do tomorrow :-)

Ok, another patch to prepare. Will do tomorrow :-)
MicheleC referenced this issue from a commit 3 weeks ago
Fix FTBFS on OMV. This resolves issue #38
MicheleC commented 3 weeks ago
Owner

@Francois could you test building with PR #41?

@Francois could you test building with PR #41?
Francois commented 2 weeks ago
Poster
Collaborator

Hello @MicheleC , thanks for the patch.
New FTBFS below:

Backtrace 
[  4%] Building C object src/imageutils/CMakeFiles/gvimageutils-static.dir/transupp.c.o
cd /tmp/BUILD.omv5000.x86_64/trinity-gwenview-14.1.2/build/src/imageutils && /usr/bin/clang  -I/tmp/BUILD.omv5000.x86_64/trinity-gwenview-14.1.2/src -I/tmp/BUILD.omv5000.x86_64/trinity-gwenview-14.1.2/build -I/usr/include/tqt3 -I/usr/include/tqt -I/opt/trinity/include/tde -I/tmp/BUILD.omv5000.x86_64/trinity-gwenview-14.1.2/src/imageutils -I/tmp/BUILD.omv5000.x86_64/trinity-gwenview-14.1.2/build/src/imageutils -Os -fomit-frame-pointer -g3 -gdwarf-4 -Wstrict-aliasing=2 -pipe -Wformat -Werror=format-security -D_FORTIFY_SOURCE=2 -fstack-protector-all --param=ssp-buffer-size=4 -m64 -mtune=generic -flto -fdebug-prefix-map=/tmp/BUILD.omv5000.x86_64/trinity-gwenview-14.1.2=. -fmacro-prefix-map=/tmp/BUILD.omv5000.x86_64/trinity-gwenview-14.1.2=. -DNDEBUG -fPIC -MD -MT src/imageutils/CMakeFiles/gvimageutils-static.dir/transupp.c.o -MF CMakeFiles/gvimageutils-static.dir/transupp.c.o.d -o CMakeFiles/gvimageutils-static.dir/transupp.c.o -c /tmp/BUILD.omv5000.x86_64/trinity-gwenview-14.1.2/src/imageutils/transupp.c
In file included from /tmp/BUILD.omv5000.x86_64/trinity-gwenview-14.1.2/src/imageutils/transupp.c:4:
/tmp/BUILD.omv5000.x86_64/trinity-gwenview-14.1.2/src/imageutils/jpegint.h:405:14: error: unknown type name 'TQ_INT32'
  405 | extern const INT32 jpeg_aritab[];
      |              ^
/tmp/BUILD.omv5000.x86_64/trinity-gwenview-14.1.2/src/imageutils/jpegint.h:19:15: note: expanded from macro 'INT32'
   19 | #define INT32   TQ_INT32
      |                 ^
1 error generated.
make[2]: *** [src/imageutils/CMakeFiles/gvimageutils-static.dir/build.make:121: src/imageutils/CMakeFiles/gvimageutils-static.dir/transupp.c.o] Error 1
make[2]: Leaving directory '/tmp/BUILD.omv5000.x86_64/trinity-gwenview-14.1.2/build'
make[1]: *** [CMakeFiles/Makefile2:908: src/imageutils/CMakeFiles/gvimageutils-static.dir/all] Error 2
make[1]: Leaving directory '/tmp/BUILD.omv5000.x86_64/trinity-gwenview-14.1.2/build'
make: *** [Makefile:149: all] Error 2
Hello @MicheleC , thanks for the patch. New FTBFS below: <details> <summary>Backtrace</summary> ``` [ 4%] Building C object src/imageutils/CMakeFiles/gvimageutils-static.dir/transupp.c.o cd /tmp/BUILD.omv5000.x86_64/trinity-gwenview-14.1.2/build/src/imageutils && /usr/bin/clang -I/tmp/BUILD.omv5000.x86_64/trinity-gwenview-14.1.2/src -I/tmp/BUILD.omv5000.x86_64/trinity-gwenview-14.1.2/build -I/usr/include/tqt3 -I/usr/include/tqt -I/opt/trinity/include/tde -I/tmp/BUILD.omv5000.x86_64/trinity-gwenview-14.1.2/src/imageutils -I/tmp/BUILD.omv5000.x86_64/trinity-gwenview-14.1.2/build/src/imageutils -Os -fomit-frame-pointer -g3 -gdwarf-4 -Wstrict-aliasing=2 -pipe -Wformat -Werror=format-security -D_FORTIFY_SOURCE=2 -fstack-protector-all --param=ssp-buffer-size=4 -m64 -mtune=generic -flto -fdebug-prefix-map=/tmp/BUILD.omv5000.x86_64/trinity-gwenview-14.1.2=. -fmacro-prefix-map=/tmp/BUILD.omv5000.x86_64/trinity-gwenview-14.1.2=. -DNDEBUG -fPIC -MD -MT src/imageutils/CMakeFiles/gvimageutils-static.dir/transupp.c.o -MF CMakeFiles/gvimageutils-static.dir/transupp.c.o.d -o CMakeFiles/gvimageutils-static.dir/transupp.c.o -c /tmp/BUILD.omv5000.x86_64/trinity-gwenview-14.1.2/src/imageutils/transupp.c In file included from /tmp/BUILD.omv5000.x86_64/trinity-gwenview-14.1.2/src/imageutils/transupp.c:4: /tmp/BUILD.omv5000.x86_64/trinity-gwenview-14.1.2/src/imageutils/jpegint.h:405:14: error: unknown type name 'TQ_INT32' 405 | extern const INT32 jpeg_aritab[]; | ^ /tmp/BUILD.omv5000.x86_64/trinity-gwenview-14.1.2/src/imageutils/jpegint.h:19:15: note: expanded from macro 'INT32' 19 | #define INT32 TQ_INT32 | ^ 1 error generated. make[2]: *** [src/imageutils/CMakeFiles/gvimageutils-static.dir/build.make:121: src/imageutils/CMakeFiles/gvimageutils-static.dir/transupp.c.o] Error 1 make[2]: Leaving directory '/tmp/BUILD.omv5000.x86_64/trinity-gwenview-14.1.2/build' make[1]: *** [CMakeFiles/Makefile2:908: src/imageutils/CMakeFiles/gvimageutils-static.dir/all] Error 2 make[1]: Leaving directory '/tmp/BUILD.omv5000.x86_64/trinity-gwenview-14.1.2/build' make: *** [Makefile:149: all] Error 2 ``` </details>
zeroability commented 2 weeks ago
Collaborator

Could it be a result of this?

These days jcopy_block_row is declared in jpegint.h, so #include <jpegint.h> should fix it. (Of course it may be even better to avoid using jcopy_block_row given it may go away in future libjpeg versions - the int in jpegint.h is for internal).

Could it be a result of this? >These days jcopy_block_row is declared in jpegint.h, so #include <jpegint.h> should fix it. (Of course it may be even better to avoid using jcopy_block_row given it may go away in future libjpeg versions - the int in jpegint.h is for internal).
MicheleC referenced this issue from a commit 2 weeks ago
Fix FTBFS on OMV. This resolves issue #38
MicheleC commented 2 weeks ago
Owner

@Francois I have updated the patch, replacing INT32 with int32_t type, which is available from c++11. TQ_INT32 is defined in a TQt header file that was not included in jpegint.h or before it.

@Francois I have updated the patch, replacing `INT32` with `int32_t` type, which is available from c++11. `TQ_INT32` is defined in a TQt header file that was not included in `jpegint.h` or before it.
MicheleC referenced this issue from a commit 2 weeks ago
Fix FTBFS on OMV. This resolves issue #38
Francois commented 2 weeks ago
Poster
Collaborator

Thanks @MicheleC , now I can build gwenvien on openmandriva.

Thanks @MicheleC , now I can build gwenvien on openmandriva.
MicheleC referenced this issue from a commit 2 weeks ago
Fix FTBFS on OMV. This resolves issue #38
MicheleC commented 2 weeks ago
Owner

PR merged and backported.

PR merged and backported.
MicheleC closed this issue 2 weeks ago
MicheleC added this to the R14.1.3 release milestone 2 weeks ago
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
r14.1.x
master
r14.0.x
v3.5.13-sru
Labels
Apply labels
Clear labels
GE/need-info
General - need additional info from contributor PR/keep-branch
Pull request - do not delete branch after merging PR/not-ok
Pull request - need fixing PR/rfc
Pull request - request for comments PR/update-trans
Pull request - update to translation files needed PR/wip
Pull request - work in progress RS/R14.0.x
Related to R14.0.x series RS/R14.1.x
Related to R14.1.x series SL/critical
Severity level - critical SL/major
Severity level - major SL/minor
Severity level - minor SL/normal
Severity level - normal SL/regression
Severity level - regression from previous version SL/trivial
Severity level - trivial SL/wishlist
Severity level - wishlist request ST/duplicate
Status - duplicate of another issue ST/invalid
Status - invalid report ST/notourproblem
Status - not our problem ST/rejected
Status - rejected ST/wontfix
Status - won't fix ST/worksforme
Status - works for me, unable to reproduce
No Label GE/need-info PR/keep-branch PR/not-ok PR/rfc PR/update-trans PR/wip RS/R14.0.x RS/R14.1.x SL/critical SL/major SL/minor SL/normal SL/regression SL/trivial SL/wishlist ST/duplicate ST/invalid ST/notourproblem ST/rejected ST/wontfix ST/worksforme
Milestone
Set milestone
Clear milestone
No items
No Milestone
R14.1.3 release
Assignees
Assign users
Clear assignees
No Assignees
4 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: TDE/gwenview#38
Write Preview
Loading…
Cancel
Save
There is no content yet.
Delete Branch '%!s(<nil>)'

Deleting a branch is permanent. It CANNOT be undone. Continue?

No
Yes