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

concurrentqueue md5sum changed #956

Closed
felrock opened this issue Feb 1, 2022 · 9 comments
Closed

concurrentqueue md5sum changed #956

felrock opened this issue Feb 1, 2022 · 9 comments
Labels
bug Something isn't working

Comments

@felrock
Copy link

felrock commented Feb 1, 2022

Description

Project in question:
rosbag2/shared_queues_vendor

concurrentqueue zip file(https://github.com/cameron314/concurrentqueue/archive/8f65a8734d77c3cc00d74c0532efca872931d3ce.zip) new md5sum is c4cc435ec9153208b764015826d40231, so it breaks the CMakeLists.txt download. It retrys a couple of times before returning a garbage error:
--- stderr: shared_queues_vendor
CMake Error at ext-concurrentqueue-stamp/download-ext-concurrentqueue.cmake:170 (message):
Each download failed!

Expected Behavior

md5 to match and build the file.

Actual Behavior

couple of re-trys and then it returns error

To Reproduce

** Steps to reproduce the behavior, e.g.
Try to build shared_queue_vendor

System (please complete the following information)

ubuntu 18.04, master

Additional context

** Add any other context about the problem here **
I suggest fixing the error message as well, it doesn't print the actual url that can't be downloaded and the problem is clearly the md5sum match but it doesn't mention that in the error message.

@felrock felrock added the bug Something isn't working label Feb 1, 2022
@Henrycious
Copy link

You can look there: same error. For me it helped to update to the newest version of CMake

Duplicate of #955

@felrock
Copy link
Author

felrock commented Feb 1, 2022

Yeah, the error roots from the md5sum change in the zip file i posted. Not sure how a update to CMake should do any different here. md5sum exists to prevent changes, and here we have a change. Might even be an attack of some sorts.

@felrock
Copy link
Author

felrock commented Feb 1, 2022

#281 Made a issue on concurrentqueue as well

@geoporus1
Copy link

@felrock good catch! It is very weird why the md5sum would change out of the blue after so long time. Maybe someone can answer this! Good you opened also an issue on concurrentqueue as well!

@clalancette
Copy link
Contributor

This still looks like a GitHub backend error, as we saw this error on multiple different packages in multiple different ways overnight. Right now, I can build shared_queues_vendor locally without changes, so I believe that this is fixed on the GitHub backend now.

@felrock
Copy link
Author

felrock commented Feb 1, 2022

How can a backend issue have an effect on a zip file? Which other repos were effected? I only noticed this one

@clalancette
Copy link
Contributor

@gavanderhoorn
Copy link

Related / follow-up on ROS Answers: Circumvent _vendor projects.

@felrock
Copy link
Author

felrock commented Feb 3, 2022

It's not even close to a follow-up discussion regarding this problem. I refereed to this issue as example as to why vendor projects suck, in my opinion ofc.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

5 participants