Skip to content

Conversation

dnadlinger
Copy link
Contributor

Previously, the code was special-cased to use stat64() on OS X for no apparent reason (I searched the logs, but couldn't find anything obvious). This also caused the new symlink() unit tests to fail.

I additionally removed further special casing that excluded certain unit tests from being run on OS X, because I can see no apparent reason for them not to work there as well (and indeed, they all pass on my OS X Lion box).

Previously, the code was special-cased to use `stat64()` on OS X for no apparent reason (I searched the logs, but couldn't find anything obvious). This also caused the new `symlink()` unit tests to fail.
I can see no obvious reason why they wouldn't work there, and indeed they all pas on my OS X Lion box.
@jmdavis
Copy link
Member

jmdavis commented Jul 30, 2011

I believe that tests were previously removed precisely because they were breaking and someone (Andrei IIRC) decided that it was easier/better to remove them in the short term. So, if you've fixed them an re-enabled them, that's great.

@jmdavis
Copy link
Member

jmdavis commented Jul 30, 2011

Well, it looks good from what I can see. So, if it passes on your machine, it should be solid. I'll merge it in.

jmdavis added a commit that referenced this pull request Jul 30, 2011
Use lstat64() on OS X like on other Posix systems.
@jmdavis jmdavis merged commit 1278f7c into dlang:master Jul 30, 2011
@jmdavis
Copy link
Member

jmdavis commented Jul 30, 2011

And thank you for taking the time to fix this. I don't have access to a Mac, so this is a big help.

@dnadlinger
Copy link
Contributor Author

Yeah, sure, no problem. I still wonder why the OS X code was special cased not to use lstat64 in the first place, though, as the docs say it existed since 10.5 just like stat64. I had another look at the commit log, and it seems like it was added as a quick build fix by Andrei because you switched the code to use lstat64 on Linux in your d8488a9, but (understandably, if you don't have access to an OS X box) omitted the declaration for OS X.

Well, let's just wait and see if any related issues pop up, I suppose…

kuettler pushed a commit to kuettler/phobos that referenced this pull request Feb 6, 2018
Hand off d.tag generation to dlang.org makefiles
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.

2 participants