VerySleepy / verysleepy

Very Sleepy, a sampling CPU profiler for Windows
http://www.codersnotes.com/sleepy
GNU General Public License v2.0
1.07k stars 104 forks source link

+ hdpi fixes #74

Closed djdron closed 4 years ago

djdron commented 4 years ago

note : you need to upgrade wxwidgets to latest master to use HDPI features

CyberShadow commented 4 years ago

Hi and thank you for your contribution.

Unfortunately, as I don't use Windows (or Very Sleepy), I can only accept a contribution if it is complete and passes the CI. So, I need to ask you that any changes required to upgrade wxWidgets are added to this pull request before it can be merged.

Could you also please update the commit messages (here and in the other PRs) to be consistent with the format already used in this project?

djdron commented 4 years ago

Seems build is OK now, but tests failed ?

CyberShadow commented 4 years ago

"Build execution time has reached the maximum allowed time for your plan (60 minutes)."

It looks like it got stuck on execution. Possibly it is showing a pop-up or error message.

Perhaps the cause can be determined by connecting via RDP: https://www.appveyor.com/docs/how-to/rdp-to-build-worker/

djdron commented 4 years ago

I need to setup appveyor to my fork ?

CyberShadow commented 4 years ago

I'm not 100% sure if it will work, but you could try to temporarily edit appveyor.yml in this pull request.

djdron commented 4 years ago

image It got stuck on this in some tests. I killed Sleepy process and tests continue.

djdron commented 4 years ago

reproduced this on local machine, trying to fix

djdron commented 4 years ago

Seems to work ok now) What did you mean to do with commit messages? remove "+" sign ?

CyberShadow commented 4 years ago

Thank you very much! This is fantastic work.

What did you mean to do with commit messages? remove "+" sign ?

Up to you if you want to do this and otherwise clean up history; alternatively I can squash everything into a single commit.

If you want to clean up history, what I would do is:

Up to you if this is something that you want to do.

djdron commented 4 years ago

Thank you very much! This is fantastic work.

You're welcome) Thanks for this beautiful profiler.

Up to you if you want to do this and otherwise clean up history; alternatively I can squash everything into a single commit.

Please do it, because i don't know how to re-arrange & combine pushed commits. Do i need to create new branch from master, cherry-pick them, combine if necessary and create new PR ? Or i need to push --force ?

See e.g. https://chris.beams.io/posts/git-commit/ for details.

Thanks for info.

Up to you if this is something that you want to do.

I don't really want