On Thu, 2012-10-18 at 15:15 +0200, Richard Marko wrote:
> fixes rhbz#864324
> 
> Signed-off-by: Richard Marko <[email protected]>
> ---
>  src/hooks/abrt_exception_handler.py.in | 21 +++++++++------------
>  1 file changed, 9 insertions(+), 12 deletions(-)
> 
> diff --git a/src/hooks/abrt_exception_handler.py.in 
> b/src/hooks/abrt_exception_handler.py.in
> index 91a9909..24877bf 100644
> --- a/src/hooks/abrt_exception_handler.py.in
> +++ b/src/hooks/abrt_exception_handler.py.in
> @@ -26,20 +26,12 @@ import sys
>  import os
>  import inspect
>  
> -try:
> -    import rpm
> -    HAVE_RPM = True
> -except ImportError as imperr:
> -    HAVE_RPM = False
> -    import syslog
> -    syslog.syslog("RPM module not available, cannot query RPM db for package 
> "\
> -                  "names")
> -
>  class RPMinfoError(Exception):
>      """Exception class for RPMdb-querying related errors"""
>      pass
>  
>  def write_dump(tb_text, tb):
> +
Why this empty line? Otherwise it looks good to me. Importing rpm really
means crazy things to happen.

-- 
Vratislav Podzimek

Anaconda Rider | Red Hat, Inc. | Brno - Czech Republic

Reply via email to