[09:05] *** apogee_ntv left
[09:06] *** apogee_ntv joined
[09:18] <Geth> ¦ MoarVM/pending-object-repossession: 2d078bbc54 | (Elizabeth Mattijsen)++ | 3 files

[09:18] <Geth> ¦ MoarVM/pending-object-repossession: Install pending object repossession target in demand_object

[09:18] <Geth> ¦ MoarVM/pending-object-repossession: 

[09:18] <Geth> ¦ MoarVM/pending-object-repossession: MVM_serialization_deserialize queues every repossessed STable for body

[09:18] <Geth> ¦ MoarVM/pending-object-repossession: deserialization and then runs the object repossession loop. Inside that

[09:18] <Geth> ¦ MoarVM/pending-object-repossession: loop, resolving a row's updated STable through read_object_table_entry

[09:18] <Geth> ¦ MoarVM/pending-object-repossession: can demand one of the compilation unit's own STables, which enters the

[09:18] <Geth> ¦ MoarVM/pending-object-repossession: work loop and deserializes the queued STable bodies. Deserializing an

[09:18] <Geth> ¦ MoarVM/pending-object-repossession: <…commit message has 55 more lines…>

[09:18] <Geth> ¦ MoarVM/pending-object-repossession: review: https://github.com/MoarVM/MoarVM/commit/2d078bbc54

[09:19] <Geth> ¦ MoarVM: lizmat++ created pull request #2046: Install pending object repossession target in demand_object

[09:19] <Geth> ¦ MoarVM: review: https://github.com/MoarVM/MoarVM/pull/2046

[10:20] <Geth> ¦ MoarVM/main: a6b33639c7 | (Elizabeth Mattijsen)++ (committed using GitHub Web editor) | 3 files

[10:20] <Geth> ¦ MoarVM/main: Install pending object repossession target in demand_object

[10:20] <Geth> ¦ MoarVM/main: 

[10:20] <Geth> ¦ MoarVM/main: MVM_serialization_deserialize queues every repossessed STable for body

[10:20] <Geth> ¦ MoarVM/main: deserialization and then runs the object repossession loop. Inside that

[10:20] <Geth> ¦ MoarVM/main: loop, resolving a row's updated STable through read_object_table_entry

[10:20] <Geth> ¦ MoarVM/main: can demand one of the compilation unit's own STables, which enters the

[10:20] <Geth> ¦ MoarVM/main: work loop and deserializes the queued STable bodies. Deserializing an

[10:20] <Geth> ¦ MoarVM/main: <…commit message has 55 more lines…>

[10:20] <Geth> ¦ MoarVM/main: review: https://github.com/MoarVM/MoarVM/commit/a6b33639c7

[13:34] *** bisectable6 left
[13:34] *** quotable6 left
[13:34] *** bloatable6 left
[13:34] *** committable6 left
[13:34] *** coverable6 left
[13:34] *** notable6 left
[13:34] *** shareable6 left
[13:34] *** tellable6 left
[13:34] *** unicodable6 left
[13:34] *** evalable6 left
[13:34] *** sourceable6 left
[13:34] *** huggable6 left
[13:34] *** greppable6 left
[13:34] *** releasable6 left
[13:34] *** linkable6 left
[13:34] *** benchable6 left
[13:34] *** nativecallable6 left
[13:39] *** bloatable6 joined
[13:39] *** releasable6 joined
[13:39] *** bisectable6 joined
[13:39] *** benchable6 joined
[13:39] *** linkable6 joined
[13:40] *** committable6 joined
[13:40] *** notable6 joined
[13:40] *** sourceable6 joined
[13:41] *** evalable6 joined
[13:41] *** quotable6 joined
[13:41] *** greppable6 joined
[13:41] *** shareable6 joined
[13:41] *** unicodable6 joined
[13:41] *** nativecallable6 joined
[13:42] *** huggable6 joined
[13:42] *** tellable6 joined
[13:42] *** coverable6 joined
[16:46] <[Coke]> I am not a fan of commits being faux-cherry picked from an outside repository

[16:46] <[Coke]> I would rather the commit had gone in with ugexe as the author via a pull request so it could have gone through CI, etc.

[16:46] <[Coke]> (rather than relying on the CI running on main)

[16:47] <[Coke]> I did this for a previous ugexe commit, where I cloned his repo, added ours as an upstream, pushed his branch to our repo, and then made a PR out of that new branch.

[16:50] <[Coke]> (I see this was done via a PR as well, so that gets us the CI, but does not get the author information correct)

[17:58] <lizmat> The reason it happened this way, is that ugexe at least appeared to be unwilling to make a PR

[18:00] <lizmat> https://irclogs.raku.org/raku-dev/gist.html?2026-09-21Z20:28,2026-09-21Z22:12,2026-09-21Z23:48

[18:00] <lizmat> please note that https://github.com/MoarVM/MoarVM/commit/2d078bbc54 states at the end:

[18:01] <lizmat> Taken from:

[18:01] <lizmat>   ugexe@cb4feec

[18:03] <[Coke]> we can pull the branch in as is, then create a pr in moarvm/moarvm ourselves from that.

[18:04] <[Coke]> (that's how I tested that originally) (that branch is probably still around)

[18:04] <[Coke]> Just for next time.

[18:04] <[Coke]> no worries about changing this one.

[18:04] <lizmat> well, I guess my git foo was insufficient to do what you suggested

[18:05] <lizmat> also, I find that just copying the code from a PR manually, helps in understanding the PR significantly

[18:05] <lizmat> it's a forced slow way of looking at the code, works for me

[22:51] <apogee_ntv> I will try to look at that prefix topic setter issue properly once I get Selkie 0.99 out, that's taking most of the hours I have in the day at the moment.

[22:51] <apogee_ntv> postfix*