All of lore.kernel.org
 help / color / mirror / Atom feed
From: Thierry.Martinez@inria.fr (Thierry Martinez)
To: cocci@systeme.lip6.fr
Subject: [Cocci] Regression in Python Support
Date: Tue, 18 Oct 2016 18:54:45 +0200	[thread overview]
Message-ID: <ow4d1ixwqay.fsf@inria.fr> (raw)
In-Reply-To: <57FCD273.9070607@redhat.com>

Hello, Michael.

This should now be fixed in trunk. Thank you for the report.

The support of --with-python had been dropped since the Python version
was determined during run-time since 1.0.5 (even if it had not been
properly documented, sorry for that). The option --with-python is now
restored in trunk to specify the name of the default Python
interpreter. The interpreter can also be changed by passing the option
--python in the command line of spatch.

Note that it is still possible that Fedora 23 and 24 are shipped with
Python versions that have not been tested yet. If such a similar error
is still raised with trunk, would you give me the output of
"python3 --version"?

Thank you again for the report.
-- 
Thierry.

Michael Stefaniuc (2016/10/11, 13:52 CEST)?:
> Hello Thierry,
>
> I've run into a regression in coccinelle 1.0.6 for the python support:
> I am getting this error on Fedora 24:
> Python error: /usr/lib64/python2.7/lib-dynload/_collectionsmodule.so:
> undefined symbol: PyObject_SetItem
>
> On Fedora 23 I get a similar error but for a different symbol.
>
> The error is strange as I expected it to use python3 as I use:
> ./configure --with-python=/usr/bin/python3
>
> That works just fine in coccinelle-1.0.5. I've bisected it to:
> # first bad commit: [62f2df8c7d35967a1eca4dcd487d59a9918dd5aa] Fixed:
> Bundles compilation and pyml update
> commit 62f2df8c7d35967a1eca4dcd487d59a9918dd5aa
> Author: Thierry Martinez <martinez@nsup.org>
> Date:   Sun Jun 26 20:22:59 2016 +0200
>
>     Fixed: Bundles compilation and pyml update
>
>     - %.cmx: %.cmo dependency (with .ml files that do not have .mli,
>       compiling the .ml into .cmo or into .cmx both write to the same .cmi
>       file, so that cannot be done concurrently)
>     - untarred pyml distribution to ease updates
>     - Python exception catched in pycocci interface
>
>
> thanks
> bye
> 	michael
> _______________________________________________
> Cocci mailing list
> Cocci at systeme.lip6.fr
> https://systeme.lip6.fr/mailman/listinfo/cocci

  reply	other threads:[~2016-10-18 16:54 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-10-11 11:52 [Cocci] Regression in Python Support Michael Stefaniuc
2016-10-18 16:54 ` Thierry Martinez [this message]
2016-10-18 19:11   ` Michael Stefaniuc
2016-10-18 20:42     ` Julia Lawall

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=ow4d1ixwqay.fsf@inria.fr \
    --to=thierry.martinez@inria.fr \
    --cc=cocci@systeme.lip6.fr \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.