don't include (ancient version of) Time::HiRes as Perl extension, since it's a core Perl module #6225
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This reverts #5616 .
Including
Time::HiRes
as extension doesn't make sense, since it's a core Perl module (that is, it's always available).Moreover, the
Time::HiRes
version that was included in #5616 is an ancient version, or some kind of customised fork of the officialTime::HiRes
.Some installations are currently failing because of this:
BioPerl
:"tv_interval" is not exported by the Time::HiRes module
proovread
:"usleep" is not exported by the Time::HiRes module
This change was done for
QUAST
(cfr. #5610), butQUAST-4.6.0-foss-2016b-Python-3.5.2.eb
builds just fine on top ofPerl/5.24.0-foss-2016b
that doesn't includeTime::HiRes
as an extension...