search

projectatomic / vagrant-service-manager

To provide the user a CLI to configure the ADB/CDK for different use cases and to provide glue between ADB/CDK and the user's developer environment.
GNU General Public License v2.0
18 stars 16 forks source link

Fix #384: don't execute os_variant for unsupported box #385

Closed coolbrg closed 8 years ago

coolbrg commented 8 years ago

Fix #384

Fixed the regression due to https://github.com/projectatomic/vagrant-service-manager/commit/e5f4d4d0e030bbd12d513746e1034489b9c86b1c

LalatenduMohanty commented 8 years ago

@budhrg we should put some tests for this PR, may be some unit tests. Otherwise LGTM

coolbrg commented 8 years ago

@LalatenduMohanty @navidshaikh : Done with logging error as debug log which is only visible during --debug option.

coolbrg commented 8 years ago

retest this please

coolbrg commented 8 years ago

retest this please

naina-verma commented 8 years ago

@budhrg ,LGTM!