Issue #8403 has been updated by Jeff McCune. Status changed from In Topic Branch Pending Merge to Merged - Pending Release
# Merged into master # Dan reviewed, tech mailed. Merged as: <pre> commit ac2e54024e38104c32b4083460243da59047e092 Merge: 81f0a50 73b147d Author: Jeff McCune <[email protected]> Date: Wed Jul 13 16:00:19 2011 -0700 Merge branch 'ticket/master/8403_stop_using_uuidgen' * ticket/master/8403_stop_using_uuidgen: (#8403) Add guid gem requirement to README (#8403) Do not require the uuidgen executable commit 73b147d3b6d65340850e2308be453f7c76d90766 Author: Jeff McCune <[email protected]> Date: Wed Jul 13 15:56:36 2011 -0700 (#8403) Add guid gem requirement to README This change adds the two gem requirements CloudPack has to the README file. Dan mentioned this should be documented in the code review. Reviewed-by: Dan Bode commit 1b7a2d99437abbbb4f8ccfa2db5f4b19cefb8d6e Author: Jeff McCune <[email protected]> Date: Wed Jul 13 15:08:28 2011 -0700 (#8403) Do not require the uuidgen executable The uuidgen executable is platform specific. This change uses the guid gem instead, which must be installed now for CloudPack to function. Reviewed-by: Dan Bode </pre> ---------------------------------------- Bug #8403: Cloud Pack should not use uuidgen executable https://projects.puppetlabs.com/issues/8403 Author: Jeff McCune Status: Merged - Pending Release Priority: Normal Assignee: Jeff McCune Category: cloudpack Target version: 0.6.0 Keywords: Branch: This is platform specific. There's a gem out there to calculate UUID's. Use that. <pre> if not test('f', '/usr/bin/uuidgen') raise "/usr/bin/uuidgen does not exist; please install uuidgen." elsif not test('x', '/usr/bin/uuidgen') raise "/usr/bin/uuidgen is not executable; please change that file's permissions." end certname = `/usr/bin/uuidgen`.downcase.chomp </pre> -- You have received this notification because you have either subscribed to it, or are involved in it. To change your notification preferences, please click here: http://projects.puppetlabs.com/my/account -- You received this message because you are subscribed to the Google Groups "Puppet Bugs" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/puppet-bugs?hl=en.
