Opened 5 weeks ago

Closed 5 weeks ago

Last modified 5 weeks ago

#71078 closed defect (fixed)

valkey @8.0.1: Build error

Reported by: GarrettAlbright (Garrett Albright) Owned by: mohd-akram (Mohamed Akram)
Priority: Normal Milestone:
Component: ports Version: 2.10.2
Keywords: Cc:
Port: valkey

Description

Valkey 8.0.1 is failing to build. This is happening both on my work machine which has 8.0.0 installed and on my laptop which has no previous version installed. Both are M1 machines.

The attached main.log file seems to indicate issues related to Lua. Lua doesn't seem to be a dependency though. I do have the standard "lua" package (@5.3.6) and some related stuff installed, if that helps.

Attachments (1)

main.log (198.4 KB) - added by GarrettAlbright (Garrett Albright) 5 weeks ago.

Download all attachments as: .zip

Change History (6)

Changed 5 weeks ago by GarrettAlbright (Garrett Albright)

Attachment: main.log added

comment:1 Changed 5 weeks ago by jmroot (Joshua Root)

Cc: makr@… removed
Owner: set to mohd-akram
Status: newassigned

comment:2 Changed 5 weeks ago by ryandesign (Ryan Carsten Schmidt)

It looks like valkey might have a bundled copy of lua. It could be getting confused by the MacPorts lua that you have installed, inadvertently using parts of MacPorts lua instead of its own version. If so, that is a bug that we should fix. To verify whether that's the case, try this:

sudo port clean valkey
sudo port -f deactivate lua
sudo port install valkey

You can sudo port activate lua again afterward if you like.

comment:3 Changed 5 weeks ago by mohd-akram (Mohamed Akram)

Resolution: fixed
Status: assignedclosed

In 80fe9a62668d13226326509c23da58d0cce1e546/macports-ports (master):

valkey: fix build conflict with lua

Fixes: #71078

comment:4 Changed 5 weeks ago by mohd-akram (Mohamed Akram)

This was already fixed but it was reverted when I updated Valkey to 8.0.1 while cleaning up the portfile. I added the same code back. I hadn't seen #40656 so the fix seemed weird and probably outdated, but -isystem makes a bit more sense now.

comment:5 Changed 5 weeks ago by GarrettAlbright (Garrett Albright)

The port builds for me now. Thank you!

Note: See TracTickets for help on using tickets.