Project

General

Profile

Actions
Copy link

Bug #5174

closed

Use of a wrong conditional to detect if we are building on sles in several package specfile

Bug #5174: Use of a wrong conditional to detect if we are building on sles in several package specfile

Added by Matthieu CERDA about 12 years ago. Updated over 11 years ago.

Status:
Released
Priority:
1 (highest)
Assignee:
Jonathan CLARKE
Category:
Packaging
Target version:
2.11.0~beta2
Pull Request:
https://github.com/Normation/rudder-p...
Severity:
UX impact:
User visibility:
Effort required:
Priority:
Name check:
Fix check:
Regression:

Description

This conditionnal is wrong:

%if 0%{?sles}

it should be replaced with:

%if 0%{?sles_version}

This problem impacts the dependencies of the new webapp rudder handling, and ncf-api-virtualenv

Updated by Matthieu CERDA about 12 years ago Actions
Copy link
 #1

  • Project changed from Rudder to 34
  • Status changed from In progress to Pending technical review
  • Assignee changed from Matthieu CERDA to Jonathan CLARKE
  • % Done changed from 0 to 100
  • Pull Request set to https://github.com/Normation/rudder-packages/pull/421

PR is ready !

Updated by Vincent MEMBRÉ about 12 years ago Actions
Copy link
 #2

  • Status changed from Pending technical review to Pending release

This was fixed on Friday 27/06 by commit:bbf8a68113e9b46dd8e4b68e0bb9eab35b13d349

Updated by Vincent MEMBRÉ about 12 years ago Actions
Copy link
 #3

  • Subject changed from 2.11 SLES RPMs sometimes use a wrong conditionnal to detect if we are building on sles to Use of a wrong conditionnal to detect if we are building on sles in several package specfile

Updated by Vincent MEMBRÉ about 12 years ago Actions
Copy link
 #4

  • Subject changed from Use of a wrong conditionnal to detect if we are building on sles in several package specfile to Use of a wrong conditional to detect if we are building on sles in several package specfile

Updated by Vincent MEMBRÉ about 12 years ago Actions
Copy link
 #5

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 2.11.0~beta2 (announcement , changelog), which were released today.

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

  • Project changed from 34 to Rudder
  • Category set to Packaging
Actions
Copy link

Also available in: PDF Atom