Re: [boost] [random] Merging to master for 1.75 release

On Nov 24, 2020, at 12:47 PM, Edward Diener via Boost wrote:

On 11/24/2020 2:10 PM, Marshall Clow via Boost wrote:

...

On Nov 24, 2020, at 10:55 AM, Steven Watanabe via Boost wrote:

...

AMDG

On 11/18/20 11:18 AM, Edward Diener via Boost wrote:

...

On 11/18/2020 10:18 AM, Glen Fernandes via Boost wrote:

...

On Wed, Nov 18, 2020 at 5:55 AM Edward Diener wrote:

...

Is there any possibility of merging 'develop' to 'master' for the random library for the 1.75 release ? The 'master' branch has not been updated for 2 years, or since the 1.70 release. While I am not claiming all invaluable fixes I know my fix for Embarcadero C++ clang-based support and another fix for correctly auto-linking random_device is among the 'develop' branch fixes which have never made it to the 'master' branch.

I'm assuming this is for Steven (who is the active maintainer of Random). i.e. Unlike the other repositories which we've helped out with, that are either under CMT or no active maintainer.

A maintainer of a library has the right to do what he wants. But it does not seem that Steven is aware of the situation. Also others may have write access to 'random' for all I know, and be able to review the merge and make the change. In the 'develop' regression tests for 'random' there are no issues which I can see in the results and the last change was over 4 months ago.

I can merge it now if it's okay with the release managers.

I looked at the commits since the last merge, and only this one looks problematic for a post-beta merge. (We try not to add new features after the beta) https://github.com/boostorg/random/commit/b8d5df31168a07e04a175ace751f0defa7... https://github.com/boostorg/random/commit/b8d5df31168a07e04a175ace751f0defa7...<https://github.com/boostorg/random/commit/b8d5df31168a07e04a175ace751f0defa7... https://github.com/boostorg/random/commit/b8d5df31168a07e04a175ace751f0defa7...> All the rest are fine

The commit you cite above does not change anything for the end-user. All the fix does is use type_traits rather than mpl in a number of cases, which removes the mpl dependency for the 'random' library, and all those cases are private implementation details which does not affect the end-user's use of the library in any way. This commit is hardly adding a new feature to the 'random' library.

Then I copy/pasted the wrong URL; sorry. https://github.com/boostorg/random/commit/1182a4071e6bcdaa94e4c49e4a220885b0... https://github.com/boostorg/random/commit/1182a4071e6bcdaa94e4c49e4a220885b0... This looks like a new feature to me. (I am willing to be convinced otherwise, though) — Marshall