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

No release tags, broken dependencies #6

Open
ypvypv opened this issue Dec 2, 2020 · 4 comments
Open

No release tags, broken dependencies #6

ypvypv opened this issue Dec 2, 2020 · 4 comments

Comments

@ypvypv
Copy link

ypvypv commented Dec 2, 2020

Hi,

Just wanted to let you know that you introduced a CMAKE 3.15 hard requirement from external/cpr a week ago. It broke our build that is pulling from this repo.

One way to solve this problem would be to start tagging releases which we can sync to. You have no releases. I am going to git pull to a commit I suppose.

Thanks,
Yuri

@johniez
Copy link
Collaborator

johniez commented Dec 3, 2020

Hi,
sorry for that, the dependency on cmake 3.15 is quite strong on older distributions, I understand that.

I tagged the commit before, you can use https://github.com/seznam/httpmockserver/releases/tag/version-0.1 instead of commit hash itself.

@ypvypv
Copy link
Author

ypvypv commented Dec 3, 2020

Hi again,

Actually the problem is worse than I thought. Your tag should still break because you are not pulling tagged versions of cpr and the other library. The external dependencies keep moving even though you tagged your release. Maybe you could pin all your dependencies (and cpr to a version with <= CMAKE 3.10 ). Eventually these dependencies will break your own build.

Cheers,
Yuri

@johniez
Copy link
Collaborator

johniez commented Dec 3, 2020

git clone https://github.com/seznam/httpmockserver/
cd httpmockserver
git checkout version-0.1 -b old-cmake
git submodule update --init --recursive
head -1 external/cpr/CMakeLists.txt

Gives me cmake_minimum_required(VERSION 2.8.7) as it should. Tag points to old versions of externals submodules. But maybe you are using it differently?

@johniez
Copy link
Collaborator

johniez commented Dec 3, 2020

But anyway, depending on the cpr is really unnecessary here, because it is used in a test case only. It could be made directly by libcurl and this strong dependecy will be gone. I will probably change that.

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

No branches or pull requests

2 participants