tspivey / yugioh-game

Text-based Yu-Gi-Oh MUD
https://allinaccess.com/game/
MIT License
33 stars 23 forks source link

Added soundpack command (used by the new soundpack) #276

Closed JessicaTegner closed 8 months ago

JessicaTegner commented 1 year ago

This pr adds a soundpack command.

Right now this only contains version (so soundpack version abc) but can be expanded later if we want.

This goes with the new mush soundpack that calls soundpack version on login to check for updates. It's the server that tells the player if their soundpack is out of date.

Timtam commented 1 year ago

Just as a side note, I don't like a soundpack command here. That feels like we're mixing up two independent projects. You'd always need to push and restart (!) the server for a soundpack update announcement, that doesn't feel right in the slightest. You can achieve the same by checking values on the GitHub repository of the soundpack if that is what you want. BTW, what is it all about with the new soundpack, why not continue to maintain the one that already exists, its public over all: https://github.com/Timtam/yugioh-soundpack/

JessicaTegner commented 1 year ago

yep good point as @tspivey also said. Also figured that the soundpack wasn't being updated anymore and haven't heard of anyone seeing you around

Timtam commented 1 year ago

Me being busy and not playing ygo right now doesn't mean that I don't maintain my projects over here on GitHub, which is probably a thing for many open source devs out there. Just propose a PR and i'll gladly accept it if its useful. I also showed up recently and fixed some of the public decks, will hopefully continue to do so sooner or later again.

JessicaTegner commented 1 year ago

I would have nothing against sending prs over. The same argument I had for TSP I'll tell you. I'm trying to streamline and automate as many things about the mud as possible , since the current devs (you and tsp) aren't really around (and in tsps case, doesn't really like the game).

I'm simply trying to make everyones life easier

Timtam commented 1 year ago

Thats fine, I don't see anyone arguing against that. Thats exactly why I said that contributing to the current soundpack is so much more efficient, it saves your time, instead of doing everything from scratch you can just build on top of what is already there, and GitHub as a network allows us to stay connected.

tspivey commented 1 year ago

Once we get a soundpack updater in place, #272 should be as easy as a merge into both soundpack and game, then ask players to update.

Timtam commented 1 year ago

I'm still against moving part of the update process of a project into another project, namely the soundpack into the game itself. Why not build a soundpack-only solution? I've done something like similar already and it works totally fine. You can gather some inspiration from it here: https://github.com/Timtam/Avalon/blob/master/worlds/plugins/avalon/updater.xml That'd be much cleaner and future proof, as we don't need to update two independent projects in the case that something, whatever it may be, changes in the future.

JessicaTegner commented 1 year ago

the issue with that would be that you can't override files while they are in use. That's why it's splitted up into a separate process

Timtam commented 1 year ago

Yeah, thats not the issue that troubles me here. You already got a bat in place which downloads and replaces the soundpack don't you? That can just stay as it is, I just don't like the soundpack version check and download on the server side.