Skip to content

Commit

Permalink
imp - Don't invoke msbuild directly
Browse files Browse the repository at this point in the history
---

For build scripts, invoking dotnet with msbuild is taking us back to the days where we would build "msbuild" for Mono in a PPA explicitly using mono-msbuild in April 2021. We've changed the dotnet command to be more current as the "msbuild" approach is so 2021.

---

Type: imp
Breaking: False
Doc Required: False
Backport Required: False
Part: 1/1
  • Loading branch information
AptiviCEO committed Aug 22, 2024
1 parent 28f54bd commit 0c27839
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 6 deletions.
6 changes: 3 additions & 3 deletions tools/build.cmd
Original file line number Diff line number Diff line change
Expand Up @@ -23,14 +23,14 @@ if "%releaseconfig%" == "" set releaseconfig=Release

:download
echo Downloading packages...
"%ProgramFiles%\dotnet\dotnet.exe" msbuild "..\ChemiStar.sln" -t:restore -p:Configuration=%releaseconfig%
"%ProgramFiles%\dotnet\dotnet.exe" restore "..\ChemiStar.sln" --configuration %releaseconfig%
if %errorlevel% == 0 goto :build
echo There was an error trying to download packages (%errorlevel%).
goto :finished

:build
echo Building...
"%ProgramFiles%\dotnet\dotnet.exe" msbuild "..\ChemiStar.sln" -p:Configuration=%releaseconfig%
echo Building ChemiStar...
"%ProgramFiles%\dotnet\dotnet.exe" build "..\ChemiStar.sln" --configuration %releaseconfig%
if %errorlevel% == 0 goto :success
echo There was an error trying to build (%errorlevel%).
goto :finished
Expand Down
6 changes: 3 additions & 3 deletions tools/build.sh
Original file line number Diff line number Diff line change
Expand Up @@ -32,15 +32,15 @@ fi

# Download packages
echo Downloading packages...
"$dotnetpath" msbuild "../ChemiStar.sln" -t:restore -p:Configuration=$releaseconf
"$dotnetpath" restore "../ChemiStar.sln" --configuration $releaseconf
if [ ! $? == 0 ]; then
echo Download failed.
exit 1
fi

# Build
echo Building...
"$dotnetpath" msbuild "../ChemiStar.sln" -p:Configuration=$releaseconf
echo Building ChemiStar...
"$dotnetpath" build "../ChemiStar.sln" --configuration $releaseconf
if [ ! $? == 0 ]; then
echo Build failed.
exit 1
Expand Down

0 comments on commit 0c27839

Please sign in to comment.