Skip to content

Add linux-armv7l-musl#31

Merged
vweevers merged 2 commits into
prebuild:masterfrom
alexojegu:linux-armv7-musl
Feb 2, 2024
Merged

Add linux-armv7l-musl#31
vweevers merged 2 commits into
prebuild:masterfrom
alexojegu:linux-armv7-musl

Conversation

@alexojegu
Copy link
Copy Markdown
Contributor

This PR add a docker images:

  • linux-armv7-musl: armv7 image with musl as libc

@alexojegu
Copy link
Copy Markdown
Contributor Author

alexojegu commented Nov 22, 2023

It is based on dockcross/linux-armv7l-musl. I have named the image as linux-armv7-musl, if it seems more appropriate I can change the name to linux-armv7l-musl.

The same in the Dockerfile, I have used:

ENV PREBUILD_ARCH arm
ENV PREBUILD_ARMV 7

But if it is more appropriate I can use (I'm not sure if this would work):

ENV PREBUILD_ARCH arm
ENV PREBUILD_ARMV 7l

@vweevers
Copy link
Copy Markdown
Member

I assume dockcross had a reason for using arm7l in this image name (unlike others like dockcross/linux-armv7 - we just copied those names). Any guesses as to why? I don't know what the L represents or why it's significant for one image and not others.

@alexojegu
Copy link
Copy Markdown
Contributor Author

alexojegu commented Nov 22, 2023

I changed the name of everything from linux-armv7-musl to linux-armv7l-musl but kept the PREBUILD_ARMV as 7, if you want I can also change it to 7l.

I don't know what the L means either. This seems suggest: armv7l is little-endian v7.

@alexojegu alexojegu changed the title Add linux-armv7-musl Add linux-armv7l-musl Nov 22, 2023
@alexojegu
Copy link
Copy Markdown
Contributor Author

alexojegu commented Dec 5, 2023

Excuse me @vweevers, I don't know if I understood you correctly before. Are you waiting for me to modify PREBUILD_ARMV or just for you to get time?

@vweevers
Copy link
Copy Markdown
Member

vweevers commented Feb 2, 2024

just for you to get time?

just that :)

@vweevers vweevers merged commit 4696dd2 into prebuild:master Feb 2, 2024
@vweevers
Copy link
Copy Markdown
Member

vweevers commented Feb 2, 2024

2.2.0

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