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

Changes for new webinterface #9

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Changes for new webinterface #9

wants to merge 6 commits into from

Conversation

anfelor
Copy link
Member

@anfelor anfelor commented Mar 30, 2022

Hi @makarius, hi @adelon,

I have recently adopted the current Naproche master branch for the new webinterface at https://naproche.github.io/
For this I have made the following changes:

  1. GHCJS does not support the network package. Unfortunately, this means that it is necessary to split Naproche into three different packages: app, Isabelle and src. The parts of the Isabelle sources that do not use network are now found under src/Isabelle, while those that use network are under Isabelle. Last year, I was able to keep the Isabelle sources together in my dev branch by creating an Isabelle package that shared its sources with the naproche package. But this is not possible here, as the sources now describe common types like Bytes and GHC would get confused if Bytes from the Isabelle package is the same thing as Bytes from the Naproche package.
  2. I defined common APIs like MonadFile, MessageExchangeContext and RunProverContext that abstract from these system calls. There is a Console context with a basic implementation in src/ itself and a PIDE context in app/ (which depends on Isabelle/).
  3. I made some small changes so that the project would work for GHC, like removing the use of the new ImportQualifiedPost extension and adding a MonadFail instance in the parser.

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