Skip to content

dmd.root.filename: Define PATH_MAX if missing in druntime#9382

Merged
dlang-bot merged 1 commit intodlang:stablefrom
ibuclaw:missingpathmax
Feb 23, 2019
Merged

dmd.root.filename: Define PATH_MAX if missing in druntime#9382
dlang-bot merged 1 commit intodlang:stablefrom
ibuclaw:missingpathmax

Conversation

@ibuclaw
Copy link
Member

@ibuclaw ibuclaw commented Feb 23, 2019

If #9261 introduced a regression, this would squash it (though dmd should really fix their release process).

@ibuclaw ibuclaw added the Severity:Regression PRs that fix regressions label Feb 23, 2019
@dlang-bot
Copy link
Contributor

Thanks for your pull request, @ibuclaw!

Bugzilla references

Your PR doesn't reference any Bugzilla issue.

If your PR contains non-trivial changes, please reference a Bugzilla issue or create a manual changelog.

Testing this PR locally

If you don't have a local development environment setup, you can use Digger to test this PR:

dub fetch digger
dub run digger -- build "stable + dmd#9382"

Copy link
Contributor

@thewilsonator thewilsonator left a comment

Choose a reason for hiding this comment

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

Should this go to stable?

@jacob-carlborg
Copy link
Contributor

Should this go to stable?

Yes, #9261 is in stable.

@dlang-bot dlang-bot merged commit a934a23 into dlang:stable Feb 23, 2019
@ibuclaw ibuclaw deleted the missingpathmax branch February 23, 2019 12:13
@jondegenhardt
Copy link

This appears to have fixed Issue 19684: "DMD 2.085.0-beta.1: DMD cannot find dmd.conf if invoked via symlink (OSX)".

I tested this by downloading the updated 2.085-beta and the issue no longer surfaces. Closing the bug.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants