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 missing php dependencies for nginx image #8

Merged
merged 2 commits into from
Oct 14, 2018
Merged

Add missing php dependencies for nginx image #8

merged 2 commits into from
Oct 14, 2018

Conversation

languitar
Copy link
Contributor

Without these dependencies, baikal cannot be used.

@ckulka
Copy link
Owner

ckulka commented Jun 17, 2018

Hi @languitar , thanks for the PR - when do you get the error messages that these dependencies are missing? I went through my logs, but couldn't find anything.

Also, could you please also sort the dependencies in alphabetical order and add them to the other Dockerfiles as well?

@ckulka ckulka self-assigned this Jun 17, 2018
@ckulka ckulka added the bug label Jun 17, 2018
Without these dependencies, baikal cannot be used.
@languitar
Copy link
Contributor Author

Sorry for the delay. I have added the dependencies also to the arm nginx image. I am not sure about the apache versions, because I don't use them an I am not sure what is already included there.

I have migrated an existing installation of baikal to the docker image and without these dependencies, first launch failed immediately. I don't have the logs available anymore, though. :/

@languitar
Copy link
Contributor Author

Maybe @dober knows more about what fails without those dependencies?

@dober
Copy link

dober commented Sep 23, 2018

Sure!
Without this deps we got an error:

https://davserver/card.php/addressbooks/user/contacts

Error: Class 'DOMDocument' not found in /var/www/baikal/vendor/sabre/dav/lib/DAV/Server.php:256 Stack trace: #0 /var/www/baikal/Core/Frameworks/Baikal/Core/Server.php(123): Sabre\DAV\Server->exec() #1 /var/www/baikal/html/card.php(62): Baikal\Core\Server->start() #2 {main}

@mehdicopter
Copy link

mehdicopter commented Oct 13, 2018

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

Successfully merging this pull request may close these issues.

None yet

4 participants