Win mac fixes (Attnam#668)

* Update build-msys2-nonwizard.yml

Install mingw-w64-i686-pcre

* Update build-msys2-wizmode.yml

Install mingw-w64-i686-pcre

* Update error.cpp

* Update rawbit.cpp

* Update message.cpp

* Update item.cpp

rm sprintf

* Update game.cpp

rm sprintf

* Update game.cpp

* Update cmdcraft.cpp

* Update whandler.cpp

* Update game.cpp

Add forgotten comma

* Mac hdiutil resource busy workaround

Mit komplementen:
fredowski/osxbundler@67d668c

* Specify release artifact filename more explicitly

Various updates to 3rd party yaml versions

* Bump actions/checkout to v4

* Update build-osx-wizmode.yml

Author: ryfactor

.github/workflows/build-msys2-nonwizard.yml

@@ -8,13 +8,13 @@ jobs:
       run:
         shell: msys2 {0}
     steps:
-      - uses: actions/checkout@v2
+      - uses: actions/checkout@v4
       - uses: msys2/setup-msys2@v2
         with:
           msystem: MINGW32
           release: false
           update: false
-          install: git mingw-w64-i686-gcc make mingw-w64-i686-cmake mingw-w64-i686-libpng mingw-w64-i686-SDL2 mingw-w64-i686-SDL2_mixer
+          install: git mingw-w64-i686-gcc make mingw-w64-i686-cmake mingw-w64-i686-libpng mingw-w64-i686-SDL2 mingw-w64-i686-SDL2_mixer mingw-w64-i686-pcre
       - name: CI-Build
         run: |
           cd D:/a/ivan/ivan

.github/workflows/build-msys2-wizmode.yml

@@ -8,13 +8,13 @@ jobs:
       run:
         shell: msys2 {0}
     steps:
-      - uses: actions/checkout@v2
+      - uses: actions/checkout@v4
       - uses: msys2/setup-msys2@v2
         with:
           msystem: MINGW32
           release: false
           update: false
-          install: git mingw-w64-i686-gcc make mingw-w64-i686-cmake mingw-w64-i686-libpng mingw-w64-i686-SDL2 mingw-w64-i686-SDL2_mixer
+          install: git mingw-w64-i686-gcc make mingw-w64-i686-cmake mingw-w64-i686-libpng mingw-w64-i686-SDL2 mingw-w64-i686-SDL2_mixer mingw-w64-i686-pcre
       - name: Get tag name, print
         shell: bash
         run: |
@@ -39,10 +39,10 @@ jobs:
 
       # Attention: you perform the release step when you push a tag, neat huh?
       - name: Release
-        uses: softprops/action-gh-release@v1
+        uses: softprops/action-gh-release@v2
         if: startsWith(github.ref, 'refs/tags/')
         with:
-          files: ${{ github.workspace }}\*.zip
+          files: ${{ github.workspace }}\IVAN-${{env.RELEASE_VERSION}}-win.zip
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
           #GITHUB_REPOSITORY: attnam/ivan

.github/workflows/build-osx-wizmode.yml

@@ -46,7 +46,7 @@ jobs:
           key="$(grep -o "'https://[^']*'" ci/osx/requirements.sh | openssl dgst -sha256 -hex)"
           echo "CACHE_KEY=${key#* }" >> $GITHUB_ENV
       - name: Reuse cached files
-        uses: actions/cache@v2
+        uses: actions/cache@v4
         with:
           path: |
             ~/Thirdparty/Frameworks

FeLib/Source/error.cpp

@@ -123,7 +123,7 @@ void globalerrorhandler::Abort(cchar* Format, ...)
 
   va_list AP;
   va_start(AP, Format);
-  vsprintf(Buffer, Format, AP);
+  vsnprintf(Buffer, sizeof(Buffer), Format, AP);
   va_end(AP);
 
   strcat(Buffer, BugMsg);

FeLib/Source/rawbit.cpp

@@ -423,7 +423,7 @@ void rawbitmap::Printf(bitmap* Bitmap, v2 Pos, packcol16 Color, cchar* Format, .
 
   va_list AP;
   va_start(AP, Format);
-  vsprintf(Buffer, Format, AP);
+  vsnprintf(Buffer, sizeof(Buffer), Format, AP);
   va_end(AP);
 
   fontcache::const_iterator Iterator = FontCache.find(Color);
@@ -467,7 +467,7 @@ void rawbitmap::PrintfUnshaded(bitmap* Bitmap, v2 Pos, packcol16 Color, cchar* F
 
   va_list AP;
   va_start(AP, Format);
-  vsprintf(Buffer, Format, AP);
+  vsnprintf(Buffer, sizeof(Buffer), Format, AP);
   va_end(AP);
 
   fontcache::const_iterator Iterator = FontCache.find(Color);

FeLib/Source/whandler.cpp

@@ -340,7 +340,7 @@ void ShowFPS(){ //TODO still flickers sometimes cuz of silhouette?
       static v2 v2Margin(iMargin,iMargin);
       static char c[100];
 
-      sprintf(c,"FPS:ls=%.1f,insta=%.1f",globalwindowhandler::GetFPS(false),globalwindowhandler::GetFPS(true));
+      snprintf(c, sizeof(c), "FPS:ls=%.1f,insta=%.1f", globalwindowhandler::GetFPS(false), globalwindowhandler::GetFPS(true));
       int iDistX = strlen(c)*8 + 10 + 100; // +10 to look good, + 100 cuz of silhouette area
       v2 v2Pos = RES-v2(iDistX,RES.Y)+v2Margin;
       v2 v2Size(iDistX, 8+iMargin*2);

Main/Source/cmdcraft.cpp

@@ -3057,7 +3057,7 @@ void updateCraftDesc(){
   float fSkill=craftcore::CraftSkill(PLAYER); //TODO should this dynamic value show too where stats are?
   festring fsSkill="Crafting Proficiency: ";  // It's actually different from skills, so don't call it a skill.
   static char cSkill[20];
-  sprintf(cSkill, "%.1f",fSkill);
+  snprintf(cSkill, sizeof(cSkill), "%.1f",fSkill);
   fsSkill<<cSkill;
 
   festring fsDesc=fsSkill;

Main/Source/game.cpp

@@ -371,7 +371,7 @@ void game::InitScript()
     for (int f = 0; f <= 99; f++) //additional dungeon files
     {
       char bnum[32];
-      sprintf(bnum, "Script/dungeon_%02d.dat", f);
+      snprintf(bnum, sizeof(bnum), "Script/dungeon_%02d.dat", f);
       inputfile ifl(game::GetDataDir()+bnum, &game::GetGlobalValueMap(), false);
       if (ifl.IsOpen())
       {
@@ -5866,7 +5866,7 @@ truth game::ValidateCustomCmdKey(int iNewKey, int iIgnoreIndex, bool bMoveKeys)
 festring IntToHexStr(int i)
 {
   static char hexbuf[100];
-  sprintf(hexbuf, "0x%04X", i);
+  snprintf(hexbuf, sizeof(hexbuf), "0x%04X", i);
   festring fs;fs=hexbuf;
   return fs;
 }

Main/Source/item.cpp

@@ -869,7 +869,7 @@ void item::AddInventoryEntry(ccharacter*, festring& Entry, int Amount, truth Sho
       Entry << " " << GetVolume() * Amount << "cm3"; //the item can be seen therefore it's volume guessed already
       if(GetSecondaryMaterial()==NULL){ //simple items like ingots sticks etc
         static char density[20];
-        sprintf(density, "%.1f", GetWeight()/(float)GetVolume());
+        snprintf(density, sizeof(density), "%.1f", GetWeight()/(float)GetVolume());
         Entry << " " << density << "g/cm3"; //the item can be seen and weighted already so this just helps avoiding having to mentally calc density for every item
         if(game::WizardModeIsActive()) //TODO || Char-> possess item <materialmanual*>
           Entry << " " << GetStrengthValue() << "str"; //this is special info tho.

Main/Source/message.cpp

@@ -53,7 +53,7 @@ void msgsystem::AddMessage(cchar* Format, ...)
 
   va_list AP;
   va_start(AP, Format);
-  vsprintf(Message, Format, AP);
+  vsnprintf(Message, sizeof(Message), Format, AP);
   va_end(AP);
 
   festring Buffer(Message);