RetroPie forum home
    • Recent
    • Tags
    • Popular
    • Home
    • Docs
    • Register
    • Login
    Please do not post a support request without first reading and following the advice in https://retropie.org.uk/forum/topic/3/read-this-first

    Eduke bug in toxic dump with the second shrink room.

    Scheduled Pinned Locked Moved Help and Support
    eduke3dduke nukem 3dshrinkertoxic dumplevel4
    19 Posts 3 Posters 1.8k Views
    Loading More Posts
    • Oldest to Newest
    • Newest to Oldest
    • Most Votes
    Reply
    • Reply as topic
    Log in to reply
    This topic has been deleted. Only users with topic management privileges can see it.
    • mituM
      mitu Global Moderator
      last edited by

      Thanks for the saves, I'll give them a try. Please remove the archive.

      D 1 Reply Last reply Reply Quote 0
      • S
        sleve_mcdichael
        last edited by sleve_mcdichael

        Can confirm the bug, shrink ray splashes against the alcove and does not reach the player. This commit is supposed to fix it (haven't tested. Don't know how to make a patch.)

        D mituM 3 Replies Last reply Reply Quote 0
        • D
          deFaezer @sleve_mcdichael
          last edited by

          @sleve_mcdichael

          Thanks, this link seems like a good place to start.

          It seems that the file /home/pi/installEduke32/eduke32/source/duke3d/src/actors.cpp has changed quite a bit in two years.
          is going to be a puzzle for me.

          https://drive.google.com/file/d/1NckPDHazkyzPYArHz79fR9u9gNJyGcq5/view?usp=sharing

          1 Reply Last reply Reply Quote 0
          • D
            deFaezer @sleve_mcdichael
            last edited by deFaezer

            @sleve_mcdichael

            update:

            update /home/pi/installEduke32/eduke32/source/duke3d/src/actors.cpp is not part of retropie but something I added myself to try and overwrite the existing.....

            My mistake!

            1 Reply Last reply Reply Quote 0
            • mituM
              mitu Global Moderator @sleve_mcdichael
              last edited by

              @sleve_mcdichael said in Eduke bug in toxic dump with the second shrink room.:

              This commit is supposed to fix it ...

              Yes, it does fix the issue with the shrinker ray getting stuck

              I've sent a patch to fix this, once merged you'll need to update your RetroPie-Setup script and re-install from source eduke32.

              S 1 Reply Last reply Reply Quote 1
              • S
                sleve_mcdichael @mitu
                last edited by

                @mitu works, thanks. I downloaded the patch & script from your repo, and can now get back out of that second shrink room.

                1 Reply Last reply Reply Quote 0
                • D
                  deFaezer @mitu
                  last edited by

                  @mitu

                  So how can i fix this problem…, how to do this?

                  S 1 Reply Last reply Reply Quote 0
                  • S
                    sleve_mcdichael @deFaezer
                    last edited by sleve_mcdichael

                    @defaezer @defaezer said in Eduke bug in toxic dump with the second shrink room.:

                    So how can i fix this problem…, how to do this?

                    The easy way is: wait until this pull request is merged, then update your RetroPie-Setup script, and reinstall (from source) eduke32.

                    (Aaand, it's merged. Ignore the following, just update your script & reinstall eduke.)

                    If you don't want to wait, download the patch and the updated script from mitu's repo.

                    Patch: https://raw.githubusercontent.com/cmitu/RetroPie-Setup/eduke32-bugfix/scriptmodules/ports/eduke32/0005-e1m4-shrinker-bug.patch. Put it in RetroPie-Setup/scriptmodules/ports/eduke32/.

                    Updated script: https://raw.githubusercontent.com/cmitu/RetroPie-Setup/eduke32-bugfix/scriptmodules/ports/eduke32.sh. Put it in RetroPie-Setup/scriptmodules/ports/, replacing the eduke32.sh that is already there.

                    Once those two files are in place, again you reinstall from source eduke32 from the setup script.

                    After it's installed, cd back into your RetroPie-Setup directory and do a git reset --hard to undo your changes, or else you won't be able to update the setup script in the future (you will get an error that your local changes would be overwritten and you need to commit or stash them first.)

                    (Edit: when you do the hard reset, it doesn't touch the patch because the repo doesn't know that it exists since it's not part of it. When the repo does get updated later and then attempts to add the patch, you might get the error again. Just delete it, either now or whenever it becomes an issue.)

                    D 1 Reply Last reply Reply Quote 1
                    • S
                      sleve_mcdichael
                      last edited by sleve_mcdichael

                      @mitu @buzz reinstalling this reintroduced a minor bug that I had squashed previously in my own setup, in which the log file is saved to $HOME where it sticks out like a sore thumb. The log is saved to {cwd}, wherever that happens to be so all it needs is a pushd into the config dir in the emulator command, before the game is launched. This should fix it, I think:

                      https://github.com/RetroPie/RetroPie-Setup/pull/3473

                      1 Reply Last reply Reply Quote 0
                      • D
                        deFaezer @sleve_mcdichael
                        last edited by

                        @sleve_mcdichael, @mitu
                        You guys are amazing, thank you so much, and I'm not only happy that it works, but I've also learned something.

                        1 Reply Last reply Reply Quote 0
                        • S
                          sleve_mcdichael
                          last edited by

                          @sleve_mcdichael said in Eduke bug in toxic dump with the second shrink room.:

                          reinstalling this reintroduced a minor bug

                          @mitu it also re-downloaded the shareware game data, even though I had the registered version files in place already. But since my filenames are in ALL CAPS (extracted from the DOS disk), they weren't detected by the script.

                          Aside from just, fixing my own local files and forgetting about it, what's the best way to handle this? Is there any way to make the test [[ ! -f "$dest/duke3d.grp" ]] case-insensitive?

                          I know that in find and grep you can use the -i flag to ignore case, and how in test/[ you can compare strings with, for example ${var1,,} or ${var2^^}, but how can I do something similar but with filenames?

                          I suppose it could just be [[ ! -f "$dest/duke3d.grp" && ! -f "$dest/DUKE3D.GRP" ]], but this still wouldn't catch mixed-case like Duke3d.grp if someone had that, and I'm sure we don't want to test for all 2^(8)=256 combinations.

                          Or, would we be better off just rename-ing everything to lowercase first and adding that to the dependencies, as in wolf4sdl?

                          mituM 1 Reply Last reply Reply Quote 0
                          • mituM
                            mitu Global Moderator @sleve_mcdichael
                            last edited by mitu

                            @sleve_mcdichael said in Eduke bug in toxic dump with the second shrink room.:

                            I suppose it could just be [[ ! -f "$dest/duke3d.grp" && ! -f "$dest/DUKE3D.GRP" ]], but this still wouldn't catch mixed-case like Duke3d.grp if someone had that, and I'm sure we don't want to test for all 2^(8)=256 combinations.

                            Sometimes it's just easier to rename the file. Steam and GOG releases have lowercase names for .grp files, so most of installations would be covered.

                            1 Reply Last reply Reply Quote 0
                            • First post
                              Last post

                            Contributions to the project are always appreciated, so if you would like to support us with a donation you can do so here.

                            Hosting provided by Mythic-Beasts. See the Hosting Information page for more information.