Update .gitmodules

[designgears]

Updated to use https, don't have access via ssh.

Please make sure you have the correct access rights
and the repository exists.
fatal: clone of 'git@github.com:switchbrew/libnx.git' into submodule path '/c/nx-sd/components/EdiZon/libs/libnx' failed
Failed to clone 'libs/libnx'. Retry scheduled
Cloning into '/c/nx-sd/components/EdiZon/libs/libnx'...
Permission denied (publickey).
fatal: Could not read from remote repository.

[HookedBehemoth]

I don't get how that even matters for devs. That subrepo is for the CI and I doubt you have LIBNX defined to be in that directory.

[designgears]

It just get's passed over when I compile myself, but I have my scripts setup to compile a lot of things so I have --recursive set as most stuff has submodules. Doesn't hurt anything in my case, it just says it can't connect and moves along because I have the correct libnx build in the docker instance already.

Just seemed more user friendly to have it use https since CI doesn't benefit from it being ssh either.

[AnalogMan151]

Having it be HTTPS would be easier when the make file is updated to make use of the submodule libnx as it doesn’t require people to generate and add an SSH key to their account.

[designgears]

Looks like TRAVIS even has issues with the git user, that would remove the need for that little sed hack in there as well.

[igromanru]

In my opinion git should be used with SSH instead of HTTPS.
Not only that you have to enter your password and/or it will be saved by git, what is a potential security risk, if someone manage to hijack your PC, but it's also easier to authenticate yourself over all different git services without using the same username and password on each platform (GitHub, GitLab, BitBucket, your own hosted git etc.).

But in the first place, @WerWolv should decide what he is using on daily basis. It can be pretty annoying to enter the HTTPS password for a single repo, if you're using ssh-agent for all other repositories.

[designgears]

No need for any sort of security there, it's pulling a public repo in.

[igromanru]

I'm talking about generally usage.
But yeah. In this case it doesn't matter as long nobody wants to replace libnx with an own fork to make some changes.

[WerWolv]

For me it really doesn't matter. As it is right now, the submodule isn't used even used by travis properly. Also in the rewrite this submodule will be removed as well