Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 607890 - dev-tex/latex2html-2017.2: Can't use 'defined(@array)' (Maybe you should just omit the defined()?) at /usr/lib64/latex2html/styles/natbib.perl line 1454, <STYLES> line 4.
Summary: dev-tex/latex2html-2017.2: Can't use 'defined(@array)' (Maybe you should just...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: TeX project
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-02-01 08:53 UTC by Bernd Feige
Modified: 2017-02-02 10:34 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Bernd Feige 2017-02-01 08:53:39 UTC
Thanks for the upgrade to 2017.2!
Problem with using natbib...
It seems that upstream has the defined(@array) problem in yet more places :-(
Comment 1 Alexis Ballier gentoo-dev 2017-02-01 13:32:39 UTC
when does this happen ?

the patch changes specifically this like, does it happen if you upgrade latex2html first ?
Comment 2 Bernd Feige 2017-02-01 14:57:44 UTC
(In reply to Alexis Ballier from comment #1)
> when does this happen ?
> 
> the patch changes specifically this like, does it happen if you upgrade
> latex2html first ?

I'm not sure what you mean. Of course I upgraded to latex2html-2017.2 first. /usr/lib64/latex2html/styles/natbib.perl with the offending line

if (defined @$style) {

belongs to the package and, of course, is invoked if the LaTeX package "natbib" is used in the translated .tex file.

If you mean the patch for the previous error
https://bugs.gentoo.org/show_bug.cgi?id=604184 -
that one was in versions/frame.pl and there it was patched...
Comment 3 Alexis Ballier gentoo-dev 2017-02-01 20:29:42 UTC
mine looks like this:

    local($style)="citestyle_$2";
    if (@$style) {
        ($CITE_OPEN_DELIM,
         $CITE_CLOSE_DELIM,
         $CITE_ENUM,
  


and the latex2html-tests.patch applied by the ebuild indeed patches this:

Index: latex2html-2017.2/styles/natbib.perl
===================================================================
--- latex2html-2017.2.orig/styles/natbib.perl
+++ latex2html-2017.2/styles/natbib.perl
@@ -1451,7 +1451,7 @@ sub do_cmd_citestyle {
     local($_) = @_;
     s/$next_pair_pr_rx//o;
     local($style)="citestyle_$2";
-    if (defined @$style) {
+    if (@$style) {
 	($CITE_OPEN_DELIM,
 	 $CITE_CLOSE_DELIM,
 	 $CITE_ENUM,
Comment 4 Bernd Feige 2017-02-02 10:34:46 UTC
(In reply to Alexis Ballier from comment #3)
> mine looks like this:

Okay, after today's update (and I may be up to 1 day delayed since I update from deltas) it is fixed here too. Could it be that a previous version of latex2html-tests.patch did not contain the natbib patch and landed in the previous delta?

Anyway, marking "Resolved" now, thanks!