Re: [csw-devel] [PATCH] Add support for reading password from a file by default

2011-04-22 Thread Ben Walton
Excerpts from Maciej Bliziński's message of Fri Apr 22 19:02:36 -0400 2011: > LGTM Ok, here's a pull request then since this branch isn't yet public. The http-auth branch in ~bwalton/opencsw-git contains the commit. Thanks -Ben -- Ben Walton Systems Programmer - CHASS University of Toronto C:416

Re: [csw-devel] [PATCH] Add support for reading password from a file by default

2011-04-22 Thread Maciej Bliziński
2011/4/22 Ben Walton : > Before asking the user for a password to submit their package, look > for it in /etc/opt/csw/releases/auth/$username.  Fall back to getpass > on if there is any IOError while reading from the file. > > Signed-off-by: Ben Walton > --- >  gar/v2/lib/python/csw_upload_pkg.py

[csw-devel] SF.net SVN: gar:[14384] csw/mgar/pkg/bacula/trunk/Makefile

2011-04-22 Thread bdwalton
Revision: 14384 http://gar.svn.sourceforge.net/gar/?rev=14384&view=rev Author: bdwalton Date: 2011-04-22 20:03:44 + (Fri, 22 Apr 2011) Log Message: --- bacula/trunk: add legacy config migrations Modified Paths: -- csw/mgar/pkg/bacula/trunk/Makefile Mod

[csw-devel] SF.net SVN: gar:[14383] csw/mgar/gar/v2/gar.mk

2011-04-22 Thread bdwalton
Revision: 14383 http://gar.svn.sourceforge.net/gar/?rev=14383&view=rev Author: bdwalton Date: 2011-04-22 19:46:49 + (Fri, 22 Apr 2011) Log Message: --- gar/v2: hide echo for the banner merge command Modified Paths: -- csw/mgar/gar/v2/gar.mk Modified: c

[csw-devel] SF.net SVN: gar:[14382] csw/mgar/pkg/bacula/trunk/files

2011-04-22 Thread bdwalton
Revision: 14382 http://gar.svn.sourceforge.net/gar/?rev=14382&view=rev Author: bdwalton Date: 2011-04-22 19:02:28 + (Fri, 22 Apr 2011) Log Message: --- bacula/trunk: drop the legacy pre/post scripts as the functionality is wholly moved to CAS now Removed Paths: --

[csw-devel] SF.net SVN: gar:[14381] csw/mgar/pkg/php5/trunk/Makefile

2011-04-22 Thread bdwalton
Revision: 14381 http://gar.svn.sourceforge.net/gar/?rev=14381&view=rev Author: bdwalton Date: 2011-04-22 18:07:11 + (Fri, 22 Apr 2011) Log Message: --- php5/trunk: CSWpostgresql is a required build dep Modified Paths: -- csw/mgar/pkg/php5/trunk/Makefile

[csw-devel] [PATCH] Add support for reading password from a file by default

2011-04-22 Thread Ben Walton
Before asking the user for a password to submit their package, look for it in /etc/opt/csw/releases/auth/$username. Fall back to getpass on if there is any IOError while reading from the file. Signed-off-by: Ben Walton --- gar/v2/lib/python/csw_upload_pkg.py | 11 ++- 1 files changed,

Re: [csw-devel] [PATCH] Add support for reading password from a file by default

2011-04-22 Thread Maciej Bliziński
2011/4/22 Ben Walton : > Excerpts from Ben Walton's message of Fri Apr 22 08:10:23 -0400 2011: > >> +    password = getpass.getpass("{0}'s pkg release password> >> ".format(username)) > > FYI, this bug in python is annoying. :( > > http://bugs.python.org/issue11236 Yes, annoying indeed. There we

[csw-devel] SF.net SVN: gar:[14380] csw/mgar/pkg/php5/trunk/Makefile

2011-04-22 Thread bdwalton
Revision: 14380 http://gar.svn.sourceforge.net/gar/?rev=14380&view=rev Author: bdwalton Date: 2011-04-22 12:57:48 + (Fri, 22 Apr 2011) Log Message: --- php5/trunk: update ini file handling to use preserveconf (still need to migrate this) Modified Paths: --

[csw-devel] SF.net SVN: gar:[14379] csw/mgar/pkg/php5/trunk/Makefile

2011-04-22 Thread bdwalton
Revision: 14379 http://gar.svn.sourceforge.net/gar/?rev=14379&view=rev Author: bdwalton Date: 2011-04-22 12:50:12 + (Fri, 22 Apr 2011) Log Message: --- php5/trunk: update sqlite library dep Modified Paths: -- csw/mgar/pkg/php5/trunk/Makefile Modified:

[csw-devel] SF.net SVN: gar:[14378] csw/mgar/pkg/php5/trunk/Makefile

2011-04-22 Thread bdwalton
Revision: 14378 http://gar.svn.sourceforge.net/gar/?rev=14378&view=rev Author: bdwalton Date: 2011-04-22 12:47:38 + (Fri, 22 Apr 2011) Log Message: --- php5/trunk: set merge rules for the modulated build Modified Paths: -- csw/mgar/pkg/php5/trunk/Makefi

Re: [csw-devel] [PATCH] Add support for reading password from a file by default

2011-04-22 Thread Ben Walton
Excerpts from Ben Walton's message of Fri Apr 22 08:10:23 -0400 2011: > +password = getpass.getpass("{0}'s pkg release password> > ".format(username)) FYI, this bug in python is annoying. :( http://bugs.python.org/issue11236 Thanks -Ben -- Ben Walton Systems Programmer - CHASS University o

Re: [csw-devel] [PATCH] Add support for reading password from a file by default

2011-04-22 Thread Maciej Bliziński
2011/4/22 Ben Walton : >> To get the password, you can: >> >> password = af.read() # reads the whole file >> >> and with stripping: >> >> password = af.read().strip() >> >> (strips ws from both sides) > > Do we want that though?  It shouldn't hurt, but it also shouldn't be > necessary.  I'm only lo

[csw-devel] [PATCH] Add support for reading password from a file by default

2011-04-22 Thread Ben Walton
Before asking the user for a password to submit their package, look for it in /etc/opt/csw/releases/auth/$username. Fall back to getpass on any exception. Signed-off-by: Ben Walton --- gar/v2/lib/python/csw_upload_pkg.py | 11 ++- 1 files changed, 10 insertions(+), 1 deletions(-) dif

Re: [csw-devel] [PATCH] Add support for reading password from a file by default

2011-04-22 Thread Ben Walton
Excerpts from Maciej Bliziński's message of Fri Apr 22 02:03:34 -0400 2011: > 2011/4/22 Ben Walton : > > +  try: > > +    af = open(authfile, 'r') > > +    password = af.readline().rstrip() > > Try using the 'with' syntax: > > try: > with open(authfile, 'r') as af: > password = af... Ah, o