[Rpm-maint] [rpm-software-management/rpm] Add tests for HDRID, PKGID and SOURCEPKGID (PR #3560)

[Panu Matilainen]

@pmatilai commented on this pull request.



> +RPMTEST_CHECK([[
+sid=$(rpm -qp --qf "%{pkgid}\n" ${RPMTEST}/build/SRPMS/attrtest-1.0-1.src.rpm)
+pid=$(rpm -qp --qf "%{pkgid}\n" ${RPMTEST}/build/RPMS/4/noarch/attrtest-1.0-1.noarch.rpm)
+spid=$(rpm -qp --qf "%{sourcepkgid}\n" ${RPMTEST}/build/RPMS/4/noarch/attrtest-1.0-1.*.rpm)
+hid=$(rpm -qp --qf "%{hdrid}\n" ${RPMTEST}/build/RPMS/4/noarch/attrtest-1.0-1.noarch.rpm)
+shid=$(rpm -qp --qf "%{hdrid}\n" ${RPMTEST}/build/SRPMS/attrtest-1.0-1.src.rpm)
+
+# only validate the general characteristics here to avoid pinning
+echo -n ${sid} | wc -c
+echo -n ${pid} | wc -c
+echo -n ${spid} | wc -c
+echo -n ${hid} | wc -c
+echo -n ${shid} | wc -c
+[ "${sid}" = "${spid}" ] || exit 1
+[ "${sid}" != "${pid}" ] || exit 1
+[ "${pid}" != "${sid}" ] || exit 1

Oh. I think the idea was to compare to spid but there's been a lossy transmission :sweat_smile:
Good spotting, will fix.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3560#discussion_r1950635479
You are receiving this because you are subscribed to this thread.

Message ID: <rpm-software-management/rpm/pull/3560/review/2608367183 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20250211/03139058/attachment.htm>