dev@glassfish.java.net

Re: pom review for adding the inject apis in the javaee apis module

From: Sahoo <Sahoo_at_Sun.COM>
Date: Tue, 06 Oct 2009 14:16:54 +0530

Ludo,

Can you send us the diffs that you have committed for review?

Thanks,
Sahoo

Ludovic Champenois wrote:
> On 10/4/09 10:08 PM, Sahoo wrote:
>> Please give me some more time to review this change. I need some time
>> to answer why javax.inject were packaged as part of
>> webbeans-osgi-bundle.jar.
>>
> I will but meanwhile, I aboslutely need to do the check in for the
> NetBeans 6.8 beta maven support...as the published jars are broken.
> Will revisit later if needed... Sorry about that, this is a p0 for nb.
> Thanks,
> Ludo
>> Sahoo
>>
>> Ludovic Champenois wrote:
>>> On 10/3/09 8:53 AM, Ludovic Champenois wrote:
>>>> See the diff. Maybe Sahoo or the webbeans owners can comment while
>>>> Jane is on vacation?
>>>
>>> This seems also to work...Maybe cleaner?
>>>
>>> # Following Index: paths are relative to:
>>> /Users/ludo/acvs/v3/javaee-api/javax.javaee
>>> # This patch can be applied using context Tools: Patch action on
>>> respective folder.
>>> # It uses platform neutral UTF-8 encoding and \n newlines.
>>> # Above lines and this line are ignored by the patching process.
>>> Index: pom.xml
>>> --- pom.xml Base (BASE)
>>> +++ pom.xml Locally Modified (Based On LOCAL)
>>> @@ -260,7 +260,7 @@
>>> </dependency>
>>> <dependency>
>>> <groupId>org.jboss.webbeans</groupId>
>>> - <artifactId>jsr299-api</artifactId>
>>> + <artifactId>webbeans-osgi-bundle</artifactId>
>>> <version>${webbeans.version}</version>
>>> <optional>true</optional>
>>> </dependency>
>>>
>>>
>>>
>>>>
>>>> I am not sure about the correct version for javax.inject but this
>>>> new dependency is necessary to add the javax.inject apis into the
>>>> javaee-api jars, necessary to produced the stripped api jar (only
>>>> apis, no impl)...
>>>> Currently, the build of this stripped jar is failing with missing
>>>> Scope class.
>>>>
>>>> Related question: why the java.inject APIs are inside
>>>> webbeans-osgi-bundle.jar and not in a separate javax.... jar?
>>>> I need to refresh the posted stripped api jars asap (they are used
>>>> for Maven EE 6 projects, and currently not functioning)...
>>>>
>>>> # Following Index: paths are relative to:
>>>> /Users/ludo/acvs/v3/javaee-api/javax.javaee
>>>> # This patch can be applied using context Tools: Patch action on
>>>> respective folder.
>>>> # It uses platform neutral UTF-8 encoding and \n newlines.
>>>> # Above lines and this line are ignored by the patching process.
>>>> Index: pom.xml
>>>> --- pom.xml Base (BASE)
>>>> +++ pom.xml Locally Modified (Based On LOCAL)
>>>> @@ -265,6 +265,12 @@
>>>> <optional>true</optional>
>>>> </dependency>
>>>> <dependency>
>>>> + <groupId>javax.inject</groupId>
>>>> + <artifactId>inject-api</artifactId>
>>>> + <version>1.0.0-PRD</version>
>>>> + <optional>true</optional>
>>>> + </dependency>
>>>> + <dependency>
>>>> <groupId>com.sun.messaging.mq</groupId>
>>>> <artifactId>jaxm-api</artifactId>
>>>> <version>4.3</version>
>>>>
>>>>
>>>> ---------------------------------------------------------------------
>>>> To unsubscribe, e-mail: dev-unsubscribe_at_glassfish.dev.java.net
>>>> For additional commands, e-mail: dev-help_at_glassfish.dev.java.net
>>>>
>>>
>>>
>>> ---------------------------------------------------------------------
>>> To unsubscribe, e-mail: dev-unsubscribe_at_glassfish.dev.java.net
>>> For additional commands, e-mail: dev-help_at_glassfish.dev.java.net
>>>
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: dev-unsubscribe_at_glassfish.dev.java.net
>> For additional commands, e-mail: dev-help_at_glassfish.dev.java.net
>>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe_at_glassfish.dev.java.net
> For additional commands, e-mail: dev-help_at_glassfish.dev.java.net
>