Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Merge Review: regexp https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226366 overholt@xxxxxxxxxx changed: What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review? |fedora-review+ ------- Additional Comments From overholt@xxxxxxxxxx 2007-02-09 16:39 EST ------- (In reply to comment #6) > > > X source files match upstream > > > . I can't find the tarball. Also, Source0 can be the actual URL ending with the > > > tar.gz. > > Really? > > Sorry, I accidentally copied that from another review :) The md5sums match. > > > X BuildRequires are proper > > > . why is jpackage-utils in Requires(pre,post)? > > According to the guidelines, all directories created by the package must be > > owned by the package > > Yes, I agree with your reasoning but let's just remove the javadoc symlinking in > %post{,un} and then these requirements can go away. Okay, this isn't holding up the review, but I still don't like it :). > > > X final provides and requires are sane > > > Do we need a 'java' dependency somewhere? Does the (erroneous, I think) > > > Requires(pre,post) on jpackage-utils imply a regular Requires on it? Do we > > > need things in coreutils (rpm, ln) in Requires(post,postun)? > > Added the Requires on java I asked about the Requires(x) on coreutils things and the answer was to err on the safe side so those are fine. I don't like the JPackage-style %{__rm} but again, that's not going to hold up the review. APPROVED. Thanks, Vivek! As per https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225928#c7 , please rebuild this package in Brew and when I've confirmed that the updated package has hit Rawhide, I'll close this bug as RAWHIDE. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug, or are watching the QA contact. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review