azihassan / devilutionX

Diablo build for modern operating systems
Other
4 stars 1 forks source link

Switch to unpacked MPQs #15

Open azihassan opened 2 weeks ago

azihassan commented 2 weeks ago

I was on the fence about using unpacked MPQs but this settles it for me. I'm hoping for some load time improvements too

azihassan commented 2 days ago

Reopening this because I suspect it's causing some strange visual issues. I'm gonna recompile the dreamcast branch with set(UNPACKED_MPQS OFF) to make sure

image

azihassan commented 2 days ago

I was cloning from main unpack_and_minify_mpq from main, but after switching to the 1.5.1 tag it started works again. I'll try bisecting before opening an issue tomorrow

image

azihassan commented 1 day ago

Bisect result

git bisect start
# status: waiting for both good and bad commits
# bad: [d354283df819f3551eb8034795413d445e005add] .github/workflows: Update upload-artifact
git bisect bad d354283df819f3551eb8034795413d445e005add
# status: waiting for good commit(s), bad commit known
# good: [e7ff2622ae733a5d7ae3d73c923cff593df6f19c] Bump zlib to 1.3
git bisect good e7ff2622ae733a5d7ae3d73c923cff593df6f19c
# good: [1f7c4458600566c121fd6e72bc7c0abf97f400c9] Remove unused sounds
git bisect good 1f7c4458600566c121fd6e72bc7c0abf97f400c9
# bad: [91802dadc309794c591d08ebc63d148c1ad4b68c] Bump dvl_gfx
git bisect bad 91802dadc309794c591d08ebc63d148c1ad4b68c
# good: [2e2b5666ca9441cf5a29fa53f1355832b98dd347] Remove bhit.wav (unused)
git bisect good 2e2b5666ca9441cf5a29fa53f1355832b98dd347
# first bad commit: [91802dadc309794c591d08ebc63d148c1ad4b68c] Bump dvl_gfx

Something changed in the underlying library between commits https://github.com/diasurgical/devilutionx-graphics-tools/commit/64c0629480598c6ebdd25c527082606c1ae952a1 and https://github.com/diasurgical/devilutionx-graphics-tools/commit/ae39773dc2932b82328507bb51f79fa13b561ec0

azihassan commented 1 day ago

Commit 5f260010b0af26e9c453931f42edcb02ce894a16 segfaults with diabdat.mpq, commit https://github.com/diasurgical/devilutionx-graphics-tools/commit/ae39773dc2932b82328507bb51f79fa13b561ec0 unpacks it but causes the bug above