← Back to index | Original Bugzilla link

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

https://github.com/dlang/phobos/pull/4902

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