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

musl: add 0 large file patch #30777

Closed
wants to merge 1 commit into from
Closed

musl: add 0 large file patch #30777

wants to merge 1 commit into from

Conversation

dkwo
Copy link
Contributor

@dkwo dkwo commented May 10, 2021

  • musl: add patch for 0 largefile
  • musl: revbump

This would allow e.g. building firefox so that it is usable with musl 1.2.2

General

Have the results of the proposed changes been tested?

  • I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
  • I generally don't use the affected packages but briefly tested this PR

Builds fine on x86_64-musl.

@dkwo dkwo changed the title musl: add 0 large file patch [wip] musl: add 0 large file patch May 10, 2021
@dkwo
Copy link
Contributor Author

dkwo commented May 10, 2021

@ericonr If you find it ok, perhaps this could be merged before the ff update.

@dkwo dkwo changed the title [wip] musl: add 0 large file patch musl: add 0 large file patch May 10, 2021
Copy link
Member

@ericonr ericonr left a comment

Choose a reason for hiding this comment

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

It should be a single commit and the message should be musl: backport O_LARGE fix with an explanation in the body: "musl 1.2.2 started defining O_LARGEFILE to an actual value for x86_64, which breaks the firefox seccomp filter when built with older musl headers. backporting this change and building firefox with it allows one to use a local more up to date musl version with firefox from repos"

@ericonr
Copy link
Member

ericonr commented May 10, 2021

Hmm, we'd have to check if this breaks chromium or the other mozilla packages as well...

musl 1.2.2 started defining O_LARGEFILE to an actual value for x86_64, which breaks the firefox seccomp filter when built with older musl headers. backporting this change and building firefox with it allows one to use a local more up to date musl version with firefox from repos
@dkwo
Copy link
Contributor Author

dkwo commented May 10, 2021

Hmm, we'd have to check if this breaks chromium or the other mozilla packages as well...

Do you mean building or using them?
(As far as using, they seem to work fine with the musl patch.)

@ericonr
Copy link
Member

ericonr commented May 10, 2021

Yeah, using. Because the thing is, by backporting this musl change, we are introducing the issue with musl 1.2.2 to our musl package. So anything that needed to be rebuilt because of musl 1.2.2 in other distros also needs to rebuilt if we merge this in.

@dkwo
Copy link
Contributor Author

dkwo commented May 10, 2021 via email

@ericonr ericonr mentioned this pull request May 12, 2021
2 tasks
@ericonr
Copy link
Member

ericonr commented May 12, 2021

Firefox-esr and Thunderbird, at least.

In the opposite direction, I made: https://github.com/ericonr/void-packages/tree/musl-1.2.0

@dkwo
Copy link
Contributor Author

dkwo commented May 13, 2021

Your idea is probably smarter.
Thunderbird does not need a revbump thouhg.

@dkwo
Copy link
Contributor Author

dkwo commented May 13, 2021

I'm a bit confused: if I build your solution, I still get crash in firefox sandbox with syscal 16, as mentioned in #29632
What am I doing wrong?

@ericonr
Copy link
Member

ericonr commented May 13, 2021

Are you using firefox from repos or from when you built with this patch? There's a chance it's broken when built with the patch but running without it.

@dkwo
Copy link
Contributor Author

dkwo commented May 13, 2021 via email

@dkwo
Copy link
Contributor Author

dkwo commented May 13, 2021

Edit: sorry, my bad, I had to delete previous musl build, now it works as expected.

@dkwo
Copy link
Contributor Author

dkwo commented May 13, 2021

I think your solution is better, so let's close this.
Thank you.

@dkwo dkwo closed this May 13, 2021
@ericonr
Copy link
Member

ericonr commented May 13, 2021

You can build with -f to overwrite the existing package build ;)

@dkwo dkwo deleted the musl-ff branch May 13, 2021 14:35
@dkwo
Copy link
Contributor Author

dkwo commented May 13, 2021 via email

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 11, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants