Skip to content
This repository has been archived by the owner on Apr 8, 2019. It is now read-only.

GTNPORTAL-3318 - Added EAP 6.2 profile #716

Closed
wants to merge 1 commit into from

Conversation

jpkrohling
Copy link
Contributor

Added a profile for EAP 6.2. This required a few changes in the code, which should be completely backwards compatible.

Note that the artifact jboss-as-subsystem-test is now a POM artifact, instead of a JAR (as it was previously). The replacement (for our case) is jboss-as-subsystem-test-framework. So, the new eap620 profile overrides a new property, that holds the name of the artifact to use. For the existing profiles, it will fall back to the old name.

The change in CdiContextExtensionProcessor is due to changes in Weld. The constant WeldAttachments.PORTABLE_EXTENSIONS look like this in the version of Weld that we are currently using:

public static final AttachmentKey<AttachmentList<Metadata<Extension>>> PORTABLE_EXTENSIONS = AttachmentKey.createList(Metadata.class);

So, the usage of the constant was replaced by a simple use of the value of the constant.

Added a profile for EAP 6.2. This required a few changes in the code, which should be completely backwards compatible.
@gatein-bot
Copy link

Can one of the admins verify this patch?

@jpkrohling
Copy link
Contributor Author

http://torii.gva.redhat.com/jenkins/job/gatein-portal-jbossas711-prbuilder/6/

There's one test failure, but it's not related to this change (probably related to GTNPORTAL-2751 , as per @ppalaga ).

@ppalaga
Copy link
Contributor

ppalaga commented Dec 9, 2013

Merged to master.

@ppalaga ppalaga closed this Dec 9, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants