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

Feature/ubuntu 20.04 #5

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

Conversation

dbendlin
Copy link

@dbendlin dbendlin commented Sep 9, 2020

A small change to match Dockerfile with README file

ywarnier and others added 14 commits October 13, 2015 20:23
…nd chamilo-lms -1.11.x, this file is only processed at docker build, still need to find out how to everride its content when using dockr run =/
…p instructions to get some errors, I've also changed the host and domain used by the current example.
…es, file include ARG instructions to override some values that were before hardcoded, still need to find out how to make the env variables override values when issuing docker run. This file downloads chamilo, composer and chash from github and then performs composer update producing a 3.5GB size docker image. I'm going to summit another pull request that avoids this steps. Finally openssh its removed , docker exec -it ${container_name} /bin/bash could be used in the docker host to get into the running container. Entrypoint inmplemented so container can run in detached mode
…ust use one downloaded pron githubs repo =)
@dbendlin
Copy link
Author

dbendlin commented Sep 9, 2020

Somehow I keep missing the right target branch to merge

@ywarnier
Copy link
Member

ywarnier commented Sep 9, 2020

No need to close and reopen PRs. If you just update the branch (from which you sent the PR) on your repo, it will update the PR automatically ;-)
We might take some time to review your PR. Not a bit priority right now, but we'll try to do that soon.

…uded composer and php-fpm from ubuntu apt repositories
@dbendlin
Copy link
Author

No need to close and reopen PRs. If you just update the branch (from which you sent the PR) on your repo, it will update the PR automatically ;-)
We might take some time to review your PR. Not a bit priority right now, but we'll try to do that soon.

Thakns for the tip, still learning to the right way to contribute ;-)

Redirected git clone to a fork and branch of chamilo/chamilo-lms in my personal repository, this forked project intruduces a few configuration and programming changes that help me get chamilo-lms up and running on a docker container and authenticating users against our OpenLDAP service.
tag name replaced by the correct one
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.

3 participants