Bug 1042660 - Please use jboss standard version format for properties in uberfire(ie use "version.groupid" instead of "groupid.version"
Summary: Please use jboss standard version format for properties in uberfire(ie use "v...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: JBoss BPMS Platform 6
Classification: Retired
Component: Build and Assembly
Version: 6.0.0
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: CR1
: 6.0.0
Assignee: Alexandre Porcelli
QA Contact: Lukáš Petrovický
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-12-13 02:18 UTC by Ryan Zhang
Modified: 2014-08-06 20:13 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2014-08-06 20:13:01 UTC
Type: Bug


Attachments (Terms of Use)

Description Ryan Zhang 2013-12-13 02:18:14 UTC
Description of problem:
In uberfire component, the properties defined as:
...
    <javax.inject.version>1</javax.inject.version>
    <cdi-api.version>1.0-SP4</cdi-api.version>
    <weld-spi.version>1.1.Final</weld-spi.version>
    <weld.version>1.1.13.Final</weld.version>
...

But in community bom , ipbom and jboss eap bom, the properties are defined as:
...
    <version.org.jboss.weld.weld>1.1.13.Final-redhat-1</version.org.jboss.weld.weld>
    <version.org.jboss.weld.weld-api>1.1.Final-redhat-2</version.org.jboss.weld.weld-api>

...

Please use the same version format and the same properties values for the same artifact in uberfire.


Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

Comment 2 Ryan Zhang 2013-12-13 05:14:54 UTC
Please notice that there are several artifacts versions defined in uberfire are different from the ones defined in community bom.
For example:
lucene
mockito-all
helix
....

Please also make sure the common artifact use the same version besides the version format.

Comment 3 Ryan Zhang 2013-12-13 05:19:07 UTC
Kindly remind that mvel version:2.1.Beta8 is also different.

Comment 5 Ryan Zhang 2013-12-13 11:51:48 UTC
A side build problem:
uberfire contains lucene-core in the dependency tree (am I correct?)

Should we also explicitly define it in dependencyManagment section?

Comment 7 Alexandre Porcelli 2014-01-09 20:59:26 UTC
Fix pushed to master and product branch of uberfire repository.

As commit message says, still room for improvement on pom's - but for now i think it's fine.

(master) https://github.com/droolsjbpm/uberfire/commit/405bae6eb
(0.3.x) https://github.com/droolsjbpm/uberfire/commit/72fde1ea0

Comment 8 Lukáš Petrovický 2014-01-19 18:14:15 UTC
VERIFIED by checking the commits.


Note You need to log in before you can comment on or make changes to this bug.