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

Support for pikaur #275

Closed
t-animal opened this issue Nov 5, 2024 · 3 comments · Fixed by #276
Closed

Support for pikaur #275

t-animal opened this issue Nov 5, 2024 · 3 comments · Fixed by #276
Assignees
Labels
feature/request New feature or request
Milestone

Comments

@t-animal
Copy link

t-animal commented Nov 5, 2024

Description

Support for pikaur as an additional aur pacman wrapper.

Motivation and context

AUR wrappers show diffs to the last installed version of the installation scripts when installing a new version. This is critical for security. When arch-update uses a different package manager than what the user uses otherwise, these updates becomes misaligned. I don't know if it is feasible to just support all wrappers that there are, but I for one use pikaur and hope that it could be added as a supported wrapper. I guess, that if they're compatible with pacman, adding them should be relatively easy.

Maybe a more generic option could also be viable. Like setting a path to a wrapper in the config file instead of naming the wrapper.

@t-animal t-animal added the feature/request New feature or request label Nov 5, 2024
@Antiz96
Copy link
Owner

Antiz96 commented Nov 5, 2024

Hi,

Thanks a lot for the feature request!

AUR wrappers show diffs to the last installed version of the installation scripts when installing a new version. This is critical for security. When arch-update uses a different package manager than what the user uses otherwise, these updates becomes misaligned.

This is a fair point! However...

I don't know if it is feasible to just support all wrappers that there are,

...it's definitely not feasible to support every AUR helpers / pacman wrapper there is. 😅

but I for one use pikaur and hope that it could be added as a supported wrapper. I guess, that if they're compatible with pacman, adding them should be relatively easy.

It's indeed important that they share options with pacman itself (to ease their support integration within Arch-Update) but, most importantly, they should also be compatible with each other in terms of options, to avoid complicating the different scripts / libraries with numerous conditionals statements.

Luckily, it seems like pikaur shares most options with yay and paru (at least the ones we are interested in for Arch-Update, at first glance).
I still need to study that deeper and make some tests before confirming if pikaur support could be added or not though.

I have a lot to do in the incoming days / weeks, so I can't promise I'll be able to do that soon, but I'll try my best!
I'll let you know 😉

@Antiz96 Antiz96 added this to the v3.4.0 milestone Nov 6, 2024
@Antiz96
Copy link
Owner

Antiz96 commented Nov 6, 2024

Turns out I got time to make a few quick tests today and adding support for pikaur should be doable!

I'm starting to work on this.
Thanks again for the feature request :)

Antiz96 added a commit that referenced this issue Nov 6, 2024
Add support for the [pikaur](https://github.com/actionless/pikaur) AUR helper (in additon of `paru` and `yay`) for AUR packages support.

Closes #275
@Antiz96 Antiz96 closed this as completed in 7b5d08b Nov 6, 2024
@Antiz96
Copy link
Owner

Antiz96 commented Nov 6, 2024

Support for pikaur added in #276 and included in the v3.4.0 release 😉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature/request New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants