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

Ubuntu/Debian Linux install broken #1527

Closed
time-killer-games opened this Issue Feb 2, 2019 · 7 comments

Comments

Projects
None yet
2 participants
@time-killer-games
Copy link
Contributor

time-killer-games commented Feb 2, 2019

I deleted my enigma-dev folder for an attempt at fetching a fresh install.

I used the usual script from the Ubuntu/Debian install page for my Ubuntu (bionic beaver):
https://enigma-dev.org/docs/Wiki/Install:Linux

During installation, many errors of the same exact missing header show up in the terminal:
https://pastebin.com/EhZ3wriY

Then LGM opens with the following:
screenshot from 2019-02-02 16-38-50

So until that header issue gets resolved, I can't build the *.so file for LGM it appears.

@RobertBColton

This comment has been minimized.

Copy link
Member

RobertBColton commented Feb 2, 2019

As of #1448 (a1aa34d) we now depend on Google Protocol Buffers. I will make an announcement. Check that same guide again to see how to install it.
https://enigma-dev.org/docs/Wiki/Install:Linux#Source_Code

@RobertBColton

This comment has been minimized.

Copy link
Member

RobertBColton commented Feb 2, 2019

Sorry, also, I had to fix one more thing, the script is not supposed to be cd'ing to the CompilerSource directory. It should just run make in the root enigma-dev folder where CompilerSource is the default rule. That way it does not miss the protobuf dependency.
https://enigma-dev.org/docs/wiki/index.php?title=Install%3ALinux&action=historysubmit&diff=32061&oldid=32060

@time-killer-games

This comment has been minimized.

Copy link
Contributor Author

time-killer-games commented Feb 2, 2019

I deleted my previous comment. I stand corrected. While the same errors are present in the terminal - LGM now works. Thank you @RobertBColton !

@RobertBColton

This comment has been minimized.

Copy link
Member

RobertBColton commented Feb 2, 2019

You're welcome, I'll now make an announcement!

@time-killer-games

This comment has been minimized.

Copy link
Contributor Author

time-killer-games commented Feb 2, 2019

@RobertBColton Ok, I thought it was working because it was taking longer with LGM's compile form, but it is still failing to build the library.

Here's the new terminal log:
https://pastebin.com/Y2dhxYzr

@RobertBColton

This comment has been minimized.

Copy link
Member

RobertBColton commented Feb 2, 2019

Ok, that's Josh's fault, allow me to explain. Most of those are actually warnings we need to clean up. The real error is this:

compiler/compile.cpp:433:56: error: no match for ‘operator[]’ (operand types are ‘const google::protobuf::RepeatedPtrField<buffers::resources::Timeline_Moment>’ and ‘int’)
         wto << "  curr[" << game.timelines[i].moments()[j].step()
                                                        ^

It's Josh's fault because he made the change to use the [] subscript operator instead of Get which isn't in older versions of Protobuf's library. Ubuntu notoriously has outdated protocol buffers, so you need to update your protocol buffers basically... maybe from a PPA. We will probably also send another pull request now to remove the subscript operator and just make it compatible with older protobuf. Sorry about this.

@RobertBColton RobertBColton reopened this Feb 2, 2019

@time-killer-games

This comment has been minimized.

Copy link
Contributor Author

time-killer-games commented Feb 3, 2019

fixed. Just do:

sudo add-apt-repository ppa:maarten-fonville/protobuf
sudo apt-get update

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment