New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[JBERET-369] Create a BOM for using JBeret in Java SE standalone envi… #164
Conversation
758c6f3
to
1426942
Compare
jberet-se-bom/pom.xml
Outdated
<version.org.jberet>1.4.4.Final</version.org.jberet> | ||
<version.org.jberet.support>1.3.4.Final</version.org.jberet.support> | ||
<version.org.spec.javax.batch>2.0.0.Final</version.org.spec.javax.batch> | ||
<version.org.spec.javax.transaction>2.0.0.CR1</version.org.spec.javax.transaction> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
org.spec.javax... seems odd. Can you use the same property name as in jberet-parent, such as
version.org.jboss.spec.javax.batch.jboss-batch-api_1.0_spec
version.org.jboss.spec.javax.transaction.jboss-transaction-api_1.2_spec
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi, @chengfang I've updated the changes as per master PR, please review it.
Thanks.
4127c74
to
d221a90
Compare
jberet-se-bom/pom.xml
Outdated
<groupId>org.codehaus.woodstox</groupId> | ||
<artifactId>stax2-api</artifactId> | ||
<version>${version.org.codehaus.woodstox.stax2-api}</version> | ||
</dependency> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
aalto-xml and stax2-api should not be necessary. They are optional if users want to use them instead of the default JDK parser. For most users, these 2 dependencies are not needed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@chengfang changes modified. Thanks.
pom.xml
Outdated
|
||
<!-- Used in SE bom --> | ||
<version.org.codehaus.woodstox.woodstox-core-asl>4.4.1</version.org.codehaus.woodstox.woodstox-core-asl> | ||
<version.org.apache.qpid>0.10.0</version.org.apache.qpid> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same as above, these 2 props are not used by most users and I think should not be in BOM.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@chengfang changes modified. Thanks.
d221a90
to
3872ba9
Compare
Issue: https://issues.redhat.com/browse/JBERET-369
master PR #163