atosl icon indicating copy to clipboard operation
atosl copied to clipboard

fix armv7 bug on linux

Open ccarnero opened this issue 10 years ago • 8 comments

Hi, I was facing this problem on my linux box: https://github.com/facebook/atosl/issues/4

So I fixed using the following code (all credit to @hawflakes) and now is working: https://gist.github.com/hawflakes/9677613

  • I had to update the config.mk file to avoid warnings to be treat as errors

Best regards Christian

ccarnero avatar Jun 04 '14 14:06 ccarnero

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please sign up at https://code.facebook.com/cla - and if you have received this in error or have any questions, please drop us a line at [email protected]. Thanks!

facebook-github-bot avatar Jun 04 '14 14:06 facebook-github-bot

Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks!

facebook-github-bot avatar Jun 04 '14 14:06 facebook-github-bot

Cool.. I'm not crazy about the hack but I don't have a better solution at the moment. But could you clean up the patch a bit if you want it included? As in, don't just disable -Werror :)

Likely there is just some bug where we aren't properly calculating offsets in a fat binary, but I haven't had time to look into it.

zlandau avatar Jun 04 '14 16:06 zlandau

I'm not crazy about the fix, either. But it's been so long since I've looked at DWARF and this got me unstuck.

hawflakes avatar Jun 05 '14 21:06 hawflakes

Ping :-)

LegNeato avatar Sep 25 '14 20:09 LegNeato

Can you try https://github.com/facebook/atosl/pull/18 I think it may fix your issue.

zlandau avatar Sep 25 '14 20:09 zlandau

Did #18 fix your issue @hawflakes ?

LegNeato avatar Feb 19 '15 04:02 LegNeato

Thank you for reporting this issue and appreciate your patience. We've notified the core team for an update on this issue. We're looking for a response within the next 30 days or the issue may be closed.

ghost avatar Aug 04 '15 18:08 ghost