Project

General

Profile

Bug #10162

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

Rudder apache common conf is not packaged correctly on rpm

Added by Vincent MEMBRÉ over 3 years ago. Updated over 3 years ago.

Status:
Released
Priority:
N/A
Category:
Packaging
Target version:
Severity:
User visibility:
Effort required:
Priority:

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

#1

Updated by Vincent MEMBRÉ over 3 years ago

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

Updated by Vincent MEMBRÉ over 3 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
#3

Updated by Vincent MEMBRÉ over 3 years ago

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

Updated by Vincent MEMBRÉ over 3 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)
#5

Updated by Vincent MEMBRÉ over 3 years ago

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

Updated by Vincent MEMBRÉ over 3 years ago

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

Updated by Vincent MEMBRÉ over 3 years ago

  • Status changed from Pending release to Released

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

Also available in: Atom PDF