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

Website Broken Link Report #3818

Closed
Joombalaya opened this issue Apr 30, 2019 · 4 comments

Comments

@Joombalaya
Copy link

commented Apr 30, 2019

The link https://support.microsoft.com/kb/140584/en-us (in INSTALL.md) for "How to link with the correct C Run-Time CRT library" is currently down. I am not sure if this link needs to be removed or updated to point to more current Windows information.

@bagder

This comment has been minimized.

Copy link
Member

commented Apr 30, 2019

Since Microsoft removed it, we should probably not try to link to it but I found it archived here:

https://jeffpar.github.io/kbarchive/kb/140/Q140584/
https://web.archive.org/web/20140320202443/http://support.microsoft.com/kb/140584

Is the info still relevant? Should we maintain our own docs for it?

@Joombalaya

This comment has been minimized.

Copy link
Author

commented Apr 30, 2019

I am a novice when it comes to cURL. I simply noticed as a matter of course when going through the documents for the first time, so I do not know if the information is still relevant. I imagine users who compile on the Windows environment need to be aware of the caveats associated with DLL vs Static regardless.

@jay

This comment has been minimized.

Copy link
Member

commented May 1, 2019

Is the info still relevant? Should we maintain our own docs for it?

It has some outdated info, it says the default switch is single threaded but in any recent compiler it's the multithreaded crt dll (release /MD, debug /MDd). The crt mixing part is also covered in How To Use the C Run-Time and the compiler switches are documented elsewhere so we could link to that instead

diff --git a/docs/INSTALL.md b/docs/INSTALL.md
index f2f9322..e1a0a3c 100644
--- a/docs/INSTALL.md
+++ b/docs/INSTALL.md
@@ -122,7 +122,7 @@ libressl.
  understanding if you are not going to follow the advice given above.
 
  - [How To Use the C Run-Time](https://support.microsoft.com/kb/94248/en-us)
- - [How to link with the correct C Run-Time CRT library](https://support.microsoft.com/kb/140584/en-us)
+ - [Run-Time Library Compiler Options](https://docs.microsoft.com/en-us/cpp/build/reference/md-mt-ld-use-run-time-library)
  - [Potential Errors Passing CRT Objects Across DLL Boundaries](https://msdn.microsoft.com/en-us/library/ms235460)
 
 If your app is misbehaving in some strange way, or it is suffering from
@bagder

This comment has been minimized.

Copy link
Member

commented May 2, 2019

Sounds good to me!

@bagder bagder closed this in 4d8461a May 12, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Aug 10, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
3 participants
You can’t perform that action at this time.