Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Breakage with Binutils 2.35 (or maybe 2.34?) #236

Closed
mhatta opened this issue Aug 16, 2020 · 3 comments · Fixed by #240
Closed

Breakage with Binutils 2.35 (or maybe 2.34?) #236

mhatta opened this issue Aug 16, 2020 · 3 comments · Fixed by #240

Comments

@mhatta
Copy link
Contributor

mhatta commented Aug 16, 2020

Looks like #ifdef bfd_section_flags trick in opencog/util/backtrace-symbols.c doesn't work anymore with Binutils 2.35 (or did it work with 2.34?)

#ifndef bfd_get_section_flags works, but I'm not sure it's the way to go. If it's okay, I can provide a PR.

@linas
Copy link
Member

linas commented Aug 16, 2020

Provide a pull req.

I recently saw some generic library that does stack traces, .. I've already forgotten it's name and can't find it instantly .. the long term solution might be to port to that.

Stack-tracing is a debugging tool, it mostly does not affect ordinary users, and I use it only as a prelim to taking a closer look with gdb ... anyway, crashes are rare...

@amebel
Copy link
Contributor

amebel commented Oct 5, 2020

@linas I tested #237 with binutils 2.34 (on ubuntu 20.04) and 2.30 (on ubuntu 18.04) and this repo builds and tests passes. What failures are you referring to @ #237 (comment) ?

@linas
Copy link
Member

linas commented Oct 6, 2020

Hi @amebel -- there seem to be two incompatible versions of binutils, one version has an extra argument in a handful of calls; the other version does not. The current code tries to handle both versions with some #ifdef magic. The problem with #237 is that it defeats the #ifdef by making both version work the same way. That's why it's "obviously wrong".

Oh, and I am on Debian stable, and #237 breaks that ... I have GNU assembler (GNU Binutils for Debian) 2.31.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants