Steps to reproduce: 1. Extract stage3 20140116 hardened nomultilib 2. chroot 3. revdep-pax -r 4. 50% chance to fail hard with the following output: sh-4.2# revdep-pax -r Traceback (most recent call last): File "/usr/sbin/revdep-pax", line 654, in <module> main() File "/usr/sbin/revdep-pax", line 643, in main run_reverse(verbose, executable_only) File "/usr/sbin/revdep-pax", line 311, in run_reverse sv = '%s\n\t%s ( %s )' % (sv, elf, elf_str_flags) UnboundLocalError: local variable 'sv' referenced before assignment 50%, because there's a random chance (based on unordered list order) that this bug will not be triggered. Prometheanfire made a patch couple days ago, will upload.
Thanks for the catch! I wonder what's changed because I must have run revdep-pax -r a million times and never hit that.
Created attachment 368244 [details, diff] elfix-pep8 and more
Created attachment 368264 [details, diff] pep8
Created attachment 368266 [details, diff] use before define
(In reply to Matthew Thode ( prometheanfire ) from comment #3) > Created attachment 368264 [details, diff] [details, diff] > pep8 Applied (In reply to Matthew Thode ( prometheanfire ) from comment #4) > Created attachment 368266 [details, diff] [details, diff] > use before define This doesn't seem right. It changes the logic. Why does it give the same error reporting?
Created attachment 368566 [details, diff] fixes use before define, now more streamlined with comments
Okay committed.
