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

ssu: Make sure the ssu cache is up-to-date. #3

Merged
merged 1 commit into from
Nov 23, 2021
Merged

ssu: Make sure the ssu cache is up-to-date. #3

merged 1 commit into from
Nov 23, 2021

Conversation

abranson
Copy link
Member

@abranson abranson commented Nov 22, 2021

[ssu] Make sure the ssu cache is up-to-date. Fixes JB#54650

Add RPM file triggers to:

  • Touch any new ssu config files so it's newer than the cache
  • Remove caches when relevant files are removed.

rpm/ssu.spec Show resolved Hide resolved
Copy link
Contributor

@saukko saukko left a comment

Choose a reason for hiding this comment

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

LGTM, one comment though.

[ssu] Make sure the ssu cache is up-to-date. Fixes JB#54650

Add RPM file triggers to:
- Touch any new ssu config files so it's newer than the cache
- Remove caches when relevant files are removed.
@abranson abranson merged commit d0aea62 into master Nov 23, 2021
@abranson abranson deleted the jb56434 branch November 23, 2021 12:06
@Olf0
Copy link

Olf0 commented Apr 12, 2023

Because I was analysing the changes in detail for the third time, only to forget soon after what I comprehended, I wrote a minimalistic documentation for this PR.

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