deemru / Chromium-Gost

Chromium с поддержкой алгоритмов ГОСТ
https://www.cryptopro.ru/forum2/default.aspx?g=posts&t=9991
MIT License
398 stars 43 forks source link

CFBundleName should be changed #31

Closed core-code closed 3 years ago

core-code commented 3 years ago

CFBundleName in the Info.plist should be changed from "Chromium" to "Chromium-Gost"

deemru commented 3 years ago

We are not in depth of mac features, why it "should be"? Are there any cons?

core-code commented 3 years ago

well.

CFBundleName is the name of the app. if you leave it at 'Chromium' your app is indistuingishable from upstream original Chromium. you've already re-named it in the file-system so it makes sense to also adjust the bundle name.

other Chromium forks like Chedot, Epic, ipDTL, Iron RocketNav and Superbird have also adjusted their CFBundleName.

deemru commented 3 years ago

So technically (for an user) there are no real cons and renaming is needed just to make sense and be distinguishable in some mac internals.

We try to update this value in a next release. Thanks.

core-code commented 3 years ago

awesome thanks

deemru commented 3 years ago

https://github.com/deemru/chromium-gost/commit/af6e1389ed74d43335cab0737eedcf1381762597#diff-1442bb1852f322463b7846df55f396f109ac43811a75b761e426865fe0c1327aR72

done?

core-code commented 3 years ago

you are changing the CFBundleIdentifier instead of the CFBundleName. that fullfills the objective (make it distuingishable) but i cannot guarantee that there are no negative side effects in this case.

deemru commented 3 years ago

This is it? https://github.com/deemru/chromium-gost/commit/bfdcc37eb328c704ecb536f8a242892dbeed3dbb#diff-1442bb1852f322463b7846df55f396f109ac43811a75b761e426865fe0c1327aR80

core-code commented 3 years ago

looks good thanks. will give a definitive confirmation when the next build is out

deemru commented 3 years ago

Next build: https://github.com/deemru/chromium-gost/releases/download/90.0.4430.72/chromium-gost-90.0.4430.72-macos-amd64.tar.bz2

core-code commented 3 years ago

thanks, i had a look. i can confirm that the CFBundleName has been changed, which is what the bug report was about ("Chromium" => "Chromium-Gost")

i can also confirm that the build has a changed CFBundleIdentifier ("org.chromium.Chromium" => "ru.cryptopro.chromium-gost")