[Bug 426883] Review Request: brazil - Extremely small footprint Java HTTP stack

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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: brazil - Extremely small footprint Java HTTP stack


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


overholt@xxxxxxxxxx changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |tcallawa@xxxxxxxxxx
         AssignedTo|overholt@xxxxxxxxxx         |fedora@xxxxxxxxxxxxxx
             Status|ASSIGNED                    |NEEDINFO
               Flag|                            |needinfo?(tcallawa@xxxxxxxxx
                   |                            |m)




------- Additional Comments From overholt@xxxxxxxxxx  2008-04-17 10:51 EST -------
Nice job, Mat!  A very clean package.  Here's my review.  Everything's good to
go pending spot's legal approval of the fetching (see the last question below,
spot).  Assuming that is given the go-ahead, this package is APPROVED.

MUST items that either have comments or need looking into:
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
* verify source and patches (md5sum matches upstream, know what the patches do)
  - the tarball I created using your script didn't have the same md5sum as
    yours, but a recursive diff of the exploded tarball resulted in no
    differences so I'll assume it's a timestamp thing
? specfile is legible
  - two grammar nit-picks (feel free to ignore my pedantry if you wish ;) :
    "URL based" -> "URL-based"
    "java" -> "Java"

Questions:

- does upstream not provide any build mechanism?
- have you considered offering upstream your build.xml?
- your signal-handling patch doesn't affect runtime, right?
- is the script for fetching the source acceptable to Fedora "legal" (CCing
  spot)?  To download myself I had to click through to accept the SPL.

-- 
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.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]