From 2333696650bd0973fa2508d6b0afe5e9c1334545 Mon Sep 17 00:00:00 2001 From: Stig Sandbeck Mathisen Date: Sat, 4 Oct 2014 19:32:33 +0200 Subject: [PATCH] Syntax check ksh --- .travis.yml | 2 +- t/test.t | 9 ++++++--- 2 files changed, 7 insertions(+), 4 deletions(-) diff --git a/.travis.yml b/.travis.yml index 602e8ffa..fba41a51 100644 --- a/.travis.yml +++ b/.travis.yml @@ -2,7 +2,7 @@ language: perl install: - sudo apt-get update - - sudo apt-get --no-install-recommends install devscripts python ruby php5-cli gawk pylint + - sudo apt-get --no-install-recommends install devscripts python ruby php5-cli gawk ksh pylint - sudo apt-get --no-install-recommends install pkg-config libdb-dev libvirt-dev libexpat-dev # - Munin/Plugin.pm is in "munin-node" on precise - sudo apt-get --no-install-recommends install munin-node diff --git a/t/test.t b/t/test.t index 2d02b413..8b507715 100644 --- a/t/test.t +++ b/t/test.t @@ -52,6 +52,10 @@ sub process_file { "checkbashisms" ); }; } + elsif ( $interpreter =~ m{/bin/ksh} ) { + ok( check_file_with( [ 'ksh', '-n', $file ] ), + $filename . " ksh syntax check" ); + } elsif ( $interpreter =~ m{/bin/bash} ) { ok( check_file_with( [ 'bash', '-n', $file ] ), $filename . " bash syntax check" ); @@ -62,7 +66,7 @@ sub process_file { } elsif ( $interpreter =~ m{python} ) { ok( check_file_with( - [ 'pylint', '--errors-only', '--report=no', $file ] + [ 'pylint', '--rcfile=/dev/null', '--errors-only', '--report=no', $file ] ), $filename . " python syntax check" ); @@ -98,8 +102,7 @@ sub check_file_with { return 1; } else { - diag($stdout); - diag($stderr); + diag(sprintf("\nCommand: %s\n\nSTDOUT:\n\n%s\n\nSTDERR:\n\n%s\n\n", join(" ", @{$check_command}), $stdout, $stderr)); return; } }