Skip to content

Conversation

@JeffBezanson
Copy link
Member

closes #25590

@JeffBezanson JeffBezanson added the stdlib Julia's standard library label Jan 30, 2018
@vtjnash
Copy link
Member

vtjnash commented Jan 30, 2018

Looking at the PR, this seems like too much code division. I think we should just keep these couple of functions in Random.

@JeffBezanson
Copy link
Member Author

But nothing else in Random has to do with UUIDs, and nothing in UUIDs has to do with random number generation (it's only a user of it). I don't see why those would need to be grouped together.

@StefanKarpinski
Copy link
Member

Also, this is going to end up being a package that implements all five UUID generation methods and therefore depends on SHA and MD5 (which will be, as discussed, moved into stdlib). Presumably one doesn't want Random to gratuitously depend on those packages at that point.

@JeffBezanson JeffBezanson merged commit 0f95988 into master Jan 31, 2018
@JeffBezanson JeffBezanson deleted the jb/uuid branch January 31, 2018 03:25
@oxinabox
Copy link
Contributor

Shall a new issue be created: "Add UUID-XYZ functionality to stdlib",
with move SHA.jl and MD5.jl as subtasks?

@StefanKarpinski
Copy link
Member

Sure. I would make the tasks "UUID: add uuid3", etc.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stdlib Julia's standard library

Projects

None yet

Development

Successfully merging this pull request may close these issues.

what to do with UUID?

5 participants