Skip to content

Fix Linux build#84

Closed
sgorpi wants to merge 1 commit intodamiensellier:masterfrom
sgorpi:FixLinuxBuild
Closed

Fix Linux build#84
sgorpi wants to merge 1 commit intodamiensellier:masterfrom
sgorpi:FixLinuxBuild

Conversation

@sgorpi
Copy link

@sgorpi sgorpi commented Apr 22, 2025

Thx for making Ctrl up to date @damiensellier !

To compile on Ubuntu 24.04 I had to fix a few minor issues in the makefile:

  • Change to a new location for JUCE's arch_detection.cpp
  • Add a few missing library dependencies

For reference, I had to install the following packages (maybe more are needed but I had those installed on my system already):

  • binutils-dev
  • libsframe1

With that it compiles and runs.

@sgorpi
Copy link
Author

sgorpi commented Apr 22, 2025

By the way, this should fix issue #8

@damiensellier
Copy link
Owner

That's awesome! Thanks for the PR, I'll add it manually to 5.6.32 and the Master branch. I'm not a LINUX user, so I rely 100% on you ;)

Thanks for your help Hedde!

Damien

damiensellier added a commit that referenced this pull request Apr 22, 2025
* FIX LINUX MAKEFILE from @sgorpi PR #84
damiensellier added a commit that referenced this pull request Apr 22, 2025
* FIX LINUX MAKEFILE from @sgorpi PR #84
damiensellier added a commit that referenced this pull request Apr 22, 2025
* FIX LINUX MAKEFILE from @sgorpi PR #84
@damiensellier
Copy link
Owner

damiensellier commented Apr 22, 2025

There was a lot of files updated for LINUX a long time ago I did not merged to CtrlrX yet.
I referenced this update in another issue : #26

RomanKubiak#400

I was wondering if CtrlrX was working properly on your LINUX install or it required some other updates?

Let me know, thanks

Damien

@damiensellier
Copy link
Owner

I'll close the PR because I merged it by hand since Xcode can't see any changes in the Builds/Linux/ folder that is not synced up. But it's already on CtrlrX Master and 5.6.32.

@sgorpi
Copy link
Author

sgorpi commented Apr 23, 2025

There was a lot of files updated for LINUX a long time ago I did not merged to CtrlrX yet. I referenced this update in another issue : #26

RomanKubiak#400

I was wondering if CtrlrX was working properly on your LINUX install or it required some other updates?

Let me know, thanks

Damien

I saw that issue+PR also indeed. It ditched the Makefile for CMake (which may be a wise choice for the future as JUCE also builds with it nowadays), and a whole lot of LUA changes. To my surprise I didn't need any source code changes on my machine (it has liblua5.2 and liblua5.4 installed I see). I've only tested CtrlrX with 1 panel so far, and that worked... I'll test a bit more over time and if I find anything I'll post an issue or PR.

I was wondering if you have any GitHub actions to automatically compile & package for different platforms and configs. If so, you could add a Linux build (I see you can choose specific distro versions for example) and see if that works. That may also give good automated feedback for future changes.

Cheers, H

@sgorpi sgorpi deleted the FixLinuxBuild branch April 23, 2025 03:08
@damiensellier
Copy link
Owner

Hi, I never used a CI/CD pipeline, I guess setting up a platform with github from the projucer makefile would be the easy way.
We already have so much trouble to generate macOS binaries for Arm vs Intel etc if it could run from the get go through an online process that would be incredible.

I'll give it a try with GitHub Actions when I'll get free time. If you think you can help with the appropriate settings to compile for LINUX, let me know.

@sgorpi
Copy link
Author

sgorpi commented Apr 23, 2025

To be honest I also don't have experience setting up a CI/CD pipeline, but I'm interested to help out (but with limited time available). You can find my email address via my commits.

I know there's examples out there we can learn from (GitHub supplies nice examples, and for example from Surge XT, probably many more to find). And the 'runners' do support macOS intel and ARM it seems (see docs)

It looks like this script was actually meant for CI/CD: https://github.com/damiensellier/CtrlrX/blob/master/Scripts/post-commit . It does run and package CtrlrX on my PC, so that may be a good start?

dnaldoog pushed a commit to dnaldoog/CtrlrX that referenced this pull request Jan 5, 2026
dnaldoog pushed a commit to dnaldoog/CtrlrX that referenced this pull request Jan 5, 2026
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