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 lhl2617/VSLilyPond suite #752

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

Conversation

lbssousa
Copy link
Contributor

  • I have read the note above about PRs contributing or fixing extensions
  • I have tried reaching out to the extension maintainers about publishing this extension to Open VSX (if not, please create an issue in the extension's repo using this template).
  • This extension has an OSI-approved OSS license (we don't accept proprietary extensions in this repository)

Description

This PR adds https://github.com/lhl2617/VSLilyPond suite to the extension marketplace

@filiptronicek
Copy link
Collaborator

Hi there @lbssousa. Could you please look into the logs of the failed CI and convert this to a draft in the meantime if this takes some more effort to set up?

@lbssousa lbssousa marked this pull request as draft April 2, 2024 18:50
@lbssousa
Copy link
Contributor Author

lbssousa commented Apr 2, 2024

I'm trying to debug CI errors. Reading the logs, I've found it's failing at this point, when trying to bundle python-ly for extension VSLilyPond-formatter:

$ staticx /tmp/repository/main/ly/dist-linux/python-ly /tmp/repository/main/ly/dist-linux/python-ly.static
Traceback (most recent call last):
  File "ly/build.py", line 71, in <module>
    cmd(f"staticx {bin_path} {bin_path}.static")
  File "ly/build.py", line 31, in cmd
    raise Exception(
Exception: Command 'staticx /tmp/repository/main/ly/dist-linux/python-ly /tmp/repository/main/ly/dist-linux/python-ly.static' exited with code 2, expected 0

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

Successfully merging this pull request may close these issues.

2 participants