emacspy dynamic module - support request

emacspy dynamic module - support request

[813gan]

[-- Attachment #1: Type: text/plain, Size: 1064 bytes --]

Hello Everyone.

I recently forked https://github.com/813gan/emacspy.
It's dynamic module that allows interacting with python using libpython (no subprocess).
My intention is to allow Emacs modes to use python modules including bindings of external libraries.

I'm facing problems with --module-assertions.
With mentioned flag enabled emacs is printing
"Emacs module assertion: Emacs value not found in 57 values of 1 environments"
and dumps core.

Crash is reproducible with Makefile target test_module_assertions (https://github.com/813gan/emacspy/blob/f61275c6d3adec7947d292b8a9df25551e40c869/Makefile#L68)

I have absolutely no idea what is going on there. Only thing i discovered is that commenting line
https://github.com/813gan/emacspy/blob/f61275c6d3adec7947d292b8a9df25551e40c869/emacspy.pyx#L140
prevents crash. It's obviously not a solution.

Otherwise module appears to work;
Despite that i don't want to ship it as i assume emacs have good reason to crash.

I would appreciate any help. If you need more information, please contact me.

Happy Hacking!

[-- Attachment #2: Type: text/html, Size: 2132 bytes --]

Re: emacspy dynamic module - support request

[Richard Stallman]

[[[ To any NSA and FBI agents reading my email: please consider    ]]]
[[[ whether defending the US Constitution against all enemies,     ]]]
[[[ foreign or domestic, requires you to follow Snowden's example. ]]]

  > My intention is to allow Emacs modes to use python modules including
  > bindings of external libraries.

This sort of thing risks encouraging distributing software that
dynamically links Emacs with nonfree libraries.  It is the same risk
that exists with dynamic modules written in C.

For dynamic modules written in C, we've taken some measures to
discourage thta.  Does emacspy take the same measures?  If not,
could you modify it to take them?

Another point.  emacspy is hosted on GitHub, which is a site that
requires nonfree software for most kinds of use.  In our evaluation,
https://www.gnu.org/software/repo-criteria-evaluation.html, it gets
grade F.

Could you possibly host your variant on a different repo site?

That same page reviews some other repo sites.


-- 
Dr Richard Stallman (https://stallman.org)
Chief GNUisance of the GNU Project (https://gnu.org)
Founder, Free Software Foundation (https://fsf.org)
Internet Hall-of-Famer (https://internethalloffame.org)

Re: emacspy dynamic module - support request

[813gan]

> > My intention is to allow Emacs modes to use python modules including
> > bindings of external libraries.
> 
> This sort of thing risks encouraging distributing software that
> dynamically links Emacs with nonfree libraries. It is the same risk
> that exists with dynamic modules written in C.
> 
> For dynamic modules written in C, we've taken some measures to
> discourage thta. Does emacspy take the same measures? If not,
> could you modify it to take them?

Emacspy currently does not validate licenses of Python modules.
I have plan to add hi-level Elisp API on top of existing C/cython functions.
Making such license validation would be easy to do inside this hi-level API.
I assume malicious users trying to bypass validation are not concern.

> Could you possibly host your variant on a different repo site?

Sure. I pushed https://notabug.org/813gan/emacspy

Re: emacspy dynamic module - support request

[Eli Zaretskii]

> Date: Sun, 25 Aug 2024 01:25:20 +0000
> From: 813gan <813gan@protonmail.com>
> 
> I recently forked https://github.com/813gan/emacspy.
> It's dynamic module that allows interacting with python using libpython (no subprocess).
> My intention is to allow Emacs modes to use python modules including bindings of external libraries.
> 
> I'm facing problems with --module-assertions.
> With mentioned flag enabled emacs is printing
> "Emacs module assertion: Emacs value not found in 57 values of 1 environments"
> and dumps core.
> 
> Crash is reproducible with Makefile target test_module_assertions
> (https://github.com/813gan/emacspy/blob/f61275c6d3adec7947d292b8a9df25551e40c869/Makefile#L68)
> 
> I have absolutely no idea what is going on there. Only thing i discovered is that commenting line 
> https://github.com/813gan/emacspy/blob/f61275c6d3adec7947d292b8a9df25551e40c869/emacspy.pyx#L140 
> prevents crash. It's obviously not a solution.
> 
> Otherwise module appears to work;
> Despite that i don't want to ship it as i assume emacs have good reason to crash.
> 
> I would appreciate any help. If you need more information, please contact me.

Philipp, are you able to help 813gan out?

Re: emacspy dynamic module - support request

[Richard Stallman]

[[[ To any NSA and FBI agents reading my email: please consider    ]]]
[[[ whether defending the US Constitution against all enemies,     ]]]
[[[ foreign or domestic, requires you to follow Snowden's example. ]]]

  > Emacspy currently does not validate licenses of Python modules.
  > I have plan to add hi-level Elisp API on top of existing C/cython functions.
  > Making such license validation would be easy to do inside this hi-level API.
  > I assume malicious users trying to bypass validation are not concern.

that sounds like a good method, as long as you release the dynamic module
and the Lisp code tpgether as one package.

  > > Could you possibly host your variant on a different repo site?

  > Sure. I pushed https://notabug.org/813gan/emacspy

We don't have an evaluation of that, but ISTR it i smuch better
than GitHub and GitLab.  Thanks.

-- 
Dr Richard Stallman (https://stallman.org)
Chief GNUisance of the GNU Project (https://gnu.org)
Founder, Free Software Foundation (https://fsf.org)
Internet Hall-of-Famer (https://internethalloffame.org)