Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 714018 - dev-libs/libixion-0.15.0 : ../.../model_context.hpp: error: field nullptr_t has incomplete type ixion::model_context::input_cell
Summary: dev-libs/libixion-0.15.0 : ../.../model_context.hpp: error: field nullptr_t h...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: Gentoo musl team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-03-22 17:44 UTC by Toralf Förster
Modified: 2020-03-23 00:34 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
emerge-info.txt (emerge-info.txt,17.54 KB, text/plain)
2020-03-22 17:44 UTC, Toralf Förster
Details
dev-libs:libixion-0.15.0:20200322-173610.log (dev-libs:libixion-0.15.0:20200322-173610.log,33.52 KB, text/plain)
2020-03-22 17:44 UTC, Toralf Förster
Details
emerge-history.txt (emerge-history.txt,276.84 KB, text/plain)
2020-03-22 17:44 UTC, Toralf Förster
Details
environment (environment,87.52 KB, text/plain)
2020-03-22 17:44 UTC, Toralf Förster
Details
etc.portage.tbz2 (etc.portage.tbz2,20.93 KB, application/x-bzip)
2020-03-22 17:44 UTC, Toralf Förster
Details
logs.tbz2 (logs.tbz2,11.52 KB, application/x-bzip)
2020-03-22 17:44 UTC, Toralf Förster
Details
temp.tbz2 (temp.tbz2,22.89 KB, application/x-bzip)
2020-03-22 17:44 UTC, Toralf Förster
Details
libixion-0.15.0-nullptr_t.patch (libixion-0.15.0-nullptr_t.patch,459 bytes, patch)
2020-03-22 23:20 UTC, Mark Wright
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Toralf Förster gentoo-dev 2020-03-22 17:44:20 UTC
   64 |         input_cell(nullptr_t);
      |                   ^
../../include/ixion/model_context.hpp:64:20: error: field nullptr_t has incomplete type ixion::model_context::input_cell
   64 |         input_cell(nullptr_t);
      |                    ^~~~~~~~~
../../include/ixion/model_context.hpp:51:28: note: definition of struct ixion::model_context::input_cell is not complete until the closing brace

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

  This is an unstable amd64 chroot image at a tinderbox (==build bot)
  name: 17.0_musl-20200316-165821

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

gcc-config -l:
 [1] x86_64-gentoo-linux-musl-9.3.0 *
clang version 10.0.0 
Target: x86_64-gentoo-linux-musl
Thread model: posix
InstalledDir: /usr/lib/llvm/10/bin
/usr/lib/llvm/10
10.0.0
Available Python interpreters, in order of preference:
  [1]   python3.8
  [2]   python3.7
  [3]   python3.6
  [4]   python2.7 (fallback)
Available Ruby profiles:
  [1]   ruby24 (with Rubygems)
  [2]   ruby25 (with Rubygems) *
Available Rust versions:
  [1]   rust-1.41.1 *

timestamp of HEAD at this tinderbox image:
/var/db/repos/gentoo	Sun Mar 22 16:14:27 UTC 2020
/var/db/repos/musl	Sun Mar 22 15:02:57 UTC 2020

emerge -qpvO dev-libs/libixion
[ebuild     U ] dev-libs/libixion-0.15.0 [0.14.1] USE="threads -debug -python -static-libs" PYTHON_SINGLE_TARGET="python3_6 -python3_7 -python3_8"
Comment 1 Toralf Förster gentoo-dev 2020-03-22 17:44:23 UTC
Created attachment 624510 [details]
emerge-info.txt
Comment 2 Toralf Förster gentoo-dev 2020-03-22 17:44:26 UTC
Created attachment 624512 [details]
dev-libs:libixion-0.15.0:20200322-173610.log
Comment 3 Toralf Förster gentoo-dev 2020-03-22 17:44:28 UTC
Created attachment 624514 [details]
emerge-history.txt
Comment 4 Toralf Förster gentoo-dev 2020-03-22 17:44:31 UTC
Created attachment 624516 [details]
environment
Comment 5 Toralf Förster gentoo-dev 2020-03-22 17:44:34 UTC
Created attachment 624518 [details]
etc.portage.tbz2
Comment 6 Toralf Förster gentoo-dev 2020-03-22 17:44:36 UTC
Created attachment 624520 [details]
logs.tbz2
Comment 7 Toralf Förster gentoo-dev 2020-03-22 17:44:40 UTC
Created attachment 624522 [details]
temp.tbz2
Comment 8 Mark Wright gentoo-dev 2020-03-22 23:20:14 UTC
Created attachment 624658 [details, diff]
libixion-0.15.0-nullptr_t.patch

There is a similar error when compiling with clang 10.0.0, so hopefully the proposed patch (by me) will also fix it with musl.  I have not tested it with musl.
Comment 9 Andreas Sturmlechner gentoo-dev 2020-03-22 23:22:05 UTC
Please send it upstream.
Comment 10 Jory A. Pratt gentoo-dev 2020-03-23 00:04:30 UTC
(In reply to Mark Wright from comment #8)
> Created attachment 624658 [details, diff] [details, diff]
> libixion-0.15.0-nullptr_t.patch
> 
> There is a similar error when compiling with clang 10.0.0, so hopefully the
> proposed patch (by me) will also fix it with musl.  I have not tested it
> with musl.

From a8fe2c542b16ee30723ab3d7f3b643ee479e9723 Mon Sep 17 00:00:00 2001
From: orbea <orbea@riseup.net>
Date: Fri, 23 Aug 2019 08:50:28 -0700
Subject: [PATCH] Fix build with clang.

---
 include/ixion/model_context.hpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/ixion/model_context.hpp b/include/ixion/model_context.hpp
index 6ff5b67..dc27ed0 100644
--- a/include/ixion/model_context.hpp
+++ b/include/ixion/model_context.hpp
@@ -61,7 +61,7 @@ public:
         } value;
 
         /** Initializes the cell to be empty. */
-        input_cell(nullptr_t);
+        input_cell(std::nullptr_t);
         /** Boolean cell value. */
         input_cell(bool b);
         /** The char array must be null-terminated. */
-- 
2.24.1


Patch has been around for ever.
Comment 11 Michael 'veremitz' Everitt 2020-03-23 00:08:27 UTC
(In reply to Jory A. Pratt from comment #10)
> (In reply to Mark Wright from comment #8)
> > Created attachment 624658 [details, diff] [details, diff] [details, diff]
> > libixion-0.15.0-nullptr_t.patch
> > 
> > There is a similar error when compiling with clang 10.0.0, so hopefully the
> > proposed patch (by me) will also fix it with musl.  I have not tested it
> > with musl.
> 
> From a8fe2c542b16ee30723ab3d7f3b643ee479e9723 Mon Sep 17 00:00:00 2001
> From: orbea <orbea@riseup.net>
> Date: Fri, 23 Aug 2019 08:50:28 -0700
> Subject: [PATCH] Fix build with clang.
> 
> ---
>  include/ixion/model_context.hpp | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/include/ixion/model_context.hpp
> b/include/ixion/model_context.hpp
> index 6ff5b67..dc27ed0 100644
> --- a/include/ixion/model_context.hpp
> +++ b/include/ixion/model_context.hpp
> @@ -61,7 +61,7 @@ public:
>          } value;
>  
>          /** Initializes the cell to be empty. */
> -        input_cell(nullptr_t);
> +        input_cell(std::nullptr_t);
>          /** Boolean cell value. */
>          input_cell(bool b);
>          /** The char array must be null-terminated. */
> -- 
> 2.24.1
> 
> 
> Patch has been around for ever.

Ok so why the hell hasn't it been pushed upstream??!?!
Comment 12 Jory A. Pratt gentoo-dev 2020-03-23 00:24:11 UTC
(In reply to Andreas Sturmlechner from comment #9)
> Please send it upstream.

patch has already landed upstream commit a8fe2c542b16ee30723ab3d7f3b643ee479e9723
Comment 13 Michael 'veremitz' Everitt 2020-03-23 00:31:00 UTC
(In reply to Jory A. Pratt from comment #12)
> (In reply to Andreas Sturmlechner from comment #9)
> > Please send it upstream.
> 
> patch has already landed upstream commit
> a8fe2c542b16ee30723ab3d7f3b643ee479e9723

ACK - https://gitlab.com/ixion/ixion/-/commit/a8fe2c542b16ee30723ab3d7f3b643ee479e9723

Since this is already 6 months ago .. why is this package failing still?! Looks like we need an -rX release or snapshot...
Comment 14 Andreas Sturmlechner gentoo-dev 2020-03-23 00:34:14 UTC
don't panic, it is upstream is all I need to know...
Comment 15 Larry the Git Cow gentoo-dev 2020-03-23 00:34:26 UTC
The bug has been closed via the following commit(s):

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

commit 5cd6c37430359b936182b26f47605e396043239c
Author:     Andreas Sturmlechner <asturm@gentoo.org>
AuthorDate: 2020-03-23 00:33:20 +0000
Commit:     Andreas Sturmlechner <asturm@gentoo.org>
CommitDate: 2020-03-23 00:33:20 +0000

    dev-libs/libixion: Fix build with musl/clang
    
    Closes: https://bugs.gentoo.org/714018
    Package-Manager: Portage-2.3.94, Repoman-2.3.21
    Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org>

 .../files/libixion-0.15.0-musl-clang.patch         | 24 ++++++++++++++++++++++
 dev-libs/libixion/libixion-0.15.0.ebuild           |  2 ++
 2 files changed, 26 insertions(+)