Opened 7 years ago

Closed 5 years ago

#11841 closed Bugs (duplicate)

advapi does not exist on windows phone

Reported by: Michael Lacher <milacher@…> Owned by: James E. King, III
Milestone: Boost 1.66.0 Component: uuid
Version: Boost 1.59.0 Severity: Showstopper
Keywords: Cc:

Description

Boost 1.59.0 introduces a hard dependency on advapi32.lib in "boost/uuid/see_rng.hpp" which does not exist on windows phone. This makes this library unusable for all windows phone builds.

Change History (8)

comment:1 by erika@…, 6 years ago

This is still valid for 1.62.0

comment:2 by James E. King, III <jking@…>, 5 years ago

If you use either of the constructors of basic_random_generator which take a reference or a pointer to a UniformRandomNumberGenerator, those do not call detail::seed and therefore should be compatible. Given I don't currently have access to a windows phone build environment, do you have any local mods (perhaps similar to the _WIN32_WCE ifdefs) that resolve the issue?

comment:3 by James E. King, III, 5 years ago

Milestone: To Be DeterminedBoost 1.66.0
Owner: changed from Andy Tompkins to James E. King, III

I believe if you use the following code it will resolve your issue, please try and see:

boost::mt19937 ran; boost::uuids::uuid uuid = boost::uuids::random_generator(ran)();

Further, it looks like Windows Phone is essentially dead now, so I am resolving this as wontfix. Feel free to reopen (with supporting comment, of course) if you disagree or you are not able to work around it.

comment:4 by James E. King, III, 5 years ago

Resolution: wontfix
Status: newclosed

comment:5 by anonymous, 5 years ago

Resolution: wontfix
Status: closedreopened

comment:6 by James E. King, III, 5 years ago

This issue was reported again here:

https://github.com/boostorg/uuid/issues/24

The workaround provided was not sufficient so I submitted a PR to modify how we generate entropy for the default random generator constructor.

https://github.com/boostorg/uuid/pull/25

comment:7 by James E. King, III, 5 years ago

Given this is already being tracked in github issues I am going to resolve this one. Please follow it in github issues.

https://github.com/boostorg/uuid/issues/24

Current plans are to get boostorg/winapi to be UWP compatible, and then to update boost::random_device to be UWP compatible (and make the library header-only too), and then we won't need seed_rng any more in uuid.

comment:8 by James E. King, III, 5 years ago

Resolution: duplicate
Status: reopenedclosed
Note: See TracTickets for help on using tickets.