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

Add Arch Linux #838

Merged
merged 1 commit into from
May 11, 2024
Merged

Add Arch Linux #838

merged 1 commit into from
May 11, 2024

Conversation

taotieren
Copy link
Contributor

No description provided.

Copy link

codecov bot commented Mar 23, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 57.97%. Comparing base (4f93acb) to head (df3f5f8).
Report is 10 commits behind head on main.

❗ Current head df3f5f8 differs from pull request most recent head 6836c37. Consider uploading reports for the commit 6836c37 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #838      +/-   ##
==========================================
- Coverage   57.98%   57.97%   -0.01%     
==========================================
  Files          75       75              
  Lines       10560    10560              
==========================================
- Hits         6123     6122       -1     
- Misses       4437     4438       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@rnijveld
Copy link
Collaborator

I'm not in the Arch ecosystem myself, is yay enough of a default that we can assume Arch Linux users will always use it and know how to install it?

@taotieren
Copy link
Contributor Author

I'm not in the Arch ecosystem myself, is yay enough of a default that we can assume Arch Linux users will always use it and know how to install it?我自己不在 Arch 生态系统中,我们是否可以假设 Arch Linux 用户会一直使用它并知道如何安装它?

The packet address of the AUR for sudo-rs is: https://aur.archlinux.org/packages/sudo-rs
arch users can install via AUR, generally speaking, arch users will understand where to install from when they see this command line, if it is not clear, add an AUR connection address.

@pvdrz
Copy link
Collaborator

pvdrz commented Mar 26, 2024

I'm not in the Arch ecosystem myself, is yay enough of a default that we can assume Arch Linux users will always use it and know how to install it?

No, it isn't.

However, the important point here is that sudo-rs is available in the arch user repos (AUR), which is how community packages that aren't part of the official repos are installed. Just like how Fedora has Copr and OpenSuse has OBS. The only difference is that there's no default tool to interact with the AUR

I think a clearer way to put this would be to say that there's a sudo-rs package in the AUR, add a link to https://aur.archlinux.org/packages/sudo-rs and probably a link to https://wiki.archlinux.org/title/AUR_helpers as well.

@taotieren
Copy link
Contributor Author

@pvdrz Thanks for the advice. I'll submit the new changes.

@pvdrz pvdrz enabled auto-merge April 24, 2024 13:24
@pvdrz
Copy link
Collaborator

pvdrz commented May 9, 2024

@taotieren could you rebase this PR so CI passes and we can merge it? Thanks 👏

auto-merge was automatically disabled May 10, 2024 09:35

Head branch was pushed to by a user without write access

@taotieren taotieren force-pushed the arch branch 4 times, most recently from 9882e6a to dd53f53 Compare May 10, 2024 10:29
@squell
Copy link
Member

squell commented May 10, 2024

@taotieren I can lend a hand in fixing the CI updates; maybe it's good if we separate the "fix the CI" (which is not your fault that it is complaining) from the "add Arch Linux" stuff.

@squell
Copy link
Member

squell commented May 10, 2024

CC PR #844. If that's merged, rebasing this on main should do the trick. I've included your CI-related commit there.

@squell squell added documentation Improvements or additions to documentation minor minor issue, PR without an issue labels May 10, 2024
@taotieren
Copy link
Contributor Author

@taotieren I can lend a hand in fixing the CI updates; maybe it's good if we separate the "fix the CI" (which is not your fault that it is complaining) from the "add Arch Linux" stuff.

Thanks!

@pvdrz pvdrz added this pull request to the merge queue May 11, 2024
Merged via the queue into trifectatechfoundation:main with commit 90e2385 May 11, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation minor minor issue, PR without an issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants