Mailing-List: contact deltaspike-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: deltaspike-dev@incubator.apache.org
Received-SPF: pass (nike.apache.org: domain of lightguard.jp@gmail.com
 designates 209.85.217.175 as permitted sender)
MIME-Version: 1.0
In-Reply-To: 
 <CAE9L6G3Uz6Oe5So5EYXm3anStbNxYz-OACDX+v1UY0F-dHnedA@mail.gmail.com>
References: 
 <2136975679.14810.1332689968331.JavaMail.tomcat@hel.zones.apache.org>
 <F1B47A77-6676-4C15-AE68-4AFCBAF91EAE@redhat.com>
 <CAGJtJfEvKFCB8at3XpdzuSKgXH8BR7Q=5B+QY=m9YR6QCVkNLA@mail.gmail.com>
 <68D4BD4F-E203-499C-8C65-B39B3607C27C@redhat.com>
 <1332695036.93982.YahooMailNeo@web171504.mail.ir2.yahoo.com>
 <767725A1-C53F-48E1-BA28-B5CBDA413CAA@redhat.com>
 <CAGJtJfFmc4c2LN5wxBmePy0o+K8Ca=oJ_ZTN01OvyxTik83ycg@mail.gmail.com>
 <78276B7F-B8A0-44B4-858B-39FA9CC3A45D@redhat.com>
 <0B5A1FAA-7E6A-45A7-9539-50ED383C3466@redhat.com>
 <0A4A6F32-B8B4-4ABA-B2D8-581E34F2A923@gmail.com>
 <CAE9L6G3Uz6Oe5So5EYXm3anStbNxYz-OACDX+v1UY0F-dHnedA@mail.gmail.com>
From: Jason Porter <lightguard.jp@gmail.com>
Date: Mon, 26 Mar 2012 15:07:18 -0600
Message-ID: 
 <CAF9TksMO+EkYCFs4usF1Qyk8ATn0QocOdiR_5Rg6aOnhHpFYUQ@mail.gmail.com>
Subject: Re: [jira] [Created] (DELTASPIKE-129) re-visit visibility of
 AnnotationBuilder, ImmutableInjectionPoint,
 InjectableMethod and ParameterValueRedefiner
To: deltaspike-dev@incubator.apache.org, gudnabrsam@gmail.com
Content-Type: multipart/alternative; boundary=bcaec5485810a062ef04bc2bc335

--bcaec5485810a062ef04bc2bc335
Content-Type: text/plain; charset=UTF-8

It could, I sort of envisioned that's what Core was for.

On Mon, Mar 26, 2012 at 15:01, Matt Benson <gudnabrsam@gmail.com> wrote:

> Could it be that certain classes belong in some DS artifact that is
> meant to serve as a toolbox for extension authors, then?
>
> Matt
>
> On Sun, Mar 25, 2012 at 1:40 PM, Jason Porter <lightguard.jp@gmail.com>
> wrote:
> > For now, the wiki is as good as anywhere else.
> >
> > Sent from my iPhone
> >
> > On Mar 25, 2012, at 12:03, Pete Muir <pmuir@redhat.com> wrote:
> >
> >> Ok, I see that they are not used. So, what is the objection to these
> classes? No clear use case? If so, where do I document the use cases?
> >>
> >> IMO they are all useful things for extension authors.
> >>
> >> On 25 Mar 2012, at 18:15, Pete Muir wrote:
> >>
> >>> Maybe this is just a cultural mismatch. Do Apache projects expect
> people to rely on the "API" packages and Implementation packages when
> writing code?
> >>>
> >>> Anyway, this goes back to my original question. How do you reduce the
> visibility of these classes without affecting the API. Other classes expose
> them via methods, so it's not as simple as "just reduce the visibility"...
> >>>
> >>> On 25 Mar 2012, at 18:12, Gerhard Petracek wrote:
> >>>
> >>>> imo they shouldn't be part of the api and i'm not sure if they fit in
> the
> >>>> spi package, because you don't need them to customize deltaspike.
> >>>> they are just helpers which are even quite special for extensions
> authors.
> >>>>
> >>>> regards,
> >>>> gerhard
> >>>>
> >>>>
> >>>>
> >>>> 2012/3/25 Pete Muir <pmuir@redhat.com>
> >>>>
> >>>>> Yes, this is definitely all squarely aimed at extension authors and
> not
> >>>>> end user apps IMO.
> >>>>>
> >>>>> On 25 Mar 2012, at 18:03, Mark Struberg wrote:
> >>>>>
> >>>>>> Is this useful for Extension implementers? If so we might think
> about
> >>>>> putting them into spi packages?
> >>>>>>
> >>>>>> LieGrue,
> >>>>>> strub
> >>>>>>
> >>>>>>
> >>>>>>
> >>>>>> ----- Original Message -----
> >>>>>>> From: Pete Muir <pmuir@redhat.com>
> >>>>>>> To: deltaspike-dev@incubator.apache.org
> >>>>>>> Cc:
> >>>>>>> Sent: Sunday, March 25, 2012 6:36 PM
> >>>>>>> Subject: Re: [jira] [Created] (DELTASPIKE-129) re-visit visibility
> of
> >>>>> AnnotationBuilder, ImmutableInjectionPoint, InjectableMethod and
> >>>>> ParameterValueRedefiner
> >>>>>>>
> >>>>>>>
> >>>>>>> On 25 Mar 2012, at 17:30, Gerhard Petracek wrote:
> >>>>>>>
> >>>>>>>> hi pete,
> >>>>>>>>
> >>>>>>>> that would be possible e.g. with AnnotationBuilder. however, it
> isn't
> >>>>>>>> possible with all of them.
> >>>>>>>
> >>>>>>> Why?
> >>>>>>>
> >>>>>>>> -> we already moved internal helpers to
> >>>>>>>> org.apache.deltaspike.core.util
> >>>>>>>>> if< we need them in the api-module.
> >>>>>>>> they might not provide a stable api (over time) or are quite
> special.
> >>>>>>>> we moved them there to remove the visibility via an organizational
> >>>>>>> approach.
> >>>>>>>
> >>>>>>> I have no problem with this approach.
> >>>>>>>
> >>>>>>> Perhaps you could expand on what you mean here then? Do you mean
> extract
> >>>>>>> interfaces from these classes and move the implementation to core?
> >>>>>>>
> >>>>>>>
> >>>>>>>
> >>>>>>> I can't see how you can reduce the visibility without changing the
> API?
> >>>>>>>
> >>>>>>>>
> >>>>>>>> regards,
> >>>>>>>> gerhard
> >>>>>>>>
> >>>>>>>>
> >>>>>>>>
> >>>>>>>> 2012/3/25 Pete Muir <pmuir@redhat.com>
> >>>>>>>>
> >>>>>>>>> I assume you mean the visibility of the constructors of
> >>>>>>> AnnotationBuilder,
> >>>>>>>>> ImmutableInjectioPoint, InjectableMethod, and ParameterValue?
> >>>>>>>>>
> >>>>>>>>>
> >>>>>>>>> Begin forwarded message:
> >>>>>>>>>
> >>>>>>>>>> From: "Gerhard Petracek (Created) (JIRA)"
> >>>>>>> <jira@apache.org>
> >>>>>>>>>> Subject: [jira] [Created] (DELTASPIKE-129) re-visit visibility
> of
> >>>>>>>>> AnnotationBuilder, ImmutableInjectionPoint, InjectableMethod and
> >>>>>>>>> ParameterValueRedefiner
> >>>>>>>>>> Date: 25 March 2012 16:39:27 GMT+01:00
> >>>>>>>>>> To: deltaspike-dev@incubator.apache.org
> >>>>>>>>>>
> >>>>>>>>>> re-visit visibility of AnnotationBuilder,
> ImmutableInjectionPoint,
> >>>>>>>>> InjectableMethod and ParameterValueRedefiner
> >>>>>>>>>>
> >>>>>>>>>
> >>>>>>>
> >>>>>
> ---------------------------------------------------------------------------------------------------------------
> >>>>>>>>>>
> >>>>>>>>>>              Key: DELTASPIKE-129
> >>>>>>>>>>              URL:
> >>>>>>>>> https://issues.apache.org/jira/browse/DELTASPIKE-129
> >>>>>>>>>>          Project: DeltaSpike
> >>>>>>>>>>       Issue Type: Task
> >>>>>>>>>>       Components: Core
> >>>>>>>>>> Affects Versions: 0.1-incubating
> >>>>>>>>>>         Reporter: Gerhard Petracek
> >>>>>>>>>>         Assignee: Jason Porter
> >>>>>>>>>>          Fix For: 0.2-incubating
> >>>>>>>>>>
> >>>>>>>>>>
> >>>>>>>>>> ... since those classes aren't intended to be used by users, we
> >>>>>>> should
> >>>>>>>>> re-visit them.
> >>>>>>>>>> if we can't keep them package-private, we could move them to
> >>>>>>> the
> >>>>>>>>> util-package (like we did with ClassDeactivation now
> >>>>>>> ClassDeactivationUtils)
> >>>>>>>>>>
> >>>>>>>>>> --
> >>>>>>>>>> This message is automatically generated by JIRA.
> >>>>>>>>>> If you think it was sent incorrectly, please contact your JIRA
> >>>>>>>>> administrators:
> >>>>>>>>>
> >>>>>>>
> >>>>>
> https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
> >>>>>>>>>> For more information on JIRA, see:
> >>>>>>>>> http://www.atlassian.com/software/jira
> >>>>>>>>>>
> >>>>>>>>>>
> >>>>>>>>>
> >>>>>>>>>
> >>>>>>>
> >>>>>
> >>>>>
> >>>
> >>
>


-- 
Jason Porter
http://lightguard-jp.blogspot.com
http://twitter.com/lightguardjp

Software Engineer
Open Source Advocate
Author of Seam Catch - Next Generation Java Exception Handling

PGP key id: 926CCFF5
PGP key available at: keyserver.net, pgp.mit.edu

--bcaec5485810a062ef04bc2bc335--