Hi Tino,
thanks for the hint which reduced the number of test suite failures to
one as you can see in Salsa CI[1]. In case someone might find a
solution for this one we might upload - if not I wonder whether this
package is a candidate for removal.
Please note: I have no interest into this package at all - just hunting
for Python3.12 issues.
Kind regards
Andreas.
[1]https://salsa.debian.org/science-team/segyio/-/jobs/5338110
Am Wed, Feb 21, 2024 at 11:42:58AM +0100 schrieb Tino Didriksen:
> std::uint16_t is in #include <cstdint>
>
> stdint.h is the C header, which doesn't import the symbols to the std::
> namespace. In general, the headers Standard C++ imports from Standard C
> snips the .h and prefixes c, so stdint.h -> cstdint, stdio.h -> cstdio, etc.
>
> -- Tino Didriksen
>
>
> On Wed, 21 Feb 2024 at 11:16, Andreas Tille <andr...@an3as.eu> wrote:
>
> > Hi,
> >
> > I've found in the set of patches for segyio other cherry-picked patches
> > to adapt to certain Python3.x versions[1]. The patch kindly suggested
> > by s3v to fix this bug[2] would simply be another cherry-pick from upstream
> > who has meanwhile released a couple of new versions incorporating all
> > patches we seem to need for Python3.12 - thus by upgrading to latest
> > upstream the current bug would be fixed.
> >
> > Usually I would do this in case of team maintained packages but I faced
> > some problems with latest upstream and thus I simply created a branch
> > version_1.9.12 with all proposed changes including current packaging
> > standards and fixing a further bug. Unfortunately the build system is
> > all but smooth compared to other packages and I finally stumbled upon
> > a C++ conversion issue[4] which is not solved by simply adding
> > #include <stdint.h>
> > and my further attempt leads to a test suite issue which seems to
> > indicate that my poor C++ understanding is wrong.
> >
> > So I'm giving up here by leaving two questions:
> >
> > 1. Anybody up for fixing this package and bringing it to latest
> > upstream?
> > 2. Alternatively do we want to drop this package from Debian?
> >
> > Kind regards
> > Andreas.
> >
> >
> > [1]
> > https://salsa.debian.org/science-team/segyio/-/tree/master/debian/patches?ref_type=heads
> > [2] https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1055728#14
> > [3]
> > https://salsa.debian.org/science-team/segyio/-/tree/version_1.9.12?ref_type=heads
> > [4]
> > https://salsa.debian.org/science-team/segyio/-/blob/version_1.9.12/debian/patches/gcc-13.patch?ref_type=heads#L6-9
> >
> > --
> > http://fam-tille.de
> >
> >
--
http://fam-tille.de
