From zsong at openjdk.org Thu Jul 3 20:56:48 2025 From: zsong at openjdk.org (Zhao Song) Date: Thu, 3 Jul 2025 20:56:48 GMT Subject: RFR: 2510: Bug closed due to inactivity even though there was "activity" Message-ID: <8ASTfNy1jJJjaRe8EfR_OzYDfYgDGy-2MyNFz9ZlBtU=.9e166f5c-4d10-44e8-8f80-461e326544b9@github.com> Currently, the PullRequestPrunerBot considers only comments, reviews, and review comments as activity. Therefore, it can sometimes mistakenly close a user's pull request. In this patch, I am trying to let PullRequestPrunerBot counts commits, pr state changes as valid activities. ------------- Commit messages: - update - update - update - update - 2510 Changes: https://git.openjdk.org/skara/pull/1723/files Webrev: https://webrevs.openjdk.org/?repo=skara&pr=1723&range=00 Issue: https://bugs.openjdk.org/browse/SKARA-2510 Stats: 135 lines in 12 files changed: 123 ins; 0 del; 12 mod Patch: https://git.openjdk.org/skara/pull/1723.diff Fetch: git fetch https://git.openjdk.org/skara.git pull/1723/head:pull/1723 PR: https://git.openjdk.org/skara/pull/1723 From jlahoda at openjdk.org Fri Jul 4 08:43:28 2025 From: jlahoda at openjdk.org (Jan Lahoda) Date: Fri, 4 Jul 2025 08:43:28 GMT Subject: RFR: Merging JShell's file locations mapping into the 'compiler' label, and deleting mapping for 'kulla'. Message-ID: There's a proposal to dissolve the Kulla project (the project that introduced JShell into JDK), and use the `compiler-dev` mailing list instead of `kulla-dev`: https://mail.openjdk.org/pipermail/kulla-dev/2025-June/005928.html The idea in this PR is to merge the file mappings used for JShell into the `compiler` label, so that changes touching JShell get the `compiler` label, and the e-mails are sent to `compiler-dev`. Please let me now if there's something better than can be done, or what more is needed to stop using the `kulla-dev` list. Thanks! ------------- Commit messages: - Merging JShell's file locations mapping into the 'compiler' label, and deleting mapping for 'kulla'. Changes: https://git.openjdk.org/skara/pull/1724/files Webrev: https://webrevs.openjdk.org/?repo=skara&pr=1724&range=00 Stats: 14 lines in 1 file changed: 6 ins; 8 del; 0 mod Patch: https://git.openjdk.org/skara/pull/1724.diff Fetch: git fetch https://git.openjdk.org/skara.git pull/1724/head:pull/1724 PR: https://git.openjdk.org/skara/pull/1724 From zsong at openjdk.org Mon Jul 7 16:44:58 2025 From: zsong at openjdk.org (Zhao Song) Date: Mon, 7 Jul 2025 16:44:58 GMT Subject: RFR: Merging JShell's file locations mapping into the 'compiler' label, and deleting mapping for 'kulla'. In-Reply-To: References: Message-ID: On Fri, 4 Jul 2025 08:40:32 GMT, Jan Lahoda wrote: > There's a proposal to dissolve the Kulla project (the project that introduced JShell into JDK), and use the `compiler-dev` mailing list instead of `kulla-dev`: > https://mail.openjdk.org/pipermail/kulla-dev/2025-June/005928.html > > The idea in this PR is to merge the file mappings used for JShell into the `compiler` label, so that changes touching JShell get the `compiler` label, and the e-mails are sent to `compiler-dev`. > > Please let me now if there's something better than can be done, or what more is needed to stop using the `kulla-dev` list. > > Thanks! The change looks good to me. But please file an issue for it in SKARA project for it. https://bugs.openjdk.org/projects/SKARA ------------- PR Comment: https://git.openjdk.org/skara/pull/1724#issuecomment-3045882521 From jlahoda at openjdk.org Mon Jul 7 16:51:09 2025 From: jlahoda at openjdk.org (Jan Lahoda) Date: Mon, 7 Jul 2025 16:51:09 GMT Subject: RFR: 2535: Merge JShell's file locations mapping into the 'compiler' label, and deleting mapping for 'kulla' In-Reply-To: References: Message-ID: On Fri, 4 Jul 2025 08:40:32 GMT, Jan Lahoda wrote: > There's a proposal to dissolve the Kulla project (the project that introduced JShell into JDK), and use the `compiler-dev` mailing list instead of `kulla-dev`: > https://mail.openjdk.org/pipermail/kulla-dev/2025-June/005928.html > > The idea in this PR is to merge the file mappings used for JShell into the `compiler` label, so that changes touching JShell get the `compiler` label, and the e-mails are sent to `compiler-dev`. > > Please let me now if there's something better than can be done, or what more is needed to stop using the `kulla-dev` list. > > Thanks! Thanks. I opened: https://bugs.openjdk.org/browse/SKARA-2535 please let me know if something should be improved. ------------- PR Comment: https://git.openjdk.org/skara/pull/1724#issuecomment-3045906303 From zsong at openjdk.org Mon Jul 7 16:55:29 2025 From: zsong at openjdk.org (Zhao Song) Date: Mon, 7 Jul 2025 16:55:29 GMT Subject: RFR: 2535: Merge JShell's file locations mapping into the 'compiler' label, and deleting mapping for 'kulla' In-Reply-To: References: Message-ID: On Fri, 4 Jul 2025 08:40:32 GMT, Jan Lahoda wrote: > There's a proposal to dissolve the Kulla project (the project that introduced JShell into JDK), and use the `compiler-dev` mailing list instead of `kulla-dev`: > https://mail.openjdk.org/pipermail/kulla-dev/2025-June/005928.html > > The idea in this PR is to merge the file mappings used for JShell into the `compiler` label, so that changes touching JShell get the `compiler` label, and the e-mails are sent to `compiler-dev`. > > Please let me now if there's something better than can be done, or what more is needed to stop using the `kulla-dev` list. > > Thanks! Marked as reviewed by zsong (Reviewer). ------------- PR Review: https://git.openjdk.org/skara/pull/1724#pullrequestreview-2994692798 From jlahoda at openjdk.org Tue Jul 8 13:51:21 2025 From: jlahoda at openjdk.org (Jan Lahoda) Date: Tue, 8 Jul 2025 13:51:21 GMT Subject: RFR: 2535: Merge JShell's file locations mapping into the 'compiler' label, and deleting mapping for 'kulla' In-Reply-To: References: Message-ID: On Mon, 7 Jul 2025 16:42:52 GMT, Zhao Song wrote: >> There's a proposal to dissolve the Kulla project (the project that introduced JShell into JDK), and use the `compiler-dev` mailing list instead of `kulla-dev`: >> https://mail.openjdk.org/pipermail/kulla-dev/2025-June/005928.html >> >> The idea in this PR is to merge the file mappings used for JShell into the `compiler` label, so that changes touching JShell get the `compiler` label, and the e-mails are sent to `compiler-dev`. >> >> Please let me now if there's something better than can be done, or what more is needed to stop using the `kulla-dev` list. >> >> Thanks! > > The change looks good to me. But please file an issue for it in SKARA project. https://bugs.openjdk.org/projects/SKARA @zhaosongzs - can I issue the integrate command, or should I wait for another reviewer (the comment on pre-integration checks is not clear in this case). Thanks! ------------- PR Comment: https://git.openjdk.org/skara/pull/1724#issuecomment-3049049778 From zsong at openjdk.org Tue Jul 8 16:04:14 2025 From: zsong at openjdk.org (Zhao Song) Date: Tue, 8 Jul 2025 16:04:14 GMT Subject: RFR: 2535: Merge JShell's file locations mapping into the 'compiler' label, and deleting mapping for 'kulla' In-Reply-To: References: Message-ID: On Mon, 7 Jul 2025 16:42:52 GMT, Zhao Song wrote: >> There's a proposal to dissolve the Kulla project (the project that introduced JShell into JDK), and use the `compiler-dev` mailing list instead of `kulla-dev`: >> https://mail.openjdk.org/pipermail/kulla-dev/2025-June/005928.html >> >> The idea in this PR is to merge the file mappings used for JShell into the `compiler` label, so that changes touching JShell get the `compiler` label, and the e-mails are sent to `compiler-dev`. >> >> Please let me now if there's something better than can be done, or what more is needed to stop using the `kulla-dev` list. >> >> Thanks! > > The change looks good to me. But please file an issue for it in SKARA project. https://bugs.openjdk.org/projects/SKARA > @zhaosongzs - can I issue the integrate command, or should I wait for another reviewer (the comment on pre-integration checks is not clear in this case). Thanks! Yes, please issue the integrate command ------------- PR Comment: https://git.openjdk.org/skara/pull/1724#issuecomment-3049506905 From duke at openjdk.org Wed Jul 9 05:11:00 2025 From: duke at openjdk.org (duke) Date: Wed, 9 Jul 2025 05:11:00 GMT Subject: RFR: 2535: Merge JShell's file locations mapping into the 'compiler' label, and deleting mapping for 'kulla' In-Reply-To: References: Message-ID: On Fri, 4 Jul 2025 08:40:32 GMT, Jan Lahoda wrote: > There's a proposal to dissolve the Kulla project (the project that introduced JShell into JDK), and use the `compiler-dev` mailing list instead of `kulla-dev`: > https://mail.openjdk.org/pipermail/kulla-dev/2025-June/005928.html > > The idea in this PR is to merge the file mappings used for JShell into the `compiler` label, so that changes touching JShell get the `compiler` label, and the e-mails are sent to `compiler-dev`. > > Please let me now if there's something better than can be done, or what more is needed to stop using the `kulla-dev` list. > > Thanks! @lahodaj Your change (at version e5aae14b55d4c21ad08cb31bda74d69711ae57d0) is now ready to be sponsored by a Committer. ------------- PR Comment: https://git.openjdk.org/skara/pull/1724#issuecomment-3051135662 From zsong at openjdk.org Thu Jul 10 17:23:26 2025 From: zsong at openjdk.org (Zhao Song) Date: Thu, 10 Jul 2025 17:23:26 GMT Subject: RFR: 2535: Merge JShell's file locations mapping into the 'compiler' label, and deleting mapping for 'kulla' In-Reply-To: References: Message-ID: On Fri, 4 Jul 2025 08:40:32 GMT, Jan Lahoda wrote: > There's a proposal to dissolve the Kulla project (the project that introduced JShell into JDK), and use the `compiler-dev` mailing list instead of `kulla-dev`: > https://mail.openjdk.org/pipermail/kulla-dev/2025-June/005928.html > > The idea in this PR is to merge the file mappings used for JShell into the `compiler` label, so that changes touching JShell get the `compiler` label, and the e-mails are sent to `compiler-dev`. > > Please let me now if there's something better than can be done, or what more is needed to stop using the `kulla-dev` list. > > Thanks! Sorry for the delayed response, I somehow forgot to sponsor your PR yesterday. ------------- PR Comment: https://git.openjdk.org/skara/pull/1724#issuecomment-3058305696 From jlahoda at openjdk.org Thu Jul 10 17:23:26 2025 From: jlahoda at openjdk.org (Jan Lahoda) Date: Thu, 10 Jul 2025 17:23:26 GMT Subject: Integrated: 2535: Merge JShell's file locations mapping into the 'compiler' label, and deleting mapping for 'kulla' In-Reply-To: References: Message-ID: On Fri, 4 Jul 2025 08:40:32 GMT, Jan Lahoda wrote: > There's a proposal to dissolve the Kulla project (the project that introduced JShell into JDK), and use the `compiler-dev` mailing list instead of `kulla-dev`: > https://mail.openjdk.org/pipermail/kulla-dev/2025-June/005928.html > > The idea in this PR is to merge the file mappings used for JShell into the `compiler` label, so that changes touching JShell get the `compiler` label, and the e-mails are sent to `compiler-dev`. > > Please let me now if there's something better than can be done, or what more is needed to stop using the `kulla-dev` list. > > Thanks! This pull request has now been integrated. Changeset: 9ee28e84 Author: Jan Lahoda Committer: Zhao Song URL: https://git.openjdk.org/skara/commit/9ee28e846c5ca8beb1d52e7069b130defdb993a6 Stats: 14 lines in 1 file changed: 6 ins; 8 del; 0 mod 2535: Merge JShell's file locations mapping into the 'compiler' label, and deleting mapping for 'kulla' Reviewed-by: zsong ------------- PR: https://git.openjdk.org/skara/pull/1724 From yzheng at openjdk.org Thu Jul 17 14:33:39 2025 From: yzheng at openjdk.org (Yudi Zheng) Date: Thu, 17 Jul 2025 14:33:39 GMT Subject: RFR: 2547: Notify graal mailing list for java man page changes Message-ID: This allows Graal developers to make adaptions to java man page changes in time ------------- Commit messages: - Notify graal mailing list for java man page changes Changes: https://git.openjdk.org/skara/pull/1725/files Webrev: https://webrevs.openjdk.org/?repo=skara&pr=1725&range=00 Issue: https://bugs.openjdk.org/browse/SKARA-2547 Stats: 1 line in 1 file changed: 1 ins; 0 del; 0 mod Patch: https://git.openjdk.org/skara/pull/1725.diff Fetch: git fetch https://git.openjdk.org/skara.git pull/1725/head:pull/1725 PR: https://git.openjdk.org/skara/pull/1725 From iris at openjdk.org Thu Jul 17 14:50:41 2025 From: iris at openjdk.org (Iris Clark) Date: Thu, 17 Jul 2025 14:50:41 GMT Subject: RFR: 2547: Notify graal mailing list for java man page changes In-Reply-To: References: Message-ID: On Thu, 17 Jul 2025 14:27:56 GMT, Yudi Zheng wrote: > This allows Graal developers to make adaptions to java man page changes in time java.md location confirmed ------------- Marked as reviewed by iris (no project role). PR Review: https://git.openjdk.org/skara/pull/1725#pullrequestreview-3029873736 From dnsimon at openjdk.org Thu Jul 17 15:03:28 2025 From: dnsimon at openjdk.org (Doug Simon) Date: Thu, 17 Jul 2025 15:03:28 GMT Subject: RFR: 2547: Notify graal mailing list for java man page changes In-Reply-To: References: Message-ID: <_Yh3w9faZBwoZJ9Xnr8SpvAB7SRaxny_iLwSlE_BUTA=.28b37945-3726-4690-a6c3-66def9f80f86@github.com> On Thu, 17 Jul 2025 14:27:56 GMT, Yudi Zheng wrote: > This allows Graal developers to make adaptions to java man page changes in time LGTM ------------- Marked as reviewed by dnsimon (no project role). PR Review: https://git.openjdk.org/skara/pull/1725#pullrequestreview-3029924777 From zsong at openjdk.org Thu Jul 17 15:38:35 2025 From: zsong at openjdk.org (Zhao Song) Date: Thu, 17 Jul 2025 15:38:35 GMT Subject: RFR: 2547: Notify graal mailing list for java man page changes In-Reply-To: References: Message-ID: On Thu, 17 Jul 2025 14:27:56 GMT, Yudi Zheng wrote: > This allows Graal developers to make adaptions to java man page changes in time Marked as reviewed by zsong (Reviewer). ------------- PR Review: https://git.openjdk.org/skara/pull/1725#pullrequestreview-3030057960 From yzheng at openjdk.org Fri Jul 18 09:07:00 2025 From: yzheng at openjdk.org (Yudi Zheng) Date: Fri, 18 Jul 2025 09:07:00 GMT Subject: RFR: 2547: Notify graal mailing list for java man page changes In-Reply-To: References: Message-ID: <7xADLJOcaZ1xnUDNJ6nB1-WOxh6QMAj0rTOYayJ5SQw=.fd320e1a-edc8-42b7-a24f-1781a871cd4d@github.com> On Thu, 17 Jul 2025 14:27:56 GMT, Yudi Zheng wrote: > This allows Graal developers to make adaptions to java man page changes in time Thanks for the reviews! ------------- PR Comment: https://git.openjdk.org/skara/pull/1725#issuecomment-3088666320 From duke at openjdk.org Fri Jul 18 09:07:00 2025 From: duke at openjdk.org (duke) Date: Fri, 18 Jul 2025 09:07:00 GMT Subject: RFR: 2547: Notify graal mailing list for java man page changes In-Reply-To: References: Message-ID: On Thu, 17 Jul 2025 14:27:56 GMT, Yudi Zheng wrote: > This allows Graal developers to make adaptions to java man page changes in time @mur47x111 Your change (at version 011e409a53904fb308a4bbd5c113049c8ab450f3) is now ready to be sponsored by a Committer. ------------- PR Comment: https://git.openjdk.org/skara/pull/1725#issuecomment-3088667746 From yzheng at openjdk.org Fri Jul 18 15:28:51 2025 From: yzheng at openjdk.org (Yudi Zheng) Date: Fri, 18 Jul 2025 15:28:51 GMT Subject: Integrated: 2547: Notify graal mailing list for java man page changes In-Reply-To: References: Message-ID: On Thu, 17 Jul 2025 14:27:56 GMT, Yudi Zheng wrote: > This allows Graal developers to make adaptions to java man page changes in time This pull request has now been integrated. Changeset: aeee3644 Author: Yudi Zheng Committer: Zhao Song URL: https://git.openjdk.org/skara/commit/aeee36444467af55436a7174e6a0a055188b8c05 Stats: 1 line in 1 file changed: 1 ins; 0 del; 0 mod 2547: Notify graal mailing list for java man page changes Reviewed-by: iris, dnsimon, zsong ------------- PR: https://git.openjdk.org/skara/pull/1725