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

pkg-config is missing in Dockerfile #5

Open
christofsteel opened this issue May 22, 2019 · 3 comments · May be fixed by #9
Open

pkg-config is missing in Dockerfile #5

christofsteel opened this issue May 22, 2019 · 3 comments · May be fixed by #9

Comments

@christofsteel
Copy link

Hi,
the Dockerfile tries to compile memcached. This failes, because pkg-config is not installed. If you add pkg-config to any apt-get command, it builds.

@MidgardLol
Copy link

Hi,
Get last zotero-prime,
change version in 2 in docker-compose.yml
make "docker-compose up -d" in zotero-prime/docker
with re2c 0.16 and pkg-config 0.29 on Debian GNU/Linux 9.9 (stretch)

i have that warning 👎
checking for re2c... no
configure: WARNING: You will need re2c 0.13.4 or later if you want to regenerate PHP parsers.
and then crash 👎
checking for pkg-config... no
pkg-config not found
configure: error: Please reinstall the pkg-config distribution
ERROR: Service 'app-zotero' failed to build: The command '/bin/sh -c DEBIAN_FRONTEND=noninteractive pecl download memcached-3.0.4 && tar xvzf memcached-3.0.4.tgz && cd memcached-3.0.4 && phpize && ./configure --enable-memcached-igbinary && make && make install' returned a non-zero code: 1

How to say it where to find them ?

@christofsteel
Copy link
Author

christofsteel commented Jun 12, 2019

Hi,
if you do exactly what I proposed, it will build. I already did a pull request to fix the issue. As long as it not accepted, you can use my fork, which only adds pkg-config. https://github.com/christofsteel/zotero-prime.

EDIT:
Your problem is the following, even if your host system (Debian) has pkg-config installed, the container has not. So you need to edit the Dockerfile and add an instruction to install it. As I was saying, my pull request does exactly that, and you can use it.

@MidgardLol
Copy link

MidgardLol commented Jun 12, 2019 via email

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 a pull request may close this issue.

2 participants