Project

General

Profile

Actions
Copy link

Bug #6169

closed

Bug #6168: ncf API hooks fails because it uses bashism but declares using shell

Error management code is a bashism

Bug #6169: Error management code is a bashism

Added by Benoît PECCATTE over 11 years ago. Updated over 11 years ago.

Status:
Released
Priority:
N/A
Assignee:
Matthieu CERDA
Category:
Web - Technique editor
Target version:
2.11.7
Pull Request:
https://github.com/Normation/rudder-p...
Severity:
UX impact:
User visibility:
Effort required:
Priority:
Name check:
Fix check:
Regression:

Description

The error management code introduced in #5866 is a bashism.

Since we are on the server, we can require bash to be present, so we should call it instead of sh.

Updated by Benoît PECCATTE over 11 years ago Actions
Copy link
 #1

  • Status changed from New to In progress

Updated by Benoît PECCATTE over 11 years ago Actions
Copy link
 #2

  • Status changed from In progress to Pending technical review
  • Assignee changed from Benoît PECCATTE to Matthieu CERDA
  • Pull Request set to https://github.com/Normation/rudder-packages/pull/596

Updated by Benoît PECCATTE over 11 years ago Actions
Copy link
 #3

  • Status changed from Pending technical review to Pending release
  • % Done changed from 0 to 100

Applied in changeset packages:rudder-packages|commit:34600fb109986b28793eb12ae5fc83edf4f073fb.

Updated by Matthieu CERDA over 11 years ago Actions
Copy link
 #4

Applied in changeset packages:rudder-packages|commit:32d729ffecf15a1ca3091b1495f9ff8c762bd0ea.

Updated by Vincent MEMBRÉ over 11 years ago Actions
Copy link
 #5

  • Target version changed from 2.11.6 to 2.11.7

Updated by Vincent MEMBRÉ over 11 years ago Actions
Copy link
 #6

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 2.11.7, which was released these days.

Actions
Copy link

Also available in: PDF Atom