ManageIQ / manageiq-smartstate

Apache License 2.0
4 stars 36 forks source link

Replace miq-uuid with uuidtools #130

Closed djberg96 closed 4 years ago

djberg96 commented 4 years ago

As far as I can tell, the only reason to require miq-uuid within this repo is to load the uuidtools library (which is require'd within the miq-uuid.rb file). The MiqUUID module doesn't appear to be used anywhere in this code (it only defines one method - clean_guid), but UUIDTools::UUID.parse_raw is used directly in several places.

So, as part of the https://github.com/ManageIQ/manageiq-gems-pending/issues/231 effort, this PR declares the uuidtools library as a dependency, and updates the various require statements to use uuidtools instead of miq-uuid.