Re: HOTFIX: x11/qt5/qtbase enable cups and double-conversion support

2020-03-15 Thread Rafael Sadowski
On Sun Mar 15, 2020 at 12:23:27PM +0300, Vadim Zhukov wrote:
> вс, 15 мар. 2020 г. в 12:13, Rafael Sadowski :
> >
> > The diff below have to go quick into the tree. This enables cups and
> > doubleconversion support again. Currently ALL Qt5 applications crash
> > when opening the "print dialog". Sorry for that but plugins are hard to
> > test!
> >
> > Following changes made:
> >
> > - Add -L/-I to CONFIGURE_ARGS this fixes cups and doubleconversion
> >   detection.
> > - Remove double configure args
> > - enable cups and doubleconversion and bump the -main package.
> >
> > This update should also unbreak the qcad build. Testing currently
> >
> > OK, objections?
> >
> > Rafael
> >
> > Index: Makefile
> > ===
> > RCS file: /cvs/ports/x11/qt5/qtbase/Makefile,v
> > retrieving revision 1.36
> > diff -u -p -u -p -r1.36 Makefile
> > --- Makefile13 Mar 2020 16:53:49 -  1.36
> > +++ Makefile15 Mar 2020 09:09:01 -
> > @@ -16,6 +16,8 @@ PKGNAME-psql =qt5-postgresql-${VERSION
> >  PKGNAME-sqlite2 =  qt5-sqlite2-${VERSION}
> >  PKGNAME-tds =  qt5-tds-${VERSION}
> >
> > +REVISON-main = 0
> > +
> >  PKG_ARCH-global =  *
> >  PKG_ARCH-examples =*
> >
> > @@ -38,14 +40,15 @@ SHARED_LIBS +=  Qt5OpenGLExtensions
> >  SHARED_LIBS +=  Qt5XcbQpa0.0 # 5.13
> >
> >  WANTLIB-main += ${COMPILER_LIBCXX} EGL GL ICE SM X11 X11-xcb Xext atk-1.0
> > -WANTLIB-main += c cairo cairo-gobject crypto dbus-1 drm execinfo fontconfig
> > -WANTLIB-main += freetype gbm gdk-3 gdk_pixbuf-2.0 gio-2.0 glib-2.0
> > -WANTLIB-main += gobject-2.0 gthread-2.0 gtk-3 harfbuzz icudata icui18n
> > -WANTLIB-main += icuio icuuc iodbc iodbcinst jpeg m pango-1.0 pangocairo-1.0
> > -WANTLIB-main += pcre2-16 png sqlite3 ssl xcb xcb-glx xcb-icccm xcb-image
> > -WANTLIB-main += xcb-keysyms xcb-randr xcb-render xcb-render-util xcb-shape
> > -WANTLIB-main += xcb-shm xcb-sync xcb-xfixes xcb-xinerama xcb-xinput
> > -WANTLIB-main += xcb-xkb xkbcommon xkbcommon-x11 z zstd
> > +WANTLIB-main += c cairo cairo-gobject crypto cups dbus-1 double-conversion
> > +WANTLIB-main += drm execinfo fontconfig freetype gbm gdk-3 gdk_pixbuf-2.0
> > +WANTLIB-main += gio-2.0 glib-2.0 gobject-2.0 gthread-2.0 gtk-3 harfbuzz
> > +WANTLIB-main += icudata icui18n icuio icuuc iodbc iodbcinst jpeg m
> > +WANTLIB-main += pango-1.0 pangocairo-1.0 pcre2-16 png sqlite3 ssl xcb
> > +WANTLIB-main += xcb-glx xcb-icccm xcb-image xcb-keysyms xcb-randr 
> > xcb-render
> > +WANTLIB-main += xcb-render-util xcb-shape xcb-shm xcb-sync xcb-xfixes
> > +WANTLIB-main += xcb-xinerama xcb-xinput xcb-xkb xkbcommon xkbcommon-x11
> > +WANTLIB-main += z zst

Here is also a copy/paste issue s/zst/zstd. Only in my email.

> >
> >  WANTLIB-global =
> >
> > @@ -81,6 +84,8 @@ LIB_DEPENDS-main =${LIB_DEPENDS} \
> > devel/pcre2 \
> > graphics/cairo \
> > graphics/gdk-pixbuf2 \
> > +   math/double-conversion \
> > +   print/cups,-libs \
> > textproc/icu4c \
> > x11/dbus \
> > x11/gtk+3 \
> > @@ -143,6 +148,10 @@ QT_QMLDIR =${QT_BASEDIR}/qml
> >
> >  CONFIGURE_ARGS +=  OPENSSL_LIBS="-L/usr/lib -lssl -lcrypto"
> >
> > +# configure
> > +CONFIGURE_ARGS+= -I "${PREFIX}/include"
> > +CONFIGURE_ARGS+= -L "${PREFIX}/lib"
> 
> Those should be "${LOCALBASE}/foo".
> 
> The PREFIX is where the currently built package will be installed.
> The LOCALBASE is where the dependencies reside.

Thanks Vadim for the review, builds also with this change.

> 
> > +
> >  # paths
> >  CONFIGURE_ARGS +=  -bindir ${QT_BINDIR} \
> > -datadir ${QT_BASEDIR} \
> > @@ -151,7 +160,7 @@ CONFIGURE_ARGS +=   -bindir ${QT_BINDIR} \
> > -headerdir ${QT_INCDIR} \
> > -libdir ${QT_BASEDIR} \
> > -plugindir ${QT_PLUGINSDIR} \
> > -   -prefix ${PREFIX} \
> > +   -prefix ${QT_BASEDIR} \
> > -qmldir ${QT_QMLDIR} \
> > -sysconfdir ${SYSCONFDIR}/xdg
> >
> > @@ -159,7 +168,6 @@ CONFIGURE_ARGS +=   -bindir ${QT_BINDIR} \
> >  CONFIGURE_ARGS +=  -confirm-license \
> > -no-compile-examples \
> > -opensource \
> > -   -openssl-linked \
> 
> Is this change intended?

Yes it was there twice. Sorry I didn't mention it.

New diff:


Index: Makefile
===
RCS file: /cvs/ports/x11/qt5/qtbase/Makefile,v
retrieving revision 1.36
diff -u -p -u -p -r1.36 Makefile
--- Makefile13 Mar 2020 16:53:49 -  1.36
+++ Makefile15 Mar 2020 09:34:23 -
@@ -16,6 +16,8 @@ PKGNAME-psql =qt5-postgresql-${VERSION
 PKGN

Re: HOTFIX: x11/qt5/qtbase enable cups and double-conversion support

2020-03-15 Thread Vadim Zhukov
вс, 15 мар. 2020 г. в 12:13, Rafael Sadowski :
>
> The diff below have to go quick into the tree. This enables cups and
> doubleconversion support again. Currently ALL Qt5 applications crash
> when opening the "print dialog". Sorry for that but plugins are hard to
> test!
>
> Following changes made:
>
> - Add -L/-I to CONFIGURE_ARGS this fixes cups and doubleconversion
>   detection.
> - Remove double configure args
> - enable cups and doubleconversion and bump the -main package.
>
> This update should also unbreak the qcad build. Testing currently
>
> OK, objections?
>
> Rafael
>
> Index: Makefile
> ===
> RCS file: /cvs/ports/x11/qt5/qtbase/Makefile,v
> retrieving revision 1.36
> diff -u -p -u -p -r1.36 Makefile
> --- Makefile13 Mar 2020 16:53:49 -  1.36
> +++ Makefile15 Mar 2020 09:09:01 -
> @@ -16,6 +16,8 @@ PKGNAME-psql =qt5-postgresql-${VERSION
>  PKGNAME-sqlite2 =  qt5-sqlite2-${VERSION}
>  PKGNAME-tds =  qt5-tds-${VERSION}
>
> +REVISON-main = 0
> +
>  PKG_ARCH-global =  *
>  PKG_ARCH-examples =*
>
> @@ -38,14 +40,15 @@ SHARED_LIBS +=  Qt5OpenGLExtensions
>  SHARED_LIBS +=  Qt5XcbQpa0.0 # 5.13
>
>  WANTLIB-main += ${COMPILER_LIBCXX} EGL GL ICE SM X11 X11-xcb Xext atk-1.0
> -WANTLIB-main += c cairo cairo-gobject crypto dbus-1 drm execinfo fontconfig
> -WANTLIB-main += freetype gbm gdk-3 gdk_pixbuf-2.0 gio-2.0 glib-2.0
> -WANTLIB-main += gobject-2.0 gthread-2.0 gtk-3 harfbuzz icudata icui18n
> -WANTLIB-main += icuio icuuc iodbc iodbcinst jpeg m pango-1.0 pangocairo-1.0
> -WANTLIB-main += pcre2-16 png sqlite3 ssl xcb xcb-glx xcb-icccm xcb-image
> -WANTLIB-main += xcb-keysyms xcb-randr xcb-render xcb-render-util xcb-shape
> -WANTLIB-main += xcb-shm xcb-sync xcb-xfixes xcb-xinerama xcb-xinput
> -WANTLIB-main += xcb-xkb xkbcommon xkbcommon-x11 z zstd
> +WANTLIB-main += c cairo cairo-gobject crypto cups dbus-1 double-conversion
> +WANTLIB-main += drm execinfo fontconfig freetype gbm gdk-3 gdk_pixbuf-2.0
> +WANTLIB-main += gio-2.0 glib-2.0 gobject-2.0 gthread-2.0 gtk-3 harfbuzz
> +WANTLIB-main += icudata icui18n icuio icuuc iodbc iodbcinst jpeg m
> +WANTLIB-main += pango-1.0 pangocairo-1.0 pcre2-16 png sqlite3 ssl xcb
> +WANTLIB-main += xcb-glx xcb-icccm xcb-image xcb-keysyms xcb-randr xcb-render
> +WANTLIB-main += xcb-render-util xcb-shape xcb-shm xcb-sync xcb-xfixes
> +WANTLIB-main += xcb-xinerama xcb-xinput xcb-xkb xkbcommon xkbcommon-x11
> +WANTLIB-main += z zst
>
>  WANTLIB-global =
>
> @@ -81,6 +84,8 @@ LIB_DEPENDS-main =${LIB_DEPENDS} \
> devel/pcre2 \
> graphics/cairo \
> graphics/gdk-pixbuf2 \
> +   math/double-conversion \
> +   print/cups,-libs \
> textproc/icu4c \
> x11/dbus \
> x11/gtk+3 \
> @@ -143,6 +148,10 @@ QT_QMLDIR =${QT_BASEDIR}/qml
>
>  CONFIGURE_ARGS +=  OPENSSL_LIBS="-L/usr/lib -lssl -lcrypto"
>
> +# configure
> +CONFIGURE_ARGS+= -I "${PREFIX}/include"
> +CONFIGURE_ARGS+= -L "${PREFIX}/lib"

Those should be "${LOCALBASE}/foo".

The PREFIX is where the currently built package will be installed.
The LOCALBASE is where the dependencies reside.

> +
>  # paths
>  CONFIGURE_ARGS +=  -bindir ${QT_BINDIR} \
> -datadir ${QT_BASEDIR} \
> @@ -151,7 +160,7 @@ CONFIGURE_ARGS +=   -bindir ${QT_BINDIR} \
> -headerdir ${QT_INCDIR} \
> -libdir ${QT_BASEDIR} \
> -plugindir ${QT_PLUGINSDIR} \
> -   -prefix ${PREFIX} \
> +   -prefix ${QT_BASEDIR} \
> -qmldir ${QT_QMLDIR} \
> -sysconfdir ${SYSCONFDIR}/xdg
>
> @@ -159,7 +168,6 @@ CONFIGURE_ARGS +=   -bindir ${QT_BINDIR} \
>  CONFIGURE_ARGS +=  -confirm-license \
> -no-compile-examples \
> -opensource \
> -   -openssl-linked \

Is this change intended?

-- 
  WBR,
  Vadim Zhukov