-
Notifications
You must be signed in to change notification settings - Fork 46
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
Integrate qgis plugin #3868
Integrate qgis plugin #3868
Conversation
Makefile
Outdated
|
||
.PHONY: docker-build-qgisserver | ||
docker-build-qgisserver: $(shell docker-required --path docker/qgisserver) docker/qgisserver/commons | ||
docker build --tag=$(DOCKER_TEST_BASE)-qgisserver:latest docker/qgisserver |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wrong tag...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed, thanks :-)
fb34222
to
b1670e8
Compare
Makefile
Outdated
docker/qgisserver/commons: commons | ||
rm --recursive --force $@ | ||
cp --recursive $< $@ | ||
touch $@ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I find simpler to use repository root as docker context
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, we relay don't wants to mix everything ...
docker/qgisserver/Dockerfile
Outdated
LABEL maintainer Camptocamp "info@camptocamp.com" | ||
|
||
COPY * /var/www/plugins/geomapfish_qgisserver/ | ||
COPY __init__.py accesscontrol.py metadata.txt /var/www/plugins/geomapfish_qgisserver/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would create a folder for the plugin
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done :-)
af69406
to
835f14e
Compare
835f14e
to
87209a3
Compare
6b14ec8
to
06bb57c
Compare
06bb57c
to
e2b1191
Compare
No description provided.