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

Added by Benoît PECCATTE over 9 years ago. Updated about 9 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.

Actions
Copy link
 #1

Updated by Benoît PECCATTE over 9 years ago

  • Status changed from New to In progress
Actions
Copy link
 #2

Updated by Benoît PECCATTE over 9 years ago

  • 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
Actions
Copy link
 #3

Updated by Benoît PECCATTE over 9 years ago

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

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

Actions
Copy link
 #4

Updated by Matthieu CERDA over 9 years ago

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

Actions
Copy link
 #5

Updated by Vincent MEMBRÉ about 9 years ago

  • Target version changed from 2.11.6 to 2.11.7
Actions
Copy link
 #6

Updated by Vincent MEMBRÉ about 9 years ago

  • 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: Atom PDF