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

auto-admin: more changes for aarch64 #3

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

linimon
Copy link

@linimon linimon commented Jun 3, 2021

ACPI sysctls and dmidecode are not available on all architectures. Guard against annoying error messages.

linimon added 4 commits June 3, 2021 16:30
dmidecode is not available on every architecture.  Suppress its error messages for such cases.
aarch64 does not have the "dev.acpi_*" sysctls.  So, skip this entire script in that case.
Otherwise the script can hang.

This change replaces an earlier one, by moving the test before the OS test.
@linimon
Copy link
Author

linimon commented Jun 4, 2021

While here, ensure PORTSDIR actually exists.

@linimon linimon closed this Jun 4, 2021
@linimon linimon reopened this Jun 4, 2021
@linimon
Copy link
Author

linimon commented Jun 4, 2021

Another fight with github.

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.

1 participant