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 the new qgis.org key using the simpler approach, also update the rest of the dockerfiles #56

Merged
merged 2 commits into from
Aug 17, 2022

Conversation

tudorbarascu
Copy link
Contributor

@tudorbarascu tudorbarascu commented Aug 14, 2022

Hello,

Simply updated the qgis.org key to all the docker files + fix a bionic -> focal typo

@wlorenzetti
Copy link
Member

Thanks @tudorbarascu, but follow qgis.org instruction key come from qgis.org site, is it correct?

@wlorenzetti
Copy link
Member

While at it, I don't understand the difference beetween Dockerfile.g3wsuite-deps.dockerfile and Dockerfile.g3wsuite-deps.dockerfile and Dockerfile.g3wsuite-deps.ltr.dockerfile as they're the same, so why should both be kept?

Simple @tudorbarascu, g3w-suite-deps il for last QGIS version available, now is 3.26 instead ltr is for last fersion for QGIS LTR edition 3.22. Into Continue Integration no LTR version is used to test with new API.

@tudorbarascu
Copy link
Contributor Author

Thanks @tudorbarascu, but follow qgis.org instruction key come from qgis.org site, is it correct?

Yes, that's the general instruction for linux. It seems that Tim puts all the time the new key to the keyserver.ubuntu.com as you can see from https://keyserver.ubuntu.com/pks/lookup?search=qgis.org&fingerprint=on&op=index

So for ubuntu the key steps are easier and as the docker images are for ubuntu I thought its simpler as it was before. Thank you

@wlorenzetti
Copy link
Member

@tudorbarascu I didn't Tim enetered kye on official Ubuntu key-server. Thanks.

@wlorenzetti wlorenzetti merged commit bb0fa67 into g3w-suite:dev Aug 17, 2022
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.

2 participants