Bug 629708

Summary:	net-print/cups-filters-1.17.2 fails to build with app-text/poppler-0.58.0
Product:	Gentoo Linux	Reporter:	Georgy Yakovlev <gyakovlev>
Component:	Current packages	Assignee:	Printing Team <printing>
Status:	RESOLVED FIXED
Severity:	normal	CC:	bkohler, carlphilippreh, che, chris, dblaci, fabio.coatti, flockmock, floppym, gaweal, gentoo, jarausch, jcallen, josef64, jouni.kosonen, jr.juiliano, kamensky.fb, kde, klaus818, krinpaus, marduk, mark+gentoobugs, Martin.vGagern, mike, nelo, nholland, office, phantom4, proteuss, randy-andy-, raphead, reagentoo, reavertm, saintdev, sam, scottsshort, unheatedgarage, vanilla.sources, volkov, xaviermiller
Priority:	Normal
Version:	unspecified
Hardware:	All
OS:	Linux
URL:	https://bugs.linuxfoundation.org/show_bug.cgi?id=1408
Whiteboard:
Package list:		Runtime testing required:	---
Bug Depends on:	641340
Bug Blocks:	629836
Attachments:	build log emerge --info cups-filters Half-working patch use std::move?

Description Georgy Yakovlev archtester

2017-09-02 23:16:45 UTC

net-print/cups-filters-1.17.2 fails to build with app-text/poppler-0.58.0



Reproducible: Always

Comment 1 Georgy Yakovlev archtester

2017-09-02 23:17:24 UTC

Created attachment 491962 [details]
build log

Comment 2 Georgy Yakovlev archtester

2017-09-02 23:17:50 UTC

Created attachment 491964 [details]
emerge --info cups-filters

Comment 3 Ostashevskyi Viktor 2017-09-03 07:11:19 UTC

I have same issue.

Comment 4 Stefan Briesenick 2017-09-03 10:12:50 UTC

confirm. building with app-text/poppler-0.57.0 works though.

Comment 5 Dmitriy Baranov 2017-09-03 11:14:33 UTC

Same problem. (~amd64)

Comment 6 klaus818 2017-09-03 11:58:17 UTC

If you have the same problem and no other helpful information, you can add yourself to the cc-list.

A message "same problem" helps nobody.

Comment 7 Dmitriy Baranov 2017-09-03 12:08:57 UTC

(In reply to klaus818 from comment #6)
> If you have the same problem and no other helpful information, you can add
> yourself to the cc-list.
> 
> A message "same problem" helps nobody.

Are you blind?

> amd64
Additional info.

Comment 8 Michael Palimaka (kensington) gentoo-dev

2017-09-03 12:25:34 UTC

me toooooo!

Comment 9 László Szalma 2017-09-03 15:33:54 UTC

As this blocks many kde packages to update I applied the workaround for now:

added this line to /etc/portage/package.mask

    =app-text/poppler-0.58.0

I also confirm the build fails for

=net-print/cups-filters-1.16.4
=net-print/cups-filters-1.17.1

too in addition to 1.17.2.

Comment 10 Ben Kohler gentoo-dev

2017-09-04 14:02:11 UTC

FYI upstream https://bugs.linuxfoundation.org/show_bug.cgi?id=1408

Comment 11 Malcolm Lashley 2017-09-04 19:09:10 UTC

Created attachment 492352 [details, diff]
Half-working patch

This is not pretty - some serious API changes - but this at least gets it to compile - I'm pretty sure it isn't 100% correct - but bannertopdf generates *something*. I lack the expertise in the CUPS system to say if everything is present and correct.

(tested with
/usr/libexec/cups/filter/bannertopdf 1 usermalc jobtitlemalc copies666 optionsarehere </usr/share/cups/data/testprint >/tmp/banner.pdf
)

Posting in case someone wants to pick it up and run with it (also posted upstream). I lack the time/energy to take it further.

Comment 12 iGentoo 2017-09-05 12:36:54 UTC

Created attachment 492426 [details, diff]
use std::move?

Comment 13 Lars Wendler (Polynomial-C) (RETIRED) gentoo-dev

2017-09-08 10:09:05 UTC

This is fixed in cups-filter-1.17.3

Comment 14 Andreas Sturmlechner gentoo-dev

2018-01-16 14:06:39 UTC

*** Bug 644752 has been marked as a duplicate of this bug. ***