Issue 371 in rietveld: upload.py needs config file or ...

0 views
Skip to first unread message

riet...@googlecode.com

unread,
Feb 18, 2012, 3:34:07 AM2/18/12
to coderev...@googlegroups.com
Status: New
Owner: tech...@gmail.com
Labels: Type-Enhancement Priority-Medium Component-Scripts

New issue 371 by tech...@gmail.com: upload.py needs config file or ...
http://code.google.com/p/rietveld/issues/detail?id=371

It is very tedious to enter the same values for upload.py every time in
command line like cc list or reviewers. There should be an easier way to
handle that. Either by .codereview config file or upload.py can request
info from server.

riet...@googlecode.com

unread,
Feb 18, 2012, 12:58:52 PM2/18/12
to coderev...@googlegroups.com

Comment #1 on issue 371 by gvanros...@gmail.com: upload.py needs config

Or you can write a 2-line wrapper shell script.

riet...@googlecode.com

unread,
Feb 18, 2012, 3:49:41 PM2/18/12
to coderev...@googlegroups.com

Comment #2 on issue 371 by tech...@gmail.com: upload.py needs config file
or ...
http://code.google.com/p/rietveld/issues/detail?id=371

I do checkouts more often than patches, so the shell script won't automate
much unless it is checked from the repository.

Gerrit has its own config file for some reason, and Rietveld can.

riet...@googlecode.com

unread,
Feb 18, 2012, 4:30:49 PM2/18/12
to coderev...@googlegroups.com

Comment #3 on issue 371 by gvanros...@gmail.com: upload.py needs config

Sure, I'm not objecting, but I'm not volunteering either. :-)

Should the config file be per directory or global (in the user's home
directory)? Should its name start with a dot or not?

riet...@googlecode.com

unread,
Feb 18, 2012, 5:00:56 PM2/18/12
to coderev...@googlegroups.com

Comment #4 on issue 371 by albrecht.andi: upload.py needs config file or ...
http://code.google.com/p/rietveld/issues/detail?id=371

Or should it use the configuration system provided by the VCS? I prefer not
to have yet another hidden file in my project dirs. Therefore I'm currently
using git/hg config and a small wrapper ATM.

Reply all
Reply to author
Forward
0 new messages