[HEADS-UP] poppler soname bump

[David Tardon]

Hi all,

I will build a new release of poppler this week, which includes soname
bump. I will take care of rebuilding the affected packages:

boomaga
calligra
cups-filters
gambas3
gdcm
inkscape
kf5-kfilemetadata
libreoffice
okular
pdf2djvu
poppler
poppler-sharp
texlive
texworks

Note that packages that only use one of the wrappers (poppler-cpp,
poppler-glib, poppler-qt{4,5}) are not affected.

D.

[Yaakov Selkowitz]

On 2017-11-06 06:13, David Tardon wrote:
>  I will build a new release of poppler this week, which includes
> soname
>  bump. I will take care of rebuilding the affected packages: 
While ultimately it would be nice to have everything use one of the
bindings as intended, but in the meantime perhaps we can still save you
some work for the next bump.

>  kf5-kfilemetadata
>  okular 
These are cases of overlinking, as they only use symbols from
libpoppler-qt5.  This can be seen in Ubuntu (which links with
-Wl,--as-needed and -Wl,--no-undefined) and Cygwin (which as a PE
platform acts similarly automatically), neither of which require libpoppler.

extra-cmake-modules should be fixed so that Poppler::Qt5 does not pull
in Poppler::Core:

--- a/find-modules/FindPoppler.cmake
+++ b/find-modules/FindPoppler.cmake
@@ -86,5 +86,4 @@
 foreach(_comp ${Poppler_known_components})
     string(TOLOWER "${_comp}" _lc_comp)
-    set(Poppler_${_comp}_component_deps "Core")
     set(Poppler_${_comp}_pkg_config "poppler-${_lc_comp}")
     set(Poppler_${_comp}_lib "poppler-${_lc_comp}")

>  poppler-sharp 
Are you sure this is necessary?  Only libpoppler-glib is mentioned in
the .dll.config, and libpoppler doesn't have a C API for the assembly to
load.

>  texworks 
This too appears to be a case of overlinking, and again neither Ubuntu
or Cygwin show a libpoppler dependency, and Debian's build log shows a
warning that it is unused.  POPPLER_BASE_LIBRARIES is used in
modules/QtPDF/CMake/Modules/FindPoppler.cmake (no relation to ECM's) to
determine if poppler was built with fontconfig support, but if you
remove it from POPPLER_LIBRARIES there it should avoid this.

-- 
Yaakov Selkowitz
Software Engineer - Platform Enablement Group
Red Hat, Inc.

[Rex Dieter]

Yaakov Selkowitz wrote:

> > kf5-kfilemetadata
> > okular
>  
>  These are cases of overlinking, as they only use symbols from
>  libpoppler-qt5.  This can be seen in Ubuntu (which links with
>  -Wl,--as-needed and -Wl,--no-undefined) and Cygwin (which as a PE
>  platform acts similarly automatically), neither of which require
>  libpoppler.
>  
>  extra-cmake-modules should be fixed so that Poppler::Qt5 does not pull
>  in Poppler::Core:
>  
>  --- a/find-modules/FindPoppler.cmake
>  +++ b/find-modules/FindPoppler.cmake
>  @@ -86,5 +86,4 @@
>   foreach(_comp ${Poppler_known_components})
>       string(TOLOWER "${_comp}" _lc_comp)
>  -    set(Poppler_${_comp}_component_deps "Core")
>       set(Poppler_${_comp}_pkg_config "poppler-${_lc_comp}")
>       set(Poppler_${_comp}_lib "poppler-${_lc_comp}") 

Thanks, I'll fix these as suggested (in rawhide)

-- Rex

[David Tardon]

Hi,

On Mon, Nov 06, 2017 at 01:10:15PM -0600, Yaakov Selkowitz wrote:
>  On 2017-11-06 06:13, David Tardon wrote:
>  > I will build a new release of poppler this week, which includes soname
>  > bump. I will take care of rebuilding the affected packages:
>  
>  While ultimately it would be nice to have everything use one of the
>  bindings as intended, 
That's not possible, unfortunately. The bindings are insufficient if the
project needs more than PDF rendering widget.

>  > poppler-sharp
>  
>  Are you sure this is necessary?  Only libpoppler-glib is mentioned in
>  the .dll.config, and libpoppler doesn't have a C API for the assembly to
>  load. 
dnf repoquery claims it is.

D.

[Yaakov Selkowitz]

On 2017-11-07 02:24, David Tardon wrote:
>  On Mon, Nov 06, 2017 at 01:10:15PM -0600, Yaakov Selkowitz wrote:
> > On 2017-11-06 06:13, David Tardon wrote:
> >> poppler-sharp
> >
> > Are you sure this is necessary?  Only libpoppler-glib is mentioned in
> > the .dll.config, and libpoppler doesn't have a C API for the assembly to
> > load.
>  
>  dnf repoquery claims it is. 
In this case it's because of the spec itself:

Requires:	poppler
Requires:	poppler-glib

The latter is clearly required, as the glib bindings are loaded by the
assembly and are listed in the .dll.config.  The same cannot be true of
libpoppler, however, as it is not listed in the .dll.config and has no C
API that could be loaded by the assembly.  Therefore, it is not clear
why poppler is explicitly listed, nor why it should necessitate a rebuild.

-- 
Yaakov Selkowitz
Software Engineer - Platform Enablement Group
Red Hat, Inc.

[David Tardon]

Hi all,

On Mon, Nov 06, 2017 at 01:13:15PM +0100, David Tardon wrote:
>  Hi all,
>  
>  I will build a new release of poppler this week, which includes soname
>  bump. I will take care of rebuilding the affected packages:
>  
>  boomaga
>  cups-filters
>  gambas3
>  gdcm
>  inkscape
>  kf5-kfilemetadata
>  libreoffice
>  okular
>  pdf2djvu
>  poppler
>  poppler-sharp
>  texworks 
Done.

>  texlive 
Still running.

>  calligra 
Build fails because of broken deps.

D.