waiverdb

#39 fix the erros in the spec file reported by rpmlint
Merged by mjia. Opened by mjia.
mjia/waiverdb upstream  into  master

Download 39.patch

Also, invoke rpmlint in our Jenkins jobs.

You don't want to commit Release: 2 here. In our source tree it only ever makes sense to have Release: 1. If you have packaging changes for Fedora those will live in Fedora's dist-git.

Let's run rpmlint across the built RPMs too. Right now in the Jenkinsfile you are doing it after we build the SRPM but before mock, so you are only testing the SRPM which is much less useful.

rebased

Good point. rpmlint is running across the built RPMs too.

This is already running across the .src.rpms as well. You don't need the step above.

Btw did you test this on Jenkins? Do we have rpmlint on the slaves?

Yes, I've tested it on Jenkins and the slave has rpmlint.

2 new commits added

  • add rpmlint into the Jenkins file
  • fix the long line error in the spec file reported by rpmlint

(LGTM)

:+1:

Pull-Request has been merged by mjia
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.