Skip to content

Conversation

devnexen
Copy link
Contributor

@devnexen devnexen commented Oct 2, 2020

related to ringbuffer, matching cmake settings to disable mlock for this platform.
Haiku does not support tls model as well.

@devnexen
Copy link
Contributor Author

devnexen commented Oct 2, 2020

Little question. If there any plan to update rpmalloc (ie its current git hash is old and sets wrongly tls model, thus the runtime_loader fails to resolve when it starts but it had been solved since) ?

@LmmsBot
Copy link

LmmsBot commented Oct 2, 2020

🤖 Hey, I'm @LmmsBot from github.com/lmms/bot and I made downloads for this pull request, click me to make them magically appear! 🎩

Linux

Windows

macOS

🤖
{"platform_name_to_artifacts": {"Linux": [{"artifact": {"title": {"title": "(AppImage)", "platform_name": "Linux"}, "link": {"link": "https://9155-15778896-gh.circle-artifacts.com/0/lmms-1.2.3-717%2Bgbfcb68f-linux-x86_64.AppImage"}}, "build_link": "https://circleci.com/gh/LMMS/lmms/9155?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link"}], "Windows": [{"artifact": {"title": {"title": "64-bit", "platform_name": "Windows"}, "link": {"link": "https://9154-15778896-gh.circle-artifacts.com/0/lmms-1.2.3-717%2Bgbfcb68feb-mingw-win64.exe"}}, "build_link": "https://circleci.com/gh/LMMS/lmms/9154?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link"}, {"artifact": {"title": {"title": "32-bit", "platform_name": "Windows"}, "link": {"link": "https://9157-15778896-gh.circle-artifacts.com/0/lmms-1.2.3-717%2Bgbfcb68feb-mingw-win32.exe"}}, "build_link": "https://circleci.com/gh/LMMS/lmms/9157?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link"}, {"artifact": {"title": {"title": "32-bit", "platform_name": "Windows"}, "link": {"link": "https://ci.appveyor.com/api/buildjobs/7g09vdwd3ylr1imp/artifacts/build/lmms-1.2.2-msvc2017-win32.exe"}}, "build_link": "https://ci.appveyor.com/project/Lukas-W/lmms/builds/35529657"}, {"artifact": {"title": {"title": "64-bit", "platform_name": "Windows"}, "link": {"link": "https://ci.appveyor.com/api/buildjobs/mc028fhus0tu6uce/artifacts/build/lmms-1.2.2-msvc2017-win64.exe"}}, "build_link": "https://ci.appveyor.com/project/Lukas-W/lmms/builds/35529657"}], "macOS": [{"artifact": {"title": {"title": "", "platform_name": "macOS"}, "link": {"link": "https://9158-15778896-gh.circle-artifacts.com/0/lmms-1.2.3-717%2Bgbfcb68feb-mac10.13.dmg"}}, "build_link": "https://circleci.com/gh/LMMS/lmms/9158?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link"}]}, "commit_sha": "b0f7ba5b91ab3bf9f2164beb1b6dbd494599d898"}

related to ringbuffer, matching cmake settings to disable mlock for this platform.
Haiku does not support tls model as well.
@JohannesLorenz
Copy link
Contributor

About rpmalloc, we still have #4752 open. We should get that done, but this is blocked as long as we still support Ubuntu 14.04.

@devnexen Do you require an rpmalloc update to get LMMS run on Haiku?

@devnexen
Copy link
Contributor Author

devnexen commented Oct 3, 2020

Right but isn't Ubuntu 14.04 EOL ? Anyway yes in that case the compiler toolchain might be old for newer rpmalloc. Yes indeed I changed locally to make it work :-)

JohannesLorenz added a commit to JohannesLorenz/lmms that referenced this pull request Oct 3, 2020
Fixes LMMS#4752
Fixes LMMS#4806 (at least, this is what we think)
Helps LMMS#5694
JohannesLorenz added a commit to JohannesLorenz/lmms that referenced this pull request Oct 4, 2020
Fixes LMMS#4752
Fixes LMMS#4806 (at least, this is what we think)
Helps LMMS#5694
@JohannesLorenz JohannesLorenz self-requested a review October 4, 2020 12:37
Copy link
Contributor

@JohannesLorenz JohannesLorenz left a comment

Choose a reason for hiding this comment

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

Looks all good. I'll merge this now. The rpmalloc fix will remain delayed until #5696 is done.

@JohannesLorenz JohannesLorenz merged commit 4efe0c8 into LMMS:master Oct 4, 2020
JohannesLorenz added a commit that referenced this pull request Oct 5, 2020
This PR updates the remote URL and content of rpmalloc.

Fixes #4752
Fixes #4806 (assumably)
Helps #5694

You must now run
```
git submodule sync --recursive
```
once to reflect this change.
@JohannesLorenz
Copy link
Contributor

@devnexen for your info, rpmalloc is updated on master now.

sdasda7777 pushed a commit to sdasda7777/lmms that referenced this pull request Jun 28, 2022
This PR updates the remote URL and content of rpmalloc.

Fixes LMMS#4752
Fixes LMMS#4806 (assumably)
Helps LMMS#5694

You must now run
```
git submodule sync --recursive
```
once to reflect this change.
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