Skip to content
This repository has been archived by the owner on May 16, 2020. It is now read-only.

Replaced hardcoded french sourceforge mirrors by generic urls. #8

Merged
merged 1 commit into from
Jan 4, 2017
Merged

Replaced hardcoded french sourceforge mirrors by generic urls. #8

merged 1 commit into from
Jan 4, 2017

Conversation

fbridault
Copy link
Member

Answer to reported issue in main repo fw4spl-org/fw4spl-doc#1
Normally there should be nothing left.

@docxu233333333333333333
Copy link

@fbridault the replacement process failed :(

@fbridault
Copy link
Member Author

fbridault commented Dec 14, 2016 •
edited
Loading

What do you mean ? Don't pay attention to the appveyor report we were testing it but it can't work correctly yet. Did you try the pull request and do you still have troubles to download some packages ? Could you name them please ?

emiliehar
emiliehar approved these changes Dec 15, 2016
View reviewed changes
Copy link
Member

@emiliehar emiliehar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It works for me on MacOS (10.11.6)

@fbridault
Copy link
Member Author

This is weird, all the packages you mention are indeed hosted on sourceforge, except png, I changed it to github in this pull-request.
Here in France the redirection works fine, in which country do you live ? Do you access to sourceforge main website ? Did you try any of the urls inside the package manually in a web browser, or better with the wget command if you run Linux ? For instance this one : http://downloads.sourceforge.net/project/ogl-math/glm-0.9.6.3/glm-0.9.6.3.zip

Thanks for your feedback

@greenjava
Copy link
Member

👍 It works for me on Windows 10 (MSVC 2015 update 3)

@fbridault fbridault merged commit 4e1e6b3 into fw4spl-org:fw4spl_0.11.0 Jan 4, 2017
@fbridault fbridault deleted the remove_french_mirrors branch January 4, 2017 14:43
fbridault pushed a commit that referenced this pull request Jun 16, 2017
@gKevin @dweckmann
feat(VTK_7.1): Updated VTK to version 7.1
69b1ceb 
This version brings a few bug fixes. (see #8)
Patches have been updates too.
I consequently had to update gdcm to version 2.6.7 and write a patch for it.
fbridault pushed a commit that referenced this pull request Jun 16, 2017
merge(feat/VTK_7_1): into dev
f34edc9 
Updated VTK to version 7.1
This version brings a few bug fixes. (see #8)

See merge request !39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@emiliehar emiliehar emiliehar approved these changes

@greenjava greenjava greenjava approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@fbridault @docxu233333333333333333 @greenjava @emiliehar