Make sure perl install dir follows a standard format

[ocaisa]

Fixes #1700

[timeu]

Doesn't this also require the exts_default_options = {'configopts': ''} statement ?

[ocaisa]

No, this is specifically for the perl installation, the perl extensions are handled by another easyblock (which is unaffected by this change)
With this change you wouldn't need either of the options you had to include in your easyconfig solution.

[timeu]

@ocaisa : Ah I understand, makes sense Thanks for clarifying !

[boegel]

@timeu Can you re-test with an unmodified Perl easyconfig (so without the changes used in #1700 to dance around the problem) with this patched easyblock (using eb --include-easyblocks perl.py ...), to confirm the fix?

[boegel]

lgtm, rebuilding Perl-5.28.1-GCCcore-8.2.0.eb on top of this didn't cause any problems...

[timeu]

@boegel Do you still need me to test it ?

[ocaisa]

@timeu Yes, can you retest please just so we know it actually does fix the problem

[timeu]

Ok tried it with the unmodified easyconfig and the patched easyblock and it seems to work:

[easybuild@stg-login-0 Perl]$ eb Perl-5.28.1-GCCcore-8.2.0.eb --include-easyblocks /software/system/easyblocks/perl.py --force
== temporary log file in case of crash /tmp/eb-lvfRPB/easybuild-4F4MKc.log
== processing EasyBuild easyconfig /software/system/easyconfigs/easybuild/easyconfigs/p/Perl/Perl-5.28.1-GCCcore-8.2.0.eb
== building and installing perl/5.28.1-gcccore-8.2.0...
== fetching files...
== creating build dir, resetting environment...
== unpacking...
...
== installing extension App::cpanminus 1.7044 (289/289)...
== restore after iterating...
== postprocessing...
== sanity checking...
== cleaning up...
== creating module...
== permissions...
== packaging...
== COMPLETED: Installation ended successfully
== Results of the build can be found in the log file(s) /software/2019/software/perl/5.28.1-gcccore-8.2.0/easybuild/easybuild-Perl-5.28.1-20190503.121419.log
== Build succeeded for 1 out of 1
== Temporary log file(s) /tmp/eb-lvfRPB/easybuild-4F4MKc.log* have been removed.
== Temporary directory /tmp/eb-lvfRPB has been removed.

[boegel]

Thanks a lot for confirming the fix @timeu!