Skip to content

Conversation

brad-anderson
Copy link
Contributor

@WalterBright This pull request relies on curl-7.24.0-dmd-win32.zip being uploaded to ftp://ftp.digitalmars.com/ before it should be merged.

@dnadlinger
Copy link
Contributor

Is there a good reason to use FTP for downloads these days? HTTP should generally cause less problems with proxies, etc.

@WalterBright
Copy link
Member

It's downloading from github now.

On 2/29/2012 10:36 AM, David Nadlinger wrote:

Is there a good reason to use FTP for downloads these days? HTTP should generally cause less problems with proxies, etc.


Reply to this email directly or view it on GitHub:
#92 (comment)

@brad-anderson
Copy link
Contributor Author

@WalterBright If you want to upload curl-7.24.0-dmd-win32.zip to GitHub instead of Digital Mars ftp I'd happily change all of the links to point at Github. I can already change the dmd.zip in the installer to point to the Github download as it is done here on the download page.

@braddr
Copy link
Member

braddr commented Feb 29, 2012

See also: my comment on the commit that adds curl to the windows build. I put the curl.lib file from this zip in place and it still fails to pass the windows phobos tests. I don't have the time to dig into what's going wrong.

@WalterBright
Copy link
Member

It's on github now.

@WalterBright
Copy link
Member

Though Brad says it doesn't pass the Phobos tests.

@andralex
Copy link
Member

andralex commented Mar 2, 2012

@WalterBright: good to go?

@brad-anderson
Copy link
Contributor Author

This link is fine to merge, I'd say. Phobos is still pending a solution though (and the installer pull, while it would work fine, is useless without the phobos change).

andralex added a commit that referenced this pull request Mar 3, 2012
@andralex andralex merged commit 4051330 into dlang:master Mar 3, 2012
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.

5 participants