duncs / clusterssh

Cluster SSH - Cluster Admin Via SSH
https://github.com/duncs/clusterssh/wiki
902 stars 79 forks source link

t/15config.t fails with perl 5.37.4 #153

Closed jplesnik closed 1 year ago

jplesnik commented 1 year ago

t/15config.t fails with perl 5.37.4:

#   Failed test 'returned ok'
#   at t/15config.t line 546.
#          got: 'die'
#     expected: 'return'

#   Failed test 'Expecting no STDERR'
#   at t/15config.t line 550.
#          got: ''
#     expected: 'Unable to write default $HOME/.clusterssh/config: Is a directory
# 
# '
# Looks like you failed 2 tests of 155.
t/15config.t ............. 
Dubious, test returned 2 (wstat 512, 0x200)
Failed 2/155 subtests 

More info on https://github.com/Perl/perl5/issues/20376

tmancill commented 1 year ago

I believe this is the same issue addressed in #150.

jplesnik commented 1 year ago

You're right. This PR solves this error.

duncs commented 1 year ago

PR merged

eserte commented 1 week ago

@duncs: Can we also get a CPAN release? In the meantime, perl 5.38 and 5.40 were released, and these perl versions do not work with the latest App-ClusterSSH release.