Skip to content

Commit 917bc16

Browse files
anhtuank7cclaude
andcommitted
fix(ci): override tsflags=nodocs so man page check passes on fedora:latest
The fedora:latest container ships with `tsflags=nodocs` set in /etc/dnf/dnf.conf to keep the image minimal. That flag tells rpm to strip files marked as documentation — including man pages — at install time. So even though pamsignal's .rpm correctly includes /usr/share/man/man8/pamsignal.8.gz (verified in the build job's "Inspect package" step), the file never lands on disk inside the container, and the next step's `ls /usr/share/man/man8/pamsignal.8*` fails with ENOENT. EL9's almalinux:9 container doesn't set this flag, which is why the el9 target passed the same check. Real Fedora desktops/servers don't ship with tsflags=nodocs; users who set it explicitly are opting out of man pages. So the fix matches what an actual user would see: pass --setopt=tsflags= on the dnf install to clear the container default. This was the fourth bug in the v0.3.0 release-packages workflow run, all traceable to the v0.3.0 packaging refactor. Commits 1286a38 fixed the empty-deb and el9 spec-parse bugs (real packaging bugs); 5bd2366 and this commit fixed two CI assertion bugs that didn't affect the artifact, only the test gate that decides whether to publish to the apt/dnf repos. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent 5bd2366 commit 917bc16

1 file changed

Lines changed: 8 additions & 1 deletion

File tree

.github/workflows/release-packages.yml

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -642,7 +642,14 @@ jobs:
642642
binary_rpm=$(ls out/pamsignal-[0-9]*.x86_64.rpm | grep -v debuginfo \
643643
| grep -v debugsource | head -1)
644644
echo "Installing $binary_rpm"
645-
dnf install -y "$binary_rpm"
645+
# Override tsflags=nodocs which fedora:latest sets in
646+
# /etc/dnf/dnf.conf to keep the container minimal — without this
647+
# override, man pages (and any %doc/%license files) get stripped
648+
# at install time, breaking the man-page presence check in the
649+
# next step. Real Fedora desktops/servers don't ship with this
650+
# default; the container does. EL9 / almalinux:9 doesn't set it,
651+
# which is why this only bites on the fedora target.
652+
dnf install -y --setopt=tsflags= "$binary_rpm"
646653
647654
- name: Verify file layout and metadata
648655
run: |

0 commit comments

Comments
 (0)