Tim Landscheidt has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/65643


Change subject: Fix typo and enable warnings.
......................................................................

Fix typo and enable warnings.

Change-Id: Ied7d1daf101313f2ef4cae90a725d37a8a79e676
---
M packages/jobutils/usr/local/bin/jsub
1 file changed, 5 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/labs/toollabs 
refs/changes/43/65643/1

diff --git a/packages/jobutils/usr/local/bin/jsub 
b/packages/jobutils/usr/local/bin/jsub
index 08f2cb8..1ab3132 100755
--- a/packages/jobutils/usr/local/bin/jsub
+++ b/packages/jobutils/usr/local/bin/jsub
@@ -1,4 +1,7 @@
-#! /usr/bin/perl
+#!/usr/bin/perl -w
+
+use strict;
+use warnings;
 
 use String::ShellQuote;
 
@@ -151,7 +154,7 @@
 
 if($continuous) {
   push @args, '-q', 'continuous';
-  push @args, '-b', $sqargs{'-b'}  if defined $qsargs{'-b'};
+  push @args, '-b', $qsargs{'-b'}  if defined $qsargs{'-b'};
   open QSUB, '|-', '/usr/bin/qsub', @args or die "\[$now\] unable to start 
qsub: $!\n";
   print QSUB "#!/bin/bash\n";
   print QSUB "while ! " . shell_quote($prog, @ARGV) . "; do\n";

-- 
To view, visit https://gerrit.wikimedia.org/r/65643
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ied7d1daf101313f2ef4cae90a725d37a8a79e676
Gerrit-PatchSet: 1
Gerrit-Project: labs/toollabs
Gerrit-Branch: master
Gerrit-Owner: Tim Landscheidt <t...@tim-landscheidt.de>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to