On Sun, Dec 15, 2019 at 11:35:31AM -0600, Bruce Dubbs via blfs-dev wrote:
> On 12/15/19 10:16 AM, DJ Lucas via blfs-dev wrote:
> > On December 15, 2019 8:14:20 AM CST, Ken Moffat via blfs-dev 
> > <[email protected]> wrote:
> > > On Fri, Dec 13, 2019 at 08:57:09AM +0000, DJ Lucas via blfs-dev wrote:
> > > > On December 12, 2019 11:30:59 PM CST, Ken Moffat via blfs-dev
> > > <[email protected]> wrote:
> > > > 
> > > But when I gave it a first go, with python symlinked to python3, the
> > > configure immediately failed:
> > > 
> > > ken@plexi /tmp/mozjs-60.8.0/mozjs-build $../js/src/configure
> > > --prefix=/usr       \
> > > >                      --with-intl-api     \
> > > >                      --with-system-zlib  \
> > > >                      --with-system-icu   \
> > > >                      --disable-jemalloc  \
> > > >                      --enable-readline
> > >   File "../js/src/../../configure.py", line 65
> > >     print("Creating config.status", file=sys.stderr)
> > 
> > Sorry, I missed this. Yes this is python3 syntax so something is calling 
> > python2 explicitly.
> 

To DJ - thanks for confirming.  The problem is always finding where
it got called explicitly.  Maybe I'll take another look to see if I
can find remaining python2 references.

> A quick check finds 93 files in mozjs-60.8.0 referencing python2.  Some
> instances:
> 
> python/mozboot/mozboot/base.py:
>   MODERN_PYTHON_VERSION = LooseVersion('2.7.3')
> 

I think that is the minimum.  The items below are possibles.

> python/mozboot/mozboot/archlinux.py:
>     SYSTEM_PACKAGES = [
>         'autoconf2.13',
>         ...
>         'python2',
>         'python2-setuptools',
>         ...
> 
> js/src/devtools/rootAnalysis/analyze.py:
>   ((os.environ.get('PYTHON', 'python2.7'),
> 
> It looks like mozjs needs a major overhaul to eliminate python2.
> 
>   -- Bruce

Indeed.  The "patch" I found was 9800 lines.  I haven't counted how
many lines are in my first attempt to update it.  I have a severe
dislike of python, as is probably already apparent (the whitespace
issue) and I mostly find the error messages unuseful - if it fails
to find a specific thing, I would hope to get a more-specific
message.

ĸen
> -- 
> http://lists.linuxfromscratch.org/listinfo/blfs-dev
> FAQ: http://www.linuxfromscratch.org/blfs/faq.html
> Unsubscribe: See the above information page

-- 
          We've all got both light and dark inside of us.
          What matters is the part we choose to act on.
                                              -- Sirius Black
-- 
http://lists.linuxfromscratch.org/listinfo/blfs-dev
FAQ: http://www.linuxfromscratch.org/blfs/faq.html
Unsubscribe: See the above information page

Reply via email to