Re: [PATCH 2/2] Log a little more info if an AuthError is catched

[Till Maas]

On Mon, Nov 03, 2014 at 06:00:25PM +0100, Pierre-Yves Chibon wrote:
>  ---
>   scripts/process-git-requests/process-git-requests | 1 +
>   1 file changed, 1 insertion(+)
>  
>  diff --git a/scripts/process-git-requests/process-git-requests
> b/scripts/process-git-requests/process-git-requests
>  index 6c41b4b..4f14a5b 100755
>  --- a/scripts/process-git-requests/process-git-requests
>  +++ b/scripts/process-git-requests/process-git-requests
>  @@ -189,6 +189,7 @@ def add_package(pkgdb, request, bug):
>                       'Approved',
>                       i)
>           except AuthError, e:
>  +            logging.debug('It says AuthError: %s' % e) 
Just a small remark: It is more efficient to write

|              logging.debug('It says AuthError: %s', e)

because then the "%" operator is not used when debug messages are not
printed.

Regards
Till