[Bug 863985] Review Request: weighttp - Small tool to benchmark webservers

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

 



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

--- Comment #11 from Eduardo Echeverria <echevemaster@xxxxxxxxx> ---
Hi Mathieu:

You are absolutely right. The issue is that I wanted applied the guidelines of
python here. Considering that wscript is a python script. So I thought:
Eduardo!! Obviously this is a program written in C, not a python script that is
have to build.
Hope apologize me for this error. u_u

Anyway there is a issue:
Spec file as given by url is not the same as in SRPM
Diff spec file in url and in SRPM
---------------------------------
--- /home/makerpm/weighttp2/863985-weighttp/srpm/weighttp.spec  2012-10-17
22:2$
+++ /home/makerpm/weighttp2/863985-weighttp/srpm-unpacked/weighttp.spec
2012-10$
@@ -19,5 +19,5 @@
 BuildRequires:    libev-devel

-# The build uses a bundled copy of waf, which requires python. If we ever
+# The build use a bundled copy of waf, which requires python. If we ever
 # start using the Fedora waf package instead, this will come in implictly.
 BuildRequires:    python

Please fix this issue, and I proceed to the formal review

Best Regards

-- 
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review



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