I meant
$cache = $tmp
unless defined $cache and len $cache;
of course. Sorry for the confusion.
Regards, Thomas
> On Sep 4, 2023, at 17:26 , Thomas Jahns wrote:
>
> Also, may I question the decision to use
>
> $cache = $tmp
>unless $cache;
>
> which results in falsy values for
Also, may I question the decision to use
$cache = $tmp
unless $cache;
which results in falsy values for $cache like '0' to be ignored while ''
works as expected?
My recommendation would be to use
$cache = $tmp
unless defined $cache and -n $cache;
instead.
Kind regards,
Okay, attached patch didn't work, I'll try an inline copy:
From a0f86d54b51710ca0843dff6d7a9f7378d5ab780 Mon Sep 17 00:00:00 2001
From: Thomas Jahns
Date: Mon, 4 Sep 2023 15:17:28 +0200
Subject: [PATCH] autom4te: Parse arguments before creating temporary
directory.
* The order before always
Of course there was a dependency. But I think I managed to find a suitable
change that again preserves the temporaries. Please see the attached patch vs.
cb6fbab55de1e9660e110857ae248a70a8b48c5b and tell me if that needs further
improvements or is acceptable as is.
Kind regards, Thomas
> On
Following up on this, I think this is because autom4te calls mktmpdir before
parsing command line arguments:
## -- ##
## Main program. ##
## -- ##
mktmpdir ('am4t');
load_configuration ($ENV{'AUTOM4TE_CFG'} || "$pkgdatadir/autom4te.cfg");
load_configuration