Re: Feedback requested: Impending changes to RunListener

On Thursday, 14 February 2013, tibord47 wrote:

> Hi Stephen,
>
> The correct groupId is com.github.stephenc.jcip.
> I already intalled and used the junit lib 4.12-snapshot in other project.
>
> The text in @ThreadSafe is similar to the one in original jcip.
> Although i would omit mentioning modifier "public" since the all problem
> of thread safety is in fields which are read or write in critical
> section=methods/constructor. So the modifier may not necessarily be public.
> It can be also protected, etc.


Look at the original spec (ie java doc) and you will see that the spec
specifically calls out public. I would be deviating from the spec if I
dropped the public. I debated re-using the original java doc text, but
decided it was better to write something equivalent in specification but
differing in wording as the spec text is under Creative Commons


> So better not to mention any modifier in the text.


I think it would be wrong to deviate from the "spec". Perhaps a
clarification from "upstream" would be best.


> Let's wait for other repliers and then to make a new deployment.
>
> Cheers,
> Tibor
>
>
> --- In [email protected] <javascript:;>, Stephen Connolly  wrote:
> >
> > To allow for the removal of licensing concerns, I have clean-room reverse
> > engineered the jcip-annotations as well:
> > https://github.com/stephenc/jcip-annotations
> >
> > Should be available from Central in the next 1-8 hours
> >
> >
> >   com.github.stephenc
> >   jcip-annotations
> >   1.0-1
> >
> >
> > This should be a drop in replacement for
> >
> >
> >     net.jcip
> >     jcip-annotations
> >     1.0
> >
> >
> > But if it isn't please let me know and I can fix and roll a 1.0-2
> >
> >
> > On 13 February 2013 09:17, Stephen Connolly
>   > wrote:
> >
> > >
> > >
> > >
> > > On 13 February 2013 01:17, Kevin Cooney  wrote:
> > >
> > >>
> > >>
> > >> On Tue, Feb 12, 2013 at 2:14 PM, tibord47  wrote:
> > >>
> > >>> **
> > >>>
> > >>>
> > >>> Hi all,
> > >>>
> > >>> ok, so in that case the standard javax annotations cannot be used.
> > >>>
> > >>> Other options are jcip and proprietary one.
> > >>>
> > >> There are a few other options that were suggested. I've summarized
> them
> > >> all here:
> > >>
> > >>
> > >>
> https://docs.google.com/document/d/1OeE5OUKgtkKHeQuA7bKldBTk-yQ83bH6BHokwifCPrE/edit#
> > >>
> > >> Feel free to add comments if I missed something. If you don't have a
> > >> Google account (or even if you do) feel free to continue to comment
> here.
> > >>
> > >>>
> > >>> I vote for jcip since the license allows us to use a dependency.
> > >>>
> > >> Stephen indicated a concern with the jcip-annotations license.
> Hopefully
> > >> he (or someone else with a concern) can explain why the license is
> > >> problematic.
> > >>
> > >
> > > I indicated that there may be a concern with the license and you'd
> want to
> > > check.
> > >
> > > We had (at former employers) an issue with the findbugs annotations
> > > (LGPL), so I created https://github.com/stephenc/findbugs-annotationsas
> > > a clean-room reverse engineered implementation (Apache License v2) of
> the
> > > same annotations specification.
> > >
> > > We weren't using the jcip-annotations, so those were not subject to the
> > > lawyers reviewing.
> > >
> > >
> > >>
> > >> -- Kevin
> > >>
> > >>
> > >
> >
> >
> > [Non-text portions of this message have been removed]
> >
>
>
>
>
> ------------------------------------
>
> Yahoo! Groups Links
>
>
>
>


[Non-text portions of this message have been removed]



------------------------------------

Yahoo! Groups Links

<*> To visit your group on the web, go to:
    http://groups.yahoo.com/group/junit/

<*> Your email settings:
    Individual Email | Traditional

<*> To change settings online go to:
    http://groups.yahoo.com/group/junit/join
    (Yahoo! ID required)

<*> To change settings via email:
    [email protected] 
    [email protected]

<*> To unsubscribe from this group, send an email to:
    [email protected]

<*> Your use of Yahoo! Groups is subject to:
    http://docs.yahoo.com/info/terms/