← Back to index | Original Bugzilla link

Bug 12125 – Win64 phobos has hard references to LIBCMT

Status
RESOLVED
Resolution
FIXED
Severity
major
Priority
P2
Component
phobos
Product
D
Version
D2
Platform
x86_64
OS
Windows
Creation time
2014-02-09T21:04:24Z
Last change time
2018-03-23T07:09:56Z
Keywords
industry, pull
Assigned to
No Owner
Creator
Manu

Comments

Comment #0 by turkeyman — 2014-02-09T21:04:24Z
Unless phobos has some dependencies on a particular flavour of MS CRT, then it should be built with /Zl to omit the reference in the object files. If phobos does depend on the particular flavour of CRT it links against, then a suite of phobos libs would need to be built to match the suite of CRT options. I don't think this is the case however. Currently, linking phobos causes CRT conflicts when linking in conjunction with other libs that depend on a different CRT.
Comment #1 by turkeyman — 2014-12-30T01:28:57Z
This should be a trivial fix, has anyone looked at it? Just add /Zl when compiling...
Comment #2 by r.sagitario — 2016-11-06T18:58:45Z
https://github.com/dlang/druntime/pull/1691 https://github.com/dlang/phobos/pull/4891
Comment #3 by github-bugzilla — 2016-11-13T02:21:59Z
Commit pushed to master at https://github.com/dlang/phobos https://github.com/dlang/phobos/commit/60cd8d2aa70e1c9cfd7c58fa42bce7345fd00b77 Merge pull request #4891 from rainers/nodefaultlib Issue 12125: do not enforce specific MS runtime
Comment #4 by github-bugzilla — 2016-12-27T13:11:25Z
Commit pushed to scope at https://github.com/dlang/phobos https://github.com/dlang/phobos/commit/60cd8d2aa70e1c9cfd7c58fa42bce7345fd00b77 Merge pull request #4891 from rainers/nodefaultlib
Comment #5 by github-bugzilla — 2017-01-07T03:02:39Z
Commit pushed to stable at https://github.com/dlang/phobos https://github.com/dlang/phobos/commit/60cd8d2aa70e1c9cfd7c58fa42bce7345fd00b77 Merge pull request #4891 from rainers/nodefaultlib
Comment #6 by github-bugzilla — 2017-01-16T23:24:57Z
Commit pushed to newCTFE at https://github.com/dlang/phobos https://github.com/dlang/phobos/commit/60cd8d2aa70e1c9cfd7c58fa42bce7345fd00b77 Merge pull request #4891 from rainers/nodefaultlib
Comment #7 by github-bugzilla — 2017-01-18T01:26:45Z
Commit pushed to stable at https://github.com/dlang/druntime https://github.com/dlang/druntime/commit/0dc5d1dd68c2bcffb9a501f2e37ff28be3abd87e Merge pull request #1691 from rainers/nodefaultlib Issue 12125: do not enforce specific MS C runtime
Comment #8 by github-bugzilla — 2017-01-18T13:11:30Z
Commit pushed to master at https://github.com/dlang/druntime https://github.com/dlang/druntime/commit/0dc5d1dd68c2bcffb9a501f2e37ff28be3abd87e Merge pull request #1691 from rainers/nodefaultlib
Comment #9 by github-bugzilla — 2017-01-24T11:54:55Z
Commit pushed to newCTFE at https://github.com/dlang/druntime https://github.com/dlang/druntime/commit/0dc5d1dd68c2bcffb9a501f2e37ff28be3abd87e Merge pull request #1691 from rainers/nodefaultlib
Comment #10 by turkeyman — 2018-03-23T06:16:48Z
is this resolved?
Comment #11 by r.sagitario — 2018-03-23T07:09:56Z
(In reply to Manu from comment #10) > is this resolved? Yes, starting with dmd 2.073.