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

Remove SystemD dependency, for non-systemd systems #2

Merged
merged 1 commit into from
Jan 5, 2024

Conversation

SputnikRocket
Copy link
Contributor

Replaced SystemD dependency with udev, as systemd provides udev on systemd systems, and eudev provides udev on non-systemd systems.

@7Ji
Copy link
Contributor

7Ji commented Jan 5, 2024

@hbiyik

This looks good to me and reasonable. I'm away from home attending my cousin's ceremony and can't verify the change on my boards. Does this look good to you?

@SputnikRocket
Copy link
Contributor Author

SputnikRocket commented Jan 5, 2024

as for the environment I expect it to run in, eg. my system, is Arch Linux ARM with runit as the primary init system. Although you do not support other Arch linux ARM forks, this would also allow it to install and run on distros like Artix Linux (ARM port)

@7Ji 7Ji merged commit 5b12e9c into 7Ji-PKGBUILDs:master Jan 5, 2024
@hbiyik
Copy link

hbiyik commented Jan 5, 2024

@7Ji this is correct, i dont know why i set it to systemd in the first place.
cograts to you and your cousin btw :)

@SputnikRocket thanks for the pr.

@SputnikRocket
Copy link
Contributor Author

👍

@7Ji
Copy link
Contributor

7Ji commented Jan 10, 2024

As @hbiyik mentioned in 7Ji/archrepo#19 (comment) :

Side note: after the recent change of mpp-git from systemd to dbus, i got an error that dbus-units dependency could not be found, i assume this is some archlinux temporary issue.

Could you elaborate on further tests about the problem? @SputnikRocket

@SputnikRocket
Copy link
Contributor Author

SputnikRocket commented Jan 10, 2024

@7Ji
This does seem to be an Archlinux ARM repo and/or systemd package issue. the dbus-units package does not seem to exist on the alarm repository. I noticed this as well when I tried to upgrade an unmodified ArchLinux ARM rootfs

@SputnikRocket
Copy link
Contributor Author

@7Ji seems to have been fixed now.

This pull request was closed.
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