On 27 Sep 2012, at 21:50, Olivier Lamy wrote:

> 2012/9/27 Jason van Zyl <[email protected]>:
>> 
>> On Sep 27, 2012, at 4:33 PM, Olivier Lamy <[email protected]> wrote:
>> 
>>> Hi,
>>> Good news.
>>> 
>>>> 
>>>>  * fix for a missing JarFile.close when scanning certain "file:" classpath 
>>>> entries
>>>>  * support for injecting the containing BundleContext when running inside 
>>>> OSGi
>>>>  * sisu-guice dependency bumped up to 3.1.2
>>> 
>>> Perso I'd like we don't use a fork of guice.
>> 
>> It's not a fork of Guice. You do understand that we can never use stock 
>> Guice without Sisu? Sisu is a set of extensions that sit on top of Guice. 
>> All of our patches except for one has been absorbed by the Guice team and 
>> Stuart is a Guice committer. Just making sure you understand the distinction 
>> in that we have to support Plexus components forever in the plugins and 
>> extensions that exist and Guice by itself is never going to do that.
> 
> I talk about https://github.com/sonatype/sisu-guice
> 
> Not a fork of guice http://code.google.com/p/google-guice/source/browse/
> 
> So I'm curious how we can named that ?

Just to clarify to the wider audience... Sisu-Guice is a fork of Guice with 
various internal enhancements, whereas Sisu is a container built on top of 
(Sisu-)Guice that provides additional external enhancements to JSR330 (such as 
support for injection of dynamic collections, classpath scanning/indexing, 
auto-binding, etc.) as well as being able to mimic the Plexus API + semantics. 
As I mentioned in my original post one of the items on the roadmap is to reduce 
the coupling between Sisu and Guice, so people can re-use all or parts of it 
with other JSR330 implementations.

Once Guice deploys its next release with the ProvisionListener API you should 
be able to use it with Sisu (I'll try running the ITs to verify this). You may 
still want to consider using Sisu-Guice for the various documented enhancements 
that have not yet been merged into Guice trunk, but I'm not going to force it 
on people. It should be just a matter of excluding and replacing the dependency 
in the parent pom where you consume Sisu.

>> 
>>> What is the status of that, when will that be possible ?
>>> 
>>>> 
>>>> I also ran the Maven ITs against this release to verify.
>>>> 
>>>> Still plenty left to do on the roadmap 
>>>> (http://wiki.eclipse.org/Sisu/Roadmap) - including cleanup of the Sisu 
>>>> API/SPI to make it more reusable and less tied to Guice for the future.
>>>> 
>>>> If anyone has questions, ideas, or wants to help out then they're very 
>>>> welcome (see support / developers links on the main page) otherwise I'll 
>>>> send the occasional note to keep everyone in the loop.
>>>> 
>>>> --
>>>> Cheers, Stuart
>>>> 
>>>> Begin forwarded message:
>>>> 
>>>>> From: Stuart McCulloch <[email protected]>
>>>>> Date: 25 September 2012 22:39:00 GMT+01:00
>>>>> To: [email protected]
>>>>> Subject: Initial contribution milestone (0.0.0.M0)
>>>>> 
>>>>> Hi folks,
>>>>> 
>>>>> Since the two main contributions were approved:
>>>>> 
>>>>> https://bugs.eclipse.org/bugs/show_bug.cgi?id=372081
>>>>> https://bugs.eclipse.org/bugs/show_bug.cgi?id=385803
>>>>> 
>>>>> and the nightly builds were in a good state:
>>>>> 
>>>>> https://hudson.eclipse.org/hudson/job/sisu-inject-nightly/10/
>>>>> https://hudson.eclipse.org/hudson/job/sisu-plexus-nightly/17/
>>>>> 
>>>>> I decided to tag our first (baseline) milestone:
>>>>> 
>>>>> http://git.eclipse.org/c/sisu/org.eclipse.sisu.inject.git/tag/?id=milestones/0.0.0.M0
>>>>> http://git.eclipse.org/c/sisu/org.eclipse.sisu.plexus.git/tag/?id=milestones/0.0.0.M0
>>>>> 
>>>>> The major difference between this and the original (pre-Eclipse) codebase 
>>>>> is the move to "org.eclipse.sisu" as the primary package and the new 
>>>>> Tycho-driven build.
>>>>> 
>>>>> To prove this was indeed the case I reworked the original Sisu build to 
>>>>> take this milestone and repackage the code back to the original 
>>>>> "org.sonatype..." packages:
>>>>> 
>>>>> https://github.com/sonatype/sisu/blob/master/legacy/containers/sisu-inject-bean/pom.xml
>>>>> https://github.com/sonatype/sisu/blob/master/legacy/containers/sisu-inject-plexus/pom.xml
>>>>> 
>>>>> I then verified the binaries, sources, and javadoc were sufficiently 
>>>>> similar to the (pre-Eclipse) 2.3.0 release and tagged this build in the 
>>>>> old repository as sisu-2.3.1.
>>>>> 
>>>>> This should give us a firm foundation as we work on improving the API/SPI.
>>>>> 
>>>>> --
>>>>> Cheers, Stuart
>>>>> 
>>>>> PS. There are plenty of tasks in the backlog if people would like to help 
>>>>> out and learn more about the Sisu codebase:
>>>>> 
>>>>> https://bugs.eclipse.org/bugs/buglist.cgi?resolution=---&classification=Technology&query_format=advanced&component=Inject&component=Plexus&product=Sisu
>>>>> 
>>>>> ;)
>>>>> 
>>>> 
>>> 
>>> 
>>> 
>>> --
>>> Olivier Lamy
>>> Talend: http://coders.talend.com
>>> http://twitter.com/olamy | http://linkedin.com/in/olamy
>>> 
>>> ---------------------------------------------------------------------
>>> To unsubscribe, e-mail: [email protected]
>>> For additional commands, e-mail: [email protected]
>>> 
>> 
>> Thanks,
>> 
>> Jason
>> 
>> ----------------------------------------------------------
>> Jason van Zyl
>> Founder & CTO, Sonatype
>> Founder,  Apache Maven
>> http://twitter.com/jvanzyl
>> ---------------------------------------------------------
>> 
>> I never make the mistake of arguing with people for whose opinions I have no 
>> respect.
>> 
>> -- Edward Gibbon
>> 
>> 
>> 
>> 
>> 
> 
> 
> 
> -- 
> Olivier Lamy
> Talend: http://coders.talend.com
> http://twitter.com/olamy | http://linkedin.com/in/olamy
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [email protected]
> For additional commands, e-mail: [email protected]
> 


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to