Bug 467482 - Build failure on aarch64 Alpine
Summary: Build failure on aarch64 Alpine
Status: RESOLVED FIXED
Alias: None
Product: valgrind
Classification: Developer tools
Component: general (other bugs)
Version First Reported In: 3.21 GIT
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Paul Floyd
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-03-17 11:22 UTC by fanquake
Modified: 2023-03-17 19:03 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
Patch to fix compile failure (1.23 KB, application/mbox)
2023-03-17 11:22 UTC, fanquake
Details

Note You need to log in before you can comment on or make changes to this bug.
Description fanquake 2023-03-17 11:22:06 UTC
Created attachment 157357 [details]
Patch to fix compile failure

aarch64 musl: fix compile failure
    
    Building master (50f65336127b5f63c8af55544ce3e6a9bc834355) on ALpine,
    aarch64, currently fails for me with:
    ```
    m_replacemalloc/vg_replace_malloc.c:1687:11: error: 'VG_MEMALIGN_MAKE_SIZE_MULTIPLE_ALIGN'
            undeclared (first use in this function); did you mean 'VG_MEMALIG_MAKE_SIZE_MULTIPLE_ALIGN'?
     1687 |       if (VG_MEMALIGN_MAKE_SIZE_MULTIPLE_ALIGN) { \
    ```
    
    Fix what looks to be a typo in vg_replace_malloc.c.
Comment 1 Paul Floyd 2023-03-17 12:10:51 UTC
That's for me. I'll apply the patch and push it tonight or this weekend.
Comment 2 Paul Floyd 2023-03-17 19:03:18 UTC
I don't have access to aarch64 musl, but it seems OK now on amd64 musl.

commit 022a9caf40ceb32b8dfe258a8f2777f6299457d3 (origin/master, origin/HEAD, master)
Author: Paul Floyd <pjfloyd@wanadoo.fr>
Date:   Fri Mar 17 19:48:08 2023 +0100

    Bug 467482 - Build failure on aarch64 Alpine
    
    Patch submitted by
            fanquake@gmail.com