Skip to content

Conversation

silverjam
Copy link
Contributor

@silverjam silverjam commented May 31, 2022

Jira: https://swift-nav.atlassian.net/browse/CPP-767

Adds a version number to the splash screen per request from apps.

Uses ImageMagick to automate. Customer installation of ImageMagick implemented here: https://github.com/swift-nav/swift-toolchains/tree/master/imagemagick

Example build here: https://github.com/swift-nav/swift-toolbox/releases/tag/v4.0.7-splashversion


swift console - splash version

Makefile.toml Outdated
[tasks.install-imagemagick]

[tasks.install-imagemagick.windows]
script = "scoop install imagemagick"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is there a certain reason you are using scoop? It looks like imagemagick might be available via Choco which is already installed on GitHub runners:

choco install imagemagick.app

Copy link
Contributor Author

Choose a reason for hiding this comment

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

scoop doesn't require admin privs to install packages so it's a little easier when trying to run the install target on Winows, but you're correct: choco works better with CI... so I'll drop scoop

@silverjam silverjam requested a review from a team June 1, 2022 22:50
@silverjam silverjam changed the title splash screen with a version splash screen with a version [CPP-767] Jun 1, 2022
@silverjam
Copy link
Contributor Author

Note sure why there are red Xs on the last commit, but there's clean build of everything here at least: https://github.com/swift-nav/swift-toolbox/actions/runs/2424923017

@silverjam silverjam merged commit dee416e into main Jun 2, 2022
@silverjam silverjam deleted the silverjam/spash-version branch June 2, 2022 19:59
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