Windows files not installed on Linux #140

Closed
alexrp opened this Issue Jul 16, 2012 · 5 comments

Comments

Projects
None yet
2 participants
Contributor

alexrp commented Jul 16, 2012

Windows files (such as core.sys.windows.windows) are not installed on Linux. This seems logical enough, but it can cause errors when using Ddoc to generate documentation. I would strongly recommend including all modules regardless of platform when installing.

Owner

klickverbot commented Jul 18, 2012

Hm, how would this cause problems with Ddoc? I don't really see a problem with installing all OS-specific files on all platforms, the added bloat should be negligible, but the description sounds rather like a bug in the frontend/druntime/your code.

Contributor

alexrp commented Jul 18, 2012

I'll investigate later today (or tomorrow) and get you the exact error. I don't know if it's a front end bug or whatever, but I know it was causing errors when generating Ddoc documentation for Windows-specific code.

Owner

klickverbot commented Jul 18, 2012

Hm, so your (or druntime's) Windows-specific code is not wrapped in a version (Windows) block, but rather assumes the files to be always present? Not sure if/how we want to support that, given how OS-specific functions are usually handled in Phobos.

Contributor

alexrp commented Jul 18, 2012

My source base has a central configuration module which uses version statements to set a bunch of enum values. The rest of the source base then uses those, so e.g.:

static if (isWindows)
{
    // Windows-specific code goes here...
}
Owner

klickverbot commented Dec 30, 2012

I think this has been fixed a while ago.

redstar pushed a commit that referenced this issue Sep 27, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment