Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Side by Side Diff: update-copyright/run_update_copyright.sh

Issue 29459580: Issue 5250 - Add copyright update script (Closed) Base URL: https://hg.adblockplus.org/codingtools
Patch Set: Minor formatting fixes Created July 4, 2017, 3:13 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
1 #!/bin/sh
Sebastian Noack 2017/07/05 15:46:52 Is there any purpose of this wrapper script, other
Vasily Kuznetsov 2017/07/05 17:38:05 I suggested to have a shell script instead of defa
Sebastian Noack 2017/07/05 19:08:25 Why do we support these arguments anyway? I guesse
Vasily Kuznetsov 2017/07/05 19:24:45 We support the arguments so that the script would
Sebastian Noack 2017/07/07 14:48:53 The wrapper script makes the whole thing less self
Vasily Kuznetsov 2017/07/07 15:55:26 I think we've been discussing this more than the i
rosie 2017/07/07 15:55:47 I don't have a strong opinion either way. Seems li
Sebastian Noack 2017/07/07 16:05:27 Fine with me.
rosie 2017/07/07 16:44:00 Done.
3 echo 'Running python3 update_copyright.py -u https://hg.adblockplus.org/ -p ssh: //hg@hg.adblockplus.org/'
5 python3 update_copyright.py -u https://hg.adblockplus.org/ -p ssh://hg@hg.adbloc kplus.org/
7 echo

Powered by Google App Engine
This is Rietveld