Skip to content

Conversation

@Revolyssup
Copy link
Contributor

No description provided.

@Revolyssup Revolyssup requested review from membphis and nic-6443 June 9, 2025 04:47
@Revolyssup Revolyssup marked this pull request as draft June 9, 2025 04:52
@Revolyssup Revolyssup marked this pull request as ready for review June 9, 2025 06:45
# build the lib and specify the storage path of the package installed
luarocks make ./apisix-master-${iteration}.rockspec --tree=/tmp/build/output/apisix/usr/local/apisix/deps --local
# To be removed after https://github.com/luarocks/luarocks/issues/1797 is fixed
luarocks --only-server https://raw.githubusercontent.com/rocks-moonscript-org/moonrocks-mirror/daab2726276e3282dc347b89a42a5107c3500567 make ./apisix-master-${iteration}.rockspec --tree=/tmp/build/output/apisix/usr/local/apisix/deps --local
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

luarocks/luarocks#1797 already closed

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

But aparrently the issue isn't fixed yet. Running on master gives the same error.
// @bzp2010

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bzp2010 What was the fix for it?

Copy link
Contributor

@nic-6443 nic-6443 Jun 9, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok, if you confirmed it's not working yet, we can merge with this workaround first.

@Revolyssup Revolyssup merged commit 6f1b7a9 into api7:master Jun 10, 2025
6 checks passed
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.

3 participants