add submodule properly this time