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

Merge 2015.5 into 2015.8 #30529

Merged
merged 8 commits into from
Jan 22, 2016
Merged

Conversation

terminalmage
Copy link
Contributor

This merge-forward brings the work done in #30484 to the 2015.8 branch, and resolves the associated merge conflicts.

terminalmage and others added 7 commits January 21, 2016 09:56
This will allow 2015.5 releases to operate successfully on Fedora 22 and
newer.
This commit adds 2015.8-specific context to the top-level docstring and
also removes yum-specific terminology from the pkg.remove function.
terminalmage added a commit to terminalmage/salt that referenced this pull request Jan 22, 2016
In the process of resolving merge conflicts in the merge-forward of
2015.5 to 2015.8 which I did in saltstack#30529, I found two changes that should
have been made in the 2015.5 branch. One was the removal of yum-specific
information in the pkg.remove docstring, and the other was needless use
of -q when deleting a version lock in pkg.unhold. We're not parsing the
output and the default loglevel is debug, so using -q here removes
helpful troubleshooting output from the log.
cachedout pushed a commit that referenced this pull request Jan 22, 2016
@cachedout
Copy link
Contributor

@terminalmage The linter isn't quite happy here.

@cachedout cachedout added the pending-changes The pull request needs additional changes before it can be merged label Jan 22, 2016
@terminalmage
Copy link
Contributor Author

Linter should be happy now.

cachedout pushed a commit that referenced this pull request Jan 22, 2016
@cachedout cachedout merged commit 1da1bb9 into saltstack:2015.8 Jan 22, 2016
@terminalmage terminalmage deleted the 2015.5-2015.8 branch January 22, 2016 17:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pending-changes The pull request needs additional changes before it can be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants