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

README.md: Warn about buggy snap packages #1524

Merged
merged 2 commits into from
Feb 15, 2021

Conversation

Enselic
Copy link
Collaborator

@Enselic Enselic commented Jan 22, 2021

As discussed in #1519, the batcat snap package is too problematic for
official endorsement, so withdraw recommendation from README.me (and
remove section from CHANGELOG.md since a release with it has not been
made yet).

For now this MR should just be seen as a sketch of how we could go about doing it, in case we end up choosing this path.

As discussed in sharkdp#1519, the batcat snap package is too problematic for
official endorsement, so withdraw recommendation from README.me (and
remove section from CHANGELOG.md since a release with it has not been
made yet).
Copy link
Collaborator

@eth-p eth-p left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think calling the existing packages "buggy" can come off as rude or callous.

It might be better to say "Existing packages may be available, but are not officially supported and may contain issues."

With the link on the word "issues".

@sharkdp sharkdp merged commit 70cf8a4 into sharkdp:master Feb 15, 2021
@sharkdp
Copy link
Owner

sharkdp commented Feb 15, 2021

Thanks!

@sharkdp
Copy link
Owner

sharkdp commented Feb 15, 2021

Unfortuntely, Ubuntu shows this if someone tries to install batcat (instead of bat, which is still the package name)

$ sudo apt install batcat
Reading package lists... Done
Building dependency tree       
Reading state information... Done

No apt package "batcat", but there is a snap with that name.
Try "snap install batcat"

@Enselic Enselic deleted the warn-about-buggy-snap-packages branch August 20, 2021 04:48
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

Successfully merging this pull request may close these issues.

3 participants