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

modified "about" window to get compiler, QT, openCV and armadillo version automatically #18

Merged
merged 2 commits into from
May 31, 2023

Conversation

atsju
Copy link
Collaborator

@atsju atsju commented Mar 17, 2023

It's not perfect as armadillo version is got dynamically at runtime while other versions are static at compile time.

Next step could be to add windows/linux version from computer it's running on.
Also adding the sha of git commit where build comes from would be great to track issues (I know very few people use their own build but this might change a bit with Linux support)

On linux:
image

With the windows build from github actions
image

@atsju
Copy link
Collaborator Author

atsju commented May 31, 2023

@gr5 I did a merge commit to avoid dismissing a potential review in progress but if you prefer I can rebase the branch to keep more linear commit history

@gr5
Copy link
Collaborator

gr5 commented May 31, 2023

I did a merge commit to

Great. Actually I can see the build is in progress now. I will build this on my own machine today just to double check everything and hope to merge this today. I'm in the middle of a bug fix right now so I'll commit that to my own personal branch first (or do a git stash) but I have errands this morning before that.

Copy link
Collaborator

@gr5 gr5 left a comment

Choose a reason for hiding this comment

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

Looks good. As an aside - note that I'm pretty sure nothing armadillo related is used (no functions, no dlls) except for an .h file that defines a few variable types. No actual armadillo functions are called. So maybe we could "fix" that situation some time.

@gr5 gr5 merged commit dec1646 into githubdoe:master May 31, 2023
@atsju atsju deleted the JST/automateVersions branch May 31, 2023 15:56
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.

2 participants