PeterMosmans / ansible-role-virtualbox-guest

Ansible role for building and installing VirtualBox guest additions
GNU General Public License v3.0
20 stars 22 forks source link

Option to detect host VirtualBox version to match GuestAdditions installation #3

Closed ijkim88 closed 8 years ago

ijkim88 commented 8 years ago

I added an option to the virtualbox_version variable, when set to auto it will detect the host's VirtualBox version and install the matching version of GuestAdditions. I decided to use a block for this change for logical grouping. I believe it would break backward compatibility since this feature was introduced in 2.0.

PeterMosmans commented 8 years ago

Hi @ijkim88 , thanks once again. It'll take me a little bit longer to test/validate these changes, but they look good so far.

I'll add your name as contributor to the README.md page.

Cheers, thanks and keep 'em coming,

Peter

PeterMosmans commented 8 years ago

I added a check to see whether the auto command was successful. For the rest: great work. I added you to README.md

Thanks,

Peter