Thanks to Scott's help, I think I understood the details a bit more. 

HV uses javax.el.ExpressionFactory/FactoryFinder which in turn uses the TCCL to find implementations and cache it in FactoryFinderFache. For WAR deployments, the cache is pre-populated by ELExpressionFactoryProcessor with the implementation from the web module mentioned by Shelly.

As this is not the case for other deployment types (e.g. RARs), tests using HV based on such deployment types fail with the exception.

So I think a separate EL module (upon which HV could depend, independently from the current deployment type) would indeed make sense.

--Gunnar



2013/4/9 Scott Marlow <smarlow@redhat.com>
Here is some of it http://pastie.org/7391504 (shows the Caused by:
"javax.el.ELException: Provider org.apache.el.ExpressionFactoryImpl not
found").

Should we create a static module that contains
org.apache.el.ExpressionFactoryImpl?  Currently, jbossweb-7.2.0.Final
contains this class.

On 04/09/2013 09:33 AM, Jaikiran Pai wrote:
> Hi Gunnar,
>
> Where are you seeing the failures? Can you point us to the stacktrace?
>
> -Jaikiran
> On Tuesday 09 April 2013 06:59 PM, Gunnar Morling wrote:
>> Hi all,
>>
>> I'm currently working on updating the AS module for Hibernate
>> Validator to the latest release (HV 5.x, Bean Validation 1.1, see [1]).
>>
>> As of this release, Hibernate Validator has a new dependency to the
>> Unified EL. So I have added this to the HV module descriptor:
>>
>>     <module name="javax.el.api"/>
>>
>> Based on test failures I'm seeing ("javax.el.ELException: Provider
>> org.apache.el.ExpressionFactoryImpl not found"), I think I need to add
>> a dependency for the actual EL implementation as well. I don't know
>> though which module that would be.
>>
>> Does anyone know which is the right one or has an example of other
>> modules depending on EL?
>>
>> Thanks,
>>
>> --Gunnar
>>
>> [1] https://issues.jboss.org/browse/AS7-6665
>>
>>
>>
>> _______________________________________________
>> jboss-as7-dev mailing list
>> jboss-as7-dev@lists.jboss.org
>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>
>
>
> _______________________________________________
> jboss-as7-dev mailing list
> jboss-as7-dev@lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>

_______________________________________________
jboss-as7-dev mailing list
jboss-as7-dev@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/jboss-as7-dev