Bug 644042 – dev-cpp/ms-gsl-0_pre20180108 : /.../multi_span:1166:57: error: conversion to long unsigned int from long int may change the sign of the result [-Werror=sign-conversion]

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Toralf Förster 2018-01-09 18:29:05 UTC]

In file included from /var/tmp/portage/dev-cpp/gsl-0_pre20180108/work/gsl-0_pre20180108/tests/multi_span_tests.cpp:21:0:
/var/tmp/portage/dev-cpp/gsl-0_pre20180108/work/gsl-0_pre20180108/include/gsl/multi_span:1166:57: error: conversion to ‘long unsigned int’ from ‘long int’ may change the sign of the result [-Werror=sign-conversion]
     struct is_multi_span_oracle<strided_span<ValueType, Rank>> : std::true_type
                                                         ^~~~
/var/tmp/portage/dev-cpp/gsl-0_pre20180108/work/gsl-0_pre20180108/include/gsl/multi_span:1276:88: error: conversion to ‘long int’ from ‘long unsigned int’ may change the sign of the result [-Werror=sign-conversion]

  -------------------------------------------------------------------

  This is an unstable amd64 chroot image at a tinderbox (==build bot)
  name: 17.0_libressl-test_20180106-200451

  -------------------------------------------------------------------

gcc-config -l:
 [1] x86_64-pc-linux-gnu-7.2.0 *

Available Python interpreters, in order of preference:
  [1]   python3.5
  [2]   python3.6 (fallback)
  [3]   python2.7 (fallback)

java-config:
The following VMs are available for generation-2:
*)	IcedTea JDK 3.6.0 [icedtea-bin-8]
Available Java Virtual Machines:
  [1]   icedtea-bin-8  system-vm

emerge -qpv dev-cpp/gsl

[Comment 1 Toralf Förster 2018-01-09 18:29:07 UTC]

Created attachment 513940 [details]
emerge-info.txt

[Comment 2 Toralf Förster 2018-01-09 18:29:10 UTC]

Created attachment 513942 [details]
dev-cpp:gsl-0_pre20180108:20180109-030841.log

[Comment 3 Toralf Förster 2018-01-09 18:29:13 UTC]

Created attachment 513944 [details]
emerge-history.txt

[Comment 4 Toralf Förster 2018-01-09 18:29:16 UTC]

Created attachment 513946 [details]
environment

[Comment 5 Toralf Förster 2018-01-09 18:29:19 UTC]

Created attachment 513948 [details]
etc.portage.tbz2

[Comment 6 Toralf Förster 2018-01-09 18:29:22 UTC]

Created attachment 513950 [details]
logs.tbz2

[Comment 7 Toralf Förster 2018-01-09 18:29:25 UTC]

Created attachment 513952 [details]
temp.tbz2

[Comment 8 Jan Henke 2018-01-09 18:53:23 UTC]

I do not see that error being generated on my machine with GCC 6.4.0, but the GCC 7 release notes also do not mention any changes in that area. Which makes this bug kind of strange from my point of view.

It also kind of looks like an upstream issue.

[Comment 9 Jan Henke 2018-01-19 18:16:17 UTC]

There is an upstream bug report about this issue: https://github.com/Microsoft/GSL/issues/525

I am monitoring that bug and will bump the ebuild, once there is a fixed revision upstream.

[Comment 10 Larry the Git Cow 2018-03-19 15:57:07 UTC]

The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=db56cbbc4241fdd4aa19a7e09987334ba6fede80

commit db56cbbc4241fdd4aa19a7e09987334ba6fede80
Author:     Jan Henke <[e-mail]>
AuthorDate: 2018-03-17 21:36:47 +0000
Commit:     Michał Górny <[e-mail]>
CommitDate: 2018-03-19 15:45:37 +0000

    dev-cpp/ms-gsl: Disable Werror when compiling tests
    
    This lead to build failures due to new warnings being emmited by GCC 7
    
    Closes: https://bugs.gentoo.org/644042
    Closes: https://github.com/gentoo/gentoo/pull/7495
    Package-Manager: Portage-2.3.24, Repoman-2.3.6

 ...s-gsl-0_pre20180304-disable_Werror-644042.patch | 27 ++++++++++++++++++++++
 dev-cpp/ms-gsl/ms-gsl-0_pre20180304.ebuild         |  1 +
 2 files changed, 28 insertions(+)