[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: Segyio is lagging behind upstream (Was: segyio ftbfs with Python 3.12)



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 <andreas@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


Reply to: