Re: AIX crashes with python3

Rob,

Can you send a pull request for your change.
That should be an easy one.
Please don't forget to add a snippet to CHANGES.txt

Thanks,
Bill

On Tue, Jul 7, 2020 at 9:51 AM Mats Wichmann <[email protected]> wrote:

> On 7/7/20 10:37 AM, Rob Boehne wrote:
>
> > I can fix it by adding this patch – this should work with any python 3.4
> > or later (as opposed to some of the other fixes that would not).
> >
> > I was about to create an issue but read I should post here first.
> >
> >
> >
> > robb@rhel7:/raid/checkouts-raid/robb/AIX-scons/scons (master)$ git diff
> >
> > diff --git a/SCons/Platform/aix.py b/SCons/Platform/aix.py
> >
> > index c91679f..247437a 100644
> >
> > --- a/SCons/Platform/aix.py
> >
> > +++ b/SCons/Platform/aix.py
> >
> > @@ -55,6 +55,7 @@ def get_xlc(env, xlc=None, packages=[]):
> >
> >          pipe = SCons.Action._subproc(env, ['lslpp', '-fc', package],
> >
> >                  stdin = 'devnull',
> >
> >                  stderr = 'devnull',
> >
> > +                universal_newlines=True,
> >
> >                  stdout = subprocess.PIPE)
> >
> >          # output of lslpp is something like this:
> >
> >          #     #Path:Fileset:File
>
> yes, scons is having problems with this topic.  we haven't had feedback
> on AIX for a while.
>
> in *MY* opinion, and I'm just one person, not sure all agree, all of the
> cases where scons talks to a commandline utility via subprocess should
> do so in text mode (which most versions of Python have called
> universal_newlines, it finally has been renamed though the old name
> continues to work). It's not like reading from files, which are
> ultimately binary: the utilities speak text, and I'm not really clear
> why Python chooses to default subprocess to binary.  If we talk to
> utilities in binary, then we have to convert to text, and decoding risks
> running into problems depending on character sets - we had a spate of
> problems with non-US locales.
>
> I had a patch making this change more gobally, but there's been a lot
> going on...
> _______________________________________________
> Scons-users mailing list
> [email protected]
> https://pairlist4.pair.net/mailman/listinfo/scons-users
>

_______________________________________________
Scons-users mailing list
[email protected]
https://pairlist4.pair.net/mailman/listinfo/scons-users