[asterisk-dev] [Code Review] 2914: Add --non-interactive flag to svnmerge so we don't have to set it in global options

Matt Jordan reviewboard at asterisk.org
Sun Oct 27 13:37:13 CDT 2013



> On Oct. 22, 2013, 2:52 p.m., Mark Michelson wrote:
> > Very smart idea. I'm surprised no one thought to do this sooner.
> 
> wdoekes wrote:
>     Thanks.
>     
>     Apparently I lack permissions to commit:
>     
>     $ cat tmp 
>     svnmerge: Add --non-interactive flag to all svn calls.
>     
>     This way, you won't have to set non-interactive in your global config.
>     
>     Review: https://reviewboard.asterisk.org/r/2914/
>     
>     $ svn commit -F tmp
>     svn: E175013: Commit failed (details follow):
>     svn: E175013: Access to '/svn/repotools/!svn/act/96882501-6fdd-435c-85ee-d10036302fe9' forbidden
>     
>     
>     Perhaps you can do the honours?

I've gone ahead and committed this, but the permission issue should now be fixed as well.


- Matt


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/2914/#review9973
-----------------------------------------------------------


On Oct. 16, 2013, 12:23 p.m., wdoekes wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/2914/
> -----------------------------------------------------------
> 
> (Updated Oct. 16, 2013, 12:23 p.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Repository: Repotools
> 
> 
> Description
> -------
> 
> Hi,
> 
> I recently deleted my .dotfiles (including the svn non-interactive setting). Now I was waiting for a stuck svnmerge.
> 
> This s/svn /svn --non-interactive / replacement should fix that in the future.
> 
> 
> Diffs
> -----
> 
>   /mergetool 858 
>   /svnmerge 858 
> 
> Diff: https://reviewboard.asterisk.org/r/2914/diff/
> 
> 
> Testing
> -------
> 
> Before the patch, merge1112 was stuck waiting for input.
> 
> After the patch, it proceeded as expected.
> 
> 
> Thanks,
> 
> wdoekes
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20131027/f3003a39/attachment.html>


More information about the asterisk-dev mailing list