You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sling.apache.org by Robert Munteanu <ro...@apache.org> on 2023/04/21 09:44:37 UTC
Dropping Java 8 support for the Sling Starter (was: [RT] Migrating the feature model tooling to Jakarta JSON)
Hi,
Changing subject to make sure we get enough attention. With Oak 1.50
requiring Java 11 it does not look like we can hold on to Java 8 for
much longer.
I filed https://issues.apache.org/jira/browse/SLING-11840 to track
this.
If anyone sees a reason to keep supporting Java 8, please say so.
Note that this does not mean that the Sling bundles need to bump the
required version to Java 11, they can keep supporting Java 8 as long as
it is desired.
Thanks,
Robert
On Thu, 2023-04-20 at 10:37 -0700, Eric Norman wrote:
> >
> > 2. Officially drop support for Java 8 in the Starter (not
> > necessarily
> > the bundles) and incorporate Parsson
> >
>
> I wouldn't have a problem with bumping the minimum java version for
> the
> starter to the minimum LTS OpenJDK version that is still receiving
> active
> support (currently version 11). It looks like active support for
> OpenJDK
> 8 ended over a year ago and it now receives only security related
> fixes.
>
> Also, Oak 1.50.0 now requires java 11 so we can't use the latest
> production
> release of oak with java 8.
>
> Regards,
> -Eric
>
> On Wed, Apr 19, 2023 at 1:21 PM Robert Munteanu <ro...@apache.org>
> wrote:
>
> > On Wed, 2023-04-19 at 16:54 +0200, Robert Munteanu wrote:
> > > I added Parsson to the contentloader 2.6.0 PR
> > >
> > >
> > > https://github.com/apache/sling-org-apache-sling-starter/pull/105
> > >
> > > Seems to work fine locally. I will split this to a different PR,
> > > just
> > > wanted to push it out there for review.
> > >
> > > Thanks,
> > > Robert
> >
> > Parsson requires Java 9, but we still run tests on Java 8
> >
> > [ERROR] [requirements-capabilities]
> > org.eclipse.parsson:jakarta.json:1.1.1: Artifact
> > org.eclipse.parsson:jakarta.json:1.1.1 requires
> > [org.eclipse.parsson.jakarta.json/1.1.1] osgi.ee;
> > filter:="(&(osgi.ee=JavaSE)(version=9))" in start level 1 but no
> > artifact is providing a matching capability in this start level.
> >
> > There are two ways out:
> >
> > 1. Look for another parser implementation
> > 2. Officially drop support for Java 8 in the Starter (not
> > necessarily
> > the bundles) and incorporate Parsson
> >
> > I slightly favour option 2, I think people deploying new versions
> > of
> > the Starter can choose a more recent version of the Java runtime.
> >
> > Thoughts?
> >
> > Robert
> >
Re: Dropping Java 8 support for the Sling Starter (was: [RT] Migrating the feature model tooling to Jakarta JSON)
Posted by Robert Munteanu <ro...@apache.org>.
On Fri, 2023-04-21 at 11:44 +0200, Robert Munteanu wrote:
> I filed https://issues.apache.org/jira/browse/SLING-11840 to track
> this.
Follow-ed up with
https://github.com/apache/sling-org-apache-sling-starter/pull/152
Thanks,
Robert
Re: Dropping Java 8 support for the Sling Starter (was: [RT] Migrating the feature model tooling to Jakarta JSON)
Posted by Konrad Windszus <kw...@apache.org>.
I created some follow up tickets for that
https://issues.apache.org/jira/browse/SLING-11843
https://issues.apache.org/jira/browse/SLING-11842
https://issues.apache.org/jira/browse/SLING-11841
Please comment directly on those.
Thanks,
Konrad
> On 21. Apr 2023, at 12:02, Stefan Seifert <St...@diva-e.com.INVALID> wrote:
>
> makes sense, go ahead!
>
> we should consider dropping java 8 for new sling module releases in general, updating to java 11 in our sling-parent. java 8 is EOL for quite a long time, we can start using new language features, and drop java 8 builds in Jenkins.
>
> stefan
>
>> -----Original Message-----
>> From: Robert Munteanu <ro...@apache.org>
>> Sent: Friday, April 21, 2023 11:45 AM
>> To: dev@sling.apache.org
>> Subject: Dropping Java 8 support for the Sling Starter (was: [RT]
>> Migrating the feature model tooling to Jakarta JSON)
>>
>> Hi,
>>
>> Changing subject to make sure we get enough attention. With Oak 1.50
>> requiring Java 11 it does not look like we can hold on to Java 8 for much
>> longer.
>>
>> I filed https://issues.apache.org/jira/browse/SLING-11840 to track this.
>>
>> If anyone sees a reason to keep supporting Java 8, please say so.
>>
>> Note that this does not mean that the Sling bundles need to bump the
>> required version to Java 11, they can keep supporting Java 8 as long as it
>> is desired.
>>
>> Thanks,
>> Robert
>>
>> On Thu, 2023-04-20 at 10:37 -0700, Eric Norman wrote:
>>>>
>>>> 2. Officially drop support for Java 8 in the Starter (not
>>>> necessarily the bundles) and incorporate Parsson
>>>>
>>>
>>> I wouldn't have a problem with bumping the minimum java version for
>>> the
>>> starter to the minimum LTS OpenJDK version that is still receiving
>>> active
>>> support (currently version 11). It looks like active support for
>>> OpenJDK
>>> 8 ended over a year ago and it now receives only security related
>>> fixes.
>>>
>>> Also, Oak 1.50.0 now requires java 11 so we can't use the latest
>>> production
>>> release of oak with java 8.
>>>
>>> Regards,
>>> -Eric
>>>
>>> On Wed, Apr 19, 2023 at 1:21 PM Robert Munteanu <ro...@apache.org>
>>> wrote:
>>>
>>>> On Wed, 2023-04-19 at 16:54 +0200, Robert Munteanu wrote:
>>>>> I added Parsson to the contentloader 2.6.0 PR
>>>>>
>>>>>
>>>>> https://github.com/apache/sling-org-apache-sling-starter/pull/105
>>>>>
>>>>> Seems to work fine locally. I will split this to a different PR,
>>>>> just
>>>>> wanted to push it out there for review.
>>>>>
>>>>> Thanks,
>>>>> Robert
>>>>
>>>> Parsson requires Java 9, but we still run tests on Java 8
>>>>
>>>> [ERROR] [requirements-capabilities]
>>>> org.eclipse.parsson:jakarta.json:1.1.1: Artifact
>>>> org.eclipse.parsson:jakarta.json:1.1.1 requires
>>>> [org.eclipse.parsson.jakarta.json/1.1.1] osgi.ee;
>>>> filter:="(&(osgi.ee=JavaSE)(version=9))" in start level 1 but no
>>>> artifact is providing a matching capability in this start level.
>>>>
>>>> There are two ways out:
>>>>
>>>> 1. Look for another parser implementation
>>>> 2. Officially drop support for Java 8 in the Starter (not
>>>> necessarily
>>>> the bundles) and incorporate Parsson
>>>>
>>>> I slightly favour option 2, I think people deploying new versions
>>>> of
>>>> the Starter can choose a more recent version of the Java runtime.
>>>>
>>>> Thoughts?
>>>>
>>>> Robert
>>>>
>
Re: Dropping Java 8 support for the Sling Starter (was: [RT] Migrating the feature model tooling to Jakarta JSON)
Posted by Konrad Windszus <kw...@apache.org>.
There was a similar discussion already at https://lists.apache.org/thread/6p86tqpldjj83zw3288ooht869yt99dr and back then we already kind of reached consensus that it is time to require Java 11 also for bundles nowadays.
Konrad
> On 21. Apr 2023, at 12:02, Stefan Seifert <St...@diva-e.com.INVALID> wrote:
>
> makes sense, go ahead!
>
> we should consider dropping java 8 for new sling module releases in general, updating to java 11 in our sling-parent. java 8 is EOL for quite a long time, we can start using new language features, and drop java 8 builds in Jenkins.
>
> stefan
>
>> -----Original Message-----
>> From: Robert Munteanu <ro...@apache.org>
>> Sent: Friday, April 21, 2023 11:45 AM
>> To: dev@sling.apache.org
>> Subject: Dropping Java 8 support for the Sling Starter (was: [RT]
>> Migrating the feature model tooling to Jakarta JSON)
>>
>> Hi,
>>
>> Changing subject to make sure we get enough attention. With Oak 1.50
>> requiring Java 11 it does not look like we can hold on to Java 8 for much
>> longer.
>>
>> I filed https://issues.apache.org/jira/browse/SLING-11840 to track this.
>>
>> If anyone sees a reason to keep supporting Java 8, please say so.
>>
>> Note that this does not mean that the Sling bundles need to bump the
>> required version to Java 11, they can keep supporting Java 8 as long as it
>> is desired.
>>
>> Thanks,
>> Robert
>>
>> On Thu, 2023-04-20 at 10:37 -0700, Eric Norman wrote:
>>>>
>>>> 2. Officially drop support for Java 8 in the Starter (not
>>>> necessarily the bundles) and incorporate Parsson
>>>>
>>>
>>> I wouldn't have a problem with bumping the minimum java version for
>>> the
>>> starter to the minimum LTS OpenJDK version that is still receiving
>>> active
>>> support (currently version 11). It looks like active support for
>>> OpenJDK
>>> 8 ended over a year ago and it now receives only security related
>>> fixes.
>>>
>>> Also, Oak 1.50.0 now requires java 11 so we can't use the latest
>>> production
>>> release of oak with java 8.
>>>
>>> Regards,
>>> -Eric
>>>
>>> On Wed, Apr 19, 2023 at 1:21 PM Robert Munteanu <ro...@apache.org>
>>> wrote:
>>>
>>>> On Wed, 2023-04-19 at 16:54 +0200, Robert Munteanu wrote:
>>>>> I added Parsson to the contentloader 2.6.0 PR
>>>>>
>>>>>
>>>>> https://github.com/apache/sling-org-apache-sling-starter/pull/105
>>>>>
>>>>> Seems to work fine locally. I will split this to a different PR,
>>>>> just
>>>>> wanted to push it out there for review.
>>>>>
>>>>> Thanks,
>>>>> Robert
>>>>
>>>> Parsson requires Java 9, but we still run tests on Java 8
>>>>
>>>> [ERROR] [requirements-capabilities]
>>>> org.eclipse.parsson:jakarta.json:1.1.1: Artifact
>>>> org.eclipse.parsson:jakarta.json:1.1.1 requires
>>>> [org.eclipse.parsson.jakarta.json/1.1.1] osgi.ee;
>>>> filter:="(&(osgi.ee=JavaSE)(version=9))" in start level 1 but no
>>>> artifact is providing a matching capability in this start level.
>>>>
>>>> There are two ways out:
>>>>
>>>> 1. Look for another parser implementation
>>>> 2. Officially drop support for Java 8 in the Starter (not
>>>> necessarily
>>>> the bundles) and incorporate Parsson
>>>>
>>>> I slightly favour option 2, I think people deploying new versions
>>>> of
>>>> the Starter can choose a more recent version of the Java runtime.
>>>>
>>>> Thoughts?
>>>>
>>>> Robert
>>>>
>
RE: Dropping Java 8 support for the Sling Starter (was: [RT] Migrating the feature model tooling to Jakarta JSON)
Posted by Stefan Seifert <St...@diva-e.com.INVALID>.
makes sense, go ahead!
we should consider dropping java 8 for new sling module releases in general, updating to java 11 in our sling-parent. java 8 is EOL for quite a long time, we can start using new language features, and drop java 8 builds in Jenkins.
stefan
> -----Original Message-----
> From: Robert Munteanu <ro...@apache.org>
> Sent: Friday, April 21, 2023 11:45 AM
> To: dev@sling.apache.org
> Subject: Dropping Java 8 support for the Sling Starter (was: [RT]
> Migrating the feature model tooling to Jakarta JSON)
>
> Hi,
>
> Changing subject to make sure we get enough attention. With Oak 1.50
> requiring Java 11 it does not look like we can hold on to Java 8 for much
> longer.
>
> I filed https://issues.apache.org/jira/browse/SLING-11840 to track this.
>
> If anyone sees a reason to keep supporting Java 8, please say so.
>
> Note that this does not mean that the Sling bundles need to bump the
> required version to Java 11, they can keep supporting Java 8 as long as it
> is desired.
>
> Thanks,
> Robert
>
> On Thu, 2023-04-20 at 10:37 -0700, Eric Norman wrote:
> > >
> > > 2. Officially drop support for Java 8 in the Starter (not
> > > necessarily the bundles) and incorporate Parsson
> > >
> >
> > I wouldn't have a problem with bumping the minimum java version for
> > the
> > starter to the minimum LTS OpenJDK version that is still receiving
> > active
> > support (currently version 11). It looks like active support for
> > OpenJDK
> > 8 ended over a year ago and it now receives only security related
> > fixes.
> >
> > Also, Oak 1.50.0 now requires java 11 so we can't use the latest
> > production
> > release of oak with java 8.
> >
> > Regards,
> > -Eric
> >
> > On Wed, Apr 19, 2023 at 1:21 PM Robert Munteanu <ro...@apache.org>
> > wrote:
> >
> > > On Wed, 2023-04-19 at 16:54 +0200, Robert Munteanu wrote:
> > > > I added Parsson to the contentloader 2.6.0 PR
> > > >
> > > >
> > > > https://github.com/apache/sling-org-apache-sling-starter/pull/105
> > > >
> > > > Seems to work fine locally. I will split this to a different PR,
> > > > just
> > > > wanted to push it out there for review.
> > > >
> > > > Thanks,
> > > > Robert
> > >
> > > Parsson requires Java 9, but we still run tests on Java 8
> > >
> > > [ERROR] [requirements-capabilities]
> > > org.eclipse.parsson:jakarta.json:1.1.1: Artifact
> > > org.eclipse.parsson:jakarta.json:1.1.1 requires
> > > [org.eclipse.parsson.jakarta.json/1.1.1] osgi.ee;
> > > filter:="(&(osgi.ee=JavaSE)(version=9))" in start level 1 but no
> > > artifact is providing a matching capability in this start level.
> > >
> > > There are two ways out:
> > >
> > > 1. Look for another parser implementation
> > > 2. Officially drop support for Java 8 in the Starter (not
> > > necessarily
> > > the bundles) and incorporate Parsson
> > >
> > > I slightly favour option 2, I think people deploying new versions
> > > of
> > > the Starter can choose a more recent version of the Java runtime.
> > >
> > > Thoughts?
> > >
> > > Robert
> > >