Packit: remove pre-sync action

The pre-sync action constantly breaks and is currently not possible to reliably test until the subsequent upstream release due to limitations in packit. The lines being added by the action script to the downstream Fedora spec were only meant to keep Fedora happy. But given that they provide no tangible benefit as github notifies us of security issues in libraries mentioned in go.mod and go.sum, along with redhat prodsec's own magic for creating security alerts, there's absolutely no point to having the pre-sync action run and add a layer of uncertainty. This commit removes the pre-sync action and `rpm/update-spec-provides.sh`. Ref: containers/podman#19232 Signed-off-by: Lokesh Mandvekar <[email protected]>
containers · Jul 14, 2023 · a4aa15f · a4aa15f
1 parent d606b8a
commit a4aa15f
Show file tree

Hide file tree

Showing 3 changed files with 0 additions and 39 deletions.
diff --git a/.packit.yaml b/.packit.yaml
@@ -35,9 +35,6 @@ jobs:
     update_release: false
     dist_git_branches:
       - fedora-all
-    actions:
-        pre-sync:
-          - "bash rpm/update-spec-provides.sh"
 
   - job: koji_build
     trigger: commit

diff --git a/rpm/skopeo.spec b/rpm/skopeo.spec
@@ -69,8 +69,6 @@ BuildRequires: glib2-devel
 BuildRequires: make
 BuildRequires: shadow-utils-subid-devel
 Requires: containers-common >= 4:1-21
-# DO NOT DELETE BELOW LINE - used for updating downstream goimports
-# vendored libraries
 
 %description
 Command line utility to inspect images and repositories directly on Docker

diff --git a/rpm/update-spec-provides.sh b/rpm/update-spec-provides.sh