Skip to content

Support creating a local JupyterLite deployment with git2cpp built from the local repo #36

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ianthomas23
Copy link
Member

This adds a new directory lite-deploy containing eveything required to build the local git2cpp source code as an
Emscripten-forge
package and create a local JupyterLite deployment that can run it in a terminal.

There are comprehensive instructions in lite-deploy/README.md. Essentially you create a new micromamba environment containing all the dependencies, use make to build everything and then make serve to serve the web site so that you can view it in a browser. If you change the git2cpp source code you can rebuild what is required using make rebuild.

The intention is that will be useful for developers working on PRs to easily see if their changes work in the JupyterLite terminal or not. I intend follow-on work to use this on demand in CI also, perhaps to deploy to Github Pages.

I will need to make new cockle and/or terminal releases next week so that we are using the latest code there, for example the changes that have added to cockle but not released yet to support identifying if coloured output is supported or not.

@ianthomas23
Copy link
Member Author

This now runs but git init and git clone fail with filesystem permission errors. So it can be used but will not be very useful until I have fixed the permissions errors which presumably needs a change in the libgit2 emscripten-forge recipe.

@ianthomas23
Copy link
Member Author

There is a temporary replacement for this in PR #37 that uses cockle rather than a full JupyterLite terminal, and that works today unlike this.

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.

1 participant