Project

General

Profile

Actions

Bug #2568

closed

rudder-webapp and rudder-inventory-endpoint use "settings-internal.xml" by default causing the build to fail outside of Normation

Added by Jonathan CLARKE over 12 years ago. Updated almost 10 years ago.

Status:
Released
Priority:
3
Assignee:
Jonathan CLARKE
Category:
Packaging
Target version:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
Name check:
Fix check:
Regression:

Description

From rudder-webapp/debian/rules (although the SPEC file uses external by default):

$ grep settings debian/rules 
    cd $(CURDIR)/SOURCES/rudder-sources/rudder-parent-pom && $(CURDIR)/SOURCES/maven2/bin/mvn -s $(CURDIR)/SOURCES/settings-internal.xml -Dmaven.test.skip=true install
    cd $(CURDIR)/SOURCES/rudder-sources/rudder-commons && $(CURDIR)/SOURCES/maven2/bin/mvn -s $(CURDIR)/SOURCES/settings-internal.xml -Dmaven.test.skip=true install
    cd $(CURDIR)/SOURCES/rudder-sources/scala-ldap && $(CURDIR)/SOURCES/maven2/bin/mvn -s $(CURDIR)/SOURCES/settings-internal.xml -Dmaven.test.skip=true install
    cd $(CURDIR)/SOURCES/rudder-sources/ldap-inventory && $(CURDIR)/SOURCES/maven2/bin/mvn -s $(CURDIR)/SOURCES/settings-internal.xml -Dmaven.test.skip=true install
    cd $(CURDIR)/SOURCES/rudder-sources/cf-clerk && $(CURDIR)/SOURCES/maven2/bin/mvn -s $(CURDIR)/SOURCES/settings-internal.xml -Dmaven.test.skip=true install
    cd $(CURDIR)/SOURCES/rudder-sources/rudder && $(CURDIR)/SOURCES/maven2/bin/mvn -s $(CURDIR)/SOURCES/settings-internal.xml -Dmaven.test.skip=true install package

This should of course be "settings-external", to enable building everywhere without a problem.

Actions #1

Updated by Jonathan CLARKE over 12 years ago

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

Applied in changeset commit:706e8fa27884b687d78f06db878ea28adf7dc906.

Actions #2

Updated by Jonathan CLARKE over 12 years ago

  • Target version changed from 2.4.0~beta1 to 2.3.8
Actions #3

Updated by Matthieu CERDA over 12 years ago

This looks valid ! All clear for me !

Actions #4

Updated by Jonathan CLARKE over 12 years ago

  • Status changed from Pending technical review to Released
Actions #5

Updated by Nicolas PERRON almost 12 years ago

  • Project changed from Rudder to 34
  • Category deleted (11)
Actions #6

Updated by Benoît PECCATTE almost 10 years ago

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

Also available in: Atom PDF