Emulator Issues #8258
open
Buildbot doesn't clean directory when files are removed.
0%
Description
The buildbot seems to build even deleted gameinis, whereas there are 1571 gameinis in the repository there are 2787 in every build the buildbot makes. While this will cause no immediate problems now, it will cause if any additional gameini pr is made. The general gameinis will be updated but if the deprecated region specific gameinis still exist they will override the general ones. Meaning no actual update will be possible until the buildbot issue is fixed.
Updated by kostamarino about 9 years ago
The reason this wasn't found before is probably because there were either additions or changes in the database, deletions were rare.
Updated by magumagu9 about 9 years ago
This isn't the first time we've run into issues with buildbots not correctly cleaning the build folder; we should just configure the buildbots to rm -rf dolphin/Binary before every build.
Updated by JMC4789 about 9 years ago
- Status changed from New to Accepted
- Priority set to Low
Updated by kostamarino about 9 years ago
- Priority changed from Low to Normal
I am changing the priority, this now causing issues by using depracated gameinis that shouldn't exist.
Updated by 
Updated by
Updated by JosJuice about 9 years ago
Issue 8318 has been merged into this issue.
Updated by JosJuice about 9 years ago
- Priority changed from Normal to High
Along with causing performance problems by using EFB2RAM for titles that don't need to be using it anymore, this is also making three Hyrule Field Speed Hacks show up for RZDE01 even though there only should be one.
Updated by kostamarino about 9 years ago
Even a one time manual cleanup would help a lot since the gameini database has been converted. This issue piles up issues with every gameini database update for those using the buildbot builds.
Updated by parlane almost 9 years ago
FWIW I have cleaned the windows builders.
Updated by JMC4789 almost 9 years ago
- Priority changed from High to Normal
- Milestone deleted (
Current)
I think for now we can remove current as the huge problem is fixed.
Updated by JosJuice over 8 years ago
- Priority changed from Normal to High
Recent PR builds (for Windows, I haven't looked at other OSes) contain the executables DSPTool.exe, Dolphin.exe, DolphinQt.exe and DolphinWX.exe. The last two of those are not supposed to exist. My guess is that spxtr's DolphinQt rewrite PRs (3221 and 3261) created these executables, making them show up in all other PRs too because the buildbot doesn't remove files properly. They are confusing for users and waste space.
Updated by parlane over 8 years ago
- Priority changed from High to Normal
Deleted windows build directories.
Updated by JosJuice over 8 years ago
That's not a sustainable solution. PR 3261 keeps recreating DolphinQt.exe every time it's rebuilt. This has already happened once, as you can see with DolphinQt.exe being present in the pr-win-x64 build 4341 of PR 3265.
Updated by JosJuice over 6 years ago
Addressed for Windows by https://github.com/dolphin-emu/sadm/commit/15b06d7ac58dce0f8a611c156fffecf47538774a. I don't know what the status is for the other OSes.