Project

General

Profile

Actions
Copy link

Bug #16471

closed

Remove unused webapp postinst parameters

Added by Félix DALLIDET over 4 years ago. Updated over 4 years ago.

Status:
Released
Priority:
N/A
Assignee:
Benoît PECCATTE
Category:
Packaging
Target version:
6.0.2
Pull Request:
https://github.com/Normation/rudder-p...
Severity:
UX impact:
User visibility:
Effort required:
Priority:
0
Name check:
Reviewed
Fix check:
Checked
Regression:

Description

We are always calling the webapp postinst with something like:

/opt/rudder/share/package-scripts/rudder-webapp-postinst "${FIRST_INSTALL}" "%{apache}" "%{apache_user}" "%{apache_group}"

But, apache_user is undefined and rudder-webapp-postinst does not use the user and group args anywhere.
We should remove them if they are not useful.

Actions
Copy link
 #1

Updated by Félix DALLIDET over 4 years ago

  • Status changed from New to In progress
  • Assignee set to Félix DALLIDET
Actions
Copy link
 #2

Updated by Félix DALLIDET over 4 years ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Félix DALLIDET to Benoît PECCATTE
  • Pull Request set to https://github.com/Normation/rudder-packages/pull/2174
Actions
Copy link
 #3

Updated by Félix DALLIDET over 4 years ago

  • Status changed from Pending technical review to Pending release
Actions
Copy link
 #4

Updated by Vincent MEMBRÉ over 4 years ago

  • Fix check changed from To do to Checked
Actions
Copy link
 #5

Updated by Alexis Mousset over 4 years ago

  • Subject changed from webapp postinst has unused parameters to Remove unused webapp postinst parameters
  • Name check changed from To do to Reviewed
Actions
Copy link
 #6

Updated by Vincent MEMBRÉ over 4 years ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 6.0.2 which was released today.

Actions
Copy link

Also available in: Atom PDF