Skip to content

Docker updates 2022 07 22#4

Closed
taniwallach wants to merge 3 commits into
openwebwork:developfrom
taniwallach:docker-updates-2022-07-22
Closed

Docker updates 2022 07 22#4
taniwallach wants to merge 3 commits into
openwebwork:developfrom
taniwallach:docker-updates-2022-07-22

Conversation

@taniwallach

Copy link
Copy Markdown
Member

Intended to make the Docker build work, and to split it into a 2 stage build, so the underlying OS packages and CPAN installs need not be redone for each rebuild of the main image.

It also updates the build instructions in README.md and points those also to use the openwebwork repository and not the drdrew42 repository.

This PR was based off the branch with the PG-2.17 updates, so needs to be merged after #2. There is only one new commit after what is in that PR.

Note: .git was removed from .dockerignore as otherwise the npm install inside lib/PG/htdocs is failing.

there that are still needed by the standalone renderer when rendering a
problem.  Requests to webwork2_files are redirected as appropriate by
the renderer.  Most go to PG/htdocs.
@taniwallach taniwallach requested review from drdrew42 and drgrice1 July 22, 2022 13:29
@drgrice1

Copy link
Copy Markdown
Member

Did you mean to target this to the develop branch?

  - 2 stage build process
  - changes needed after the PG-2.17 update and the changes to
    how codemirror is included.
  - update build instructions and other things in README.md
@taniwallach taniwallach force-pushed the docker-updates-2022-07-22 branch from 71c5787 to 243e728 Compare July 22, 2022 13:35
@taniwallach taniwallach changed the base branch from main to develop July 22, 2022 14:35
@taniwallach

Copy link
Copy Markdown
Member Author

Did you mean to target this to the develop branch?

I guess so. Sorry - changed.

@taniwallach taniwallach mentioned this pull request Sep 21, 2022
@taniwallach

Copy link
Copy Markdown
Member Author

Replaced by #8

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