[Bug 439911] Review Request: stringtemplate - A Java template engine

bugzilla at redhat.com bugzilla at redhat.com
Sat Apr 5 00:13:20 UTC 2008


Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: stringtemplate - A Java template engine


https://bugzilla.redhat.com/show_bug.cgi?id=439911


tibbs at math.uh.edu changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|fedora-review?              |fedora-review+




------- Additional Comments From tibbs at math.uh.edu  2008-04-04 20:13 EST -------
This seems to have cropped up, or did I somehow miss it earlier:
  stringtemplate.src:42: W: rpm-buildroot-usage %build rm -rf $RPM_BUILD_ROOT
Normally you don't need to erase the build root in %build, because it should be
empty before %install.  Why do you need to do that?

Otherwise, things look good:

The license text is present.
The dependencies look correct:
  stringtemplate-3.1-1.fc9.noarch.rpm
   stringtemplate = 3.1-1.fc9
  =
   java >= 1:1.6.0
   jpackage-utils

  stringtemplate-javadoc-3.1-1.fc9.noarch.rpm
   stringtemplate-javadoc = 3.1-1.fc9
  =
   java-javadoc

%check is now present, and the tests pass:
    [junit] Tests run: 279, Failures: 0, Errors: 0, Time elapsed: 0.28 sec

So, because another round trip probably isn't productive, I'll approve this on
the condition that you remove the "rm -rf $RPM_BUILD_ROOT" bit from %build.  Or,
if you need it for something, let's investigate why.

APPROVE

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.




More information about the package-review mailing list