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

Support Ice 3.7 and Ubuntu 20.04 #74

Merged
merged 6 commits into from
Aug 6, 2022
Merged

Conversation

AtomicHaggis
Copy link
Contributor

This pull request includes all changes and updates to MumPi to support newer operating systems such as Ubuntu 18.04 and 20.04. Since older versions of Ubuntu are no longer supported, support for obsolete versions of Mumble below 1.3.0 have been removed.

@AtomicHaggis
Copy link
Contributor Author

AtomicHaggis commented Jun 26, 2022

As requested, this PR has been squashed down before submission so it only contains two commits.

I have tested this version on a clean installation of Ubuntu 20.04.3 and it operates correctly.

I did not update the version number in this case, as I would rather leave that to the maintainer to decide the versioning.

@Kissaki
Copy link
Owner

Kissaki commented Jun 26, 2022

Awesome!

I will broadly look over the changes a bit later, but overall trust you on them and their functioning. :)

@AtomicHaggis
Copy link
Contributor Author

Yeah, sorry for the delay in getting the PR submitted with the squashed commits. Hopefully this revitalises the project and we can bring it up to date later with Mumble 1.4 once it goes stable.

@Kissaki
Copy link
Owner

Kissaki commented Jun 27, 2022

The second commit has a different author. I assume you want to use/get attributed with the first one; this github account?

@Kissaki Kissaki force-pushed the master branch 3 times, most recently from 4727636 to f4f9c57 Compare June 27, 2022 18:13
@AtomicHaggis
Copy link
Contributor Author

That's correct. AtomicHaggis. That other one was using a work account.

@Kissaki Kissaki force-pushed the master branch 2 times, most recently from 992fb7f to 1489ab1 Compare June 27, 2022 18:26
AtomicHaggis and others added 6 commits June 27, 2022 20:27
Update default configuration to use generated Murmur 1.3.0 with Ice 3.7 slice PHP code.
Change type references to backslash syntax

Update ServerInterface.php Ice initialization. Drop Ice 3.3 and 3.4 initialization.
@Kissaki
Copy link
Owner

Kissaki commented Jun 27, 2022

I split the changes into logical commits. The changes look good to me.

I changed the ice extension loaded and version check a bit, which you can see in 286939f.
(Better visual diff in a diff viewer that can ignore whitespace changes.)

Can you please test this resulting state works?
I do not expect breakage from my changes (otherwise just splitting and dropping whitespace changes), but better be safe with a test. :)

@AtomicHaggis
Copy link
Contributor Author

AtomicHaggis commented Jun 27, 2022

Yep, no problem. I'll grab it and test it in the morning.

@AtomicHaggis
Copy link
Contributor Author

Work's been pretty hectic. I'll try to get to this tomorrow.

@Kissaki
Copy link
Owner

Kissaki commented Jul 21, 2022

@AtomicHaggis Will you be able to test it soon(ish)? Otherwise I will merge it as-is.

@AtomicHaggis
Copy link
Contributor Author

@Kissaki Life's been a bit chaotic! I'll get to testing it this weekend. I'll let you know on Sunday.

@Kissaki Kissaki changed the title Update of MumPi to support Ice 3.7 & Ubuntu 20.04 Update to support Ice 3.7 & Ubuntu 20.04 Aug 6, 2022
@Kissaki Kissaki changed the title Update to support Ice 3.7 & Ubuntu 20.04 Support Ice 3.7 and Ubuntu 20.04 Aug 6, 2022
@Kissaki Kissaki merged commit 8d5110c into Kissaki:master Aug 6, 2022
@Kissaki
Copy link
Owner

Kissaki commented Aug 6, 2022

I am merging it as-is :)

and will tag a release version

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.

2 participants