Bug 467482

Summary: Build failure on aarch64 Alpine
Product: [Developer tools] valgrind Reporter: fanquake
Component: generalAssignee: Paul Floyd <pjfloyd>
Status: RESOLVED FIXED    
Severity: normal CC: pjfloyd
Priority: NOR    
Version First Reported In: 3.21 GIT   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:
Attachments: Patch to fix compile failure

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