[
Date Prev][
Date Next][
Thread Prev][
Thread Next][
Date Index][
Thread Index]
[
List Home]
|
Re: [glassfish-dev] Time to rename the ORB packages?
|
I’ve spoken with a worker at Jetbrains. It seems that excluding com.sun.* packages from code coverage is something they’ve done as an optimization, since almost all such code is in the JDK.
Most of the orb classes would not be used externally, but I would like to be able to verify this. Is there a way to search Maven central to see what projects depend on a particular project? The data should be there.
> On Mar 28, 2020, at 3:18 PM, Bill Shannon <bill.shannon@xxxxxxxxxx> wrote:
>
> In general, we're not renaming implementation packages.
>
> It sounds like a bug, or configuration error, that IntelliJ won't handle
> these packages. There are com.sun packages in other EE4J components.
> For example, I don't plan to rename the com.sun classed in Jakarta Mail
> because users have been using those classes for some time. While it
> may be no worse than renaming javax, it's just another cut.
>
> In your case, if the IntelliJ problem is serious for you and can't be
> fixed, then you can take the risk and rename the packages. Just be careful
> not to make things worse.
>
>
> Russell Gold wrote on 3/27/20 8:58 AM:
>> The ORB classes are still located under com.sun.corba.ee.* Aside from lack of consistency, this has now turned out to be a maintenance problem, as my IDE, Intellij IDEA specifically excludes com.sun.* packages from code coverage results, since that hierarchy is generally used for the JDK. So the question is timing. I’m thinking that the time has come to do the rename. If users of the ORB are doing things correctly, only the main ORB and ORB singleton class names should be pertinent - and I can make those extend the new class names.
>>
>> Thoughts?
>>
>> - Russ
>> _______________________________________________
>> glassfish-dev mailing list
>> glassfish-dev@xxxxxxxxxxx
>> To unsubscribe from this list, visit https://urldefense.com/v3/__https://www.eclipse.org/mailman/listinfo/glassfish-dev__;!!GqivPVa7Brio!KZY4amxSORc_tcorxmDPoScclaCEmMRnb576xjAqy1Mrs2LZ8rcvwBSVh7ingFHgOQ$
>>
