The only issue I see here is that if getVersionedStormRoot fails for some reason it will return null.  This will result in some difficult to debug issues because the classpath, etc will be set to "./storm/null/*".  It would probably be better to blow up if we cannot find the root, or pick a different default.