-
Notifications
You must be signed in to change notification settings - Fork 18
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Swap Data::UUID for PurePerl implementation? #25
Comments
Merged
fujiwara
added a commit
that referenced
this issue
May 22, 2017
Changelog diff is: diff --git a/Changes b/Changes index bcc48e3..a276bd0 100644 --- a/Changes +++ b/Changes @@ -2,6 +2,9 @@ Revision history for Fluent::Logger {{$NEXT}} +0.24 2017-05-22T02:10:23Z + * Switch to UUID::Tiny from Data::UUID (#25 #26) + 0.23 2017-05-16T04:12:25Z * Data::MessagePack::Stream(for ack option) is optional (#23 chorny #24)
I understood. I switched to use UUID::Tiny on v0.24. Thank you! |
fujiwara
added a commit
that referenced
this issue
Jun 2, 2017
Changelog diff is: diff --git a/Changes b/Changes index a276bd0..c1066c5 100644 --- a/Changes +++ b/Changes @@ -2,6 +2,9 @@ Revision history for Fluent::Logger {{$NEXT}} +0.25 2017-06-02T01:41:24Z + * Fix tests failure on Perl 5.26 (#27) + 0.24 2017-05-22T02:10:23Z * Switch to UUID::Tiny from Data::UUID (#25 #26)
This issue was closed.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hi, I've noticed you've listed Data::UUID as a dependency. Would there be any chance of you switching to UUID::Tiny instead? It removes the C dependency and, this is just my opinion, the API is more perl-ish.
The text was updated successfully, but these errors were encountered: