Re: errors with TEXINFO_XS=omit

[Patrice Dumas]

On Sun, Nov 15, 2020 at 03:27:56PM +0000, Gavin Smith wrote:
> 
> These messages indicate a problem with the Texinfo::Parser namespace.  It
> is odd because you are managing to execute code in the Texinfo/Parser.pm
> file which declares the namespace as "Texinfo::Parser", so all symbols
> (such as %parser_default_configuration) should go into that namespace.
> 
> It is just a hunch but are you creating any new Parser objects in your new
> code?

No, it is the consequence of having a syntax error in
Texinfo::ParserNonXS.

> You could also try running with TEXINFO_XS=debug to see if anything odd is
> happening, such as modules being initialised twice.

Nothing obvious.  If I do an explicit, in texi2any.pl
require Texinfo::ParserNonXS;
I get 

Missing right curly or square bracket at ../tp/Texinfo/ParserNonXS.pm line 
5723, at end of line
syntax error at ../tp/Texinfo/ParserNonXS.pm line 5723, at EOF
Compilation failed in require at ./texi2any.pl line 1021.

I found a way to have an error message by browsing about eval, using
in tp/Texinfo/XSLoader.pm line 228.

  eval "require $fallback_module";
  warn() if $@;

$ ./texi2any.pl tests/formatting/simplest.texi 
Missing right curly or square bracket at ../tp/Texinfo/ParserNonXS.pm line 
5723, at end of line
syntax error at ../tp/Texinfo/ParserNonXS.pm line 5723, at EOF
Compilation failed in require at (eval 13) line 1.
        ...caught at ../tp/Texinfo/XSLoader.pm line 228.
FORMAT_MENU not a possible customization in Texinfo::Parser::parser
expanded_formats not a possible customization in Texinfo::Parser::parser
ENABLE_ENCODING not a possible customization in Texinfo::Parser::parser
values not a possible customization in Texinfo::Parser::parser
include_directories not a possible customization in Texinfo::Parser::parser
Can't locate object method "line_error" via package "Texinfo::Parser" at 
../tp/Texinfo/ParserNonXS.pm line 3987, <FH> line 3.


Ok to commit that change?




With 
TEXINFO_XS=debug
TEXINFO_XS_PARSER=0
and without require Texinfo::ParserNonXS;

$ ./texi2any.pl tests/formatting/simplest.texi 
checking ../tp/maintain/lib/Text-Unidecode/lib/MiscXS.la
checking ../tp/maintain/lib/Unicode-EastAsianWidth/lib/MiscXS.la
checking ../tp/maintain/lib/libintl-perl/lib/MiscXS.la
checking ../tp/maintain/MiscXS.la
checking ../tp/Texinfo/XS/parsetexi/MiscXS.la
checking ../tp/Texinfo/XS/MiscXS.la
found ../tp/Texinfo/XS/MiscXS.la
../tp/Texinfo/XS/.libs/MiscXS.so loaded
looking for boot_Texinfo__MiscXS
trying to call boot_Texinfo__MiscXS...
  ...succeeded
attempting to override Texinfo::Convert::Unicode::unicode_text with 
Texinfo::MiscXS::unicode_text...
  ...succeeded
falling back to pure Perl module Texinfo::ParserNonXS
checking ../tp/maintain/lib/Text-Unidecode/lib/XSParagraph.la
checking ../tp/maintain/lib/Unicode-EastAsianWidth/lib/XSParagraph.la
checking ../tp/maintain/lib/libintl-perl/lib/XSParagraph.la
checking ../tp/maintain/XSParagraph.la
checking ../tp/Texinfo/XS/parsetexi/XSParagraph.la
checking ../tp/Texinfo/XS/XSParagraph.la
found ../tp/Texinfo/XS/XSParagraph.la
../tp/Texinfo/XS/.libs/XSParagraph.so loaded
looking for boot_Texinfo__Convert__Paragraph
trying to call boot_Texinfo__Convert__Paragraph...
  ...succeeded
attempting to override Texinfo::Convert::Plaintext::_process_text_internal with 
Texinfo::MiscXS::process_text...
  ...succeeded
values not a possible customization in Texinfo::Parser::parser
FORMAT_MENU not a possible customization in Texinfo::Parser::parser
ENABLE_ENCODING not a possible customization in Texinfo::Parser::parser
expanded_formats not a possible customization in Texinfo::Parser::parser
include_directories not a possible customization in Texinfo::Parser::parser
Can't locate object method "line_error" via package "Texinfo::Parser" at 
../tp/Texinfo/ParserNonXS.pm line 3987, <FH> line 3.