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

fix: use product.applicationName on darwin builds #215498

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

inclyc
Copy link

@inclyc inclyc commented Jun 14, 2024

product.applicationName, by default in OSS version is code-oss, We may want to keep this naming rule to be consistent with Linux oss versions.

`product.applicationName`, by default in OSS version is `code-oss`,
We may want to keep this naming rule to be consistent with Linux oss versions.
@inclyc
Copy link
Author

inclyc commented Jun 14, 2024

@microsoft-github-policy-service agree

@bpasero
Copy link
Member

bpasero commented Jun 14, 2024

Why?

@bpasero bpasero added the info-needed Issue requires more information from poster label Jun 14, 2024
@inclyc
Copy link
Author

inclyc commented Jun 16, 2024

Hi @bpasero

Why?

Because for Linux builds we currently put the "code.sh" into 'bin/' + product.applicationName. However on darwin, the name is still code for Code - OSS, this might be confusing for users on macOS with "Visual Studio Code" installed. (I suppose the name code is specific to that release right?)
The PR makes the CLI executable name be consistent with Linux version.

https://github.com/microsoft/vscode/blob/b526328f9e34109ab3582199bfbde0232606e376/build/gulpfile.vscode.js#L389C18-L389C50

@bpasero
Copy link
Member

bpasero commented Jun 19, 2024

I am not sure what this may break tbh. Maybe @joaomoreno has an idea about the impact?

Still, this PR does not address an actual issue, so I am not sure why we should have it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
info-needed Issue requires more information from poster
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants