shlint 0.1.5 → 0.1.6

checksums.yaml

@@ -0,0 +1,7 @@
+---
+SHA1:
+  metadata.gz: 2013c0961be2fbc1b5172a88f2829a98e4ef32c5
+  data.tar.gz: 73cac37e561784d853daba5fa34e67228cbf0f41
+SHA512:
+  metadata.gz: 9fc2aeb052d8f79fa0f53b11b7deca6ed005692d40a87ac0b727078146ea35a72f7bf6012f3143d1563f64b51a12d37b178d8760b682fad939856ff98d1be0da
+  data.tar.gz: bb7085b3ba647a5d11da3b3538e27c64389e753ccd6d1447546e4d082f77e4dd7ca6f6368f9c043b4da58a1ec3df647559b8bb00f031e883c0f9de7109ac82b4

data/lib/checkbashisms

@@ -33,7 +33,7 @@ Usage: $progname [-n] [-f] [-x] script ...
    or: $progname --help
    or: $progname --version
 This script performs basic checks for the presence of bashisms
-in /bin/sh scripts.
+in /bin/sh scripts and the lack of bashisms in /bin/bash ones.
 EOF
 
 my $version = <<"EOF";
@@ -74,6 +74,8 @@ if ($opt_version) { print $version; exit 0; }
 
 $opt_echo = 1 if $opt_posix;
 
+my $mode = 0;
+my $issues = 0;
 my $status = 0;
 my $makefile = 0;
 my (%bashisms, %string_bashisms, %singlequote_bashisms);
@@ -116,6 +118,8 @@ foreach my $filename (@ARGV) {
 	next;
     }
 
+    $issues = 0;
+    $mode = 0;
     my $cat_string = "";
     my $cat_indented = 0;
     my $quote_string = "";
@@ -143,9 +147,7 @@ foreach my $filename (@ARGV) {
 		next if $opt_force;
 
 		if ($interpreter =~ m,/bash$,) {
-		    warn "script $display_filename is already a bash script; skipping\n";
-		    $status |= 2;
-		    last;  # end this file
+		    $mode = 1;
 		}
 		elsif ($interpreter !~ m,/(sh|posh)$,) {
 ### ksh/zsh?
@@ -476,6 +478,11 @@ foreach my $filename (@ARGV) {
 	if ($buffered_line ne '');
 
     close C;
+
+    if ($mode && !$issues) {
+	warn "could not find any possible bashisms in bash script $filename\n";
+	$status |= 4;
+    }
 }
 
 exit $status;
@@ -483,8 +490,14 @@ exit $status;
 sub output_explanation {
     my ($filename, $line, $explanation) = @_;
 
-    warn "possible bashism in $filename line $. ($explanation):\n$line\n";
-    $status |= 1;
+    if ($mode) {
+        # When examining a bash script, just flag that there are indeed
+        # bashisms present
+        $issues = 1;
+    } else {
+        warn "possible bashism in $filename line $. ($explanation):\n$line\n";
+        $status |= 1;
+    }
 }
 
 # Returns non-zero if the given file is not actually a shell script,

metadata

@@ -1,15 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: shlint
 version: !ruby/object:Gem::Version
-  version: 0.1.5
-  prerelease: 
+  version: 0.1.6
 platform: ruby
 authors:
 - Ross Duggan
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-12-07 00:00:00.000000000 Z
+date: 2013-05-02 00:00:00.000000000 Z
 dependencies: []
 description: Checks the syntax of your shellscript against known and available shells.
 email:
@@ -28,26 +27,25 @@ files:
 - README.md
 homepage: http://github.com/duggan/shlint
 licenses: []
+metadata: {}
 post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
-  - - ! '>='
+  - - '>='
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
-  - - ! '>='
+  - - '>='
     - !ruby/object:Gem::Version
       version: 1.3.6
 requirements: []
 rubyforge_project: 
-rubygems_version: 1.8.23
+rubygems_version: 2.0.0
 signing_key: 
-specification_version: 3
+specification_version: 4
 summary: A linting tool for shell.
 test_files: []