Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

update mumble #4437

Merged
merged 6 commits into from
Oct 31, 2022
Merged

update mumble #4437

merged 6 commits into from
Oct 31, 2022

Conversation

QuintenQVD0
Copy link
Contributor

@QuintenQVD0 QuintenQVD0 commented Oct 15, 2022

This is a changed mumble egg as the do not provice a static binary anymore for the latest 1.4.x server version.
The current only way to get 1.4 running is you compile the source or run it from the alpine packages mumur what is offical by them.
This egg uses a image that has that mumble packages installed in the docker image
changes:

  • The image
  • startup cmd as it is now in PATH and does not need a ./ anymore and specify to use murmur.ini config
  • File parser to set the sqlite database path to /home/container/murmur.sqlite
  • Set the logfile to /home/container/murmur.log but notthing seeems to be writen to it
  • remove the version variable as there is no version control as the binary is now in the docker image
  • remove the logs part as it is not more used

EDIT
install script:
afbeelding

Just display Mumble as the docker images and not the full URL
@QuintenQVD0
Copy link
Contributor Author

I can join:
afbeelding

Copy link
Member

@parkervcp parkervcp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@parkervcp parkervcp mentioned this pull request Oct 28, 2022
3 tasks
@matthewpi matthewpi changed the base branch from develop to 1.0-develop October 31, 2022 16:26
@matthewpi matthewpi merged commit e21aab2 into pterodactyl:1.0-develop Oct 31, 2022
@QuintenQVD0 QuintenQVD0 deleted the patch-1 branch October 31, 2022 17:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants