Skip to content

add patch for Flye to fix compilation error on aarch64#13142

Closed
rungitta wants to merge 1 commit intoeasybuilders:developfrom
rungitta:20210614164639_new_pr_Flye283
Closed

add patch for Flye to fix compilation error on aarch64#13142
rungitta wants to merge 1 commit intoeasybuilders:developfrom
rungitta:20210614164639_new_pr_Flye283

Conversation

@rungitta
Copy link
Copy Markdown
Contributor

(created using eb --new-pr)

@rungitta
Copy link
Copy Markdown
Contributor Author

Seems like #13080 only worked for building Flye. Running the test produced an error:
ref. test: https://github.com/fenderglass/Flye/blob/flye/docs/INSTALL.md#installing-from-sourcer
ref. error: mikolmogorov/Flye#398

Patch fixes this on aarch64. However, I'm not sure this patch doesn't cause any issues on x86_64.

@zao
Copy link
Copy Markdown
Contributor

zao commented Jun 14, 2021

Test report by @zao
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
eb-u2004 - Linux Ubuntu 20.04, x86_64, Intel(R) Core(TM) i7-8700K CPU @ 3.70GHz, Python 3.8.5
See https://gist.github.com/f83e003ff209d9c82f3bbd9a07812de9 for a full test report.

@boegel boegel changed the title patch to fix the unsigned char error on aarch64 add patch for Flye to fix compilation error on aarch64 Jan 14, 2024
@boegel boegel added bug fix aarch64 Related to Arm 64-bit (aarch64) labels Jan 14, 2024
@boegel boegel added this to the 4.x milestone Jan 14, 2024
@boegel
Copy link
Copy Markdown
Member

boegel commented Jun 16, 2025

Closing this since GCC/10.2.0 is deprecated or no longer supported, see https://docs.easybuild.io/policies/toolchains

In addition, this easyconfig was archived recently, see #22954 + easybuilders/easybuild-easyconfigs-archive#11

Sorry for not getting back to this @rungitta

If this is still relevant, please consider opening a new pull request using a more recent toolchain

@boegel boegel closed this Jun 16, 2025
@boegel
Copy link
Copy Markdown
Member

boegel commented Jun 16, 2025

Installation of Flye/2.9.3-GCC-12.3.0 on aarch64/neoverse_v1 works fine, so fix here doesn't seem relevant anymore (at least not with recent compiler version)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

aarch64 Related to Arm 64-bit (aarch64) bug fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants