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 support for the aura helper #74

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

willtrnr
Copy link

@willtrnr willtrnr commented Jun 9, 2022

This is adding aura as a helper option.

However, this helper is a bit different. It works in reverse, meaning that it must be run as root and it drops privileges to run makepkg. This removes the need for the builder user and is pretty nice for Ansible use.

Also, it either uses only the AUR or only the repos for the requested packages (dependencies can be from both), there's no in between. Since this module is focused on AUR packages, I set it to use that (the -A flag instead of -S) and the aur_only option has no effect.

Feedback welcome if there's anything.

@kewlfft
Copy link
Owner

kewlfft commented Jul 8, 2023

I am a bit skeptical with all the specificities you mention. Do you confirm your request is still relevant and is working with current aura version?

@willtrnr
Copy link
Author

willtrnr commented Jul 9, 2023

It's been quite a while, I personally no longer have the specific use-case I had for an AUR helper.
However, I guess it could be useful to some other users out there.
IMO, not having to create a builder user provides much better ergonomics in an Ansible context.

And, yes, this will still work on a current aura version.

@kewlfft
Copy link
Owner

kewlfft commented Jul 9, 2023

It's adding some logic I won't have time to test, and you're not going to use it either. Thanks for the commit, I'll wait for one more user to express its interest here to add it.

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.

2 participants