-
Notifications
You must be signed in to change notification settings - Fork 23.7k
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
Subversion: Only fire changed when content has changed #41036
Subversion: Only fire changed when content has changed #41036
Conversation
repository that aren't present in the subdirectory (i.e. in a different branch or similiar) then the changes handler still kicks in even though the content of this working copy hasn't necessarily changed. Now we parse the output of svn update and fire if anything changes at all.
repository that aren't present in the subdirectory (i.e. in a different branch or similiar) then the changes handler still kicks in even though the content of this working copy hasn't necessarily changed. Now we parse the output of svn update and svn switch, firing if anything changes at all.
The test
The test
The test
|
@dsummersl is there anything particular I can do to get this more attention? It should be ready to go :) |
@mintsoft I was able to test this and confirm that this fixes the issue; thanks! Also verified that shipit |
Hmm is ansibot supposed to have merged the commit in by now @dsummersl ? |
Hmm, yes. I see a couple things: this PR is labelled with I'd suggest getting that first |
@dsummersl Ahh I see, I'll have to see if I can entice a core team member to have a look. Maybe @maxamillion could do me the favour 😉 |
rebuild_merge |
SUMMARY
Previously if you checkout only a branch of part of a subversion repostitory
and another part of the repository is updated then the change handler would
fire even if the branch/subdirectory that we're interested in hasn't changed.
This now parses the output to only indicate that there has been a change when
the actual content is changed.
ISSUE TYPE
COMPONENT NAME
Subversion
ANSIBLE VERSION
Also tested under
ADDITIONAL INFORMATION