Feature #368
closed
- Assignee changed from Amber Herold to Gabriel Lander
Gabe,
I only looked at this briefly, but this is scary to me:
while proc.poll() is None:
sys.stderr.write(".")
time.sleep(waittime)
logf.flush()
proc.wait()
I suppose when threading is implemented that will go away?
Also on lines 47 and 153 of signaturePicker.py there seems to be a superfluous i in the for loops, or did I miss something?
I think that for the release, we will most likely hide the signature tool with the config file parameter HIDE_FEATURE since we have not had a chance to test it and don't have it installed here. To enable it, just set HIDE_FEATURE to true in config.php under myamiweb.
- Hi Amber,
sorry, I copied this code directly from "apFindEM.py", feel free to fix it however you see necessary.
oh, and the "i" you see in signaturePicker.py is indeed superfluous, again copied from code somewhere else. You can take it out.
- Assignee changed from Gabriel Lander to Amber Herold
- Status changed from In Code Review to Closed
Also available in: Atom
PDF