Bug 16682 – [REG 2.072] "privatization" of symbols in std.stdio breaks DFMT

Status
RESOLVED
Resolution
FIXED
Severity
regression
Priority
P1
Component
phobos
Product
D
Version
D2
Platform
All
OS
Windows
Creation time
2016-11-12T12:16:00Z
Last change time
2017-01-16T23:24:59Z
Assigned to
nobody
Creator
b2.temp

Comments

Comment #0 by b2.temp — 2016-11-12T12:16:10Z
A few Symbols made private in 2.072.0 breaks the D tool DFMT. See: https://github.com/Hackerpilot/dfmt/issues/263 https://github.com/dlang/phobos/pull/4789 Popular D projects is the best integration test possible. We need a project tester as a continuous integration service ASAP!
Comment #1 by public — 2016-11-15T10:46:57Z
Comment #2 by code — 2016-11-16T14:48:21Z
(In reply to b2.temp from comment #0) > Popular D projects is the best integration test possible. We need a project > tester as a continuous integration service ASAP! Please avoid OT stuff in Bugzilla, anyhow we're currently testing a Jenkins instance to do this, https://ci.dawg.eu/.
Comment #3 by github-bugzilla — 2016-11-16T14:54:13Z
Commits pushed to stable at https://github.com/dlang/phobos https://github.com/dlang/phobos/commit/73b699de123a3f8d2e38f0d63735865fa78c03ce Revert "Merge pull request #4789 from JackStouffer/package2" This reverts commit c940f5a4182f018901bf6d933fc65c74f817c71a, reversing changes made to aaf31b531c735d09e2527a5213fcf6634d77086e. Fixes issue 16682 https://github.com/dlang/phobos/commit/8b1a270f367118cc6114b1d4bf0cd5781567552e Merge pull request #4902 from Dicebot/revert-regressions3 Fix issue 16682: "privatization" of symbols in std.stdio breaks DFMT
Comment #4 by github-bugzilla — 2016-12-07T16:09:06Z
Commits pushed to master at https://github.com/dlang/phobos https://github.com/dlang/phobos/commit/73b699de123a3f8d2e38f0d63735865fa78c03ce Revert "Merge pull request #4789 from JackStouffer/package2" https://github.com/dlang/phobos/commit/8b1a270f367118cc6114b1d4bf0cd5781567552e Merge pull request #4902 from Dicebot/revert-regressions3
Comment #5 by github-bugzilla — 2016-12-27T13:11:27Z
Commits pushed to scope at https://github.com/dlang/phobos https://github.com/dlang/phobos/commit/73b699de123a3f8d2e38f0d63735865fa78c03ce Revert "Merge pull request #4789 from JackStouffer/package2" https://github.com/dlang/phobos/commit/8b1a270f367118cc6114b1d4bf0cd5781567552e Merge pull request #4902 from Dicebot/revert-regressions3
Comment #6 by github-bugzilla — 2017-01-16T23:24:59Z
Commits pushed to newCTFE at https://github.com/dlang/phobos https://github.com/dlang/phobos/commit/73b699de123a3f8d2e38f0d63735865fa78c03ce Revert "Merge pull request #4789 from JackStouffer/package2" https://github.com/dlang/phobos/commit/8b1a270f367118cc6114b1d4bf0cd5781567552e Merge pull request #4902 from Dicebot/revert-regressions3