Project

General

Profile

Actions
Copy link

Bug #10162

closed

Bug #10131: rudder-relay has bad "sed" line

Rudder apache common conf is not packaged correctly on rpm

Added by Vincent MEMBRÉ almost 8 years ago. Updated almost 8 years ago.

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

Description

Two issues here,

apache file is not modified correctly on SLES:

sed -i 's%^DocumentRoot /var/www$%DocumentRoot /srv/www%' /usr/src/rudder-packages/package/BUILD//usr/src/rudder-packages/package/SOURCES/rudder-apache-relay-common.conf
sed: can't read /usr/src/rudder-packages/package/BUILD//usr/src/rudder-packages/package/SOURCES/rudder-apache-relay-common.conf: No such file or directory

And it is not found after when packaging it (ie on RHEL, but would be the same on sles if it didn't fail before):

install: cannot stat `/usr/src/rudder-packages/package/BUILD//usr/src/rudder-packages/package/SOURCES/rudder-apache-relay-common.conf': No such file or directory

I suggest we should move the sed into the prepare section, where we prepare sources for build

Actions
Copy link
 #1

Updated by Vincent MEMBRÉ almost 8 years ago

  • Status changed from New to In progress
  • Assignee set to Vincent MEMBRÉ
Actions
Copy link
 #2

Updated by Vincent MEMBRÉ almost 8 years ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Vincent MEMBRÉ to Benoît PECCATTE
  • Pull Request set to https://github.com/Normation/rudder-packages/pull/1251
Actions
Copy link
 #3

Updated by Vincent MEMBRÉ almost 8 years ago

  • Status changed from Pending technical review to New
  • Pull Request deleted (https://github.com/Normation/rudder-packages/pull/1251)
Actions
Copy link
 #4

Updated by Vincent MEMBRÉ almost 8 years ago

  • Subject changed from rudder-relay has bad "sed" line to Rudder apache common conf is not packaged correctly on rpm
  • Description updated (diff)
Actions
Copy link
 #5

Updated by Vincent MEMBRÉ almost 8 years ago

  • Status changed from New to Pending technical review
  • Pull Request set to https://github.com/Normation/rudder-packages/pull/1252
Actions
Copy link
 #6

Updated by Vincent MEMBRÉ almost 8 years ago

  • Status changed from Pending technical review to Pending release
  • % Done changed from 0 to 100
Actions
Copy link
 #7

Updated by Vincent MEMBRÉ almost 8 years ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 4.1.0~beta3 which was released today.

Actions
Copy link

Also available in: Atom PDF