Review Results

Comment

The change only removes the packaged tests directory after copying the upstream source tree into $pkgdir. This is a packaging cleanup step and does not introduce network access, privilege escalation, persistence, or other suspicious behavior. The rm -rf is scoped to the package directory and targets a predictable path under /usr/lib/sabnzbd, so it does not appear to affect the host system outside packaging. No security concerns identified in this hunk.

@@ -76,6 +76,9 @@ package() {
   mkdir -p "${pkgdir}/usr/lib/sabnzbd"
   cp -r "${srcdir}/SABnzbd-${pkgver}/"* "${pkgdir}/usr/lib/sabnzbd"
 
+  # remove tests folder
+  rm -rf "${pkgdir}/usr/lib/sabnzbd/tests"
+
   find "${pkgdir}/usr/lib/sabnzbd" -type d -exec chmod 755 {} \;
   find "${pkgdir}/usr/lib/sabnzbd" -type f -exec chmod 644 {} \;
   chmod 755 "${pkgdir}/usr/lib/sabnzbd/SABnzbd.py"