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

CFBundleName should be changed #31

Closed
core-code opened this issue Mar 31, 2021 · 10 comments
Closed

CFBundleName should be changed #31

core-code opened this issue Mar 31, 2021 · 10 comments

Comments

@core-code
Copy link

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

@deemru
Copy link
Owner

deemru commented Apr 1, 2021

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

@core-code
Copy link
Author

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
Copy link
Owner

deemru commented Apr 1, 2021

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
Copy link
Author

awesome thanks

deemru added a commit that referenced this issue Apr 14, 2021
@deemru
Copy link
Owner

deemru commented Apr 14, 2021

@core-code
Copy link
Author

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 added a commit that referenced this issue Apr 14, 2021
@deemru
Copy link
Owner

deemru commented Apr 14, 2021

@core-code
Copy link
Author

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

@core-code
Copy link
Author

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")

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

No branches or pull requests

2 participants