git: 14bd6052d998 - main - converters/wkhtmltopdf: future-proof compiler version check
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Sat, 26 Mar 2022 12:22:11 UTC
The branch main has been updated by dim (src committer): URL: https://cgit.FreeBSD.org/ports/commit/?id=14bd6052d9987e3205672fc3f2874afd16279c30 commit 14bd6052d9987e3205672fc3f2874afd16279c30 Author: Dimitry Andric <dim@FreeBSD.org> AuthorDate: 2021-05-27 21:02:06 +0000 Commit: Dimitry Andric <dim@FreeBSD.org> CommitDate: 2022-03-26 12:21:08 +0000 converters/wkhtmltopdf: future-proof compiler version check During an exp-run for llvm 14 (see bug 261742), it turned out that converters/wkhtmltopdf fails to build with clang 14: c++ -Wl,-rpath=/usr/local/lib/gcc8 -Wl,-O1 -pthread -shared -Wl,-soname,libwkhtmltox.so.0 -o libwkhtmltox.so.0.12.6 loadsettings.o logging.o multipageloader.o tempfile.o converter.o websettings.o reflect.o utilities.o pdfsettings.o pdfconverter.o outline.o tocstylesheet.o imagesettings.o imageconverter.o pdf_c_bindings.o image_c_bindings.o moc_multipageloader_p.o moc_converter_p.o moc_pdfconverter_p.o moc_imageconverter_p.o moc_pdf_c_bindings_p.o moc_image_c_bindings_p.o moc_converter.o moc_multipageloader.o moc_utilities.o moc_pdfconverter.o moc_imageconverter.o qrc_wkhtmltopdf.o -L/usr/local/lib -L/wrkdirs/usr/ports/converters/wkhtmltopdf/work/wkhtmltopdf-0.12.6/build/qt/lib -L/wrkdirs/usr/ports/converters/wkhtmltopdf/work/wkhtmltopdf-0.12.6/build/qt/plugins/codecs -lqcncodecs -L/wrkdirs/usr/ports/converters/wkhtmltopdf/work/wkhtmltopdf-0.12.6/build/qt/lib -L/usr/local/lib -lqjpcodecs -lqkrcodecs -lqtwcodecs -lQtWebKit -lQtSvg -lQtXmlPatterns -lQtGui -ljpeg -lpng -lXrender -l fontconfig -lfreetype -lXext -lX11 -lQtNetwork -lssl -lcrypto -lQtCore -lz -lm ld: error: unable to find library -lQtWebKit c++: error: linker command failed with exit code 1 (use -v to see invocation) gmake[2]: *** [Makefile:164: ../../bin/libwkhtmltox.so.0.12.6] Error 1 Ultimately this is caused by incorrect compiler version detection in wkhtmltopdf's configure script. Something similar was fixed for clang 12 in ports commit 1049d80399ac327964eca41e4188f988430c5f7c, and later for clang 13 in ports commit ad5fedf36832fd7e2447d8c4cb6b54128a2fcb20. To fix this, future-proof the compiler version check, at least until it goes higher than 99.x. PR: 262814 Approved by: pi (maintainer) MFH: 2022Q1 --- converters/wkhtmltopdf/files/patch-configure | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/converters/wkhtmltopdf/files/patch-configure b/converters/wkhtmltopdf/files/patch-configure index e891e62e62a9..167565dc496d 100644 --- a/converters/wkhtmltopdf/files/patch-configure +++ b/converters/wkhtmltopdf/files/patch-configure @@ -5,7 +5,7 @@ # Check gcc's version case "$(${QMAKE_CONF_COMPILER} -dumpversion)" in - 8*|7*|6*|5*|4*|3.4*) -+ 13*|12*|11*|10*|9*|8*|7*|6*|5*|4*|3.4*) ++ [1-9][0-9]*|[4-9]*|3.4*) ;; 3.3*) canBuildWebKit="no"