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

"<" symbol causes bad output with "uftrace dump --mermaid" #1943

Open
Nor-s opened this issue Jul 31, 2024 · 2 comments
Open

"<" symbol causes bad output with "uftrace dump --mermaid" #1943

Nor-s opened this issue Jul 31, 2024 · 2 comments

Comments

@Nor-s
Copy link

Nor-s commented Jul 31, 2024

Describe the bug

"<" symbol causes bad output when using the "flowchart": {"htmlLabels": false} mermaid config option.

To Reproduce

// helloWorld.cpp
#include <iostream>

int main()
{
    std::cout<<"Hello World";
}
$ g++ -pg helloWorld.cpp
$ uftrace record a.out
$ uftrace dump --mermaid > ./tmp.html

Screenshot

tmp.html
image

Proposal

As a temporary measure, setting "htmlLabels": true

result: "htmlLabels:true"
image
@gichoel
Copy link
Contributor

gichoel commented Jul 31, 2024

If I follow the To Reproduce step as is, I get an error saying ERROR: Can't find 'mcount' symbol in the 'a.out'. and I can't reproduce the same symptom.

To see the same symptom, I think we need to change the To reproduce step to the following, could you please change it?

$ g++ -pg helloworld.cpp
$ uftrace record a.out
$ uftrace dump --mermaid > ./tmp.html

or

$ g++ helloworld.cpp
$ uftrace -P . record a.out
$ uftrace dump --mermaid > ./tmp.html

@Nor-s
Copy link
Author

Nor-s commented Jul 31, 2024

If I follow the To Reproduce step as is, I get an error saying ERROR: Can't find 'mcount' symbol in the 'a.out'. and I can't reproduce the same symptom.

To see the same symptom, I think we need to change the To reproduce step to the following, could you please change it?

$ g++ -pg helloworld.cpp
$ uftrace record a.out
$ uftrace dump --mermaid > ./tmp.html

or

$ g++ helloworld.cpp
$ uftrace -P . record a.out
$ uftrace dump --mermaid > ./tmp.html

Thank you! I've changed it.

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

No branches or pull requests

2 participants