Fix sphinx-doc#10695: old LaTeX does not allow \@ifpackageloaded usag…

…e in body
jfbu · Jul 23, 2022 · faccc21 · faccc21
1 parent afac558
commit faccc21
Show file tree

Hide file tree

Showing 5 changed files with 24 additions and 19 deletions.
diff --git a/sphinx/texinputs/sphinx.sty b/sphinx/texinputs/sphinx.sty
@@ -685,6 +685,27 @@
 %% FRAMED ENVIRONMENTS
 %
 \RequirePackage{sphinxpackageboxes}
+% This macro is possibly executed at begin document if the check
+% whether radii setting options have been used turns out positive
+\def\spx@RequirePackage@PictIIe{%
+\IfFileExists{pict2e.sty}
+   {\RequirePackage{pict2e}}
+   {\PackageWarningNoLine{sphinx}{%
+       The package pict2e is required for rounded boxes.\MessageBreak
+       It does not seem to be available on your system.\MessageBreak
+       Options for setting radii will thus be ignored}%
+    \AtEndDocument{\PackageWarningNoLine{sphinx}{%
+       I issued a warning which may have gotten lost in the\MessageBreak
+       gigantic console output: pict2e.sty was not found,\MessageBreak
+       and radii setting options have been ignored}}%
+    \def\spx@boxes@fcolorbox@rounded{\spx@boxes@fcolorbox}%
+   }%
+}%
+% This at begin document will be executed after \spx@RequirePackage@PictIIe
+\AtBeginDocument{%
+    \@ifpackageloaded{pict2e}{\let\spx@ifpackageloaded@pictiie\@firstoftwo}%
+                             {\let\spx@ifpackageloaded@pictiie\@secondoftwo}%
+}%
 \input{sphinxlatexadmonitions.sty}
 \input{sphinxlatexliterals.sty}
 \input{sphinxlatexshadowbox.sty}

diff --git a/sphinx/texinputs/sphinxlatexadmonitions.sty b/sphinx/texinputs/sphinxlatexadmonitions.sty
@@ -162,7 +162,7 @@
           \ifdim\spx@boxes@radius@bottomleft >\z@0\fi
           1\spx@boxes@fcolorbox{##1}%
         \else
-          \@ifpackageloaded{pict2e}%
+          \spx@ifpackageloaded@pictiie
              {\ifspx@boxes@insetshadow
                 \spx@boxes@fcolorbox{##1}%
               \else

diff --git a/sphinx/texinputs/sphinxlatexliterals.sty b/sphinx/texinputs/sphinxlatexliterals.sty
@@ -164,7 +164,7 @@
           \ifdim\spx@pre@radius@bottomleft >\z@0\fi
           1\spx@boxes@fcolorbox{\box\z@}%
         \else
-          \@ifpackageloaded{pict2e}%
+          \spx@ifpackageloaded@pictiie
              {\ifspx@pre@insetshadow
                 \spx@boxes@fcolorbox{\box\z@}%
               \else

diff --git a/sphinx/texinputs/sphinxlatexshadowbox.sty b/sphinx/texinputs/sphinxlatexshadowbox.sty
@@ -79,7 +79,7 @@
           \ifdim\spx@topic@radius@bottomleft >\z@0\fi
           1\spx@boxes@fcolorbox{\box\z@}%
         \else
-          \@ifpackageloaded{pict2e}%
+          \spx@ifpackageloaded@pictiie
              {\ifspx@topic@insetshadow
                 \spx@boxes@fcolorbox{\box\z@}%
               \else

diff --git a/sphinx/texinputs/sphinxpackageboxes.sty b/sphinx/texinputs/sphinxpackageboxes.sty
@@ -72,22 +72,6 @@
 % if nothing else is changed (perhaps in future the title itself could be also
 % rendered in a rounded box?)
 
-\def\spx@RequirePackage@PictIIe{%
-\IfFileExists{pict2e.sty}
-   {\RequirePackage{pict2e}}
-   {\PackageWarningNoLine{sphinx}{%
-       The package pict2e is required for rounded boxes.\MessageBreak
-       It does not seem to be available on your system.\MessageBreak
-       Options for setting radii will thus be ignored}%
-    \AtEndDocument{\PackageWarningNoLine{sphinx}{%
-       I issued a warning which may have gotten lost in the\MessageBreak
-       gigantic console output: pict2e.sty was not found,\MessageBreak
-       and radii setting options have been ignored}}%
-   \def\spx@boxes@fcolorbox@rounded{\spx@boxes@fcolorbox}%
-   }%
-}%
-
-
 %//// \spx@boxes@fcolorbox
 % This box will have the same baseline as its argument (which is typeset in
 % horizontal mode).  It takes into account four border widths parameters, four