ECWolf - Wolfenstein 3D source port for mods

cyperghost

@tpo1990 I would suggest following

instead

function configure_ecwolf() {
    addPort "$md_id" "ecwolf-spear3d" "ECWolf - Spear3d" "$md_inst/ecwolf --data sdm --file %ROM%"
    addPort "$md_id" "ecwolf-wolf3d" "ECWolf - Wolf3d" "$md_inst/ecwolf --data wl1 --file %ROM%"

    mkRomDir "ports/wolf3d"

better use

function configure_ecwolf() {
    addPort "$md_id" "ecwolf" "Wolfenstein 3D" "pushd $romdir/ports/wolf3d; $md_inst/ecwolf --data wl1; popd"
    addPort "$md_id" "ecwolf" "Wolfenstein 3D - Spear of Destiny" "pushd $romdir/ports/wolf3d; $md_inst/ecwolf --data sod; popd"

    mkRomDir "ports/wolf3d"
    moveConfigDir "$home/.local/share/ecwolf" "$md_conf_root/ecwolf"
    moveConfigDir "$home/.config/ecwolf" "$romdir/ports/wolf3d

or just a a launch menu with some helper files ...

function configure_ecwolf() {
    addPort "$md_id" "ecwolf" "Wolfenstein 3D" "pushd $romdir/ports/wolf3d; $md_inst/ecwolf.sh %ROM%; popd"

    mkRomDir "ports/wolf3d"
    moveConfigDir "$home/.local/share/ecwolf" "$md_conf_root/ecwolf"
    moveConfigDir "$home/.config/ecwolf" "$romdir/ports/wolf3d"

    #Install Graphical Selection Menu - scriptmodule needs improvement for this
    cd "$romdir/ports"
    wget -N -q --show-progress "https://raw.githubusercontent.com/crcerror/ECWolf-RPI/master/file_dl/Wolfenstein 3D.sh"
    chown $user:$user "Wolfenstein 3D.sh"

    #Install Translator - scriptmodule needs improvement for this
    cd "$md_inst"
    wget -N -q --show-progress "https://raw.githubusercontent.com/crcerror/ECWolf-RPI/master/file_dl/ecwolf.sh"
    chmod +x "ecwolf.sh"
 
    [[ "$md_mode" == "install" ]] && game_data_ecwolf
}

that will lead to a menu like this one here in this posting

I don't want to send you an PR becaus I'm the opinion that you as creator should decide what's the best solution. I strongly modified your scriptmodule but it needs much code review (like yours, too) ---> Take a look here

cyperghost

@tpo1990
Let us talk in this gist here
https://gist.github.com/crcerror/9e1742e78d50e58d585f3735dd67e855

tpo1990

@cyperghost Good idea. Let's talk about the development in the gist instead of here.

I know that my scriptmodule needs code review, but to begin with my initial idea was just for testing out the source port and making that available for the community. Now it seems that the further progression on this suggests a final release. Maybe replacing Wolf4SDL as the primary source port for Wolfenstein 3D or as an alternative.

I really appreciate your opinion and i'm open for help on developing into making it happen. ;-)

cyperghost

@tpo1990 I have a solid solution in my pocket. It will work like the prboom script module. It will find all files placed into wolf3d directory - version does not matter and it may be expanded to future game support like corridor 7 or Black stone.

Let me post this in a few hours

cyperghost

@mitu
Can you take a look here: gist

How to proper patch sourcefiles?
Use wget, load the patch and use patch command?
Is there a better way?

The patch is needed to make ESC key work --> see here

tpo1990

@cyperghost Sounds good. It might be put to good use in other scriptmodules as well. :-)

cyperghost

@tpo1990 Yes but I think there is a better way of patching inside scriptmodules. I think some patches are in RestroPie-Setup directory for "offlline" patching - that better coding practice and makes control of code easier.

But I think you can take the gist and after some final tests you can set a PR.
I think I cleanup the code a bit and then I'm done.

cyperghost

scriptmodule is working now

get it with

wget https://gist.githubusercontent.com/crcerror/9e1742e78d50e58d585f3735dd67e855/raw/34493a8dc6f3cbb75fda8f381a8a94eb4ec67d82/ecwolf.sh

cyperghost

@tpo1990 Send you a massive PR
I tested on my Pi3, compiles and setup is fine

tpo1990

@cyperghost said in ECWolf - Wolfenstein 3D source port for mods:

@tpo1990 Yes but I think there is a better way of patching inside scriptmodules. I think some patches are in RestroPie-Setup directory for "offlline" patching - that better coding practice and makes control of code easier.

But I think you can take the gist and after some final tests you can set a PR.
I think I cleanup the code a bit and then I'm done.

I will test it out later and update the scriptmodule with your changes and see how well it goes. Then a PR for RetroPie project can be created. :-)

ExarKunIv

@tpo1990 i used what @cyperghost's script module and it compiled fine for me. but that's all i had time for. i did not do any playing as of yet

UPDATE

did a quick poke around. i see that mine does not have the ability to map exit key yet. but i dont know if its just my system and that i have the old versions still on there. even though i did uninstall it before using what cyperghost just posted

cyperghost

@ExarKunIv Check your build... it should say

ecwolf-ecwolf-5065aaefe055/android-libs/launcher/src/org/libsdl/../deps/app/SDLControllerManager.java
/home/pi/RetroPie-Setup
/home/pi/RetroPie-Setup/tmp/build/ecwolf /home/pi/RetroPie-Setup

= = = = = = = = = = = = = = = = = = = = =
Building 'ecwolf' : ECWolf - ECWolf is an advanced source port for Wolfenstein 3D, 
Spear of Destiny, and Super 3D Noah's Ark based off of the Wolf4SDL code bas.
It also supports mods from .pk3 files.
= = = = = = = = = = = = = = = = = = = = =

patching file ./src/wl_play.cpp
-- The C compiler identification is GNU 4.9.2

You see the patch is applied

Then go to config/controls... last line you'll see

tpo1990

@ExarKunIv Maybe you have downloaded @cyperghost scriptmodule before the patch was added.

Uninstall and if it still compiles without the patch, then go to /home/pi/RetroPie-Setup/scriptmodules/ports/ and delete the ecwolf.sh file, then download @cyperghost scriptmodule again and reinstall.

ExarKunIv

@cyperghost @tpo1990 ok i see whats going on. when i ran the wget that cyperghost just posted it worked fine but it put the new ecwolf in the /home/pi dir not the setup script folders

so when i ran the .sh again it was the old version.

i did not know that i needed to manual copy it in the the setup folder, which for me is no big deal

but thanks for letting that it was just on my end. :)

ExarKunIv

@cyperghost ok that worked, i did see the patch was applied but i did get a error about the shareware games or something

Archive:  /tmp/tmp.Wh1wPiis6u/wolf3d14.zip
  End-of-central-directory signature not found.  Either this file is not
  a zipfile, or it constitutes one disk of a multi-part archive.  In the
  latter case the central directory and zipfile comment will be found on
  the last disk(s) of this archive.
unzip:  cannot find zipfile directory in one of /tmp/tmp.Wh1wPiis6u/wolf3d14.zip or
        /tmp/tmp.Wh1wPiis6u/wolf3d14.zip.zip, and cannot find /tmp/tmp.Wh1wPiis6u/wolf3d14.zip.ZIP, period.
Archive:  /tmp/tmp.IR8NJazGvq/soddemo.zip
  End-of-central-directory signature not found.  Either this file is not
  a zipfile, or it constitutes one disk of a multi-part archive.  In the
  latter case the central directory and zipfile comment will be found on
  the last disk(s) of this archive.
unzip:  cannot find zipfile directory in one of /tmp/tmp.IR8NJazGvq/soddemo.zip or
        /tmp/tmp.IR8NJazGvq/soddemo.zip.zip, and cannot find /tmp/tmp.IR8NJazGvq/soddemo.zip.ZIP, period.
/home/pi/RetroPie/roms/ports/wolf3d /home/pi/RetroPie-Setup/tmp /home/pi
/home/pi/RetroPie-Setup/tmp /home/pi
/home/pi
Error running 'wget -q -O/tmp/tmp.Wh1wPiis6u/wolf3d14.zip http://maniacsvault.net/ecwolf/files/shareware/wolf3d14.zip' - returned 4
Error running 'unzip -o /tmp/tmp.Wh1wPiis6u/wolf3d14.zip -d /home/pi/RetroPie/roms/ports/wolf3d' - returned 9
Error running 'wget -q -O/tmp/tmp.IR8NJazGvq/soddemo.zip http://maniacsvault.net/ecwolf/files/shareware/soddemo.zip' - returned 4
Error running 'unzip -o /tmp/tmp.IR8NJazGvq/soddemo.zip -d /home/pi/RetroPie/roms/ports/wolf3d' - returned 9

just letting you know.

cyperghost

@ExarKunIv Well I really can't help you. The demo version are downloaded through an downloadAndExtract script introduced by helpers.sh
For me this works fine and it did so in my tests, it's the exact same command as @tpo1990 used in his 1st version of the script.

I guess it's a corrupt download of the demo files, try to obtain the files by manual and check if you can unzip them.
Or can you give the output if you use file wolf3d14.zip and file soddemo.zip

cyperghost

@tpo1990 I'm on long vacation now so I can't login to git now. But you've said there is a problem in downloading source file.

It's still available, you can get the ecwolf-ecwolf-5065aaefe055.zip by using this link and it's the same link still in the scriptmodule .... I would stick to one version as long as possible if there is no reason to change. Every history commit can be downloaded as long as the resporitory is available.
Did you receive an error message?

http://maniacsvault.net/ecwolf/files/ecwolf/1.x/

These versions are all outdated, March 2019 is the newest one

You can still obtain a proper url from newest zip or tar file by "test-extracting" the archive and then read out root path. But as I said I'm on vacation now and I can't directly test on the RPi. But as far as I see the scriptmodule works ... You can of course fork on your github and use the gitpull command then you've control of the version.

EDIT: I rather think I've removed one dependencie by accident and therefore the compiling will not work on your mashine if you build this up on a fresh install. This usecase I've never tried out to admit.

ExarKunIv

@cyperghost yeah not sure either.

i uninstalled it and deleted the wolf3d folder and reinstalled it, this time it worked just fine. not sure what that was all about.

tpo1990

@cyperghost Enjoy your vacation. I will try again with the link you gave me and see if i can sort it out in the scriptmodule that will soon get uploaded to my git repo.

I agree in that there is no reason to change the version as long as it works. I will look into which dependencie that was removed.

cyperghost

@tpo1990 Yes I'll enjoy my vacation for sure ;) Thank you.

About errors... they are stored to $HOME/RetroPie-Setups/logs from there use an log viewer or gunzip the file you're interested then you'll see if there is any. Maybe you can create a new patch for upcoming versions, you've hast to add the btn_esc and the btn_pause - line to the source code but as I said I saw no massive development in the newest code but it's up to you.
I'm looking forward into your PR for RetroPie

@ExarKunIv Yes seems like a broken download that makes the demos not being extractable. I think I will later write a menu tool, that automatically rescans for new added games, in the some logic as the installer does and maybe a "export" feature to make games directly accessable to ports section. If you want to add new games you've to check the files in ports/wolf3d and copy or change the Wolfenstein 3D (demo).ecwolf. Change the WL1 to WL6 or wl6 according the correct file extension of the full version. The .ecwolf files are called through Wolfenstein 3D*.sh files in ports section ... I think you will understand if you see the files. As I said I think I'll write a small tool that will do by automatic.