[00:02] *** reportable6 left [00:04] I just rebuilt master and cannot reproduce the error [00:05] *** reportable6 joined [00:06] i reverted the problem commit [00:06] `rakubrew build moar d8281722db && rakubrew switch moar-d8281722db` to get back to the problem commit [00:07] ah, got it. should have read a little more carefully :) [00:07] *** linkable6 left [00:08] *** linkable6 joined [00:22] *** jgaz left [01:22] *** linkable6 left [01:22] *** evalable6 left [01:24] *** linkable6 joined [01:24] *** evalable6 joined [02:21] *** frost joined [02:32] *** frost left [02:36] *** MasterDuke left [02:36] *** MasterDuke joined [02:43] *** melezhik left [03:07] *** Kaiepi joined [03:08] *** Kaipi left [05:57] *** evalable6 left [05:57] *** bisectable6 left [05:57] *** nativecallable6 left [05:57] *** releasable6 left [05:57] *** quotable6 left [05:57] *** linkable6 left [05:57] *** squashable6 left [05:57] *** coverable6 left [05:57] *** greppable6 left [05:57] *** benchable6 left [05:57] *** statisfiable6 left [05:57] *** unicodable6 left [05:57] *** committable6 left [05:57] *** tellable6 left [05:57] *** sourceable6 left [05:57] *** shareable6 left [05:57] *** notable6 left [05:57] *** reportable6 left [05:57] *** bloatable6 left [05:57] *** reportable6 joined [05:57] *** evalable6 joined [05:58] *** releasable6 joined [05:58] *** notable6 joined [05:58] *** bisectable6 joined [05:58] *** tellable6 joined [05:58] *** sourceable6 joined [05:58] *** nativecallable6 joined [05:58] *** greppable6 joined [05:58] *** coverable6 joined [05:58] *** squashable6 joined [05:59] *** bloatable6 joined [05:59] *** committable6 joined [05:59] *** statisfiable6 joined [05:59] *** unicodable6 joined [06:00] *** linkable6 joined [06:00] *** shareable6 joined [06:00] *** quotable6 joined [06:00] *** benchable6 joined [06:02] *** reportable6 left [06:04] *** reportable6 joined [06:38] ¦ rakudo: 2839cef6bd | (Patrick Böker)++ | src/core.c/CompUnit/Repository/Installation.pm6 [06:38] ¦ rakudo: Fix CURI probing on Windows [06:38] ¦ rakudo: [06:38] ¦ rakudo: libUVs stat returns unusable writability information on Windows. It [06:38] ¦ rakudo: can't be trusted for actual writeability. This means `IO::Path.w` is [06:38] ¦ rakudo: just as unusable. Thus in `CURI.can-install()` on Windows we now [06:38] ¦ rakudo: actually probe for writeability by trying to create a file. [06:38] ¦ rakudo: [06:38] ¦ rakudo: This fixes installing things on Windows with Rakudo being installed [06:38] ¦ rakudo: in "C:\Program Files\Rakudo". [06:38] ¦ rakudo: review: https://github.com/rakudo/rakudo/commit/2839cef6bd [06:38] ¦ rakudo: 798878cdd8 | (Patrick Böker)++ (committed using GitHub Web editor) | src/core.c/CompUnit/Repository/Installation.pm6 [06:38] ¦ rakudo: Merge pull request #4452 from patrickbkr/fix-win-install-write-protected [06:39] ¦ rakudo: Merge pull request #4452 from patrickbkr/fix-win-install-write-protected [06:39] ¦ rakudo: [06:39] ¦ rakudo: Fix CURI probing on Windows [06:39] ¦ rakudo: review: https://github.com/rakudo/rakudo/commit/798878cdd8 [07:00] ¦ rakudo: patrickbkr++ created pull request #4455: Create checksums for release files [07:00] ¦ rakudo: review: https://github.com/rakudo/rakudo/pull/4455 [07:08] *** patrickb joined [07:22] *** Kaiepi left [07:23] *** Kaiepi joined [07:57] *** dumarchie joined [07:57] *** dumarchie left [08:26] *** Kaiepi left [08:31] *** Kaiepi joined [09:20] *** Kaiepi left [09:23] *** frost joined [09:23] *** Kaiepi joined [10:23] *** evalable6 left [10:23] *** linkable6 left [10:25] *** evalable6 joined [10:25] *** linkable6 joined [10:27] *** Kaiepi left [10:27] *** Kaiepi joined [10:32] *** sena_kun joined [11:05] *** Kaiepi left [11:07] *** Kaiepi joined [11:54] releasable6, status [11:54] sena_kun, Next release in ≈7 days and ≈7 hours. 2 blockers. Changelog for this release was not started yet [11:54] sena_kun, Details: https://gist.github.com/5175cf3729dc80356a7083c980bdd2fa [12:02] *** reportable6 left [12:05] *** reportable6 joined [12:15] ¦ rakudo: 57885f6650 | (Patrick Böker)++ (committed using GitHub Web editor) | tools/templates/moar/Makefile.in [12:15] ¦ rakudo: Fix `rakuw.exe` to be a non-console app [12:15] ¦ rakudo: [12:15] ¦ rakudo: The *w.exe binaries are the "windowed" variants that don't open a console window. [12:15] ¦ rakudo: `rakuw.exe` was missing in the respective build system check. [12:15] ¦ rakudo: review: https://github.com/rakudo/rakudo/commit/57885f6650 [12:15] ¦ rakudo: 6eea26213b | (Patrick Böker)++ (committed using GitHub Web editor) | tools/templates/moar/Makefile.in [12:15] ¦ rakudo: Merge pull request #4451 from rakudo/windowed-rakuw [12:15] ¦ rakudo: [12:15] ¦ rakudo: Fix `rakuw.exe` to be a non-console app [12:15] ¦ rakudo: review: https://github.com/rakudo/rakudo/commit/6eea26213b [12:16] *** patrickb left [12:21] *** patrickb joined [13:06] *** frost left [13:12] *** Util_ joined [13:13] *** jdv_ joined [13:14] *** gfldex_ joined [13:15] *** [Coke]_ joined [13:18] *** [Coke] left [13:18] *** gfldex left [13:18] *** Util left [13:18] *** jdv left [13:53] ¦ nqp: 06014d5657 | (Stefan Seifert)++ | src/core/NQPRoutine.nqp [13:53] ¦ nqp: Fix reproducible build regression [13:53] ¦ nqp: [13:53] ¦ nqp: Commit d97f3084addf86a5a187d404294deda82b173b70 introduced new arrays, one of [13:53] ¦ nqp: which was assembled depending on hash order. This caused the order of some [13:53] ¦ nqp: objects to vary from build to build. [13:53] ¦ nqp: review: https://github.com/Raku/nqp/commit/06014d5657 [14:16] ¦ rakudo: gfldex++ created pull request #4460: fix gh##4458 [14:16] ¦ rakudo: review: https://github.com/rakudo/rakudo/pull/4460 [14:17] *** linkable6 left [14:17] *** linkable6 joined [14:17] *** gfldex_ is now known as gfldex [14:23] I have two Gimp image files, the backgrounds of the new Windows installer. Together they are roughly 900kb in size. Should I add them to te rakudo repo or leave them out? [14:31] patrickb, better to be a separate repo IMO? [14:31] https://github.com/Raku/marketing is maybe not the right place... [14:32] patrickb, note as we build an image for every commit the additional weight may be not so desired [14:33] The images are specific to the installer, so no generic marketing resources. But it might be the practical solution. [14:33] That's why I asked. ;-) [14:34] patrickb, we can pull them during building installers from some other repo. I agree marketing is not exactly that, just pointing out possible direction. [14:35] *** vrurg_ joined [14:36] *** vrurg left [14:37] No. The acutal bmp images I'd keep in the rakudo repo (543kb). I was asking about the Gimp files that are the base of those bmp files... [14:38] ah, then ignore me. :/ [14:50] *** vrurg_ is now known as vrurg [14:58] Hm. I could actually move the entire binary release stuff to some place else. There is no reason for those files to end up in the source releases I'd say. [14:58] But where could they go? [14:59] New repo rakudo/release-tools ? [15:00] Hm the releasable stuff is also contained in the source release... [15:29] This run started on 2021-07-16T15:28:25Z and finished in 43 minutes. [15:29] but then the release stuff should stay in sync with rakudo itself. So maybe it's best to just keep it there... [15:29] yay, a new record for Blin [15:29] yay [15:34] Were there changes to the machine? [15:36] patrickb, 5950X + Pakku. [15:36] Weee! [15:36] with zef the numbers are similar, about 1-2 hours I think [15:56] ¦ nqp/new-disp: 10131fd17d | (Jonathan Worthington)++ | src/core/dispatchers.nqp [15:56] ¦ nqp/new-disp: Generify method not found handler in nqp-meth-call [15:56] ¦ nqp/new-disp: [15:56] ¦ nqp/new-disp: Delegate to the appropriate dispatcher, such that we can have the [15:56] ¦ nqp/new-disp: calling language's method not found handler invoked. [15:56] ¦ nqp/new-disp: review: https://github.com/Raku/nqp/commit/10131fd17d [15:56] ¦ nqp/new-disp: 1169dd49f9 | (Jonathan Worthington)++ | t/nqp/097-hll.t [15:56] ¦ nqp/new-disp: Adapt test to new method not found API [15:56] ¦ nqp/new-disp: review: https://github.com/Raku/nqp/commit/1169dd49f9 [15:56] ¦ nqp/new-disp: 1d6eefd3be | (Jonathan Worthington)++ | t/nqp/097-hll.t [15:56] ¦ nqp/new-disp: Make sure test type gets a HLL set [15:56] ¦ nqp/new-disp: review: https://github.com/Raku/nqp/commit/1d6eefd3be [15:58] ¦ rakudo/new-disp: f47d8b8b84 | (Jonathan Worthington)++ | src/Perl6/bootstrap.c/BOOTSTRAP.nqp [15:58] ¦ rakudo/new-disp: Adapt to method not found handler API change [15:58] ¦ rakudo/new-disp: [15:58] ¦ rakudo/new-disp: On MoarVM, after new-disp, it now also receives the arguments. We slurp [15:58] ¦ rakudo/new-disp: them, which is conveniently backward compatible with other backends not [15:58] ¦ rakudo/new-disp: sending them along. [15:58] ¦ rakudo/new-disp: review: https://github.com/rakudo/rakudo/commit/f47d8b8b84 [16:09] *** patrickb left [16:25] *** dogbert17 joined [16:27] *** dogbert11 left [16:32] *** dogbert11 joined [16:34] *** dogbert17 left [16:39] *** patrickb joined [17:12] *** sena_kun left [17:13] *** patrickb left [17:15] *** patrickb joined [17:22] I just uploaded an MSI of Rakudo 2021.06 to rakudo.org. Testers welcome. [17:27] *** Kaiepi left [17:28] <[Coke]_> patrickb++ [17:32] *** Kaiepi joined [17:32] *** Kaiepi left [17:33] *** [Coke]_ is now known as [Coke] [17:36] *** Kaiepi joined [17:38] *** Kaipi joined [17:39] *** Kaiepi left [17:49] patrickb++ [18:02] *** reportable6 left [18:05] *** reportable6 joined [18:17] *** dogbert17 joined [18:17] *** dogbert11 left [18:33] *** dogbert11 joined [18:36] *** dogbert17 left [18:41] *** dogbert11 left [18:50] *** dogbert11 joined [18:56] *** dogbert11 left [18:57] *** dogbert11 left [18:59] *** dogbert11 joined [19:03] *** jdv_ is now known as jdv [19:08] *** squashable6 left [19:09] *** squashable6 joined [20:09] *** evalable6 left [20:09] *** linkable6 left [20:10] *** evalable6 joined [20:12] *** linkable6 joined [21:29] patrickb: https://ci.appveyor.com/project/ugexe/zef/builds/40025155 [21:32] maybe creating a file isn't sufficient and it needs to create a directory? [21:32] just a wild guess [21:32] Is C:\projects\foo writable? [21:33] That worked before I assume? [21:33] it worked before, yes [21:34] There is nothing special about that folder, right? Then why should the probing fail? Wait, does that folder possibly simply doesn't exist yet? [21:35] it would get created by CURI i believe [21:37] yeah im confused... doesnt seem like it should happen [21:37] That check is in install(). If no prior call to can-install() happens that error could actually be explained. [21:38] Because writeable-path() does not try to create the folder if it doesn't exist. [21:38] yeah i guess there would be no prior call in that zef example because it doesnt use `auto`, it explicitly overrides that with -to=inst#foo sp yeah [21:39] Wait. No, that's not true. It's the private !writeable-path() that is called. That one does actually call can-install() [21:40] *** melezhik joined [21:40] oh in that appveyor output it shows that message twice [21:40] Or is there some non-obvious behavior when a private and public method are named the same? [21:43] . [21:43] releasable: status [21:43] melezhik, Next release in ≈6 days and ≈21 hours. 3 blockers. Changelog for this release was not started yet [21:43] melezhik, Details: https://gist.github.com/3ab4f4e113dcf9fcbfdf8e95932430b5 [21:54] that foo folder actually does not exist prior to trying to install to it, IIUC. [21:54] right [21:56] self!prefix-writeable || ?(!$.prefix.e && try { $.prefix.mkdir } && $.prefix.e); [21:56] presumably it doesnt reach the mkdir there [21:56] anymore [21:56] I know [21:56] the thing caches [21:57] the very first time when the dir doesn't exist it remembers the failure [21:58] self!prefix-writeable is cached, and if its caching a failure i would expect the above to fall into the rhs of the || [22:00] Right [22:05] But still. [22:06] The negative test is cached and the dir is created afterwards. All following calls will return False. [22:07] Because the writability test fails and the test for non-existance fails as well. [22:07] maybe s{//=}{||=} [22:08] Ha! I had the same idea! [22:10] Having the check for writability before the check of existence is a performance optimization for the common path of the repo existing, correct? [22:13] https://github.com/rakudo/rakudo/commit/495ef47794506d262804839ffd9a360af1b0fcb7 [22:13] that being said i dont remember exactly why [22:14] Kind of makes sense to not blindly create loads of repos on disk when we might never actually install anything into them... [22:17] ¦ rakudo: patrickbkr++ created pull request #4461: Fix installation to to-be-created CURIS on Windows [22:17] ¦ rakudo: review: https://github.com/rakudo/rakudo/pull/4461 [22:19] ugexe: Shall we just merge and see what the CI thinks of it? (I'd need to reboot and build a Rakudo with a build time > 20 minutes.) [22:20] https://ci.appveyor.com/project/ugexe/zef/builds/40025829 [22:20] i just made a zef PR that checks out that commit [22:21] s/PR/branch/ [22:21] oops my incantation was wrong heh [22:24] *** dogbert17 joined [22:28] *** dogbert11 left [22:34] *** dogbert11 joined [22:36] *** squashable6 left [22:36] *** dogbert17 left [22:39] *** squashable6 joined [22:45] I need to go to bed. I'll have a look tomorrow how it went. [22:45] ugexe: Thanks for having a look with me. [22:46] *** patrickb left [22:48] ¦ rakudo: d668d99292 | (Patrick Böker)++ (committed using GitHub Web editor) | src/core.c/CompUnit/Repository/Installation.pm6 [22:48] ¦ rakudo: Fix installation to to-be-created CURIS on Windows (#4461) [22:48] ¦ rakudo: [22:48] ¦ rakudo: The negative test for writeability when the repo was still not existing was [22:48] ¦ rakudo: cached and spoilt all further checks. Now we only cache a successful [22:48] ¦ rakudo: writeability check. [22:48] ¦ rakudo: review: https://github.com/rakudo/rakudo/commit/d668d99292 [22:48] *** linkable6 left [22:49] *** linkable6 joined [23:25] ¦ nqp/new-disp: 1c859c3a57 | (Jonathan Worthington)++ | 2 files [23:25] ¦ nqp/new-disp: Add nqp-find-meth dispatcher [23:25] ¦ nqp/new-disp: [23:25] ¦ nqp/new-disp: Which will be used to implement findmethod/tryfindmethod/can on the [23:25] ¦ nqp/new-disp: MoarVM backend. [23:25] ¦ nqp/new-disp: review: https://github.com/Raku/nqp/commit/1c859c3a57 [23:27] *** sortiz joined [23:30] *** sortiz left [23:54] *** Merfont joined [23:54] *** Kaipi left