starred/LinuxGSM

Fork 0

mirror of https://github.com/GameServerManagers/LinuxGSM.git synced 2026-04-25 14:15:59 +03:00

[GH-ISSUE #1228] unzip is not a required dependency, yet fn_dl_extract uses it #957

New issue

Closed

opened 2026-02-27 02:54:30 +03:00 by kerem · 8 comments

kerem commented

2026-02-27 02:54:30 +03:00

Owner

Originally created by @braunsonm on GitHub (Jan 5, 2017).
Original GitHub issue: https://github.com/GameServerManagers/LinuxGSM/issues/1228

unzip is used to extract .zip files, yet LGSM does not install or check it as a dependency.

github.com/GameServerManagers/LinuxGSM@34d2c96bec/lgsm/functions/core_dl.sh (L65)

Originally created by @braunsonm on GitHub (Jan 5, 2017). Original GitHub issue: https://github.com/GameServerManagers/LinuxGSM/issues/1228 unzip is used to extract .zip files, yet LGSM does not install or check it as a dependency. https://github.com/GameServerManagers/LinuxGSM/blob/34d2c96becbe3d7a0c81688ead460cdba17abac9/lgsm/functions/core_dl.sh#L65

kerem

2026-02-27 02:54:30 +03:00

closed this issue
added the
type: feature
label

kerem commented

2026-02-27 02:54:31 +03:00

Author

Owner

@dgibbs64 commented on GitHub (Jan 5, 2017):

I dont believe there are any files that use zip within LGSM. I ensured they are all .tar.bz2 because of this zip is not a requirement. Only Unreal used to use zip until I changed it.

@dgibbs64 commented on GitHub (Jan 5, 2017): I dont believe there are any files that use zip within LGSM. I ensured they are all .tar.bz2 because of this zip is not a requirement. Only Unreal used to use zip until I changed it.

kerem commented

2026-02-27 02:54:31 +03:00

Author

Owner

@braunsonm commented on GitHub (Jan 5, 2017):

But if you have the functionality to extract zip files on that function, you need the unzip dependency.

Zip is required on this pull request:
https://github.com/GameServerManagers/LinuxGSM/pull/1224

@braunsonm commented on GitHub (Jan 5, 2017): But if you have the functionality to extract zip files on that function, you need the unzip dependency. Zip is required on this pull request: https://github.com/GameServerManagers/LinuxGSM/pull/1224

kerem commented

2026-02-27 02:54:31 +03:00

Author

Owner

@dgibbs64 commented on GitHub (Jan 5, 2017):

I see then it would be a requirement for the specific game rather than a general requirement as no servers require it apart from Multi Theft Auto.

Good work btw. Sorry im quite slow with dev stuff currently I have loads on (still). I will review your work as soon as I actually get some time :)

@dgibbs64 commented on GitHub (Jan 5, 2017): I see then it would be a requirement for the specific game rather than a general requirement as no servers require it apart from Multi Theft Auto. Good work btw. Sorry im quite slow with dev stuff currently I have loads on (still). I will review your work as soon as I actually get some time :)

kerem commented

2026-02-27 02:54:31 +03:00

Author

Owner

@braunsonm commented on GitHub (Jan 5, 2017):

But then why is tar a LGSM requirement, yet no server on master uses it?

I'm just a bit confused about the inconsistency. I will adjust it to be a MTA requirement only if that's what you'd like though.

@braunsonm commented on GitHub (Jan 5, 2017): But then why is tar a LGSM requirement, yet no server on master uses it? I'm just a bit confused about the inconsistency. I will adjust it to be a MTA requirement only if that's what you'd like though.

kerem commented

2026-02-27 02:54:31 +03:00

Author

Owner

@UltimateByte commented on GitHub (Jan 6, 2017):

SteamCMD comes in a tar file if i remember well.
BTW, @dgibbs64 unzip as a requirement is a good idea since it's a pretty common package and mods installations will usually be made from zip files.

@UltimateByte commented on GitHub (Jan 6, 2017): SteamCMD comes in a tar file if i remember well. BTW, @dgibbs64 unzip as a requirement is a good idea since it's a pretty common package and mods installations will usually be made from zip files.

kerem commented

2026-02-27 02:54:31 +03:00

Author

Owner

@cedarlug commented on GitHub (Jan 6, 2017):

Looks like UT (still?) requires it, too:

cia@cryptbookpro:~/LinuxGSM$ grep -d recurse \\.zip .
./lgsm/functions/install_server_files.sh:               fileurl="http://files.gameservermanagers.com/UnrealTournament/UnrealTournament-Server-XAN-3045522-Linux.zip"; filedir="${tmpdir}"; filename="UnrealTournament-Server-XAN-3045522-Linux.zip";  executecmd="noexecute" run="norun"; force="noforce"; md5="553fed5645a9fc623e92563049bf79f6"

@cedarlug commented on GitHub (Jan 6, 2017): Looks like UT (still?) requires it, too: ```bash cia@cryptbookpro:~/LinuxGSM$ grep -d recurse \\.zip . ./lgsm/functions/install_server_files.sh: fileurl="http://files.gameservermanagers.com/UnrealTournament/UnrealTournament-Server-XAN-3045522-Linux.zip"; filedir="${tmpdir}"; filename="UnrealTournament-Server-XAN-3045522-Linux.zip"; executecmd="noexecute" run="norun"; force="noforce"; md5="553fed5645a9fc623e92563049bf79f6" ```

kerem commented

2026-02-27 02:54:31 +03:00

Author

Owner

@dgibbs64 commented on GitHub (Jan 15, 2017):

https://github.com/GameServerManagers/LinuxGSM/pull/1229

@dgibbs64 commented on GitHub (Jan 15, 2017): https://github.com/GameServerManagers/LinuxGSM/pull/1229

kerem commented

2026-02-27 02:54:31 +03:00

Author

Owner

@lock[bot] commented on GitHub (Jul 19, 2018):

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock[bot] commented on GitHub (Jul 19, 2018): This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

kerem referenced this issue

2026-02-27 02:55:57 +03:00

[GH-ISSUE #1562] Send alert problem #1219

kerem referenced this issue

2026-02-27 02:55:57 +03:00

[GH-ISSUE #1562] Send alert problem #1219