Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0011425Openbravo ERPA. Platformpublic2009-11-19 16:382009-12-15 04:56
Reporterjpabloae 
Assigned Toshuehner 
PriorityurgentSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS20OS VersionGentoo
Product Versionpi 
Target VersionFixed in Version2.50MP10 
Merge Request Status
Review Assigned To
OBNetwork customerNo
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0011425: run.all.tests: calculate.instance.revision sets the version to ${hg.id}
DescriptionThe run.all.tests ant tasks invokes the calculate.instance.revision task, and it calculates the version incorrectly, setting it to the literal "${hg.id}". This works well in the install.source, it only fails in run.all.tests. Example build output:

calculate.instance.version:
     [echo] Instance version: ${hg.id}
      [sql] Executing commands
      [sql] 1 of 1 SQL statements executed successfully


Full output: http://builds.openbravo.com/view/devel-int/job/erp_devel_int-full-pgsql/23/consoleFull [^]
Steps To Reproduceant install.source run.all.tests
Proposed SolutionThere is no need to run calculate.instance.version in run.all.tests. It is apparently triggered by the compileSqlc task.
Additional Information
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2009-11-19 16:38jpabloaeNew Issue
2009-11-19 16:38jpabloaeAssigned To => rafaroda
2009-11-19 16:38jpabloaeOBNetwork customer => No
2009-11-19 16:39jpabloaeNote Added: 0022028
2009-11-25 19:50jpabloaeAssigned Torafaroda => shuehner
2009-11-25 19:50jpabloaeNote Added: 0022176
2009-12-10 12:21hgbotCheckin
2009-12-10 12:21hgbotNote Added: 0022623
2009-12-10 12:21hgbotStatusnew => resolved
2009-12-10 12:21hgbotResolutionopen => fixed
2009-12-10 12:21hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/8cc92d93306125e2ade1aa0988af16a71764efcd [^]
2009-12-10 15:02mtaalNote Added: 0022643
2009-12-10 15:02mtaalStatusresolved => closed
2009-12-10 15:02mtaalFixed in Version => 2.50MP10
2009-12-11 00:00anonymoussf_bug_id0 => 2912315
2009-12-15 04:56hudsonbotCheckin
2009-12-15 04:56hudsonbotNote Added: 0022779

Notes
(0022028)
jpabloae   
2009-11-19 16:39   
This is preventing the live Openbravo ERP test builds fom being usables, because the about box shows "${hg.id}" instead of the real revision, so you don't know what you're testing.
(0022176)
jpabloae   
2009-11-25 19:50   
There's a missing dependdency on "init" in the main build.xml
(0022623)
hgbot   
2009-12-10 12:21   
Repository: erp/devel/pi
Changeset: 8cc92d93306125e2ade1aa0988af16a71764efcd
Author: Stefan Hühner <stefan.huehner <at> openbravo.com>
Date: Thu Dec 10 12:21:17 2009 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/8cc92d93306125e2ade1aa0988af16a71764efcd [^]

Fixed 11425: fix instance calculation in run.*.tests targets

---
M build.xml
---
(0022643)
mtaal   
2009-12-10 15:02   
Tested and works fine now
(0022779)
hudsonbot   
2009-12-15 04:56   
---
A changeset related to this issue has been promoted to main after passing a series of tests and an OBX has been generated:

Changeset: http://code.openbravo.com/erp/devel/main/rev/e7ad8a48210d [^]
Tests: http://builds.openbravo.com/view/devel-int/ [^]
OBX: http://builds.openbravo.com/erp/obx/core/OpenbravoERP-2.50CI.15705.obx [^]
---