From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 52471 invoked by alias); 22 Jan 2020 01:56:41 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Received: (qmail 52452 invoked by uid 89); 22 Jan 2020 01:56:40 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-18.6 required=5.0 tests=AWL,BAYES_00,BODY_8BITS,FREEMAIL_ENVFROM_END_DIGIT,FREEMAIL_FROM,GARBLED_BODY,GIT_PATCH_0,GIT_PATCH_1,GIT_PATCH_2,GIT_PATCH_3,KAM_NUMSUBJECT,RCVD_IN_DNSWL_NONE,SPF_PASS autolearn=ham version=3.3.1 spammy=HX-Languages-Length:6504 X-HELO: mail-ed1-f65.google.com Received: from mail-ed1-f65.google.com (HELO mail-ed1-f65.google.com) (209.85.208.65) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Wed, 22 Jan 2020 01:56:38 +0000 Received: by mail-ed1-f65.google.com with SMTP id m8so5062989edi.13 for ; Tue, 21 Jan 2020 17:56:37 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=uhYf6/UVFEnL5Eejne2NqAIPo07juqDoeWhxZfBC/R0=; b=GG46/PGmv1fNDtoB8wPyZWt3Xfh/Vxvj0Ey6F7JTAMfErHLFlUI+Lea+guksUYTARZ azCjyARAlDTsMXQZ63gELVPE47Ig90JpgM4XnetcI4eCLi3CNHcr8zvVM1ZM5CI6RakW YGF3yhWJXav1FFZ5HU6BYPkwI+MU8wbQ0qkR52LqeU0mgDRAWJSMek6jaO8l1PoJOHT4 0kWDk5NvTqrUoThICm5R4odYocHYm2kDKavj216GFmnDpux1oy3ghsaDeJi5GoImflJQ o+WycplIofhqarbFaZiniZvEPwDDJMZ7QOdp8YQxc+KfeMf9NGJud5NL2CeM0b3f+BNy eRQQ== MIME-Version: 1.0 References: In-Reply-To: From: =?UTF-8?B?5YKF57un5pmX?= Date: Wed, 22 Jan 2020 02:04:00 -0000 Message-ID: Subject: Re: [PATCH] Modify python-config.py or replace it with official one to support python3 To: gdb-patches Cc: Christian Biesinger Content-Type: multipart/mixed; boundary="0000000000009d92a1059cb0d527" X-SW-Source: 2020-01/txt/msg00664.txt.bz2 --0000000000009d92a1059cb0d527 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Content-length: 7527 now I improve the patch to be compatible with python2.7 python 3.7 and python3.8+ index gdb/configure.ac =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D --- a/gdb/configure.ac +++ b/gdb/configure.ac @@ -834,29 +834,35 @@ else esac esac + python_config=3D if test "${python_prog}" !=3D missing; then + AC_CHECK_FILE([${python_prog}-config],[python_config=3D"${python_prog}= -config"],[python_config=3D"${python_prog} ${srcdir}/python/python-config.py"]) # We have a python program to use, but it may be too old. # Don't flag an error for --with-python=3Dauto (the default). have_python_config=3Dyes - python_includes=3D`${python_prog} ${srcdir}/python/python-config.py --includes` + python_includes=3D`${python_config} --includes` if test $? !=3D 0; then have_python_config=3Dfailed if test "${with_python}" !=3D auto; then - AC_ERROR(failure running python-config --includes) + AC_ERROR(failure running ${python-config} --includes) fi fi - python_libs=3D`${python_prog} ${srcdir}/python/python-config.py --ldfl= ags` +# For python 3.8+ there must be a "--embed" flag to show lib version. + python_libs=3D`${python_config} --ldflags --embed` if test $? !=3D 0; then - have_python_config=3Dfailed - if test "${with_python}" !=3D auto; then - AC_ERROR(failure running python-config --ldflags) + python_libs=3D`${python_config} --ldflags` + if test $? !=3D 0; then + have_python_config=3Dfailed + if test "${with_python}" !=3D auto; then + AC_ERROR(failure running ${python-config} --ldflags) + fi fi fi - python_prefix=3D`${python_prog} ${srcdir}/python/python-config.py --exec-prefix` + python_prefix=3D`${python_config} --exec-prefix` if test $? !=3D 0; then have_python_config=3Dfailed if test "${with_python}" !=3D auto; then - AC_ERROR(failure running python-config --exec-prefix) + AC_ERROR(failure running ${python-config} --exec-prefix) fi fi else =E5=82=85=E7=BB=A7=E6=99=97 =E4=BA=8E2020=E5=B9=B41= =E6=9C=8821=E6=97=A5=E5=91=A8=E4=BA=8C =E4=B8=8B=E5=8D=887:41=E5=86=99=E9= =81=93=EF=BC=9A > > ok.that is,and here is my patch. > Tested with autoconf 2.69 and python3.8. > > > index:gdb/configure.ac > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D > --- a/gdb/configure.ac > +++ b/gdb/configure.ac > @@ -834,29 +834,31 @@ > esac > esac > > + python_config=3D > if test "${python_prog}" !=3D missing; then > + AC_CHECK_FILE([${python_prog}-config],[python_config=3D"${python_pro= g}-config > --embed"],[python_config=3D"${python_prog} > ${srcdir}/python/python-config.py"]) > # We have a python program to use, but it may be too old. > # Don't flag an error for --with-python=3Dauto (the default). > have_python_config=3Dyes > - python_includes=3D`${python_prog} ${srcdir}/python/python-config.py > --includes` > + python_includes=3D`${python_config} --includes` > if test $? !=3D 0; then > have_python_config=3Dfailed > if test "${with_python}" !=3D auto; then > - AC_ERROR(failure running python-config --includes) > + AC_ERROR(failure running ${python_config} --includes) > fi > fi > - python_libs=3D`${python_prog} ${srcdir}/python/python-config.py --ld= flags` > + python_libs=3D`${python_config} --ldflags` > if test $? !=3D 0; then > have_python_config=3Dfailed > if test "${with_python}" !=3D auto; then > - AC_ERROR(failure running python-config --ldflags) > + AC_ERROR(failure running ${python_config} --ldflags) > fi > fi > - python_prefix=3D`${python_prog} ${srcdir}/python/python-config.py > --exec-prefix` > + python_prefix=3D`${python_config} --exec-prefix` > if test $? !=3D 0; then > have_python_config=3Dfailed > if test "${with_python}" !=3D auto; then > - AC_ERROR(failure running python-config --exec-prefix) > + AC_ERROR(failure running ${python_config} --exec-prefix) > fi > fi > else > > Christian Biesinger =E4=BA=8E2020=E5=B9=B41=E6=9C= =8821=E6=97=A5=E5=91=A8=E4=BA=8C =E4=B8=8A=E5=8D=888:30=E5=86=99=E9=81=93= =EF=BC=9A > > > > On Mon, Jan 20, 2020 at 8:05 AM =E5=82=85=E7=BB=A7=E6=99=97 wrote: > > > > > > The formal patch only works on OSX.And here is the patch to make the > > > crossbuild available. > > > > configure is a generated file. Please edit configure.ac instead, and > > run autoconf to regenerate configure. Make sure to use autoconf > > version 2.69. > > > > Christian > > > > > diff --git a/gdb/configure-origin b/gdb/configure > > > --- a/gdb/configure-origin > > > +++ b/gdb/configure > > > @@ -9858,21 +9858,21 @@ fi > > > # We have a python program to use, but it may be too old. > > > # Don't flag an error for --with-python=3Dauto (the default). > > > have_python_config=3Dyes > > > - python_includes=3D`${python_prog} ${srcdir}/python/python-config.py= --includes` > > > + python_includes=3D`${python_prog}-config --includes` > > > if test $? !=3D 0; then > > > have_python_config=3Dfailed > > > if test "${with_python}" !=3D auto; then > > > as_fn_error "failure running python-config --includes" "$LINENO" 5 > > > fi > > > fi > > > - python_libs=3D`${python_prog} ${srcdir}/python/python-config.py --l= dflags` > > > + python_libs=3D`${python_prog}-config --ldflags` > > > if test $? !=3D 0; then > > > have_python_config=3Dfailed > > > if test "${with_python}" !=3D auto; then > > > as_fn_error "failure running python-config --ldflags" "$LINENO" 5 > > > fi > > > fi > > > - python_prefix=3D`${python_prog} ${srcdir}/python/python-config.py -= -exec-prefix` > > > + python_prefix=3D`${python_prog}-config --exec-prefix` > > > if test $? !=3D 0; then > > > have_python_config=3Dfailed > > > if test "${with_python}" !=3D auto; then > > > > > > > > > =E5=82=85=E7=BB=A7=E6=99=97 =E4=BA=8E2020=E5=B9= =B41=E6=9C=8820=E6=97=A5=E5=91=A8=E4=B8=80 =E4=B8=8B=E5=8D=888:02=E5=86=99= =E9=81=93=EF=BC=9A > > > > > > > > By default,gdb is compiled with python2 script support on OSX. Howe= ver,when compiled with arg "--with-python=3D/path/to/python3 " to support p= ython3 ,some error occurred that seems python3 is not recognized. > > > > The reason is that the python-config.py is not the same as the offi= cial one.BTW,this issue was once referred at 20 Dec 2012 mail :https://sour= ceware.org/ml/gdb-patches/2012-12/msg00751.html. > > > > Somehow,It is not committed(maybe the maintainer forgot it due to t= he happy chrismas holiday). > > > > And here is my patch: > > > > > > > > index: b/gdb/python/python-config.py > > > > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D > > > > --- a/gdb/python/python-config.py > > > > +++ b/gdb/python/python-config.py > > > > @@ -72,7 +72,7 @@ for opt in opt_flags: > > > > libs.insert(0, '-L' + getvar('LIBPL')) > > > > elif os.name =3D=3D 'nt': > > > > libs.insert(0, '-L' + sysconfig.PREFIX + '/lib= s') > > > > - if getvar('LINKFORSHARED') is not None: > > > > + if not getvar('PYTHONFRAMEWORK'): > > > > libs.extend(getvar('LINKFORSHARED').split()) > > > > print (to_unix_path(' '.join(libs))) --0000000000009d92a1059cb0d527 Content-Type: application/octet-stream; name="python-config.patch" Content-Disposition: attachment; filename="python-config.patch" Content-Transfer-Encoding: base64 Content-ID: X-Attachment-Id: f_k5onm6vu0 Content-length: 2652 ZGlmZiAtLWdpdCBhL2dkYi9jb25maWd1cmUtb3JpZ2luLmFjIGIvZ2RiL2Nv bmZpZ3VyZS5hYwppbmRleCAwODA1ODI3Li45ZjA3ZTIzIDEwMDY0NAotLS0g YS9nZGIvY29uZmlndXJlLW9yaWdpbi5hYworKysgYi9nZGIvY29uZmlndXJl LmFjCkBAIC04MzQsMjkgKzgzNCwzNSBAQCBlbHNlCiAgICAgZXNhYwogICBl c2FjCiAKKyAgcHl0aG9uX2NvbmZpZz0KICAgaWYgdGVzdCAiJHtweXRob25f cHJvZ30iICE9IG1pc3Npbmc7IHRoZW4KKyAgICBBQ19DSEVDS19GSUxFKFsk e3B5dGhvbl9wcm9nfS1jb25maWddLFtweXRob25fY29uZmlnPSIke3B5dGhv bl9wcm9nfS1jb25maWciXSxbcHl0aG9uX2NvbmZpZz0iJHtweXRob25fcHJv Z30gJHtzcmNkaXJ9L3B5dGhvbi9weXRob24tY29uZmlnLnB5Il0pCiAgICAg IyBXZSBoYXZlIGEgcHl0aG9uIHByb2dyYW0gdG8gdXNlLCBidXQgaXQgbWF5 IGJlIHRvbyBvbGQuCiAgICAgIyBEb24ndCBmbGFnIGFuIGVycm9yIGZvciAt LXdpdGgtcHl0aG9uPWF1dG8gKHRoZSBkZWZhdWx0KS4KICAgICBoYXZlX3B5 dGhvbl9jb25maWc9eWVzCi0gICAgcHl0aG9uX2luY2x1ZGVzPWAke3B5dGhv bl9wcm9nfSAke3NyY2Rpcn0vcHl0aG9uL3B5dGhvbi1jb25maWcucHkgLS1p bmNsdWRlc2AKKyAgICBweXRob25faW5jbHVkZXM9YCR7cHl0aG9uX2NvbmZp Z30gLS1pbmNsdWRlc2AKICAgICBpZiB0ZXN0ICQ/ICE9IDA7IHRoZW4KICAg ICAgIGhhdmVfcHl0aG9uX2NvbmZpZz1mYWlsZWQKICAgICAgIGlmIHRlc3Qg IiR7d2l0aF9weXRob259IiAhPSBhdXRvOyB0aGVuCi0gICAgICAgIEFDX0VS Uk9SKGZhaWx1cmUgcnVubmluZyBweXRob24tY29uZmlnIC0taW5jbHVkZXMp CisgICAgICAgIEFDX0VSUk9SKGZhaWx1cmUgcnVubmluZyAke3B5dGhvbi1j b25maWd9IC0taW5jbHVkZXMpCiAgICAgICBmaQogICAgIGZpCi0gICAgcHl0 aG9uX2xpYnM9YCR7cHl0aG9uX3Byb2d9ICR7c3JjZGlyfS9weXRob24vcHl0 aG9uLWNvbmZpZy5weSAtLWxkZmxhZ3NgCisjIEZvciBweXRob24gMy44KyB0 aGVyZSBtdXN0IGJlIGEgIi0tZW1iZWQiIGZsYWcgdG8gc2hvdyBsaWIgdmVy c2lvbi4KKyAgICBweXRob25fbGlicz1gJHtweXRob25fY29uZmlnfSAtLWxk ZmxhZ3MgLS1lbWJlZGAKICAgICBpZiB0ZXN0ICQ/ICE9IDA7IHRoZW4KLSAg ICAgIGhhdmVfcHl0aG9uX2NvbmZpZz1mYWlsZWQKLSAgICAgIGlmIHRlc3Qg IiR7d2l0aF9weXRob259IiAhPSBhdXRvOyB0aGVuCi0gICAgICAgIEFDX0VS Uk9SKGZhaWx1cmUgcnVubmluZyBweXRob24tY29uZmlnIC0tbGRmbGFncykK KyAgICAgIHB5dGhvbl9saWJzPWAke3B5dGhvbl9jb25maWd9IC0tbGRmbGFn c2AKKyAgICAgIGlmIHRlc3QgJD8gIT0gMDsgdGhlbgorICAgICAgICBoYXZl X3B5dGhvbl9jb25maWc9ZmFpbGVkCisgICAgICAgIGlmIHRlc3QgIiR7d2l0 aF9weXRob259IiAhPSBhdXRvOyB0aGVuCisgICAgICAgICAgQUNfRVJST1Io ZmFpbHVyZSBydW5uaW5nICR7cHl0aG9uLWNvbmZpZ30gLS1sZGZsYWdzKQor ICAgICAgICBmaQogICAgICAgZmkKICAgICBmaQotICAgIHB5dGhvbl9wcmVm aXg9YCR7cHl0aG9uX3Byb2d9ICR7c3JjZGlyfS9weXRob24vcHl0aG9uLWNv bmZpZy5weSAtLWV4ZWMtcHJlZml4YAorICAgIHB5dGhvbl9wcmVmaXg9YCR7 cHl0aG9uX2NvbmZpZ30gLS1leGVjLXByZWZpeGAKICAgICBpZiB0ZXN0ICQ/ ICE9IDA7IHRoZW4KICAgICAgIGhhdmVfcHl0aG9uX2NvbmZpZz1mYWlsZWQK ICAgICAgIGlmIHRlc3QgIiR7d2l0aF9weXRob259IiAhPSBhdXRvOyB0aGVu Ci0gICAgICAgIEFDX0VSUk9SKGZhaWx1cmUgcnVubmluZyBweXRob24tY29u ZmlnIC0tZXhlYy1wcmVmaXgpCisgICAgICAgIEFDX0VSUk9SKGZhaWx1cmUg cnVubmluZyAke3B5dGhvbi1jb25maWd9IC0tZXhlYy1wcmVmaXgpCiAgICAg ICBmaQogICAgIGZpCiAgIGVsc2UK --0000000000009d92a1059cb0d527--