circular saw VS glostone #8551

Open
opened 2025-05-17 00:30:34 +02:00 by AliasAlreadyTaken · 4 comments

2025-05-17 00:19:10: ACTION[Server]: [yl_commons] AliasAlreadyTaken moves "ethereal:micro_glostone 4" to {"name":"AliasAlreadyTaken","type":"player"} [main]
2025-05-17 00:19:10: ACTION[Server]: [yl_commons] lua is using 254.0 MB
2025-05-17 00:19:12: WARNING[ConnectionSend]: con(4/1) Packet quota used up for peer_id=45052, was 512 pkts
2025-05-17 00:19:12: WARNING[ConnectionSend]: con(4/1) Packet quota used up for peer_id=45052, was 512 pkts
2025-05-17 00:19:12: WARNING[ConnectionSend]: con(4/1) Packet quota used up for peer_id=45052, was 512 pkts
2025-05-17 00:19:16: ACTION[Server]: [yl_commons] AliasAlreadyTaken takes "ethereal:micro_glostone 4" from {"name":"AliasAlreadyTaken","type":"player"} [main]
2025-05-17 00:19:16: ACTION[Server]: [yl_commons] "AliasAlreadyTaken" moves ethereal:micro_glostone 4 to moreblocks:circular_saw @ (3965,21,-9060) [recycle]
2025-05-17 00:19:16: ACTION[Main]: Server: Shutting down
2025-05-17 00:19:16: ACTION[Server]: [MOD] smart_chat : Module core: *** leaves the World.
2025-05-17 00:19:16: ACTION[Server]: AliasAlreadyTaken leaves game. List of players:
2025-05-17 00:19:21: ERROR[Main]: ServerError: AsyncErr: Lua: Runtime error from mod 'moreblocks' in callback nodemeta_inventory_OnPut(): ....0/Yourland_test/bin/../mods/moreblocks/circular_saw.lua:184: attempt to index a nil value
2025-05-17 00:19:21: ERROR[Main]: stack traceback:
2025-05-17 00:19:21: ERROR[Main]: ....0/Yourland_test/bin/../mods/moreblocks/circular_saw.lua:184: in function 'update_inventory'
2025-05-17 00:19:21: ERROR[Main]: ....0/Yourland_test/bin/../mods/moreblocks/circular_saw.lua:323: in function <....0/Yourland_test/bin/../mods/moreblocks/circular_saw.lua:303>

2025-05-17 00:19:10: ACTION[Server]: [yl_commons] AliasAlreadyTaken moves "ethereal:micro_glostone 4" to {"name":"AliasAlreadyTaken","type":"player"} [main] 2025-05-17 00:19:10: ACTION[Server]: [yl_commons] lua is using 254.0 MB 2025-05-17 00:19:12: WARNING[ConnectionSend]: con(4/1) Packet quota used up for peer_id=45052, was 512 pkts 2025-05-17 00:19:12: WARNING[ConnectionSend]: con(4/1) Packet quota used up for peer_id=45052, was 512 pkts 2025-05-17 00:19:12: WARNING[ConnectionSend]: con(4/1) Packet quota used up for peer_id=45052, was 512 pkts 2025-05-17 00:19:16: ACTION[Server]: [yl_commons] AliasAlreadyTaken takes "ethereal:micro_glostone 4" from {"name":"AliasAlreadyTaken","type":"player"} [main] 2025-05-17 00:19:16: ACTION[Server]: [yl_commons] "AliasAlreadyTaken" moves ethereal:micro_glostone 4 to moreblocks:circular_saw @ (3965,21,-9060) [recycle] 2025-05-17 00:19:16: ACTION[Main]: Server: Shutting down 2025-05-17 00:19:16: ACTION[Server]: [MOD] smart_chat : Module core: *** leaves the World. 2025-05-17 00:19:16: ACTION[Server]: AliasAlreadyTaken leaves game. List of players: 2025-05-17 00:19:21: ERROR[Main]: ServerError: AsyncErr: Lua: Runtime error from mod 'moreblocks' in callback nodemeta_inventory_OnPut(): ....0/Yourland_test/bin/../mods/moreblocks/circular_saw.lua:184: attempt to index a nil value 2025-05-17 00:19:21: ERROR[Main]: stack traceback: 2025-05-17 00:19:21: ERROR[Main]: ....0/Yourland_test/bin/../mods/moreblocks/circular_saw.lua:184: in function 'update_inventory' 2025-05-17 00:19:21: ERROR[Main]: ....0/Yourland_test/bin/../mods/moreblocks/circular_saw.lua:323: in function <....0/Yourland_test/bin/../mods/moreblocks/circular_saw.lua:303>
Member

Looking at blame, seems introduced here: 2611e64a2b

We are using stairsplus:register_custom_subset() to register glostone, which calls stairsplus.register_single(), but the variable that was introduced in that commit is updated only in stairsplus:register_micro(), which calls stairsplus.register_single() itself...

So upstream messed this up by not updating the table where needed, I guess.

Looking at blame, seems introduced here: https://gitea.your-land.de/your-land/moreblocks/commit/2611e64a2b14c1586dc283569b6780611caeced0 We are using `stairsplus:register_custom_subset()` to register glostone, which calls `stairsplus.register_single()`, but the variable that was introduced in that commit is updated only in `stairsplus:register_micro()`, which calls `stairsplus.register_single()` itself... So upstream messed this up by not updating the table where needed, I guess.
whosit added the
1. kind/bug
3. source/mod upstream
2. prio/critical
labels 2025-05-17 03:02:46 +02:00
Member
Upstream issue: https://github.com/minetest-mods/moreblocks/issues/209
Member

I basically reverted that moreblocks commit with temp_fix_circular_saw_8551 snippet.

I basically reverted that moreblocks commit with `temp_fix_circular_saw_8551` snippet.
Member

Should be fixed upstream

Should be fixed upstream
whosit added the
4. step/ready to QA test
label 2025-05-21 19:04:55 +02:00
whosit added this to the Next milestone 2025-05-21 19:05:00 +02:00
Sign in to join this conversation.
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: your-land/bugtracker#8551
No description provided.