For some context on the reasons for bringing jboss-logging into solder, please take a look at http://seamframework.org/Seam3/JBossLoggingIntegration for full details.
In a nutshell it's to enable us to remove the dependency on jboss-logging, while enabling us some flexibility in terms of creating our own annotations, that don't clash with jboss-logging and cause confusion. The reason being that the annotations within jboss-logging were never intended for developer consumption, they are intended for use within the JBoss AS container. For this reason the port into solder removes those annotations from jboss-logging, and moves them to a solder package where appropriate.
Feel free to amend the wiki page if there are other items that you feel are of merit.
Thanks
Ken
Here's the link to the incorporation of jboss-logging into solder: https://github.com/kenfinnigan/solder/tree/jboss-logging
Appreciate any comments/feedback as still new at this.
If it's easier for it to be a pull request just let me know.
There will undoubtedly be a few additions to this once I work out how i18n module will look at the best location for some of these changes, solder or i18n.
Thanks
KenOn Tue, Jan 18, 2011 at 5:59 PM, Dan Allen <dan.j.allen@gmail.com> wrote:
As a team we need to review Ken's improvements and provide feedback because it's a critical piece of Seam, not to mention it's importance to the i18n API.
Ken, it would be best to post the branch where the changes live (sorry if I'm a step behind if you already have).
- Dan Allen
Sent from my Android-powered phone:
An open platform for carriers, consumers and developersOn Jan 18, 2011 3:30 PM, "Ken Finnigan" <ken@kenfinnigan.me> wrote:
> On Mon, Jan 17, 2011 at 7:09 PM, Dan Allen <dan.j.allen@gmail.com> wrote:
>
>> On Mon, Jan 17, 2011 at 18:49, Jason Porter <lightguard.jp@gmail.com>wrote:
>>
>>> I need a Solder release (I'm working on switching to beans.xml), then
>>> I can release the alpha, and a beta quickly for Catch, or we could
>>> still keep it at alpha. Though because it's such an integrated piece
>>> ideally it should be beta.
>>>
>>> I think we should all be using the next release of solder if we can
>>> make sure it works on Glassfish 3.0.1. I'll send a follow-up email
>>> with my latest exception.
>>>
>>
>> +1
>>
>> Also, please use seam-parent 7 and seam-bom 3.0.0.b06...but stay tuned, I
>> may update to seam-parent 8 to pull in the Weld 1.1.0.Final.
>>
>> -Dan
>>
>> --
>> Dan Allen
>> Principal Software Engineer, Red Hat | Author of Seam in Action
>> Registered Linux User #231597
>>
>> http://mojavelinux.com
>> http://mojavelinux.com/seaminaction
>> http://www.google.com/profiles/dan.j.allen
>>
>> _______________________________________________
>> seam-dev mailing list
>> seam-dev@lists.jboss.org
>> https://lists.jboss.org/mailman/listinfo/seam-dev
>>
>>
> +1 to Solder release, especially after some logging/i18n changes have been
> incorporated from some work I've done with Dan.
>
> In terms of i18n, there are a couple of things I would like to get done for
> the first Beta, as it will likely change the API, that rely on the Solder
> changes. Not sure whether I will be able to get them all complete by
> Wed/Thu, though will see how I go tonight.
>
> Ken