Using submodules properly