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

2.1 fix linux #21

Open
wants to merge 3 commits into
base: 2.1
Choose a base branch
from
Open

2.1 fix linux #21

wants to merge 3 commits into from

Conversation

pierstitus
Copy link

notes on how to run and build on linux and some small fixes for 64-bit

error: cast to pointer from integer of different size
[-Werror=int-to-pointer-cast]

In strummer they are fixed, in picross they are ignored for now.
Required by package quality guidelines
@TheTechnobear
Copy link
Collaborator

I will check the build errors, but Ive not needed to include the ignore errors flag in picross or strummer,
Id need to see why you are getting it, perhaps a different version of gcc?
which version of gcc are you using? ( Im using 4.7.2/Debian and 4.9.1/Ubuntu)

deb maintainer, I'll update

notes - yeah the issue is Python 2.7 (we use 2.5 on other platforms) , it appears to only load from the LD_LIBRARY_PATH, so currently Im creating a LD_LIBRARY_PATH which includes all modules, as I don't want to pollute the libs with the agent plugins. I suspect the mid term fix, is to dynamically add these directories at startup.

btw, if you need any help/info, you can email me (address on profile)... happy to help you get up and running :)

@TheTechnobear TheTechnobear self-assigned this Apr 2, 2015
@pierstitus
Copy link
Author

I'm running 4.8.2/Ubuntu.
It seems like reasonable errors/warnings, it seems a good idea not to cast int to pointers if not necessary.

I mainly had to give EigenD a try now that Workbench and Stage are also included. And it works well I must say, only it would be nice to update the readme a bit. Some later time I might connect the Striso to hear what that does.

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