Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add package: verovio #1000

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from
Draft

Conversation

IsakTheHacker
Copy link
Contributor

Fixes #462

@IsakTheHacker
Copy link
Contributor Author

To build verovio, you need to have swig installed!

@mhsmith
Copy link
Member

mhsmith commented Oct 14, 2023

Thanks; in that case, please add a comment to the meta.yaml file giving instructions for installing swig.

@mhsmith
Copy link
Member

mhsmith commented Oct 17, 2023

Also, to add a new package, we need a test script to verify that it works. Please do the following:

  • Follow the instructions in "Testing a package" in the README.
  • Push your test script to this PR.
  • Post a comment saying which Python versions and ABIs you have tested.

@IsakTheHacker
Copy link
Contributor Author

I have added an instruction on how to install swig and I have started writing a test. When I run the app it crashes with:

2023-10-18 21:31:51.999  9798-9941  libc                    com.chaquo.python.pkgtest3           A  Fatal signal 11 (SIGSEGV), code 1 (SEGV_MAPERR), fault addr 0x8 in tid 9941 (Thread-4), pid 9798 (python.pkgtest3)
2023-10-18 21:31:52.175  9947-9947  DEBUG                   pid-9947                             A  Cmdline: com.chaquo.python.pkgtest3
2023-10-18 21:31:52.175  9947-9947  DEBUG                   pid-9947                             A  pid: 9798, tid: 9941, name: Thread-4  >>> com.chaquo.python.pkgtest3 <<<
2023-10-18 21:31:52.175  9947-9947  DEBUG                   pid-9947                             A    NOTE: Function names and BuildId information is missing for some frames due
                                                                                                      NOTE: to unreadable libraries. For unwinds of apps, only shared libraries
                                                                                                      NOTE: found under the lib/ directory are readable.
                                                                                                      NOTE: On this device, run setenforce 0 to make the libraries readable.
                                                                                                      NOTE: Unreadable libraries:
                                                                                                      NOTE:   /data/data/com.chaquo.python.pkgtest3/files/chaquopy/AssetFinder/requirements/verovio/_verovio.so

@mhsmith
Copy link
Member

mhsmith commented Dec 9, 2023

Changing to draft until tests are passing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unable to pip install verovio
2 participants