Re: configure enable xs fails because Texinfo/ModulePath.pm not found

[Gavin Smith]

On Thu, Feb 24, 2022 at 05:14:22PM +0000, Gavin Smith wrote:
> > I am not sure about the best way to fix this, maybe move the relevant
> > parts of Texinfo::XSLoader in Texinfo/XS/TestXS.pm, or have fallback
> > values for $Texinfo::ModulePath::texinfo_uninstalled and
> > $Texinfo::ModulePath::builddir if the require Texinfo::ModulePath; fails
> > set in a BEGIN block.
> 
> It was "use Texinfo::ModulePath" until a couple of days ago when I changed
> it to "require Texinfo::ModulePath", as until a change you made fairly
> recently there was no line there at all, but adding "use Texinfo::ModulePath"
> was incorrect as this led the initialization code to be run twice.  I'm not
> sure why changing it from "use" to "require" would have broken it.  I will
> try to investigate.

I've made the change you suggested: I tested this installed and uninstalled
as well as with the configure check and it works okay.  It would probably
be okay without any require line at all but it's better to provide
default values for these variables, even if they aren't used in the configure
check itself.