Meador Inge added the comment:

I don't think the current patch is correct.  It breaks the usage of `freeze.py 
-p $path` since LIBDIR and LIBRARY are blindly used.

Maybe something like the attached that respects -p and -P can be used instead?

As for the testing, I think a reasonable solution to get started with is to add 
some buildbot runs that automate the hello.py freeze on various hosts.  That 
will give us some basic coverage and then more tests can be added over time if 
need be.  I would be happy to help set that up (with given guidance on how to 
access the buildbots).

Even with the attached patch I still can't fully build (on OS X 10.7.5) because 
of issue16047:

Undefined symbols for architecture x86_64:
  "_PyInit__imp", referenced from:
      __PyImport_Inittab in config.o
ld: symbol(s) not found for architecture x86_64

I will take a look at that issue next.

----------
assignee:  -> meador.inge
Added file: http://bugs.python.org/file28519/issue11824-0.patch

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue11824>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to