Changed: Use %~X instead of %X because it removes double quotes

This commit is contained in:
kervala 2016-08-26 13:30:30 +02:00
parent 77c9be8d5b
commit d71751293b

View file

@ -822,8 +822,8 @@ void CPatchManager::createBatchFile(CProductDescriptionForClient &descFile, bool
batchRelativeDstPath = CPath::standardizeDosPath(batchRelativeDstPath); batchRelativeDstPath = CPath::standardizeDosPath(batchRelativeDstPath);
// use DSTPATH and SRCPATH variables and append filenames // use DSTPATH and SRCPATH variables and append filenames
string realDstPath = toString("%%ROOTPATH%%\"\\%s\"", batchRelativeDstPath.c_str()); string realDstPath = toString("\"%%ROOTPATH%%\\%s\"", batchRelativeDstPath.c_str());
string realSrcPath = toString("%%UNPACKPATH%%\"\\%s\"", FileName.c_str()); string realSrcPath = toString("\"%%UNPACKPATH%%\\%s\"", FileName.c_str());
content += toString(":loop%u\n", nblab); content += toString(":loop%u\n", nblab);
content += toString("attrib -r -a -s -h %s\n", realDstPath.c_str()); content += toString("attrib -r -a -s -h %s\n", realDstPath.c_str());
@ -872,7 +872,7 @@ void CPatchManager::createBatchFile(CProductDescriptionForClient &descFile, bool
if (!succeeded) if (!succeeded)
{ {
#ifdef NL_OS_WINDOWS #ifdef NL_OS_WINDOWS
patchContent += toString("del %%ROOTPATH%%\"\\patch\\%s\"\n", vFileList[i].c_str()); patchContent += toString("del \"%%ROOTPATH%%\\patch\\%s\"\n", vFileList[i].c_str());
#else #else
patchContent += toString("rm -f \"$ROOTPATH/patch/%s\"\n", vFileList[i].c_str()); patchContent += toString("rm -f \"$ROOTPATH/patch/%s\"\n", vFileList[i].c_str());
#endif #endif
@ -886,8 +886,8 @@ void CPatchManager::createBatchFile(CProductDescriptionForClient &descFile, bool
content += patchContent; content += patchContent;
content += "rd /Q /S %%ROOTPATH%%\"\\patch\"\n"; content += "rd /Q /S \"%%ROOTPATH%%\\patch\"\n";
content += "if exist %%ROOTPATH%%\"\\patch\" goto looppatch\n"; content += "if exist \"%%ROOTPATH%%\\patch\" goto looppatch\n";
#else #else
content += "rm -rf \"$ROOTPATH/patch\"\n"; content += "rm -rf \"$ROOTPATH/patch\"\n";
#endif #endif
@ -919,10 +919,10 @@ void CPatchManager::createBatchFile(CProductDescriptionForClient &descFile, bool
//use bat if windows if not use sh //use bat if windows if not use sh
#ifdef NL_OS_WINDOWS #ifdef NL_OS_WINDOWS
contentPrefix += "@echo off\n"; contentPrefix += "@echo off\n";
contentPrefix += "set RYZOM_CLIENT=%1\n"; contentPrefix += "set RYZOM_CLIENT=%~1\n";
contentPrefix += "set UNPACKPATH=%2\n"; contentPrefix += "set UNPACKPATH=%~2\n";
contentPrefix += "set ROOTPATH=%3\n"; contentPrefix += "set ROOTPATH=%~3\n";
contentPrefix += "set STARTUPPATH=%4\n"; contentPrefix += "set STARTUPPATH=%~4\n";
contentPrefix += toString("set UPGRADE_FILE=%%ROOTPATH%%\\%s\n", UpgradeBatchFilename.c_str()); contentPrefix += toString("set UPGRADE_FILE=%%ROOTPATH%%\\%s\n", UpgradeBatchFilename.c_str());
contentPrefix += "\n"; contentPrefix += "\n";
contentPrefix += "set LOGIN=%5\n"; contentPrefix += "set LOGIN=%5\n";
@ -955,12 +955,12 @@ void CPatchManager::createBatchFile(CProductDescriptionForClient &descFile, bool
#ifdef NL_OS_WINDOWS #ifdef NL_OS_WINDOWS
// launch upgrade script if present (it'll execute additional steps like moving or deleting files) // launch upgrade script if present (it'll execute additional steps like moving or deleting files)
contentSuffix += "if exist %UPGRADE_FILE% call %UPGRADE_FILE%\n"; contentSuffix += "if exist \"%UPGRADE_FILE%\" call \"%UPGRADE_FILE%\"\n";
if (wantRyzomRestart) if (wantRyzomRestart)
{ {
// client shouldn't be in memory anymore else it couldn't be overwritten // client shouldn't be in memory anymore else it couldn't be overwritten
contentSuffix += toString("start \"\" /D %%STARTUPPATH%% %%RYZOM_CLIENT%% %s %%LOGIN%% %%PASSWORD%% %%SHARDID%%\n", additionalParams.c_str()); contentSuffix += toString("start \"\" /D \"%%STARTUPPATH%%\" \"%%RYZOM_CLIENT%%\" %s %%LOGIN%% %%PASSWORD%% %%SHARDID%%\n", additionalParams.c_str());
} }
#else #else
if (wantRyzomRestart) if (wantRyzomRestart)