cutter icon indicating copy to clipboard operation
cutter copied to clipboard

Add +plugins suffix to package name to prevent confusion what is incl…

Open Rot127 opened this issue 5 months ago • 7 comments

…uded.

Your checklist for this pull request

Detailed description

Test plan (required)

Closing issues

Rot127 avatar Jul 06 '25 19:07 Rot127

Related https://github.com/rizinorg/cutter/issues/3507

karliss avatar Jul 06 '25 20:07 karliss

I am starting to get some doubts about renaming. Changing the windows .zip name is fine, but on linux it's the name of the ~executable app. Not sure about macOS whether it will change only .dmg which would be fine or the .app as well.

If only .dmg but not .app can be renamed, I guess I am fine with ugly appimage name.

karliss avatar Jul 06 '25 20:07 karliss

image

karliss avatar Jul 06 '25 20:07 karliss

Made your changes. Test release so we can check it: https://github.com/Rot127/cutter/releases/tag/test-release-pkg-plugins

Rot127 avatar Jul 07 '25 12:07 Rot127

@karliss

If only .dmg but not .app can be renamed, I guess I am fine with ugly appimage name.

This would be inconsistent though. People might come to the release page and wonder if only the MacOS version contains all plugins. If you don't want to change the executable file names (totally fine), we can also go with a note on the release page, the docs and webpage alone. Basically like the note in the README.

Rot127 avatar Jul 21 '25 10:07 Rot127

People might come to the release page and wonder if only the MacOS version contains all plugins.

Where did that come from? By .app I am referring to the macos app within the dmg not the appimage. And I already told that I am fine with ugly appimage name as long as for macos and windows it only changes external package name not the executable within. It was kind of long and ugly already before the changes.

karliss avatar Jul 21 '25 12:07 karliss

Ah sorry. I misread you. I'll change the names again and see if the executables actually change as well.

Rot127 avatar Jul 21 '25 12:07 Rot127