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

fix(add repos): Fixed sources_list_dir cleaning #28

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

Conversation

spuzirev
Copy link

Hello!

There's problem when you adding repos and use clean_sources_list_d: true they're applied every time.
I've fixed it here. This fix to be honest must be done in salt sources because pkgrepo.managed doesn't care about files it creates. But this fix make this formula working fine.

@ryanwalder
Copy link

RE: Fixed sources_list_dir cleaning #26605 raised a while ago.

Copy link

@Kassec Kassec left a comment

Choose a reason for hiding this comment

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

Line 37 : why are you supposing url is http ? We may have some keys distributed through a salt:// url.

@aboe76 aboe76 changed the title Fixed sources_list_dir cleaning fix(add repos) Fixed sources_list_dir cleaning Aug 10, 2019
@aboe76 aboe76 changed the title fix(add repos) Fixed sources_list_dir cleaning fix(add repos): Fixed sources_list_dir cleaning Aug 10, 2019
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