GNU bug report logs - #20749
11.88; Meta Bug: Long standing preview bug with TIKZ (6 years!?)

Previous Next

Full log

View this message in rfc822 format

From: Nicolas Richard <youngfrog <at> members.fsf.org>
To: Tassilo Horn <tsdh <at> gnu.org>
Cc: 20749 <at> debbugs.gnu.org
Subject: bug#20749: 11.88; Meta Bug: Long standing preview bug with TIKZ (6 years!?)
Date: Mon, 08 Jun 2015 19:42:23 +0200

[Message part 1 (text/plain, inline)]

Tassilo Horn <tsdh <at> gnu.org> writes:
>> Isn't this https://sourceforge.net/p/pgf/bugs/316/ again ?
>
> Oh, yes, seems so.  And 315 seems also related.
>
>> For some reason, it seems that upstream is not fixing it even though a
>> patch is available.
>
> Why do you think that?  AFAICS, no official has done a statement on
> that, and the issue is just one year old.

Err... yeah maybe I'm being pessimistic again.

>> FWIW I've been running with that patch for a few months have yet to
>> see a problem with it.
>
> Which one?

The "correct patch". More precisely, it's :

[patch.diff (text/x-diff, inline)]

--- /tmp/pgfutil-latex-ctan.def	2015-06-08 19:36:06.890121795 +0200
+++ /usr/local/texlive/2014/texmf-dist/tex/generic/pgf/utilities/pgfutil-latex.def	2015-06-08 19:39:09.686116394 +0200
@@ -134,16 +134,17 @@
 % material to get lost.
 
 \AtBeginDocument{
-  \@ifpackageloaded{preview}{%
-    % Ok, package loaded. Swap definitions of everyshi.sty's shipout
-    % and preview.sty's shipout:
-    \let\pgf <at> temp\pr <at> shipout% This is the original shipout
-    \let\pr <at> shipout\@EveryShipout <at> Shipout% This is what preview.sty should call
-    \let\@EveryShipout <at> Org <at> Shipout\pgf <at> temp% Everyshi should now use the original one
-  }{}%    
+\@ifpackageloaded{preview}{%
+% Ok, package loaded. Swap definitions of everyshi.sty's shipout
+% and preview.sty's shipout:
+\ifPreview
+\let\shipout\@EveryShipout <at> Org <at> Shipout%This is the null version of \shipout, created by preview and saved by everyshi
+\let\@EveryShipout <at> Org <at> Shipout\pr <at> shipout% This is the original shipout
+\let\pr <at> shipout\@EveryShipout <at> Shipout%
+\fi
+}{}%
 }
 
-
 % Guess the driver:
 
 \def\pgfutil <at> guessdriver{

[Message part 3 (text/plain, inline)]

>             And do you also save \shipout before \begin{document} and
> restore it afterwards as indicated by the comment by Kirill Müller?

I don't.

> In any case: if there's anything that we could put into preview.dtx to
> fix the issue from our side, that would be awesome.  Patch welcome!

I admit I haven't looked into it, but I'm afraid I won't be able to
help.

Nico.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.