diff mbox series

[ovs-dev,branch-2.5] travis: Drop 2.6.32 kernel build.

Message ID 20190806130602.10738-1-i.maximets@samsung.com
State Accepted
Headers show
Series [ovs-dev,branch-2.5] travis: Drop 2.6.32 kernel build. | expand

Commit Message

Ilya Maximets Aug. 6, 2019, 1:06 p.m. UTC
gcc >= 5 can't build Linux kernel 2.6.32 and this will never
change because 2.6.32 is not supported for a last few years.

TravsCI migrated to use Ubuntu Xenial by defualt with gcc 5
installed. Dropping the 2.6.32 build item to unlock green build.

Signed-off-by: Ilya Maximets <i.maximets@samsung.com>
---

Sending this patch as it was a preferred solution for a few
people in discussion here:
   https://mail.openvswitch.org/pipermail/ovs-dev/2019-August/361278.html

Two patches additionally needs to be backported for successful Travis
build:

  a7021b08b 2018-07-09 | configure: Disable -Wnull-pointer-arithmetic Clang warning. [Ben Pfaff]
  1e78e3085 2017-01-26 | libX.pc: use the correct output directory [Aaron Conole]

I could backport them along with applying this patch.

 .travis.yml | 1 -
 1 file changed, 1 deletion(-)

Comments

Aaron Conole Aug. 6, 2019, 2:31 p.m. UTC | #1
Ilya Maximets <i.maximets@samsung.com> writes:

> gcc >= 5 can't build Linux kernel 2.6.32 and this will never
> change because 2.6.32 is not supported for a last few years.
>
> TravsCI migrated to use Ubuntu Xenial by defualt with gcc 5
> installed. Dropping the 2.6.32 build item to unlock green build.
>
> Signed-off-by: Ilya Maximets <i.maximets@samsung.com>
> ---
>
> Sending this patch as it was a preferred solution for a few
> people in discussion here:
>    https://mail.openvswitch.org/pipermail/ovs-dev/2019-August/361278.html
>
> Two patches additionally needs to be backported for successful Travis
> build:
>
>   a7021b08b 2018-07-09 | configure: Disable -Wnull-pointer-arithmetic Clang warning. [Ben Pfaff]
>   1e78e3085 2017-01-26 | libX.pc: use the correct output directory [Aaron Conole]
>
> I could backport them along with applying this patch.

If you apply this, please do backport those patches.

>  .travis.yml | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/.travis.yml b/.travis.yml
> index 90aedef7e..92132e384 100644
> --- a/.travis.yml
> +++ b/.travis.yml
> @@ -30,7 +30,6 @@ env:
>    - KERNEL=3.10.96
>    - KERNEL=3.4.110
>    - KERNEL=3.2.76
> -  - KERNEL=2.6.32.70
>  
>  script: ./.travis/build.sh $OPTS
Simon Horman Aug. 9, 2019, 1:44 p.m. UTC | #2
On Tue, Aug 06, 2019 at 04:06:02PM +0300, Ilya Maximets wrote:
> gcc >= 5 can't build Linux kernel 2.6.32 and this will never
> change because 2.6.32 is not supported for a last few years.
> 
> TravsCI migrated to use Ubuntu Xenial by defualt with gcc 5
> installed. Dropping the 2.6.32 build item to unlock green build.
> 
> Signed-off-by: Ilya Maximets <i.maximets@samsung.com>
> ---
> 
> Sending this patch as it was a preferred solution for a few
> people in discussion here:
>    https://mail.openvswitch.org/pipermail/ovs-dev/2019-August/361278.html

I think its fine to drop building against v2.6.32 on Xenial - as you say
it can't work. But I would think that it is also good to exercise building
v2.6.32, f.e. by building on Trusty.

> Two patches additionally needs to be backported for successful Travis
> build:
> 
>   a7021b08b 2018-07-09 | configure: Disable -Wnull-pointer-arithmetic Clang warning. [Ben Pfaff]
>   1e78e3085 2017-01-26 | libX.pc: use the correct output directory [Aaron Conole]
> 
> I could backport them along with applying this patch.
> 
>  .travis.yml | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/.travis.yml b/.travis.yml
> index 90aedef7e..92132e384 100644
> --- a/.travis.yml
> +++ b/.travis.yml
> @@ -30,7 +30,6 @@ env:
>    - KERNEL=3.10.96
>    - KERNEL=3.4.110
>    - KERNEL=3.2.76
> -  - KERNEL=2.6.32.70
>  
>  script: ./.travis/build.sh $OPTS
>  
> -- 
> 2.17.1
>
Ilya Maximets Oct. 28, 2019, 6:17 p.m. UTC | #3
On 06.08.2019 15:06, Ilya Maximets wrote:
> gcc >= 5 can't build Linux kernel 2.6.32 and this will never
> change because 2.6.32 is not supported for a last few years.
> 
> TravsCI migrated to use Ubuntu Xenial by defualt with gcc 5
> installed. Dropping the 2.6.32 build item to unlock green build.
> 
> Signed-off-by: Ilya Maximets <i.maximets@samsung.com>
> ---
> 
> Sending this patch as it was a preferred solution for a few
> people in discussion here:
>     https://mail.openvswitch.org/pipermail/ovs-dev/2019-August/361278.html
> 
> Two patches additionally needs to be backported for successful Travis
> build:
> 
>    a7021b08b 2018-07-09 | configure: Disable -Wnull-pointer-arithmetic Clang warning. [Ben Pfaff]
>    1e78e3085 2017-01-26 | libX.pc: use the correct output directory [Aaron Conole]
> 
> I could backport them along with applying this patch.

Applied to branch-2.5 with backports.

Best regards, Ilya Maximets.
diff mbox series

Patch

diff --git a/.travis.yml b/.travis.yml
index 90aedef7e..92132e384 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -30,7 +30,6 @@  env:
   - KERNEL=3.10.96
   - KERNEL=3.4.110
   - KERNEL=3.2.76
-  - KERNEL=2.6.32.70
 
 script: ./.travis/build.sh $OPTS