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
failing downloads with dcmtk #33
Comments
Hi, Can you give any specific errors? git over http places most of the logic onto the client, so it should be not too old -- maybe it's worth checking the version and update if necessary. Enabling the git protocol is an option I will discuss with the team. |
Here is an example: http://www.cdash.org/slicer4/viewBuildError.php?buildid=1096 Logs for reference: CMake Error at DCMTK-cmake/tmp/DCMTK-gitclone.cmake:19 (message): Failed to clone repository: 'http://git.dcmtk.org/dcmtk.git' [ 42%] No patch step for 'DCMTK' |
We internally had problems also with a version 1.5.xxx which is from 2008 it seems, so this might be the problem. Nevertheless I will set up a git protocol server as an alternative since it is also more efficient than using http and very easy to configure. I am pretty busy with another task at the moment, but you can probably expect it this week. |
Sounds like a great plan. It's also good to know about these http issues with git. |
The git protocol should now be available for dcmtk, i.e.
should work now :-) |
I'm having the same failure on different machines: |
Oops, that's a classic one...forgot to open the firewall on port 9418 for the world outside OFFIS ;-) Please try again, sorry for inconvenience. |
Fantastic, that works ! |
With slicer dashboards, we have frequent (>2x a week?) dashboards that fail the step of downloading dcmtk.
I suspect that part of the reason is because the download uses the http protocol.
Would it be possible for the offis team to add the git protocol as a way to access the dcmtk repo?
If it is not feasible, then I guess we can clone the official repo on github/commontk and use that one instead, hopefully it will solve our download failure.
The text was updated successfully, but these errors were encountered: