Project

General

Profile

Actions

Backport #16918

closed

hammer: making stop.sh more portable

Added by Joshua Schmid over 7 years ago. Updated over 7 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Target version:
Release:
hammer
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

https://github.com/ceph/ceph/pull/10569

for portability/compatibility reasons I'd like to request a backport of this commit to hammer as the script will fail with syntax error when using /bin/bash as default SHELL

echo $SHELL
/bin/bash
./stop.sh

outputs:

./stop.sh: line 73: syntax error near unexpected token `<'
./stop.sh: line 73: ` done < <("${CEPH_BIN}"/rbd showmapped | tail -n +2)'

verify that it works with sh:

sh stop.sh
$?
0

the commit that fixes it:
SHA1: 27f2d806f905df1a51b11bd79f1af4c26471407a

thanks!

Actions #1

Updated by Nathan Cutler over 7 years ago

  • Target version deleted (v0.94.8)
Actions #2

Updated by Nathan Cutler over 7 years ago

  • Subject changed from making stop.sh nore portable to hammer: making stop.sh more portable
Actions #3

Updated by Nathan Cutler over 7 years ago

  • Description updated (diff)
  • Status changed from New to In Progress
Actions #4

Updated by Nathan Cutler over 7 years ago

  • Status changed from In Progress to Resolved
  • Target version set to v0.94.10
Actions

Also available in: Atom PDF