Bug 16085

Summary: system-config-printer traceback when adding appsock/jetdirect printer
Product: Mageia Reporter: David Walser <luigiwalser>
Component: RPM PackagesAssignee: Philippe Makowski <makowski.mageia>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: Normal    
Version: Cauldron   
Target Milestone: ---   
Hardware: i586   
OS: Linux   
Whiteboard:
Source RPM: system-config-printer-1.5.5-3.mga5.src.rpm CVE:
Status comment:
Attachments: signal only works in main thread

Description David Walser 2015-06-06 01:12:44 CEST
When adding a new printer and clicking AppSock/JetDirect and then putting in the IP address and clicking Forward, there was a spinning cursor and nothing happened in the GUI, but I got this traceback in the terminal:

Traceback (most recent call last):
  File "/usr/share/system-config-printer/newprinter.py", line 1611, in
  opreq_id_search_error
    self.opreq_id_search_done (opreq, list(), dict())
  File "/usr/share/system-config-printer/newprinter.py", line 1582, in
  opreq_id_search_done
    self.nextNPTab ()
  File "/usr/share/system-config-printer/newprinter.py", line 1172, in
  nextNPTab
    if not mga_printer_custom.is_installed_packages(driver_packages):
  File "/usr/share/system-config-printer/mga_printer_custom.py", line
  41, in is_installed_packages
    signal.signal (signal.SIGCHLD, signal.SIG_DFL)
ValueError: signal only works in main thread


Clicking Forward again allowed it to proceed and work correctly.

Reproducible: 

Steps to Reproduce:
David Walser 2015-06-06 01:12:53 CEST

Whiteboard: (none) => MGA5TOO

Comment 1 Philippe Makowski 2015-06-08 19:08:04 CEST
Created attachment 6714 [details]
signal only works in main thread

Can you try this patch ?
It's a temporary fix, we really need to follow upstream and use packagekit
Comment 2 David Walser 2015-06-08 19:28:49 CEST
(In reply to Philippe Makowski from comment #1)
> Created attachment 6714 [details]
> signal only works in main thread
> 
> Can you try this patch ?

I can confirm that this patch fixes the issue.  Thanks.

> It's a temporary fix, we really need to follow upstream and use packagekit

I agree.
Comment 3 Philippe Makowski 2015-06-09 20:17:46 CEST
 system-config-printer-1.5.5-4.mga5 in svn http://svnweb.mageia.org/packages?view=revision&revision=822961
Comment 4 David Walser 2015-06-13 18:51:58 CEST
Fixed in system-config-printer-1.5.5-5.mga5.  Thanks Philippe!

Status: NEW => RESOLVED
Resolution: (none) => FIXED

David Walser 2015-06-13 18:52:05 CEST

Whiteboard: MGA5TOO => (none)