Skip to content

Update changing application icons instructions for Windows#12150

Merged
mhilbrunner merged 1 commit into
godotengine:masterfrom
jklpo360:patch-1
Jul 20, 2026
Merged

Update changing application icons instructions for Windows#12150
mhilbrunner merged 1 commit into
godotengine:masterfrom
jklpo360:patch-1

Conversation

@jklpo360

@jklpo360 jklpo360 commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

I have changed the ImageMagick command to use magick instead of magick convert. Using magick convert, as directed by this article, gave me a depreciation message which read:

WARNING: The convert command is deprecated in IMv7, use "magick" instead of "convert" or "magick convert"

I looked around, and using magick convert used to have the behavior of running this command with backwards compatibility for the IMv6 way of formatting. The formatting for this command didn't change, so it would be more correct to just use magick. Additionally, this backwards compatibility was removed at some point and magick convert no longer executes and instead just gives the warning detailed above.

…tengine#12149

I have changed the ImageMagick command to use ``magick`` instead of ``magick convert``. Using ``magick convert``, as directed by this article gave me a depreciation message which read: "WARNING: The convert command is deprecated in IMv7, use "magick" instead of "convert" or "magick convert"". I looked around, and using ``magick convert`` used to run this command with backwards compatibility for the IMv6 way of formatting. The formatting for this command didn't change, so it would be more correct to just use ``magick``. Additionally, this backwards compatibility was removed at some point and ``magick convert`` no longer executes and instead just gives the warning detailed above.
@jklpo360
jklpo360 requested review from a team as code owners July 16, 2026 00:32
@Mickeon Mickeon added enhancement area:manual Issues and PRs related to the Manual/Tutorials section of the documentation topic:export labels Jul 16, 2026

@Calinou Calinou left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good to me.

Note when merging: Use Squash and merge and reword the commit message to remove the reference to the original issue, as it creates spammy references when people merge it into their forks.

@AThousandShips AThousandShips changed the title Update changing_application_icon_for_windows.rst, which resolves #12149 Update changing application icons instructions for Windows Jul 17, 2026
@mhilbrunner
mhilbrunner merged commit c8f1230 into godotengine:master Jul 20, 2026
1 check passed
@mhilbrunner

mhilbrunner commented Jul 20, 2026

Copy link
Copy Markdown
Member

Merged. Thanks and congrats on your first merged contribution! And well spotted Calinou.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:manual Issues and PRs related to the Manual/Tutorials section of the documentation enhancement topic:export

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update ImageMagick usage instructions

4 participants