Re: [ovirt-devel] Requiring Java 8 during runtime

2015-10-28 Thread Juan Hernández 
On 10/27/2015 04:04 PM, Tal Nisan wrote:
> I agree with Allon, there's no reason to have different version
> throughout the modules (again, not including GWT for obvious reasons),
> there is practically no harm I can think of in raising the version for
> all modules at once

I have updated the patch to raise the level in all the projects except
the ones that require Java 7 because of GWT:

  core: Use Java 8 as source and target
  https://gerrit.ovirt.org/46288

Please review.

> 
> On Tue, Oct 27, 2015 at 10:36 AM, Allon Mureinik  > wrote:
> 
> I think it's a really bad practice to have different modules in the
> same project with different language versions.
> 
> For GWT it seems to be required, but for the rest of the backend, I
> think we should have one patch that moves the entire project in one go.
> We can have the various maintainers ack it to show their consent.
> 
> On Mon, Oct 26, 2015 at 1:17 PM, Juan Hernández  > wrote:
> 
> On 10/26/2015 12:05 PM, Martin Mucha wrote:
> > not strictly related question: are we going to raise language level 
> as well? So we can start using JDK8 features like lambdas etc?
> >
> > M.
> >
> 
> Shortly after merging that patch I'll merge another one to raise the
> language level in the RESTAPI:
> 
>   restapi: Use Java 8 as source and target
>   https://gerrit.ovirt.org/46288
> 
> Doing the same in other components is up to their maintainers. I
> think
> that should be done everywhere, except where the use of GWT doesn't
> allow to do it.
> 
> > - Original Message -
> >> Hello,
> >>
> >> As you probably know oVirt Engine 4 will use WildFly 10, and that
> >> requires Java 8. The version of WildFly that we currently use
> is 8.2.1,
> >> and it can work with both Java 7 and Java 8. In order to ease the
> >> transition I'm about to merge the following patch, that will
> require
> >> Java 8 during runtime:
> >>
> >>   core: Require Java 8 during runtime
> >>   https://gerrit.ovirt.org/46872
> >>
> >> The implication of this for you is that you must make sure
> that you have
> >> Java 8 installed in the machine where you run your the
> engine. Fedora 22
> >> only supports Java 8, so that isn't a problem. EL6 supports
> both Java 7
> >> and Java 8, so make sure that you have Java 8 installed:
> >>
> >>   yum -y install java-1.8.0-openjdk-devel
> >>
> >> If you have objections please speak now, otherwise I will
> merge the
> >> patch tomorrow.
> >>
> >> Regards,
> >> Juan Hernandez
> >>
> >> --
> >> Dirección Comercial: C/Jose Bardasano Baos, 9, Edif. Gorbea
> 3, planta
> >> 3ºD, 28016 Madrid, Spain
> >> Inscrita en el Reg. Mercantil de Madrid – C.I.F. B82657941 -
> Red Hat S.L.
> >> ___
> >> Devel mailing list
> >> Devel@ovirt.org 
> >> http://lists.ovirt.org/mailman/listinfo/devel
> > ___
> > Devel mailing list
> > Devel@ovirt.org 
> > http://lists.ovirt.org/mailman/listinfo/devel
> >
> 
> 
> --
> Dirección Comercial: C/Jose Bardasano Baos, 9, Edif. Gorbea 3,
> planta
> 3ºD, 28016 Madrid, Spain
> Inscrita en el Reg. Mercantil de Madrid – C.I.F. B82657941 - Red
> Hat S.L.
> ___
> Devel mailing list
> Devel@ovirt.org 
> http://lists.ovirt.org/mailman/listinfo/devel
> 
> 
> 
> 
> ___
> Devel mailing list
> Devel@ovirt.org 
> http://lists.ovirt.org/mailman/listinfo/devel
> 
> 


-- 
Dirección Comercial: C/Jose Bardasano Baos, 9, Edif. Gorbea 3, planta
3ºD, 28016 Madrid, Spain
Inscrita en el Reg. Mercantil de Madrid – C.I.F. B82657941 - Red Hat S.L.
___
Devel mailing list
Devel@ovirt.org
http://lists.ovirt.org/mailman/listinfo/devel

Re: [ovirt-devel] Requiring Java 8 during runtime

2015-10-27 Thread Tal Nisan 
I agree with Allon, there's no reason to have different version throughout
the modules (again, not including GWT for obvious reasons), there is
practically no harm I can think of in raising the version for all modules
at once

On Tue, Oct 27, 2015 at 10:36 AM, Allon Mureinik 
wrote:

> I think it's a really bad practice to have different modules in the same
> project with different language versions.
>
> For GWT it seems to be required, but for the rest of the backend, I think
> we should have one patch that moves the entire project in one go.
> We can have the various maintainers ack it to show their consent.
>
> On Mon, Oct 26, 2015 at 1:17 PM, Juan Hernández 
> wrote:
>
>> On 10/26/2015 12:05 PM, Martin Mucha wrote:
>> > not strictly related question: are we going to raise language level as
>> well? So we can start using JDK8 features like lambdas etc?
>> >
>> > M.
>> >
>>
>> Shortly after merging that patch I'll merge another one to raise the
>> language level in the RESTAPI:
>>
>>   restapi: Use Java 8 as source and target
>>   https://gerrit.ovirt.org/46288
>>
>> Doing the same in other components is up to their maintainers. I think
>> that should be done everywhere, except where the use of GWT doesn't
>> allow to do it.
>>
>> > - Original Message -
>> >> Hello,
>> >>
>> >> As you probably know oVirt Engine 4 will use WildFly 10, and that
>> >> requires Java 8. The version of WildFly that we currently use is 8.2.1,
>> >> and it can work with both Java 7 and Java 8. In order to ease the
>> >> transition I'm about to merge the following patch, that will require
>> >> Java 8 during runtime:
>> >>
>> >>   core: Require Java 8 during runtime
>> >>   https://gerrit.ovirt.org/46872
>> >>
>> >> The implication of this for you is that you must make sure that you
>> have
>> >> Java 8 installed in the machine where you run your the engine. Fedora
>> 22
>> >> only supports Java 8, so that isn't a problem. EL6 supports both Java 7
>> >> and Java 8, so make sure that you have Java 8 installed:
>> >>
>> >>   yum -y install java-1.8.0-openjdk-devel
>> >>
>> >> If you have objections please speak now, otherwise I will merge the
>> >> patch tomorrow.
>> >>
>> >> Regards,
>> >> Juan Hernandez
>> >>
>> >> --
>> >> Dirección Comercial: C/Jose Bardasano Baos, 9, Edif. Gorbea 3, planta
>> >> 3ºD, 28016 Madrid, Spain
>> >> Inscrita en el Reg. Mercantil de Madrid – C.I.F. B82657941 - Red Hat
>> S.L.
>> >> ___
>> >> Devel mailing list
>> >> Devel@ovirt.org
>> >> http://lists.ovirt.org/mailman/listinfo/devel
>> > ___
>> > Devel mailing list
>> > Devel@ovirt.org
>> > http://lists.ovirt.org/mailman/listinfo/devel
>> >
>>
>>
>> --
>> Dirección Comercial: C/Jose Bardasano Baos, 9, Edif. Gorbea 3, planta
>> 3ºD, 28016 Madrid, Spain
>> Inscrita en el Reg. Mercantil de Madrid – C.I.F. B82657941 - Red Hat S.L.
>> ___
>> Devel mailing list
>> Devel@ovirt.org
>> http://lists.ovirt.org/mailman/listinfo/devel
>>
>>
>>
>
> ___
> Devel mailing list
> Devel@ovirt.org
> http://lists.ovirt.org/mailman/listinfo/devel
>
___
Devel mailing list
Devel@ovirt.org
http://lists.ovirt.org/mailman/listinfo/devel

Re: [ovirt-devel] Requiring Java 8 during runtime

2015-10-27 Thread Allon Mureinik 
I think it's a really bad practice to have different modules in the same
project with different language versions.

For GWT it seems to be required, but for the rest of the backend, I think
we should have one patch that moves the entire project in one go.
We can have the various maintainers ack it to show their consent.

On Mon, Oct 26, 2015 at 1:17 PM, Juan Hernández  wrote:

> On 10/26/2015 12:05 PM, Martin Mucha wrote:
> > not strictly related question: are we going to raise language level as
> well? So we can start using JDK8 features like lambdas etc?
> >
> > M.
> >
>
> Shortly after merging that patch I'll merge another one to raise the
> language level in the RESTAPI:
>
>   restapi: Use Java 8 as source and target
>   https://gerrit.ovirt.org/46288
>
> Doing the same in other components is up to their maintainers. I think
> that should be done everywhere, except where the use of GWT doesn't
> allow to do it.
>
> > - Original Message -
> >> Hello,
> >>
> >> As you probably know oVirt Engine 4 will use WildFly 10, and that
> >> requires Java 8. The version of WildFly that we currently use is 8.2.1,
> >> and it can work with both Java 7 and Java 8. In order to ease the
> >> transition I'm about to merge the following patch, that will require
> >> Java 8 during runtime:
> >>
> >>   core: Require Java 8 during runtime
> >>   https://gerrit.ovirt.org/46872
> >>
> >> The implication of this for you is that you must make sure that you have
> >> Java 8 installed in the machine where you run your the engine. Fedora 22
> >> only supports Java 8, so that isn't a problem. EL6 supports both Java 7
> >> and Java 8, so make sure that you have Java 8 installed:
> >>
> >>   yum -y install java-1.8.0-openjdk-devel
> >>
> >> If you have objections please speak now, otherwise I will merge the
> >> patch tomorrow.
> >>
> >> Regards,
> >> Juan Hernandez
> >>
> >> --
> >> Dirección Comercial: C/Jose Bardasano Baos, 9, Edif. Gorbea 3, planta
> >> 3ºD, 28016 Madrid, Spain
> >> Inscrita en el Reg. Mercantil de Madrid – C.I.F. B82657941 - Red Hat
> S.L.
> >> ___
> >> Devel mailing list
> >> Devel@ovirt.org
> >> http://lists.ovirt.org/mailman/listinfo/devel
> > ___
> > Devel mailing list
> > Devel@ovirt.org
> > http://lists.ovirt.org/mailman/listinfo/devel
> >
>
>
> --
> Dirección Comercial: C/Jose Bardasano Baos, 9, Edif. Gorbea 3, planta
> 3ºD, 28016 Madrid, Spain
> Inscrita en el Reg. Mercantil de Madrid – C.I.F. B82657941 - Red Hat S.L.
> ___
> Devel mailing list
> Devel@ovirt.org
> http://lists.ovirt.org/mailman/listinfo/devel
>
>
>
___
Devel mailing list
Devel@ovirt.org
http://lists.ovirt.org/mailman/listinfo/devel

Re: [ovirt-devel] Requiring Java 8 during runtime

2015-10-26 Thread Juan Hernández 
On 10/26/2015 12:05 PM, Martin Mucha wrote:
> not strictly related question: are we going to raise language level as well? 
> So we can start using JDK8 features like lambdas etc?
> 
> M.
> 

Shortly after merging that patch I'll merge another one to raise the
language level in the RESTAPI:

  restapi: Use Java 8 as source and target
  https://gerrit.ovirt.org/46288

Doing the same in other components is up to their maintainers. I think
that should be done everywhere, except where the use of GWT doesn't
allow to do it.

> - Original Message -
>> Hello,
>>
>> As you probably know oVirt Engine 4 will use WildFly 10, and that
>> requires Java 8. The version of WildFly that we currently use is 8.2.1,
>> and it can work with both Java 7 and Java 8. In order to ease the
>> transition I'm about to merge the following patch, that will require
>> Java 8 during runtime:
>>
>>   core: Require Java 8 during runtime
>>   https://gerrit.ovirt.org/46872
>>
>> The implication of this for you is that you must make sure that you have
>> Java 8 installed in the machine where you run your the engine. Fedora 22
>> only supports Java 8, so that isn't a problem. EL6 supports both Java 7
>> and Java 8, so make sure that you have Java 8 installed:
>>
>>   yum -y install java-1.8.0-openjdk-devel
>>
>> If you have objections please speak now, otherwise I will merge the
>> patch tomorrow.
>>
>> Regards,
>> Juan Hernandez
>>
>> --
>> Dirección Comercial: C/Jose Bardasano Baos, 9, Edif. Gorbea 3, planta
>> 3ºD, 28016 Madrid, Spain
>> Inscrita en el Reg. Mercantil de Madrid – C.I.F. B82657941 - Red Hat S.L.
>> ___
>> Devel mailing list
>> Devel@ovirt.org
>> http://lists.ovirt.org/mailman/listinfo/devel
> ___
> Devel mailing list
> Devel@ovirt.org
> http://lists.ovirt.org/mailman/listinfo/devel
> 


-- 
Dirección Comercial: C/Jose Bardasano Baos, 9, Edif. Gorbea 3, planta
3ºD, 28016 Madrid, Spain
Inscrita en el Reg. Mercantil de Madrid – C.I.F. B82657941 - Red Hat S.L.
___
Devel mailing list
Devel@ovirt.org
http://lists.ovirt.org/mailman/listinfo/devel

Re: [ovirt-devel] Requiring Java 8 during runtime

2015-10-26 Thread Martin Mucha 
not strictly related question: are we going to raise language level as well? So 
we can start using JDK8 features like lambdas etc?

M.

- Original Message -
> Hello,
> 
> As you probably know oVirt Engine 4 will use WildFly 10, and that
> requires Java 8. The version of WildFly that we currently use is 8.2.1,
> and it can work with both Java 7 and Java 8. In order to ease the
> transition I'm about to merge the following patch, that will require
> Java 8 during runtime:
> 
>   core: Require Java 8 during runtime
>   https://gerrit.ovirt.org/46872
> 
> The implication of this for you is that you must make sure that you have
> Java 8 installed in the machine where you run your the engine. Fedora 22
> only supports Java 8, so that isn't a problem. EL6 supports both Java 7
> and Java 8, so make sure that you have Java 8 installed:
> 
>   yum -y install java-1.8.0-openjdk-devel
> 
> If you have objections please speak now, otherwise I will merge the
> patch tomorrow.
> 
> Regards,
> Juan Hernandez
> 
> --
> Dirección Comercial: C/Jose Bardasano Baos, 9, Edif. Gorbea 3, planta
> 3ºD, 28016 Madrid, Spain
> Inscrita en el Reg. Mercantil de Madrid – C.I.F. B82657941 - Red Hat S.L.
> ___
> Devel mailing list
> Devel@ovirt.org
> http://lists.ovirt.org/mailman/listinfo/devel
___
Devel mailing list
Devel@ovirt.org
http://lists.ovirt.org/mailman/listinfo/devel

[ovirt-devel] Requiring Java 8 during runtime

2015-10-26 Thread Juan Hernández 
Hello,

As you probably know oVirt Engine 4 will use WildFly 10, and that
requires Java 8. The version of WildFly that we currently use is 8.2.1,
and it can work with both Java 7 and Java 8. In order to ease the
transition I'm about to merge the following patch, that will require
Java 8 during runtime:

  core: Require Java 8 during runtime
  https://gerrit.ovirt.org/46872

The implication of this for you is that you must make sure that you have
Java 8 installed in the machine where you run your the engine. Fedora 22
only supports Java 8, so that isn't a problem. EL6 supports both Java 7
and Java 8, so make sure that you have Java 8 installed:

  yum -y install java-1.8.0-openjdk-devel

If you have objections please speak now, otherwise I will merge the
patch tomorrow.

Regards,
Juan Hernandez

-- 
Dirección Comercial: C/Jose Bardasano Baos, 9, Edif. Gorbea 3, planta
3ºD, 28016 Madrid, Spain
Inscrita en el Reg. Mercantil de Madrid – C.I.F. B82657941 - Red Hat S.L.
___
Devel mailing list
Devel@ovirt.org
http://lists.ovirt.org/mailman/listinfo/devel