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

Build systemd for all Linuxes #5379

Merged
merged 4 commits into from
Apr 29, 2020
Merged

Conversation

albertvaka
Copy link
Contributor

What does this PR do?

Add the systemd build tag for all Linux distros.

Motivation

Most Linux distros have adopted systemd.

@albertvaka albertvaka added this to the 7.20.0 milestone Apr 21, 2020
@albertvaka albertvaka requested review from a team as code owners April 21, 2020 12:26
@albertvaka albertvaka changed the title Assume all Linux distros have systemd nowadays Build systemd for all Linuxes Apr 21, 2020
Comment on lines -96 to -116
def get_distro_exclude_tags():
"""
Get tags that should be excluded for current distro.
"""
distro_name = distro.id().lower()
exclude = []
if distro_name not in REDHAT_DEBIAN_SUSE_DIST:
exclude.extend(REDHAT_DEBIAN_SUSE_ONLY_TAGS)
return exclude


Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this means we're not using distro anymore anywhere (I only see it imported but not used in this file & the tasks/android.py file). We may be able to get rid of this dependency then.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will remove it.

@albertvaka albertvaka force-pushed the albertvaka/systemd-everywhere branch from a44668d to a377fbc Compare April 22, 2020 18:58
@albertvaka albertvaka merged commit 56d0395 into master Apr 29, 2020
@albertvaka albertvaka deleted the albertvaka/systemd-everywhere branch April 29, 2020 16:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants