Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 26 Mar 2022 12:22:11 GMT
From:      Dimitry Andric <dim@FreeBSD.org>
To:        ports-committers@FreeBSD.org, dev-commits-ports-all@FreeBSD.org, dev-commits-ports-main@FreeBSD.org
Subject:   git: 14bd6052d998 - main - converters/wkhtmltopdf: future-proof compiler version check
Message-ID:  <202203261222.22QCMBks066493@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
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"



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202203261222.22QCMBks066493>