Skip to content

(Release 30) Correct src.pro for Windows#357

Merged
vadi2 merged 5 commits intoMudlet:release_30from
Nyyrazzilyss:release_30
Dec 6, 2016
Merged

(Release 30) Correct src.pro for Windows#357
vadi2 merged 5 commits intoMudlet:release_30from
Nyyrazzilyss:release_30

Conversation

@Nyyrazzilyss
Copy link
Copy Markdown
Contributor

This (only) corrects the src.pro file to build properly with Windows. It links the proper libraries, and removes reference to external links/includes

This corrects the libraries linked in src.pro for Windows so SQL
functions work properly. It also removes reference to external
libraries/includes.
This makes two changes to sound playback. mpMusicBox is moved from 4
variables into an expandable QList. An unlimited number of sounds can be
played simultaenously.

The lua playSoundFile command has an additional argument added to it for
volume (0-100). It defaults to 100 if not present for backwards
compatability.
@SlySven
Copy link
Copy Markdown
Member

SlySven commented Dec 3, 2016

@vadi2 if this gets your approval I think you will want to use the "Squash and Merge" option to lose the two sets of add+revert commits and end up just with net effect of just the first commit. I think @Nyyrazzilyss will then want to cherry-pick the second and third to his separate sound enhancement PR...

@vadi2
Copy link
Copy Markdown
Member

vadi2 commented Dec 6, 2016

Still seems to compile and run - thanks for the fix!

@vadi2 vadi2 merged commit 0866fa3 into Mudlet:release_30 Dec 6, 2016
mehulmathur16 pushed a commit to mehulmathur16/Mudlet that referenced this pull request Feb 16, 2024
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.

3 participants