> I commited your patch into 1.5 branch.
> Thanks

Great!

> Just a note to C code:
>
> for windows compatibility we try to avoid using mixed declaration and
> code, like:

Sorry. I am normally aware of that and keep it out of pure C code as
it is only allowed in C++.

Is ths line ok?

    int dlopenMode = RTLD_NOW | RTLD_GLOBAL;

It comes before other declarations but is a constant initialization.

> That was only change I made. Otherwise it seems ok. Could you please
> test the code directly from 1.5 branch if it still woks?

I will and report back here when done.

> If you want to port AIX to trunk I would appreciate that. The code
> changed recently. I made some refactoring. For code specific to
> Linux/Solaris/AIX there is a variable  'is_unix'.

That sounds good. I also remember having seen other variables like
this to avoid constantly duplicated (and possible inconsistent) calls
to 'sys.platform.startswith(<something>)'.

> Do you think you could write some text for the manual? I would welcome
> some info about:
> - what and how to install tools on AIX necessary for pyinstaller to work
> - how to compile bootloader on AIX (in case there is something special
> to be done by users).

I am not sure I will have time this week. I don't think I did anything
special (apart from installing setuptools). But I use GCC for
compiling the bootloader, and installing GCC on AIX can sometimes be a
pain (my colleagues says). I know we have been building GCC from
sources and that is not a simple task.

In any case, I will read the existing manual and see if adding AIX
specific information/instructions make sense.

Cheers,
Martin

-- 
You received this message because you are subscribed to the Google Groups 
"PyInstaller" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/pyinstaller?hl=en.

Reply via email to