keeleysam / instadmg

Automatically exported from code.google.com/p/instadmg
0 stars 0 forks source link

Global name 'e' is not defined #51

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Execute instaup2date with the --process option.

What is the expected output? What do you see instead?
InstaUp2Date should execute, then run an instance of InstaDMG. Following 
execution, InstaUp2Date should clean up temp folders. At the cleanup step, an 
error is thrown: "Unable to process the folder: 
"/private/tmp/InstaDMGTempFolder.7oced-" got error: global name 'e' is not 
defined"

What version of InstaDMG are you running? What revision number (this should
be the second line of your package log)?
InstaDMG r318 (InstaUp2Date r322)

What OS are you running on (booted from)?
10.5.8

What OS are you installing?
10.5.8

Please provide any additional information below.
Error is likely found on line 107 of tempFolderManager.py; suspect the line 
should read "except Exception as e:" instead of as it currently reads, "except 
Exception, e:"

Original issue reported on code.google.com by loadsta...@gmail.com on 1 Aug 2010 at 9:17

GoogleCodeExporter commented 9 years ago
Thanks for the report, but it was not line 107 (which is the correct format for 
Python 2.5. Your version only works in 2.6 and above). The problem was in line 
184 where I had "except:" with no type or catch value. Then the next line could 
not print the value.

Fixed in rev328.

I would be interested in knowing what the error was, as I am trying to make 
sure that this can clean up anything.

Original comment by kuehn.k...@gmail.com on 5 Aug 2010 at 2:32

GoogleCodeExporter commented 9 years ago
Tested rev 328, and I can confirm that this is fixed. Thanks! 

(I can now see the exception that is being thrown, [Errno 66] Directory not 
empty: '/private/tmp/InstaDMGTempFolder.3Vhnt3/InstaUp2DateFolder-kH3AGr/'.)

Original comment by loadsta...@gmail.com on 6 Aug 2010 at 3:50