okular prints landscape oriented document in portrait

[P]

Binary package hint: kdegraphics

I am trying to print a pdf file landscape oriented. It shows up correctly in okular but prints in portrait regardless of my printer options.

The same pdf printed with xpdf is correctly printed in landscape without touching printer options (xpdf uses lpr to print).

You can check bug #504168 in the cups error log (created when hpijs crashes) the options passed by okular to cups.

My printer is a HP deskjet 970cxi.

ProblemType: Bug
Architecture: i386
Date: Thu Jan 7 10:53:47 2010
DistroRelease: Ubuntu 9.10
InstallationMedia: Kubuntu 9.10 "Karmic Koala" - Release i386 (20091028.5)
Package: okular 4:4.3.4-0ubuntu1~karmic1
ProcEnviron:
 LANGUAGE=
 PATH=(custom, user)
 LANG=fr_FR.UTF-8
 SHELL=/bin/bash
ProcVersionSignature: Ubuntu 2.6.31-16.53-generic
SourcePackage: kdegraphics
Uname: Linux 2.6.31-16-generic i686
XsessionErrors:
 (firefox:2519): GLib-WARNING **: g_set_prgname() called multiple times
 (polkit-gnome-authentication-agent-1:2596): GLib-CRITICAL **: g_once_init_leave: assertion `initialization_value != 0' failed
 (firefox:10375): GLib-WARNING **: g_set_prgname() called multiple times

[Fabio Papa (papaf)]

Version: (using Devel)
OS: Linux
Installed from: Compiled sources

When working with landscape pdf, chosing landscape when printing actually prints the document in portrait. Chosing portrait prints the document in landscape.

I think that the portrait/landscape setting in the printing window actually is more like a rotate command: portrait leaves the document alone while landscape rotates it by 90 degrees. This is not always correct, obvioulsy in my case it's not.

Okular I used is from the latest RC debian packages, running on Ubuntu Intrepid.

[Pino Toscano (pinotree)]

Couls you please provide a sample document showing the issue?

[Fabio Papa (papaf)]

Created attachment 30416
Example pdf file creating the problem

As requested, this is the file I first encountered the problem with.

[Mail-to-wrt (mail-to-wrt)]

I observe the same bug, printing a landscape pdf, the content will not be rotated so that it fits the page but is rather cut of at the right side of the paper and at the bottom there is free white space

System: OpenSuse 11.0 and 11.1
Source: OpenSuse BuildService KDE4:Factory
Version: 0.8 (KDE 4.2.0)
Architecture i586 and x86_64
Cups: cups-1.3.9-7.1 (on OpenSuse 11.1)

When printing to pdf, the resulting pdf will be landscape as well, when printing to ps, the preview in dolphin shows it (correctly rotated) as portrait and when I then open it in okular again, it will be shown as landscape (correctly rotated again)

[Jonathan Thomas (echidnaman)]

Hi there!

Thanks for reporting this bug! Your bug seems to be a problem with the KDE program itself, and not with our KDE packages. While we appreciate your issue, it would be better if it was tracked at https://bugs.kde.org, so that the KDE developers can deal with this speedily and have direct communication with you as the reporter for more effective debugging.

Thanks!

[Rodschaffter (rodschaffter)]

User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.2.6) Gecko/20100626 SUSE/3.6.6-1.2 Firefox/3.6.6

Cannot print this file in the proper orientation with either HP 6940 or Samsung ML-2851ND:

http://www.wpi.edu/Images/CMS/WPI/WPImapJan09.pdf

It is properly displayed in Print Preview, and looking at the Printer dialog it is set to Landscape, but it prints Portrait, cutting off the right side of the page.

Both printers are network printers.

Reproducible: Always

Steps to Reproduce:
1.
2.
3.

[Ctrippe-c (ctrippe-c)]

Please try printing in portrait layout, my guess is that this is bug
https://bugs.kde.org/show_bug.cgi?id=181290

Can you confirm?

[Rodschaffter (rodschaffter)]

Yes, it prints perfectly when set to Portrait.

[Ctrippe-c (ctrippe-c)]

Thanks for the quick response. Closing as UPSTREAM.

[Vladimir Psenicka (vladimir-psenicka)]

Hi.

Will be update released to KDE 4.4.4 in opensuse 11.3?

[Marius B. Kotsbak (mariusko)]

This problem is still there in Natty. If the document is formated in landscape mode, the default in printer settings is landscape, but selecting portrait instead prints the landscape format correctly.

[Harald Sitter (apachelogger)]

As Jonathan indicated, the problem needs to be resolved upstream in okular rather than in Kubuntu. So closing bug in favor of the upstream report.

[Ian (ifreecarve)]

does anyone have a link to the upstream bug so i can follow that?

[Marius B. Kotsbak (mariusko)]

Look at "assigned to" above.

[Pino Toscano (pinotree)]

(In reply to comment #108)
> This is just ridiculous.

Just for you, really.

> What's the point of keeping binary compatibility in a
> library Okular is the only user of?

We try to have also backends written by 3rd parties.

> What other excuses are you going to come up with not to fix this bug?

Oh come on, I didn't say that won't be accepted at all, just that it will accepted once it is fixed to not break the okular core BC. Reading that as "excuse" is just wrong on your side.

[Albert Astals Cid (aacid)]

Does the patch not break djvu, dvi and ps that use the same method you modified but you are not passing/calculating the correct value? If you do not need to pass/calculate the value, doesn't that mean that is a bug just in the pdf backend and you could just fix it there without modifying so much stuff?

[Kevin-kofler (kevin-kofler)]

My patch fixes all the backends to pass the correct value.

As for binary compatibility, do we need binary compatibility also for the protected functions in FilePrinter or only for the public ones?

[Kevin-kofler (kevin-kofler)]

(Falk's original patch was incomplete and only touched the PDF backend. My version which fixes all backends is at https://git.reviewboard.kde.org/r/101513/ . I'm currently adding all the deprecated overloads to keep BC.)

[Albert Astals Cid (aacid)]

Protected methods are part of the API, so yes.

[Pino Toscano (pinotree)]

(In reply to comment #111)
> As for binary compatibility, do we need binary compatibility also for the
> protected functions in FilePrinter or only for the public ones?

For the public ones for sure.
About the protected methods: they should have not added that way at all,
originally, this is something that slipped in.
Most probably you can just add the new functions as static inside the .cpp
(inside an anonymous namespace to be safe, even).

[Kevin-kofler (kevin-kofler)]

I updated my review request to keep binary compatibility:
https://git.reviewboard.kde.org/r/101513/

[Kevin-kofler (kevin-kofler)]

Git commit e001fbab55aceef62ad9ca2bb87a170dacfe3fdd by Kevin Kofler.
Committed on 06/06/2011 at 00:40.
Pushed by kkofler into branch 'master'.

Fix landscape documents getting printed in portrait format if "Landscape" is selected in the print dialog (the default).

Partly based on a patch by Falk from KDE bug #181290.

BUG: 181290
REVIEW: 101513

M +10 -1 core/document.h
M +1 -1 generators/djvu/generator_djvu.cpp
M +57 -7 core/fileprinter.h
M +22 -0 core/document.cpp
M +5 -2 generators/dvi/dviexport.h
M +2 -2 generators/dvi/dviRenderer.h
M +1 -0 generators/poppler/generator_pdf.cpp
M +5 -3 generators/dvi/dviexport.cpp
M +60 -11 core/fileprinter.cpp
M +2 -2 generators/dvi/dviRenderer.cpp
M +1 -1 generators/spectre/generator_ghostview.cpp
M +1 -1 generators/dvi/generator_dvi.cpp
M +1 -18 part.cpp

http://commits.kde.org/okular/e001fbab55aceef62ad9ca2bb87a170dacfe3fdd

[FriedChicken (domlyons)]

Thanks!

[Kevin-kofler (kevin-kofler)]

I forgot to put in a FIXED-IN tag, so let's make it clear: This fix will be in 4.8.0.

4.7.0 has already been released, and the fix requires new APIs in the Okular core, so I'm probably not supposed to backport it for 4.7.x.

[Francesco+kde (francesco+kde)]

APPLAUSES !1!

[Helm-volker (helm-volker)]

I looked forward to see this solution, I appricate it dearly and I can hardly wait to see kde4.8.

But does anybody see a possibilty to port this solution in some way to kde 4.4.5. Since we use a debian distribution in our office, we really could make use of it. Since I'm no developer I don't now where to look.

[Dav1dblunk3tt (dav1dblunk3tt)]

I'd like to see a backport to this - the problem is very annoying and it will take some of us years to get to 4.8...

[Andras Mantia (amantia)]

For 4.7 there might be chances to backport (if Kevin wants to do it), for 4.4.x, I doubt anyone will do from KDE. Ask the Debian packagers to include the patch in their releases (if it applies cleanly at all).

[Kevin-kofler (kevin-kofler)]

I'd like this backported also, but I don't want to do this if the Okular maintainers don't want it done, it's not my program.

[Samuel Wolf (samuel-wolf)]

Still have this problem on Ubuntu 10.04 :-(

[Marius B. Kotsbak (mariusko)]

Slu: if you look at the top of the page, nobody claims it is fixed in Ubuntu. I reopened the bug for the Ubuntu package as it is now possible to implement the upstream fix in Ubuntu.

[Marius B. Kotsbak (mariusko)]

"Version Fixed In: 4.8.0" and Ubuntu Oneiric coming soon has currently 4.7.0, so it would not have this fixed.

[pierre5933 (pierre-5933)]

With suse 11.4 kde and suse 12.1 I have the same problem with okular

If I try to printa pdf document with landscape configuration I get a vertical printing oth left part of the document , in the same time if I do the same with acroread I have no problem

for your information same problem with libreoffice , but good behaviour with openoffice or firefox or thunderbird

[pierre5933 (pierre-5933)]

the problem I describe in comment 24 is for okular version 0.12 kde version 4.6 and also for version kde 4.7

[Kevin-kofler (kevin-kofler)]

It will be fixed in 4.8.0.

The problem with backporting it is that it has to add functions to the okularcore API, and AIUI the Okular maintainers don't want API additions there in a release branch. In any case, I don't want to make them without their OK and they haven't replied one way or the other. Thus, I ask again: Okular developers, are you OK with me backporting this to 4.7.x or do you want me NOT to backport it? (I'd at least like an answer, even if it is a "no".)

[Albert Astals Cid (aacid)]

I answered already ages ago. No new API in stable releases.

[Kevin-kofler (kevin-kofler)]

And meanwhile we will leave this BUG unfixed in our stable releases. :-/

(IMHO the KDE policy really needs to be changed so that bug fixes MUST be backported rather than that they can be backported as now.)

So send all complaints about this bug not being fixed in 4.7.x directly to the Okular developers, I cannot do anything about it.

[Kevin-kofler (kevin-kofler)]

That, or nag your distro to backport the fix, or switch to Fedora, which has been shipping the fix for months.

[Marius B. Kotsbak (mariusko)]

The patch http://bugsfiles.kde.org/attachment.cgi?id=59734 seems to apply nice to the current 4.7.2 version (with one trivial patch error), but I don't get it built:

dpkg-gensymbols: warning: some new symbols appeared in the symbols file: see diff output below
dpkg-gensymbols: warning: some symbols or patterns disappeared in the symbols file: see diff output below
dpkg-gensymbols: warning: debian/libokularcore1/DEBIAN/symbols doesn't match completely debian/libokularcore1.symbols
--- debian/libokularcore1.symbols (libokularcore1_4:4.7.2-0ubuntu1_i386)
+++ dpkg-gensymbolsOLv6eB 2011-11-11 17:39:47.614182253 +0100
@@ -95,21 +95,25 @@
  _ZN6Okular11FilePrinter10deleteFileER8QPrinterNS0_16FileDeletePolicyERK7QString@Base 4:4.4.5
  _ZN6Okular11FilePrinter10printErrorEi@Base 4:4.6.1
  _ZN6Okular11FilePrinter10printFilesER8QPrinterRK11QStringListNS0_16FileDeletePolicyE@Base 4:4.4.5
- _ZN6Okular11FilePrinter11cupsOptionsER8QPrinter@Base 4:4.4.5

Is this easy to fix?

[Jonathan Thomas (echidnaman)]

It depends. The easy, ostensible fix would be to just remove that line from the symbols file, and the package would build. That may or may not be a good idea, however. A library's .symbols file contains a list of functions that the library exposes to the world. Applications that link against a library use these functions, as they are described at the time the application is built. If these function symbols change, though, applications using the old symbols will crash if they are run against the newer version of the library.

The purpose of a symbol file is to track changes within a library to detect and prevent these incompatibilities between different versions of a library. In this case, a function appears to have been removed as a result of the patch. If this really is the case, then any application using libokularcore1 currently in the archive would crash if it tries to invoke this function.

Looking at the patch[1], the cupsOptions() function being referenced by the symbol has not been removed entirely, but has been prefixed with the KDE_DEPRECATED macro to trigger a build-time warning about the use of the function. I imagine that in the build output of the package, there are also symbols being added. I'd be interested to see if a cupsOptions symbol pops up in the symbols added. If the symbol hasn't changed too much, it might still be compatible with existing applications. If not, it would break applications and other libraries currently using libokularcore1.

So, in summary, it may be just as simple as updating the .symbols file by removing missing symbols and adding new ones, but this might break the binary compatibility of the library. For this reason, it may not be feasible to issue an update with this patch as an official post release update. (Which I suspect is why the Okular maintainers haven't backported it to the KDE 4.7.x branch).

[Jonathan Thomas (echidnaman)]

Oh, and of course, this is what I was referencing with [1]: https://projects.kde.org/projects/kde/kdegraphics/okular/repository/revisions/e001fbab55aceef62ad9ca2bb87a170dacfe3fdd/diff/core/fileprinter.h

[Ctrippe-c (ctrippe-c)]

This bug is still present with okular form KDe 4.7 in openSUSE 12.1. but it is fixed upstream for KDE 4.8. see https://bugs.kde.org/show_bug.cgi?id=181290#c116 Fedora also uses the patch for 4.7 IMHO we should do the same.

A test package is available from
http://download.opensuse.org/repositories/home:/christiantrippe:/branches:/KDE:/Distro:/Stable/openSUSE_12.1

@Maintenace Team: Are you fine with an update?
Taking Will into CC to make sure he also agrees.

[Ctrippe-c (ctrippe-c)]

Setting Needinfo.

[Bwiedemann (bwiedemann)]

This is an autogenerated message for OBS integration:
This bug (625122) was mentioned in
https://build.opensuse.org/request/show/91021 Factory / okular

[Wstephenson-9 (wstephenson-9)]

Fine by me.

[Mindaugas B. (embar-super)]

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

[Mindaugas B. (embar-super)]

At last Okular printed my landscape pages correctly!
Thanks.

I think it is important fix,
and online update should be provided (at least for openSUSE 12.1).

[Mindaugas B. (embar-super)]

openSUSE has already backported patch.
More details from https://bugzilla.novell.com/show_bug.cgi?id=625122 in https://build.opensuse.org/request/show/91021

If you have openSUSE 11.4 you can install
okular-4.7.3-21.1.i586.rpm or okular-4.7.3-21.1.x86_64.rpm RPM package from
https://build.opensuse.org/package/binaries?package=okular&project=KDE%3ADistro%3AFactory&repository=openSUSE_11.4

(in http://download.opensuse.org/repositories/KDE:/Distro:/Factory/openSUSE_11.4/ repository at the moment is still not-pached okular-4.7.3-20)

[pierre5933 (pierre-5933)]

Thanks

in opensuse 11.4 kde I have tried to instal okular 4.7.3.21 with one click instal , But there is a problem :
one depenciy is not deliver ( libqt4-x11...) I have accepted to break the dependency but of course Okular is not launched when I try to do it

And for kde 4.7 deliver with 4.72.31 ???

Thanks

[Mindaugas B. (embar-super)]

Pierre,
Maybe you want to add additional repository, like
http://download.opensuse.org/repositories/KDE:/Release:/47/openSUSE_11.4/
to solve dependencies.

[pierre5933 (pierre-5933)]

I have installed the repository for kde 4.7 as you wrote , but it seems that the dependebcy which is necessary is not delivered .
 So I prefer to stay in my situation and wait when Okular will bee usable in better conditions.. or use Evinc instead of okular the bug is not young and it is shade not to have it coorected in the normal situation

And Is not normal that the new version of suse 12.1 use not the las good version of okular for exemple

Thanks for the informations

[Bbrunner-u (bbrunner-u)]

Christian, could you do a submitrequest to openSUSE:12.1:Update:Test please. A SWAMPID for 12.1 isn't necessary. Thanks.

[Wstephenson-9 (wstephenson-9)]

No swamp ids needed for 12.1 updates? since when? If so, sorry for all the spam to maintenance@ I just created. Should I just SR everything to Update:Test?

[Wstephenson-9 (wstephenson-9)]

Bring on the approval then.

Oh, and do I need to produce a patchinfo still?

[Wstephenson-9 (wstephenson-9)]

created request id 96569

Submitted, update running, marking as fixed.

[Bwiedemann (bwiedemann)]

This is an autogenerated message for OBS integration:
This bug (625122) was mentioned in
https://build.opensuse.org/request/show/96569 12.1 / okular

[Myriam Schweingruber (myriam)]

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

[Marius B. Kotsbak (mariusko)]

That is true. From the upstream bug report: "The problem with backporting it is that it has to add functions to the
okularcore API, and AIUI the Okular maintainers don't want API additions there
in a release branch."

Anyway, Opensuse has made a patch for KDE/Okular 4.7.3: https://build.opensuse.org/request/show/91021 that maybe is adaptable.

[Marius B. Kotsbak (mariusko)]

And the Opensuse bug report (I was not allowed to refer to it): https://bugzilla.novell.com/show_bug.cgi?id=625122

[Marius B. Kotsbak (mariusko)]

The Opensuse patch applies cleanly (to my Okular 4.7.4 sources), and seems to build fine, but I get a file not found error on "debian/tmp/usr/lib/kde4/okularGenerator_ghostview.so" on

"dh_install: cp -a debian/tmp/usr/lib/kde4/okularGenerator_ghostview.so debian/okular//usr/lib/kde4/ returned exit code 1"

which I also get on the unpatched source. Any ideas?

[Marius B. Kotsbak (mariusko)]

I suppose this is the cause:

-----------------------------------------------------------------------------
-- The following OPTIONAL packages could NOT be located on your system.
-- Consider installing them to enable more features from this software.
-----------------------------------------------------------------------------
   * libspectre (0.2 or higher) <http://libspectre.freedesktop.org/wiki/>
     A PostScript rendering library
     Support for PS files in okular.

but I have libspectre-dev package installed...

[Marius B. Kotsbak (mariusko)]

The above problem seems to be just using the 4.7.4 ppa packages. Using the current 4.7.3 packages, the patch

https://api.opensuse.org/public/source/openSUSE:Factory/okular/fix_print_with_landscape.diff?rev=391e43683d5baf75768f695f4c1148c7&

applies and builds fine (just with some warnings about the symbols). Okular also starts. I have not tried printing yet.

[Marius B. Kotsbak (mariusko)]

Ubuntu Precise ships Okular 4.8.2, and "Version Fixed In: 4.8.0", so this should now be fixed.

[Granjow (simon-eu)]

Short question, in which version of Okular will this bug be fixed? Only have 4.7.4 in the debian repositories and just ran across this bug again (like every second time I try to print something :P), so I'm really looking forward to it.

[Kevin-kofler (kevin-kofler)]

As already written several times in the previous comments and indicated in the "fixed by" field, this was fixed in 4.8.0.

The 4.7 branch is no longer maintained, and back when it was, the Okular maintainers refused to backport the fix because it makes additions to the API of an Okular library only used by Okular itself and by Okular backends. They wanted Okular backends compiled against 4.7.4 to also work with 4.7.0 (forward compatibility) and found that more important than fixing this bug.

[Albert Astals Cid (aacid)]

Yes, we the Okular maintainer eat babies for breakfast.

Kevin, get over it, you always can't win and complaining forever won't help you.

[Kevin-kofler (kevin-kofler)]

I didn't complain, I just explained to the user WHY his Okular is still broken.

[Granjow (simon-eu)]

Thanks – I read most previous posts, and I also saw that it was fixed in 4.8; At this place, thanks for the fix.
What I wondered, maybe not expressed that well, I've now upgraded to KDE 4.8 and my Okular shows as version 0.13.3 and in apt as 4.7.4 still, but in the About dialog of Okular it also says «Using KDE Development Platform 4.8.3 (4.8.3)»; Is the fix already included in this version or only in 0.14?

[Albert Astals Cid (aacid)]

If it shows as 4.7.4 in apt it's because it's version 4.7.4, otherwise the packagers have done a huge mistake.

[Kevin-kofler (kevin-kofler)]

You appear to have version 4.7.4 of Okular, but version 4.8.3 of kdelibs. (That is possible, because kdelibs is backwards-compatible, so an older Okular on a newer kdelibs should always work.) The bug and its fix are in Okular, not in kdelibs, so the Okular version is the one that matters.

[Granjow (simon-eu)]

Okay, thanks for the information! Will try to keep that in mind for the next print :) Or not – saw that 4.8.4 just arrived! Cool.

[Rernen (rernen)]

i am running opensuse 13.1 and i can not print in landscape mode with okular
# rpm -qf /usr/bin/okular
okular-4.11.5-150.1.x86_64
# rpm -qf /usr/bin/kde4
kdebase4-runtime-4.11.5-482.6.x86_64

[Rivasgilbie122 (rivasgilbie122)]

*** Removed by KDE Sysadmin due to violation of abuse policies ***

[Shmerl (shmerl)]

I was just hit by this bug, so it's not fixed.

I'm using 4:15.08.3 (Debian testing).

[Albert Astals Cid (aacid)]

Don't expect a "me too" in a bug with 147 comments to cause the developers to do something.

If you're convinced you have this bug, open a new bug with clear instructions of what you do and how to reproduce it.