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

PKGBUILD: avoid stripping the eask binary #249

Merged
merged 1 commit into from
May 21, 2024

Conversation

Hi-Angel
Copy link
Contributor

Node puts scripts to some ELF sections, so we can't strip these. Strictly speaking, judging by the offsets they seem to be put into .dynstr section, but leaving that section unstripped doesn't make the error

    Pkg: Error reading from file

…go away. Unless someone is motivated to dig into the list of sections to be put to STRIP_BINARIES, let's just disable it.

Fixes: #248

Node puts scripts to some ELF sections, so we can't strip these.
Strictly speaking, judging by the offsets they seem to be put into
.dynstr section, but leaving that section unstripped doesn't make the
error

        Pkg: Error reading from file

…go away. Unless someone is motivated to dig into the list of sections
to be put to `STRIP_BINARIES`, let's just disable it.

Fixes: emacs-eask#248
@Hi-Angel Hi-Angel changed the title PKGBUILD: avoid stripping the PKGBUILD binary PKGBUILD: avoid stripping the eask binary May 21, 2024
@jcs090218 jcs090218 merged commit 96a3392 into emacs-eask:master May 21, 2024
138 of 143 checks passed
@jcs090218
Copy link
Member

Thank you!

@Hi-Angel
Copy link
Contributor Author

And thank you too! 😊

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.

[BUG] Cannot install on Arch
2 participants