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

Feature lowcode linux build scripts update #168

Open
wants to merge 7 commits into
base: Cog
Choose a base branch
from
Open

Feature lowcode linux build scripts update #168

wants to merge 7 commits into from

Conversation

ronsaldo
Copy link
Contributor

@ronsaldo ronsaldo commented Nov 9, 2017

Updating the Linux Lowcode VM building scripts.

@ronsaldo
Copy link
Contributor Author

ronsaldo commented Nov 9, 2017

I see this error. Do not integrate yet.
libtool: link: gcc -m64 -shared -fPIC -DPIC .libs/SDL2DisplayPlugin.o -lSDL2 -m64 -O2 -msse2 -Wl,-z -Wl,now -Wl,-soname -Wl,SDL2DisplayPlugin.so -o .libs/SDL2DisplayPlugin.so

/usr/bin/ld: cannot find -lSDL2

@ronsaldo
Copy link
Contributor Author

ronsaldo commented Nov 9, 2017

I see that the SDL2DisplayPlugin is not being built on Linux since ages. I will try to fix this problem in a separate issue and pull request.

@nicolas-cellier-aka-nice
Copy link
Contributor

So should we wait a bit for integration?
(IMO there's low risk of collision with other devs anyway)

@ronsaldo
Copy link
Contributor Author

ronsaldo commented Nov 9, 2017

Not anymore :), if possible integrate it now. As I said, I will try to fix the plugin problem in a separate issue/PR. I will probably have to contact Esteban. It looks like I am not the only one interested on this.

@krono
Copy link
Member

krono commented Oct 18, 2018

@ronsaldo can you merge from Cog or Rebase so we see whether it still works? then we could merge.

@krono
Copy link
Member

krono commented Sep 10, 2020

@ronsaldo any news here?

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