Skip to content

Added bridging functionallity to PHP. - #247

Open
kalaspuffar wants to merge 2 commits into
daisy:mainfrom
kalaspuffar:php_bridge
Open

kalaspuffar wants to merge 2 commits into
daisy:mainfrom
kalaspuffar:php_bridge

Conversation

@kalaspuffar

Copy link
Copy Markdown

This PR will add a couple of functions to the code so you can add this library as a PHP module and run it directly from the PHP code. I've seen that other glue codes have separate repositories, so this might not be how to implement this. I am still very early in my Rust journey, so I've not looked into multi-repository solutions.

I also did a small bug fix so the library doesn't have the name liblibmathcat.so

Best regards
Daniel

@NSoiffer

Copy link
Copy Markdown
Collaborator

Apologies for not seeing this earlier.

The way I have been dealing with language bindings is to have a separate repo (e.g, MathCATForC, MathCATForPython) that has wrappers for that language. The idea was not to "burden" the main library with these bindings. I don't think the current setup of using separate repos is a good idea and welcome suggestions for how to manage this better.

Two ideas that have come up:

If you have any thoughts on either approach or another approach, I'd appreciate your feedback. I have heard from some people that I should avoid github submodules.

Also see #193 for another approach to language bindings.

@rileeki rileeki moved this from Triage to In review in MathCAT Project Board Jul 28, 2026
@moritz-gross moritz-gross added enhancement New feature or request integrations regarding other software such as screen readers, operating systems, ... labels Sep 16, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request integrations regarding other software such as screen readers, operating systems, ...

Projects

Status: In review

Development

Successfully merging this pull request may close these issues.

4 participants