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

suspicious-package: convert to on_system blocks #137611

Merged
merged 2 commits into from
Dec 13, 2022

Conversation

miccal
Copy link
Contributor

@miccal miccal commented Dec 13, 2022

No description provided.

@bevanjkay
Copy link
Member

@miccal @p-linnane In these instances I think we have two options with livecheck. We could keep the livecheck block as a separate instance in each on_{system} block, or we could move it to where livecheck usually is and just split it in half. It adds two additional on_{system} blocks, but it reduces the repeated livecheck skip blocks.

@p-linnane
Copy link
Member

@bevanjkay @miccal I am a proponent of keeping everything arch/OS specific in the top on_system blocks, and all shared lines below those blocks. I think it's cleaner because it lets you see what is arch/OS specific in one spot. I'm open to the alternative, my main concern is that we make sure we're consistent with whatever we choose.

One thing that helped me make up my mind was a comment from @Rylan12 earlier:

The point of on_system is simply to apply certain stanzas on certain os versions, not to determine which versions are supported.

@p-linnane p-linnane merged commit 4528938 into master Dec 13, 2022
@p-linnane p-linnane deleted the miccal-suspicious-package branch December 13, 2022 03:19
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