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

ppc cross compile #471

Merged
merged 63 commits into from
Nov 2, 2017
Merged

ppc cross compile #471

merged 63 commits into from
Nov 2, 2017

Conversation

vb216
Copy link
Member

@vb216 vb216 commented Sep 30, 2017

Slowly getting there.. ffmpeg, cuda, and leptonica outstanding.

ffmpeg - not picking up the earlier built libs, i've tried a few different angles but not cracked it yet
cuda - its using x86_64 cuda in the docker image, but when it comes to linking it cant find the ppc64 .so files. might be a case of just bundling them and manually copying in to the docker image
leptonica - a little similar to ffmpeg, not finding a zlib its happy with

@vb216
Copy link
Member Author

vb216 commented Oct 22, 2017

Hm, with cuda, its now failing due to not finding lcuda below. But, I can only see a libcuda.so in the stubs, has something changed in v9? Or am I missing something else.

Compiling /home/travis/build/javacpp-presets/cuda/target/classes/org/bytedeco/javacpp/linux-ppc64le/libjnicublas.so
[INFO] powerpc64le-linux-gnu-g++ -I/usr/local/cuda-9.0/include/ -I/usr/lib/jvm/java-8-openjdk-amd64/include -I/usr/lib/jvm/java-8-openjdk-amd64/include/linux /home/travis/build/javacpp-presets/cuda/target/classes/org/bytedeco/javacpp/jnicublas.cpp -mcpu=power8 -m64 -O3 -s -Wl,-rpath,$ORIGIN/ -Wl,-z,noexecstack -Wl,-Bsymbolic -Wall -fPIC -shared -o libjnicublas.so -L/usr/local/cuda-9.0/lib64/ -Wl,-rpath,/usr/local/cuda-9.0/lib64/ -lcublas -lcuda -lcudart
/usr/lib/gcc-cross/powerpc64le-linux-gnu/5/../../../../powerpc64le-linux-gnu/bin/ld: cannot find -lcuda

@saudet
Copy link
Member

saudet commented Oct 22, 2017 via email

@vb216
Copy link
Member Author

vb216 commented Oct 29, 2017

Hopefully this is it finished.

else
cd ..
rm -rf tesseract-3.05.01
git clone -q --branch=3.05 https://github.com/tesseract-ocr/tesseract.git tesseract-3.05.01
Copy link
Member

@saudet saudet Oct 29, 2017

Choose a reason for hiding this comment

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

There's only been 18 commits between 3.05 and 3.05.01: tesseract-ocr/tesseract@3.05.01...3.05
We can probably just make a patch to revert the offending one...

@saudet
Copy link
Member

saudet commented Oct 29, 2017

Could we put the content of ci/hdf5 into a patch and in the main hdf5 subdirectory itself?

if [[ "$PROJ" =~ cuda ]]; then
echo "Setting up for cuda build"
#cp /usr/include/cudnn.h /usr/local/cuda/include/
python $TRAVIS_BUILD_DIR/ci/gDownload.py 0B2xpvMUzviShdFlrT3FtNXFSRW8 $HOME/downloads/cudappc64.tar
Copy link
Member

Choose a reason for hiding this comment

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

How did you create that cudappc64.tar? Maybe we can just put the few commands for this here...

@vb216
Copy link
Member Author

vb216 commented Oct 30, 2017

Yeah I was just trading off time vs getting something out there. They're all possible, though the last one its quite a pain - its deconstructing the .deb file and repackaging it up, I guess could be put in a neat one liner but I didn't see anything immediate. Won't be able to do anything for a few weeks

@saudet saudet merged commit a20d074 into bytedeco:master Nov 2, 2017
@saudet
Copy link
Member

saudet commented Nov 2, 2017

This is working remarkably well, great job! While fixing a couple of things, I also discovered an unrelated issue with Leptonica not linking to the image libraries, so that's good too.

Now that this is done, as far as CI goes, I think the priority would be to update the http://bytedeco.org/builds/ page with recent information about Travis CI and AppVeyor. Feel free to start sending pull requests there when you get some time in a few weeks, but no rush, take your time. Thanks!

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.

None yet

2 participants