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

Fix base image for ARM support #54

Closed
wants to merge 1 commit into from
Closed

Conversation

khasim
Copy link

@khasim khasim commented Aug 8, 2018

While trying to build Rocket.chat for ARM64 we found the Fibers package missing. There is a pre-built Fibers package for x64, but it has to be built from source on arm64. However, the build fails with -slim node image from which some tools like python are missing.

Therefore we request the Rocket.chat project switch to using the non slim version of node image.

The issue was reported earlier here : #48

While trying to build Rocket.chat for ARM64 we found the Fibers package missing. There is a pre-built Fibers package for x64, but it has to be built from source on arm64. However, the build fails with -slim node image from which some tools like python are missing.

Therefore we request the Rocket.chat project switch to using the non slim version of node image.

The issue was reported earlier here : RocketChat#48
@CLAassistant
Copy link

CLAassistant commented Aug 8, 2018

CLA assistant check
All committers have signed the CLA.

@khasim
Copy link
Author

khasim commented Aug 14, 2018

Hello, can some one kindly review the PR ? Is there any better way to achieve this ? I mean support for ARMv8 ? I tried to add delta packages on top of slim base image but that doesn't fetch packages like Python etc for ARMv8.

@lag-linaro
Copy link

This is not the right approach. I shall follow-up with a PR to superseded this.

Please close it for now.

@lag-linaro
Copy link

Submitted #60 to supersede this PR.

@khasim (or one of the project maintainers) could you please close this?

@lag-linaro
Copy link

Could someone please close this PR?

@Sing-Li
Copy link
Member

Sing-Li commented Aug 13, 2019

Yes. I will close this. See the currently working approach to build here:

https://github.com/RocketChat/Rocket.Chat.Embedded.arm64/tree/develop/docker

@Sing-Li Sing-Li closed this Aug 13, 2019
@lag-linaro
Copy link

Why have you created a new project instead of making this one multi-arch?

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.

4 participants