Re: [PATCH] dumper: allow compiling with binutils 2.47
Jon Turney <[email protected]> Mon, 3 Aug 2026 15:04:55 +0100
| Newsgroups | gmane.os.cygwin.patches |
|---|---|
| Message-ID | <[email protected]> |
On 28/07/2026 11:39, Johannes Schindelin via GitGitGadget wrote: > From: Johannes Schindelin <[email protected]> > > In the latest version, binutils' BFD no longer declares a `bfd_boolean`, > and the `bfd_record_phdr()` function accepts plain `bool` parameters > instead. > > Let's accommodate for that to avoid these build failures: The obvious question I would ask is "does it still build with older bfd", to which the answer seems to be "yes". So, applied. Thanks! > > .../winsup/utils/dumper.cc: In member function 'int dumper::prepare_core_dump()': > .../winsup/utils/dumper.cc:830:7: error: 'bfd_boolean' was not declared in this scope; did you mean 'boolean'? > 830 | bfd_boolean filehdr = 0; > | ^~~~~~~~~~~ > | boolean > .../winsup/utils/dumper.cc:831:18: error: expected ';' before 'phdrs' > 831 | bfd_boolean phdrs = 0; > | ^~~~~~ > | ; > [...] > > Signed-off-by: Johannes Schindelin <[email protected]> > --- > dumper: allow compiling with binutils 2.47 > [...]