Skip to content

stdout: vector of delegates, elf: custom section for symbols#835

Merged
fwsGonzo merged 22 commits intoincludeos:devfrom
fwsGonzo:dev
Oct 4, 2016
Merged

stdout: vector of delegates, elf: custom section for symbols#835
fwsGonzo merged 22 commits intoincludeos:devfrom
fwsGonzo:dev

Conversation

@fwsGonzo
Copy link
Copy Markdown
Member

... adds support for backtrace and stack sampling on multiboot, including on GCE (I believe)

Copy link
Copy Markdown
Contributor

@alfreb alfreb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Big one thanks. I'm a bit concerned about the weak symbol solution for the default print handlers; if injectables decide to use this it will effectively override any print handlers set by the service. But still, this is much better than what we have now.

@alfreb
Copy link
Copy Markdown
Contributor

alfreb commented Sep 29, 2016

@mnordsletten @fwsGonzo looks like the test fails due to objcopy being the wrong version or something. Any quick fix in sight?

@mnordsletten
Copy link
Copy Markdown
Contributor

@alfred-bratterud I was planning on changing all the pull requests machines to Ubuntu 16.04 today, I will see if that fixes this specific issue.

@mnordsletten
Copy link
Copy Markdown
Contributor

Ok to test

@mnordsletten
Copy link
Copy Markdown
Contributor

The network test failed due to the dhcp test failing. I'll just run it again here.
Ok to test

Comment thread .gitignore
*.out
*.log
\#*
\*.*
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Atom behaves strangely when these exist in any gitignores, so we can't use them until fixed

@fwsGonzo
Copy link
Copy Markdown
Member Author

fwsGonzo commented Oct 4, 2016

test please

@fwsGonzo fwsGonzo merged commit 4887518 into includeos:dev Oct 4, 2016
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 this pull request may close these issues.

4 participants