-
Notifications
You must be signed in to change notification settings - Fork 121
Revert "Add license for bibliographic data" #1221
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
Conversation
FTR:
|
cc @Flowdalic |
I've no idea how to obtain the used Dockerfile, but I suspect that it is because
which does not copy refs-LICENSE and in general it seems to be a bad idea to copy files from the repo over, as it is fragile (as demonstrated by this issue). |
Try this: https://github.com/xsf/xeps/blob/master/Dockerfile |
We were asked [1] to state that the XEP bibliographic data is openly available and free of charge if we want it to be consumed by third parties. This weas previously merged as 7477388 ("Add license for bibliographic data"), but unfortunately later reverted in 59b2a5c ("Revert "Add license for bibliographic data"", xsf#1221). However, the fix for the broken docker build is trivial: simply also copy refs-LICENSE into the container. This commit does that. Fixes xsf#1219. 1: ietf-tools/bibxml-service#302 (comment)
We were asked [1] to state that the XEP bibliographic data is openly available and free of charge if we want it to be consumed by third parties. This weas previously merged as 7477388 ("Add license for bibliographic data"), but unfortunately later reverted in 59b2a5c ("Revert "Add license for bibliographic data"", xsf#1221). However, the fix for the broken docker build is trivial: simply also copy refs-LICENSE into the container. This commit does that. Fixes xsf#1219. 1: ietf-tools/bibxml-service#302 (comment)
Reverts #1220
It breaks the docker build and I don't have the time to investigate it right now.