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

ARM build #449

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

ARM build #449

wants to merge 2 commits into from

Conversation

kakulukia
Copy link
Contributor

The ta-lib install script is a tiny bit out of date (last modified 2006-07-02) so this additional download enables building the docker image on ARM architecture to be able to use it on a RaspbarryPi.

…07-02) so this additional download enables building the docker image on ARM architecture to us it on a RaspbarryPi.
@saleh-mir
Copy link
Member

Hi Andy, thank you. I'm not exactly a Docker ninja, so I need some help with verifying this one.

Did you test it on both ARM and x86 architectures to verify this change?

@kakulukia
Copy link
Contributor Author

No, i did not test it on windows as the change is just about updating those two build files. Yes, ChatGPT did cleanup the Dockerfile and added a few comments but basically its unchanged. Make some more tests tonights - also have a windows box here.

@saleh-mir
Copy link
Member

That would be awesome. Thank you Andy.

@kakulukia
Copy link
Contributor Author

ERROR: jesse_live-0.48.0-cp39-cp39-manylinux_2_17_x86_64.manylinux2014_x86_64.whl is not a supported wheel on this platform.

The live Plugin needs to also be compiled for the ARM platform. Can you do that?

@kakulukia
Copy link
Contributor Author

Those build errors came from the main branch as i did not introduce code changes .. :/

@kakulukia
Copy link
Contributor Author

=> ERROR [jesse_basic_env 7/11] RUN cd /tmp && ls -l && /tmp/install_ta-lib.sh && rm -r /tmp/ta-lib 0.2s

[jesse_basic_env 7/11] RUN cd /tmp && ls -l && /tmp/install_ta-lib.sh && rm -r /tmp/ta-lib:
0.216 total 1580
0.216 -rwxr-xr-x 1 root root 1407 May 25 09:21 install_ta-lib.sh
0.216 -rwxr-xr-x 1 root root 1330299 May 25 09:09 ta-lib-0.4.0-src.tar.gz
0.216 -rw------- 1 root root 281617 May 14 09:03 tmphrh1f073cacert.pem
0.217 /bin/sh: 1: /tmp/install_ta-lib.sh: not found


Dockerfile:18

16 | # Install TA-lib
17 | COPY docker_build_helpers/* /tmp/
18 | >>> RUN cd /tmp && ls -l && /tmp/install_ta-lib.sh && rm -r /tmp/ta-lib
19 | ENV LD_LIBRARY_PATH=/usr/local/lib:$LD_LIBRARY_PATH
20 |

ERROR: failed to solve: process "/bin/sh -c cd /tmp && ls -l && /tmp/install_ta-lib.sh && rm -r /tmp/ta-lib" did not complete successfully: exit code: 127

This is what i get when trying to build the image on Windows, but i dont have a clue what its trying to tell me.
ls -l shows that the install script has been copied over and is set to be executable .. yet sh cant find it.
I dont know why .. this step works just fine on OSX.

But docker on my mac will start up an arm ubuntu which does not have a live-plugin.

This means i cant actually test this new image. :(

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