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

Zed binary has inconsistent naming on Linux #12306

Closed
1 task done
Pi-Cla opened this issue May 26, 2024 · 5 comments
Closed
1 task done

Zed binary has inconsistent naming on Linux #12306

Pi-Cla opened this issue May 26, 2024 · 5 comments
Labels
defect [core label] linux

Comments

@Pi-Cla
Copy link
Contributor

Pi-Cla commented May 26, 2024

Check for existing issues

  • Completed

Describe the bug / provide steps to reproduce it

There have at least been two seperate PRs #12218 and #12301 which have resulted from the documentation not saying that zed binaries on Linux are named zed rather than Zed.

Either make all binaries be zed, make the bundle-linux script and .desktop file refer to Zed, or document this difference between Linux and MacOS/Windows.

I prefer the first option the most because most programs have a lowercased binary name. Option two at least gets rid of the inconsistency and it is still preferable to option three which only acknowledges it is there.

Environment

Linux

If applicable, add mockups / screenshots to help explain present your vision of the feature

No response

If applicable, attach your ~/Library/Logs/Zed/Zed.log file to this issue.

No response

@Pi-Cla Pi-Cla added admin read Pending admin review defect [core label] triage Maintainer needs to classify the issue labels May 26, 2024
@muhdsalm
Copy link

Maybe because of #12290 we should do Option 2?

@libewa
Copy link

libewa commented May 26, 2024

Capitalized program names are – in my opinion – a bad idea, especially on Linux, where there's an actual difference between zed and Zed. We could maybe rename the binary to zed-editor like nixpkgs did for the package.

@Moshyfawn Moshyfawn added linux and removed triage Maintainer needs to classify the issue labels May 26, 2024
@dpvpro
Copy link

dpvpro commented May 27, 2024

Renaming to zed-editor better decision

@WuJune
Copy link

WuJune commented Jun 3, 2024

Renaming to zed-editor better decision

Agree, or zed-bin, both of them better than Zed in common linux situation.

@ConradIrwin
Copy link
Member

Fixed by #12622

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
defect [core label] linux
Projects
None yet
Development

No branches or pull requests

8 participants