Re: [edk2] [PATCH v2 1/2] CryptoPkg/OpensslLib: Fix the documentation about submodule update

2018-04-13 Thread Laszlo Ersek
On 04/12/18 12:03, Long, Qin wrote: > > Ah, "it's wrong here" means "the existence of "--remote" in original > suggested command is wrong". > "It's important" looks also make sense to address the "update" goal. I can > update that, if old message will cause confusion. With the commit message

Re: [edk2] [PATCH v2 1/2] CryptoPkg/OpensslLib: Fix the documentation about submodule update

2018-04-12 Thread Long, Qin
Ah, "it's wrong here" means "the existence of "--remote" in original suggested command is wrong". "It's important" looks also make sense to address the "update" goal. I can update that, if old message will cause confusion. Best Regards & Thanks, LONG, Qin -Original Message- From:

Re: [edk2] [PATCH v2 1/2] CryptoPkg/OpensslLib: Fix the documentation about submodule update

2018-04-12 Thread Laszlo Ersek
Hello Qin, On 04/12/18 05:08, Long Qin wrote: > This patch is to drop "--remote" option from the original suggested > submodule update command ("$ git submodule update --recursive > --remote") in HOWTO document. > > "--remote" option will integrate changes from the upstream subproject > with the

[edk2] [PATCH v2 1/2] CryptoPkg/OpensslLib: Fix the documentation about submodule update

2018-04-11 Thread Long Qin
This patch is to drop "--remote" option from the original suggested submodule update command ("$ git submodule update --recursive --remote") in HOWTO document. "--remote" option will integrate changes from the upstream subproject with the submodules's "current HEAD", instead of using the edk2