Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] rr: update to 5.7.0.
@ 2024-03-01  5:53 Nopey
  2024-03-01 19:39 ` [PR PATCH] [Updated] " Nopey
                   ` (11 more replies)
  0 siblings, 12 replies; 13+ messages in thread
From: Nopey @ 2024-03-01  5:53 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 879 bytes --]

There is a new pull request by Nopey against master on the void-packages repository

https://github.com/Nopey/void-packages rr-update
https://github.com/void-linux/void-packages/pull/49015

rr: update to 5.7.0.
<!-- Uncomment relevant sections and delete options which are not applicable -->

#### Testing the changes
- I tested the changes in this PR: **YES**

linux-6.patch is no longer needed, as it has been merged upstream.

an update-check script has been added for rr, it checks for their latest GitHub release

#### Local build testing
- I built this PR locally for my native architecture, x86_64-glibc
<!--
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - aarch64-musl (CROSS)
  - armv7l (CROSS)
  - armv6l-musl (CROSS)
-->

A patch file from https://github.com/void-linux/void-packages/pull/49015.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-rr-update-49015.patch --]
[-- Type: text/x-diff, Size: 4077 bytes --]

From 39264e849b6ddb9441cffac5517fa4f7583a2528 Mon Sep 17 00:00:00 2001
From: Magnus Larsen <golgothasTerror101@gmail.com>
Date: Thu, 29 Feb 2024 20:39:12 -0800
Subject: [PATCH] rr: update to 5.7.0.

---
 srcpkgs/rr/patches/linux-6.patch | 63 --------------------------------
 srcpkgs/rr/template              |  4 +-
 srcpkgs/rr/update                |  2 +
 3 files changed, 4 insertions(+), 65 deletions(-)
 delete mode 100644 srcpkgs/rr/patches/linux-6.patch
 create mode 100644 srcpkgs/rr/update

diff --git a/srcpkgs/rr/patches/linux-6.patch b/srcpkgs/rr/patches/linux-6.patch
deleted file mode 100644
index 2fdffee39af454..00000000000000
--- a/srcpkgs/rr/patches/linux-6.patch
+++ /dev/null
@@ -1,63 +0,0 @@
-From 2979c60ef8bbf7c940afd90172ddc5d8863f766e Mon Sep 17 00:00:00 2001
-From: Robert O'Callahan <robert@ocallahan.org>
-Date: Thu, 25 Aug 2022 20:40:59 +1200
-Subject: [PATCH] Avoid creating a struct with elements after
- ethtool_sset_info's variable-length-array.
-
-Resolves #3369
----
- src/record_syscall.cc | 27 +++++++++++++--------------
- 1 file changed, 13 insertions(+), 14 deletions(-)
-
-diff --git a/src/record_syscall.cc b/src/record_syscall.cc
-index 2e4cdec352..48544e1df8 100644
---- a/src/record_syscall.cc
-+++ b/src/record_syscall.cc
-@@ -1450,17 +1450,17 @@ template <typename Arch> void get_ethtool_gstrings_arch(RecordTask* t) {
-   AutoRemoteSyscalls remote(t);
- 
-   // Do a ETHTOOL_GSSET_INFO to get the number of strings
--  struct SingleStringSet {
--    ethtool_sset_info et;
--    uint32_t data;
--  };
--  SingleStringSet sss;
--  sss.et.cmd = ETHTOOL_GSSET_INFO;
--  sss.et.reserved = 0;
--  sss.et.sset_mask = 1 << et_gstrings.string_set;
--  AutoRestoreMem sss_mem(remote, &sss, sizeof(sss));
--
--  ifreq.ifr_ifru.ifru_data = sss_mem.get();
-+  ethtool_sset_info et;
-+  et.cmd = ETHTOOL_GSSET_INFO;
-+  et.reserved = 0;
-+  et.sset_mask = 1 << et_gstrings.string_set;
-+  std::vector<uint8_t> buffer;
-+  buffer.resize(sizeof(et) + sizeof(uint32_t));
-+  memcpy(buffer.data(), &et, sizeof(et));
-+  memset(buffer.data() + sizeof(et), 0, sizeof(uint32_t));
-+  AutoRestoreMem et_mem(remote, buffer.data(), buffer.size());
-+
-+  ifreq.ifr_ifru.ifru_data = et_mem.get();
-   AutoRestoreMem ifr_mem(remote, &ifreq, sizeof(ifreq));
- 
-   long ret = remote.syscall(regs.original_syscallno(), regs.arg1(),
-@@ -1470,8 +1470,7 @@ template <typename Arch> void get_ethtool_gstrings_arch(RecordTask* t) {
-     return;
-   }
- 
--  sss = t->read_mem(sss_mem.get().cast<SingleStringSet>());
--
-+  uint32_t data = t->read_mem((et_mem.get() + sizeof(et)).cast<uint32_t>());
-   // Now do the ETHTOOL_GSTRINGS call
-   ret = remote.syscall(regs.original_syscallno(), regs.arg1(), SIOCETHTOOL,
-       regs.arg3());
-@@ -1479,7 +1478,7 @@ template <typename Arch> void get_ethtool_gstrings_arch(RecordTask* t) {
-   if (ret < 0) {
-     return;
-   }
--  t->record_remote(orig_gstrings, sizeof(ethtool_gstrings) + ETH_GSTRING_LEN*sss.data);
-+  t->record_remote(orig_gstrings, sizeof(ethtool_gstrings) + ETH_GSTRING_LEN*data);
- }
- 
- static void get_ethtool_gstrings(RecordTask* t) {
diff --git a/srcpkgs/rr/template b/srcpkgs/rr/template
index 122a5f436fd75c..cbcc556c1a7396 100644
--- a/srcpkgs/rr/template
+++ b/srcpkgs/rr/template
@@ -1,6 +1,6 @@
 # Template file for 'rr'
 pkgname=rr
-version=5.6.0
+version=5.7.0
 revision=2
 archs="i686 x86_64"
 build_style=cmake
@@ -12,7 +12,7 @@ maintainer="Orphaned <orphan@voidlinux.org>"
 license="MIT, BSD-2-Clause"
 homepage="http://rr-project.org/"
 distfiles="https://github.com/mozilla/rr/archive/${version}.tar.gz"
-checksum=a75262ae622c2c91d3391401d3da4e1dc9379b5afea5c40c9ea93f6905706c87
+checksum=f7affaaee7316effd3f0924c94bc452ef50d372b94470cafe8568efd88451447
 nopie=yes
 
 case "$XBPS_MACHINE" in
diff --git a/srcpkgs/rr/update b/srcpkgs/rr/update
new file mode 100644
index 00000000000000..9f93c8bd9d3461
--- /dev/null
+++ b/srcpkgs/rr/update
@@ -0,0 +1,2 @@
+site=https://github.com/rr-debugger/rr/releases/latest
+pattern='href="/rr-debugger/rr/releases/tag/\K[\d\.]+(?=")'

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PR PATCH] [Updated] rr: update to 5.7.0.
  2024-03-01  5:53 [PR PATCH] rr: update to 5.7.0 Nopey
@ 2024-03-01 19:39 ` Nopey
  2024-03-01 19:40 ` Nopey
                   ` (10 subsequent siblings)
  11 siblings, 0 replies; 13+ messages in thread
From: Nopey @ 2024-03-01 19:39 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 884 bytes --]

There is an updated pull request by Nopey against master on the void-packages repository

https://github.com/Nopey/void-packages rr-update
https://github.com/void-linux/void-packages/pull/49015

rr: update to 5.7.0.
<!-- Uncomment relevant sections and delete options which are not applicable -->

#### Testing the changes
- I tested the changes in this PR: **YES**

linux-6.patch is no longer needed, as it has been merged upstream.

an update-check script has been added for rr, it checks for their latest GitHub release

#### Local build testing
- I built this PR locally for my native architecture, x86_64-glibc
<!--
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - aarch64-musl (CROSS)
  - armv7l (CROSS)
  - armv6l-musl (CROSS)
-->

A patch file from https://github.com/void-linux/void-packages/pull/49015.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-rr-update-49015.patch --]
[-- Type: text/x-diff, Size: 4150 bytes --]

From b462046a6f85f0a3d3e5c173296e5885f4f1a53c Mon Sep 17 00:00:00 2001
From: Magnus Larsen <golgothasTerror101@gmail.com>
Date: Thu, 29 Feb 2024 20:39:12 -0800
Subject: [PATCH] rr: update to 5.7.0.

---
 srcpkgs/rr/patches/linux-6.patch | 63 --------------------------------
 srcpkgs/rr/template              |  6 +--
 srcpkgs/rr/update                |  2 +
 3 files changed, 5 insertions(+), 66 deletions(-)
 delete mode 100644 srcpkgs/rr/patches/linux-6.patch
 create mode 100644 srcpkgs/rr/update

diff --git a/srcpkgs/rr/patches/linux-6.patch b/srcpkgs/rr/patches/linux-6.patch
deleted file mode 100644
index 2fdffee39af454..00000000000000
--- a/srcpkgs/rr/patches/linux-6.patch
+++ /dev/null
@@ -1,63 +0,0 @@
-From 2979c60ef8bbf7c940afd90172ddc5d8863f766e Mon Sep 17 00:00:00 2001
-From: Robert O'Callahan <robert@ocallahan.org>
-Date: Thu, 25 Aug 2022 20:40:59 +1200
-Subject: [PATCH] Avoid creating a struct with elements after
- ethtool_sset_info's variable-length-array.
-
-Resolves #3369
----
- src/record_syscall.cc | 27 +++++++++++++--------------
- 1 file changed, 13 insertions(+), 14 deletions(-)
-
-diff --git a/src/record_syscall.cc b/src/record_syscall.cc
-index 2e4cdec352..48544e1df8 100644
---- a/src/record_syscall.cc
-+++ b/src/record_syscall.cc
-@@ -1450,17 +1450,17 @@ template <typename Arch> void get_ethtool_gstrings_arch(RecordTask* t) {
-   AutoRemoteSyscalls remote(t);
- 
-   // Do a ETHTOOL_GSSET_INFO to get the number of strings
--  struct SingleStringSet {
--    ethtool_sset_info et;
--    uint32_t data;
--  };
--  SingleStringSet sss;
--  sss.et.cmd = ETHTOOL_GSSET_INFO;
--  sss.et.reserved = 0;
--  sss.et.sset_mask = 1 << et_gstrings.string_set;
--  AutoRestoreMem sss_mem(remote, &sss, sizeof(sss));
--
--  ifreq.ifr_ifru.ifru_data = sss_mem.get();
-+  ethtool_sset_info et;
-+  et.cmd = ETHTOOL_GSSET_INFO;
-+  et.reserved = 0;
-+  et.sset_mask = 1 << et_gstrings.string_set;
-+  std::vector<uint8_t> buffer;
-+  buffer.resize(sizeof(et) + sizeof(uint32_t));
-+  memcpy(buffer.data(), &et, sizeof(et));
-+  memset(buffer.data() + sizeof(et), 0, sizeof(uint32_t));
-+  AutoRestoreMem et_mem(remote, buffer.data(), buffer.size());
-+
-+  ifreq.ifr_ifru.ifru_data = et_mem.get();
-   AutoRestoreMem ifr_mem(remote, &ifreq, sizeof(ifreq));
- 
-   long ret = remote.syscall(regs.original_syscallno(), regs.arg1(),
-@@ -1470,8 +1470,7 @@ template <typename Arch> void get_ethtool_gstrings_arch(RecordTask* t) {
-     return;
-   }
- 
--  sss = t->read_mem(sss_mem.get().cast<SingleStringSet>());
--
-+  uint32_t data = t->read_mem((et_mem.get() + sizeof(et)).cast<uint32_t>());
-   // Now do the ETHTOOL_GSTRINGS call
-   ret = remote.syscall(regs.original_syscallno(), regs.arg1(), SIOCETHTOOL,
-       regs.arg3());
-@@ -1479,7 +1478,7 @@ template <typename Arch> void get_ethtool_gstrings_arch(RecordTask* t) {
-   if (ret < 0) {
-     return;
-   }
--  t->record_remote(orig_gstrings, sizeof(ethtool_gstrings) + ETH_GSTRING_LEN*sss.data);
-+  t->record_remote(orig_gstrings, sizeof(ethtool_gstrings) + ETH_GSTRING_LEN*data);
- }
- 
- static void get_ethtool_gstrings(RecordTask* t) {
diff --git a/srcpkgs/rr/template b/srcpkgs/rr/template
index 122a5f436fd75c..cae8aee8106943 100644
--- a/srcpkgs/rr/template
+++ b/srcpkgs/rr/template
@@ -1,7 +1,7 @@
 # Template file for 'rr'
 pkgname=rr
-version=5.6.0
-revision=2
+version=5.7.0
+revision=1
 archs="i686 x86_64"
 build_style=cmake
 configure_args="-DBUILD_TESTS=OFF -DCMAKE_CXX_STANDARD=14"
@@ -12,7 +12,7 @@ maintainer="Orphaned <orphan@voidlinux.org>"
 license="MIT, BSD-2-Clause"
 homepage="http://rr-project.org/"
 distfiles="https://github.com/mozilla/rr/archive/${version}.tar.gz"
-checksum=a75262ae622c2c91d3391401d3da4e1dc9379b5afea5c40c9ea93f6905706c87
+checksum=f7affaaee7316effd3f0924c94bc452ef50d372b94470cafe8568efd88451447
 nopie=yes
 
 case "$XBPS_MACHINE" in
diff --git a/srcpkgs/rr/update b/srcpkgs/rr/update
new file mode 100644
index 00000000000000..9f93c8bd9d3461
--- /dev/null
+++ b/srcpkgs/rr/update
@@ -0,0 +1,2 @@
+site=https://github.com/rr-debugger/rr/releases/latest
+pattern='href="/rr-debugger/rr/releases/tag/\K[\d\.]+(?=")'

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: rr: update to 5.7.0.
  2024-03-01  5:53 [PR PATCH] rr: update to 5.7.0 Nopey
  2024-03-01 19:39 ` [PR PATCH] [Updated] " Nopey
@ 2024-03-01 19:40 ` Nopey
  2024-03-02 16:00 ` [PR REVIEW] " classabbyamp
                   ` (9 subsequent siblings)
  11 siblings, 0 replies; 13+ messages in thread
From: Nopey @ 2024-03-01 19:40 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 201 bytes --]

New comment by Nopey on void-packages repository

https://github.com/void-linux/void-packages/pull/49015#issuecomment-1973805339

Comment:
reset rr template revision to 1 as advised by the CI pipeline

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PR REVIEW] rr: update to 5.7.0.
  2024-03-01  5:53 [PR PATCH] rr: update to 5.7.0 Nopey
  2024-03-01 19:39 ` [PR PATCH] [Updated] " Nopey
  2024-03-01 19:40 ` Nopey
@ 2024-03-02 16:00 ` classabbyamp
  2024-03-02 16:00 ` classabbyamp
                   ` (8 subsequent siblings)
  11 siblings, 0 replies; 13+ messages in thread
From: classabbyamp @ 2024-03-02 16:00 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 244 bytes --]

New review comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/49015#discussion_r1509997478

Comment:
```suggestion
distfiles="https://github.com/rr-debugger/rr/archive/${version}.tar.gz"
```

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PR REVIEW] rr: update to 5.7.0.
  2024-03-01  5:53 [PR PATCH] rr: update to 5.7.0 Nopey
                   ` (2 preceding siblings ...)
  2024-03-02 16:00 ` [PR REVIEW] " classabbyamp
@ 2024-03-02 16:00 ` classabbyamp
  2024-03-02 21:50 ` [PR PATCH] [Updated] " Nopey
                   ` (7 subsequent siblings)
  11 siblings, 0 replies; 13+ messages in thread
From: classabbyamp @ 2024-03-02 16:00 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 176 bytes --]

New review comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/49015#discussion_r1509997505

Comment:
works fine without this

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PR PATCH] [Updated] rr: update to 5.7.0.
  2024-03-01  5:53 [PR PATCH] rr: update to 5.7.0 Nopey
                   ` (3 preceding siblings ...)
  2024-03-02 16:00 ` classabbyamp
@ 2024-03-02 21:50 ` Nopey
  2024-03-02 21:51 ` [PR REVIEW] " Nopey
                   ` (6 subsequent siblings)
  11 siblings, 0 replies; 13+ messages in thread
From: Nopey @ 2024-03-02 21:50 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 884 bytes --]

There is an updated pull request by Nopey against master on the void-packages repository

https://github.com/Nopey/void-packages rr-update
https://github.com/void-linux/void-packages/pull/49015

rr: update to 5.7.0.
<!-- Uncomment relevant sections and delete options which are not applicable -->

#### Testing the changes
- I tested the changes in this PR: **YES**

linux-6.patch is no longer needed, as it has been merged upstream.

an update-check script has been added for rr, it checks for their latest GitHub release

#### Local build testing
- I built this PR locally for my native architecture, x86_64-glibc
<!--
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - aarch64-musl (CROSS)
  - armv7l (CROSS)
  - armv6l-musl (CROSS)
-->

A patch file from https://github.com/void-linux/void-packages/pull/49015.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-rr-update-49015.patch --]
[-- Type: text/x-diff, Size: 4264 bytes --]

From 50a93a5057cd858decf6948c4356047f6060e210 Mon Sep 17 00:00:00 2001
From: Magnus Larsen <golgothasTerror101@gmail.com>
Date: Thu, 29 Feb 2024 20:39:12 -0800
Subject: [PATCH] rr: update to 5.7.0.

---
 srcpkgs/rr/patches/linux-6.patch | 63 --------------------------------
 srcpkgs/rr/template              |  8 ++--
 srcpkgs/rr/update                |  2 +
 3 files changed, 6 insertions(+), 67 deletions(-)
 delete mode 100644 srcpkgs/rr/patches/linux-6.patch
 create mode 100644 srcpkgs/rr/update

diff --git a/srcpkgs/rr/patches/linux-6.patch b/srcpkgs/rr/patches/linux-6.patch
deleted file mode 100644
index 2fdffee39af454..00000000000000
--- a/srcpkgs/rr/patches/linux-6.patch
+++ /dev/null
@@ -1,63 +0,0 @@
-From 2979c60ef8bbf7c940afd90172ddc5d8863f766e Mon Sep 17 00:00:00 2001
-From: Robert O'Callahan <robert@ocallahan.org>
-Date: Thu, 25 Aug 2022 20:40:59 +1200
-Subject: [PATCH] Avoid creating a struct with elements after
- ethtool_sset_info's variable-length-array.
-
-Resolves #3369
----
- src/record_syscall.cc | 27 +++++++++++++--------------
- 1 file changed, 13 insertions(+), 14 deletions(-)
-
-diff --git a/src/record_syscall.cc b/src/record_syscall.cc
-index 2e4cdec352..48544e1df8 100644
---- a/src/record_syscall.cc
-+++ b/src/record_syscall.cc
-@@ -1450,17 +1450,17 @@ template <typename Arch> void get_ethtool_gstrings_arch(RecordTask* t) {
-   AutoRemoteSyscalls remote(t);
- 
-   // Do a ETHTOOL_GSSET_INFO to get the number of strings
--  struct SingleStringSet {
--    ethtool_sset_info et;
--    uint32_t data;
--  };
--  SingleStringSet sss;
--  sss.et.cmd = ETHTOOL_GSSET_INFO;
--  sss.et.reserved = 0;
--  sss.et.sset_mask = 1 << et_gstrings.string_set;
--  AutoRestoreMem sss_mem(remote, &sss, sizeof(sss));
--
--  ifreq.ifr_ifru.ifru_data = sss_mem.get();
-+  ethtool_sset_info et;
-+  et.cmd = ETHTOOL_GSSET_INFO;
-+  et.reserved = 0;
-+  et.sset_mask = 1 << et_gstrings.string_set;
-+  std::vector<uint8_t> buffer;
-+  buffer.resize(sizeof(et) + sizeof(uint32_t));
-+  memcpy(buffer.data(), &et, sizeof(et));
-+  memset(buffer.data() + sizeof(et), 0, sizeof(uint32_t));
-+  AutoRestoreMem et_mem(remote, buffer.data(), buffer.size());
-+
-+  ifreq.ifr_ifru.ifru_data = et_mem.get();
-   AutoRestoreMem ifr_mem(remote, &ifreq, sizeof(ifreq));
- 
-   long ret = remote.syscall(regs.original_syscallno(), regs.arg1(),
-@@ -1470,8 +1470,7 @@ template <typename Arch> void get_ethtool_gstrings_arch(RecordTask* t) {
-     return;
-   }
- 
--  sss = t->read_mem(sss_mem.get().cast<SingleStringSet>());
--
-+  uint32_t data = t->read_mem((et_mem.get() + sizeof(et)).cast<uint32_t>());
-   // Now do the ETHTOOL_GSTRINGS call
-   ret = remote.syscall(regs.original_syscallno(), regs.arg1(), SIOCETHTOOL,
-       regs.arg3());
-@@ -1479,7 +1478,7 @@ template <typename Arch> void get_ethtool_gstrings_arch(RecordTask* t) {
-   if (ret < 0) {
-     return;
-   }
--  t->record_remote(orig_gstrings, sizeof(ethtool_gstrings) + ETH_GSTRING_LEN*sss.data);
-+  t->record_remote(orig_gstrings, sizeof(ethtool_gstrings) + ETH_GSTRING_LEN*data);
- }
- 
- static void get_ethtool_gstrings(RecordTask* t) {
diff --git a/srcpkgs/rr/template b/srcpkgs/rr/template
index 122a5f436fd75c..78e954a46bae29 100644
--- a/srcpkgs/rr/template
+++ b/srcpkgs/rr/template
@@ -1,7 +1,7 @@
 # Template file for 'rr'
 pkgname=rr
-version=5.6.0
-revision=2
+version=5.7.0
+revision=1
 archs="i686 x86_64"
 build_style=cmake
 configure_args="-DBUILD_TESTS=OFF -DCMAKE_CXX_STANDARD=14"
@@ -11,8 +11,8 @@ short_desc="Nondeterministic debugger"
 maintainer="Orphaned <orphan@voidlinux.org>"
 license="MIT, BSD-2-Clause"
 homepage="http://rr-project.org/"
-distfiles="https://github.com/mozilla/rr/archive/${version}.tar.gz"
-checksum=a75262ae622c2c91d3391401d3da4e1dc9379b5afea5c40c9ea93f6905706c87
+distfiles="https://github.com/rr-debugger/rr/archive/${version}.tar.gz"
+checksum=f7affaaee7316effd3f0924c94bc452ef50d372b94470cafe8568efd88451447
 nopie=yes
 
 case "$XBPS_MACHINE" in
diff --git a/srcpkgs/rr/update b/srcpkgs/rr/update
new file mode 100644
index 00000000000000..9f93c8bd9d3461
--- /dev/null
+++ b/srcpkgs/rr/update
@@ -0,0 +1,2 @@
+site=https://github.com/rr-debugger/rr/releases/latest
+pattern='href="/rr-debugger/rr/releases/tag/\K[\d\.]+(?=")'

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PR REVIEW] rr: update to 5.7.0.
  2024-03-01  5:53 [PR PATCH] rr: update to 5.7.0 Nopey
                   ` (4 preceding siblings ...)
  2024-03-02 21:50 ` [PR PATCH] [Updated] " Nopey
@ 2024-03-02 21:51 ` Nopey
  2024-03-02 21:52 ` Nopey
                   ` (5 subsequent siblings)
  11 siblings, 0 replies; 13+ messages in thread
From: Nopey @ 2024-03-02 21:51 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 164 bytes --]

New review comment by Nopey on void-packages repository

https://github.com/void-linux/void-packages/pull/49015#discussion_r1510082102

Comment:
Good catch, fixed.

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PR REVIEW] rr: update to 5.7.0.
  2024-03-01  5:53 [PR PATCH] rr: update to 5.7.0 Nopey
                   ` (5 preceding siblings ...)
  2024-03-02 21:51 ` [PR REVIEW] " Nopey
@ 2024-03-02 21:52 ` Nopey
  2024-03-02 21:53 ` Nopey
                   ` (4 subsequent siblings)
  11 siblings, 0 replies; 13+ messages in thread
From: Nopey @ 2024-03-02 21:52 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 311 bytes --]

New review comment by Nopey on void-packages repository

https://github.com/void-linux/void-packages/pull/49015#discussion_r1510082184

Comment:
The `update` file is so `./xbps-src update-check rr` can detect when a new version of rr comes out.
No update scripts are strictly needed for building void packages

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PR REVIEW] rr: update to 5.7.0.
  2024-03-01  5:53 [PR PATCH] rr: update to 5.7.0 Nopey
                   ` (6 preceding siblings ...)
  2024-03-02 21:52 ` Nopey
@ 2024-03-02 21:53 ` Nopey
  2024-03-02 21:54 ` classabbyamp
                   ` (3 subsequent siblings)
  11 siblings, 0 replies; 13+ messages in thread
From: Nopey @ 2024-03-02 21:53 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 382 bytes --]

New review comment by Nopey on void-packages repository

https://github.com/void-linux/void-packages/pull/49015#discussion_r1510082184

Comment:
Should I not add update-check scripts to packages when updating them?
The `update` file is so `./xbps-src update-check rr` can detect when a new version of rr comes out.
No update scripts are strictly needed for building void packages

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PR REVIEW] rr: update to 5.7.0.
  2024-03-01  5:53 [PR PATCH] rr: update to 5.7.0 Nopey
                   ` (7 preceding siblings ...)
  2024-03-02 21:53 ` Nopey
@ 2024-03-02 21:54 ` classabbyamp
  2024-03-02 23:19 ` [PR PATCH] [Updated] " Nopey
                   ` (2 subsequent siblings)
  11 siblings, 0 replies; 13+ messages in thread
From: classabbyamp @ 2024-03-02 21:54 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 297 bytes --]

New review comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/49015#discussion_r1510082320

Comment:
I know, I'm saying it works without the update file. the file is for overrides in the case that update-check doesn't autodetect things properly 

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PR PATCH] [Updated] rr: update to 5.7.0.
  2024-03-01  5:53 [PR PATCH] rr: update to 5.7.0 Nopey
                   ` (8 preceding siblings ...)
  2024-03-02 21:54 ` classabbyamp
@ 2024-03-02 23:19 ` Nopey
  2024-03-02 23:19 ` [PR REVIEW] " Nopey
  2024-03-03  0:23 ` [PR PATCH] [Merged]: " classabbyamp
  11 siblings, 0 replies; 13+ messages in thread
From: Nopey @ 2024-03-02 23:19 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 884 bytes --]

There is an updated pull request by Nopey against master on the void-packages repository

https://github.com/Nopey/void-packages rr-update
https://github.com/void-linux/void-packages/pull/49015

rr: update to 5.7.0.
<!-- Uncomment relevant sections and delete options which are not applicable -->

#### Testing the changes
- I tested the changes in this PR: **YES**

linux-6.patch is no longer needed, as it has been merged upstream.

an update-check script has been added for rr, it checks for their latest GitHub release

#### Local build testing
- I built this PR locally for my native architecture, x86_64-glibc
<!--
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - aarch64-musl (CROSS)
  - armv7l (CROSS)
  - armv6l-musl (CROSS)
-->

A patch file from https://github.com/void-linux/void-packages/pull/49015.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-rr-update-49015.patch --]
[-- Type: text/x-diff, Size: 3905 bytes --]

From 0ef549b469731b69e71fe066b9637a709c117f88 Mon Sep 17 00:00:00 2001
From: Magnus Larsen <golgothasTerror101@gmail.com>
Date: Thu, 29 Feb 2024 20:39:12 -0800
Subject: [PATCH] rr: update to 5.7.0.

---
 srcpkgs/rr/patches/linux-6.patch | 63 --------------------------------
 srcpkgs/rr/template              |  8 ++--
 2 files changed, 4 insertions(+), 67 deletions(-)
 delete mode 100644 srcpkgs/rr/patches/linux-6.patch

diff --git a/srcpkgs/rr/patches/linux-6.patch b/srcpkgs/rr/patches/linux-6.patch
deleted file mode 100644
index 2fdffee39af454..00000000000000
--- a/srcpkgs/rr/patches/linux-6.patch
+++ /dev/null
@@ -1,63 +0,0 @@
-From 2979c60ef8bbf7c940afd90172ddc5d8863f766e Mon Sep 17 00:00:00 2001
-From: Robert O'Callahan <robert@ocallahan.org>
-Date: Thu, 25 Aug 2022 20:40:59 +1200
-Subject: [PATCH] Avoid creating a struct with elements after
- ethtool_sset_info's variable-length-array.
-
-Resolves #3369
----
- src/record_syscall.cc | 27 +++++++++++++--------------
- 1 file changed, 13 insertions(+), 14 deletions(-)
-
-diff --git a/src/record_syscall.cc b/src/record_syscall.cc
-index 2e4cdec352..48544e1df8 100644
---- a/src/record_syscall.cc
-+++ b/src/record_syscall.cc
-@@ -1450,17 +1450,17 @@ template <typename Arch> void get_ethtool_gstrings_arch(RecordTask* t) {
-   AutoRemoteSyscalls remote(t);
- 
-   // Do a ETHTOOL_GSSET_INFO to get the number of strings
--  struct SingleStringSet {
--    ethtool_sset_info et;
--    uint32_t data;
--  };
--  SingleStringSet sss;
--  sss.et.cmd = ETHTOOL_GSSET_INFO;
--  sss.et.reserved = 0;
--  sss.et.sset_mask = 1 << et_gstrings.string_set;
--  AutoRestoreMem sss_mem(remote, &sss, sizeof(sss));
--
--  ifreq.ifr_ifru.ifru_data = sss_mem.get();
-+  ethtool_sset_info et;
-+  et.cmd = ETHTOOL_GSSET_INFO;
-+  et.reserved = 0;
-+  et.sset_mask = 1 << et_gstrings.string_set;
-+  std::vector<uint8_t> buffer;
-+  buffer.resize(sizeof(et) + sizeof(uint32_t));
-+  memcpy(buffer.data(), &et, sizeof(et));
-+  memset(buffer.data() + sizeof(et), 0, sizeof(uint32_t));
-+  AutoRestoreMem et_mem(remote, buffer.data(), buffer.size());
-+
-+  ifreq.ifr_ifru.ifru_data = et_mem.get();
-   AutoRestoreMem ifr_mem(remote, &ifreq, sizeof(ifreq));
- 
-   long ret = remote.syscall(regs.original_syscallno(), regs.arg1(),
-@@ -1470,8 +1470,7 @@ template <typename Arch> void get_ethtool_gstrings_arch(RecordTask* t) {
-     return;
-   }
- 
--  sss = t->read_mem(sss_mem.get().cast<SingleStringSet>());
--
-+  uint32_t data = t->read_mem((et_mem.get() + sizeof(et)).cast<uint32_t>());
-   // Now do the ETHTOOL_GSTRINGS call
-   ret = remote.syscall(regs.original_syscallno(), regs.arg1(), SIOCETHTOOL,
-       regs.arg3());
-@@ -1479,7 +1478,7 @@ template <typename Arch> void get_ethtool_gstrings_arch(RecordTask* t) {
-   if (ret < 0) {
-     return;
-   }
--  t->record_remote(orig_gstrings, sizeof(ethtool_gstrings) + ETH_GSTRING_LEN*sss.data);
-+  t->record_remote(orig_gstrings, sizeof(ethtool_gstrings) + ETH_GSTRING_LEN*data);
- }
- 
- static void get_ethtool_gstrings(RecordTask* t) {
diff --git a/srcpkgs/rr/template b/srcpkgs/rr/template
index 122a5f436fd75c..78e954a46bae29 100644
--- a/srcpkgs/rr/template
+++ b/srcpkgs/rr/template
@@ -1,7 +1,7 @@
 # Template file for 'rr'
 pkgname=rr
-version=5.6.0
-revision=2
+version=5.7.0
+revision=1
 archs="i686 x86_64"
 build_style=cmake
 configure_args="-DBUILD_TESTS=OFF -DCMAKE_CXX_STANDARD=14"
@@ -11,8 +11,8 @@ short_desc="Nondeterministic debugger"
 maintainer="Orphaned <orphan@voidlinux.org>"
 license="MIT, BSD-2-Clause"
 homepage="http://rr-project.org/"
-distfiles="https://github.com/mozilla/rr/archive/${version}.tar.gz"
-checksum=a75262ae622c2c91d3391401d3da4e1dc9379b5afea5c40c9ea93f6905706c87
+distfiles="https://github.com/rr-debugger/rr/archive/${version}.tar.gz"
+checksum=f7affaaee7316effd3f0924c94bc452ef50d372b94470cafe8568efd88451447
 nopie=yes
 
 case "$XBPS_MACHINE" in

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PR REVIEW] rr: update to 5.7.0.
  2024-03-01  5:53 [PR PATCH] rr: update to 5.7.0 Nopey
                   ` (9 preceding siblings ...)
  2024-03-02 23:19 ` [PR PATCH] [Updated] " Nopey
@ 2024-03-02 23:19 ` Nopey
  2024-03-03  0:23 ` [PR PATCH] [Merged]: " classabbyamp
  11 siblings, 0 replies; 13+ messages in thread
From: Nopey @ 2024-03-02 23:19 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 195 bytes --]

New review comment by Nopey on void-packages repository

https://github.com/void-linux/void-packages/pull/49015#discussion_r1510101207

Comment:
Didn't know it did that, neat!
Removed `update`.

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PR PATCH] [Merged]: rr: update to 5.7.0.
  2024-03-01  5:53 [PR PATCH] rr: update to 5.7.0 Nopey
                   ` (10 preceding siblings ...)
  2024-03-02 23:19 ` [PR REVIEW] " Nopey
@ 2024-03-03  0:23 ` classabbyamp
  11 siblings, 0 replies; 13+ messages in thread
From: classabbyamp @ 2024-03-03  0:23 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 730 bytes --]

There's a merged pull request on the void-packages repository

rr: update to 5.7.0.
https://github.com/void-linux/void-packages/pull/49015

Description:
<!-- Uncomment relevant sections and delete options which are not applicable -->

#### Testing the changes
- I tested the changes in this PR: **YES**

linux-6.patch is no longer needed, as it has been merged upstream.

an update-check script has been added for rr, it checks for their latest GitHub release

#### Local build testing
- I built this PR locally for my native architecture, x86_64-glibc
<!--
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - aarch64-musl (CROSS)
  - armv7l (CROSS)
  - armv6l-musl (CROSS)
-->

^ permalink raw reply	[flat|nested] 13+ messages in thread

end of thread, other threads:[~2024-03-03  0:23 UTC | newest]

Thread overview: 13+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-03-01  5:53 [PR PATCH] rr: update to 5.7.0 Nopey
2024-03-01 19:39 ` [PR PATCH] [Updated] " Nopey
2024-03-01 19:40 ` Nopey
2024-03-02 16:00 ` [PR REVIEW] " classabbyamp
2024-03-02 16:00 ` classabbyamp
2024-03-02 21:50 ` [PR PATCH] [Updated] " Nopey
2024-03-02 21:51 ` [PR REVIEW] " Nopey
2024-03-02 21:52 ` Nopey
2024-03-02 21:53 ` Nopey
2024-03-02 21:54 ` classabbyamp
2024-03-02 23:19 ` [PR PATCH] [Updated] " Nopey
2024-03-02 23:19 ` [PR REVIEW] " Nopey
2024-03-03  0:23 ` [PR PATCH] [Merged]: " classabbyamp

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).