[Bug 491084] Review Request: medusa - parallel brute forcing pasword cracker

bugzilla at redhat.com bugzilla at redhat.com
Fri Mar 20 12:42:42 UTC 2009


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


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





--- Comment #10 from Miroslav Suchy <msuchy at redhat.com>  2009-03-20 08:42:34 EDT ---
OK. It build now.
Although there is huge list of compilers warnings. (will be nice to fix, but
not necessery).
There is also warning:
checking for openssl/ssl.h... no
configure: WARNING:  *** OpenSSL header files required for SSL support. ***
    Many of the Medusa modules depend on the OpenSSL library and header files.
If
    multiple modules are unexpectedly disabled, this is likely the cause. Make
sure
    to install libssl-dev, openssl-devel or whatever package your distribution
uses
    to distribute these files.
not sure if you would like to enable it by specifying additional requirements.

Rpmlint still say a lot of errors:
Do not use
 rm -rf ${buildroot}
use either 
 rm -Rf $RPM_BUILD_ROOT
or 
 rm -rf %{buildroot}

medusa.src: W: mixed-use-of-spaces-and-tabs (spaces: line 24, tab: line 11)
The specfile mixes use of spaces and tabs for indentation, which is a cosmetic
annoyance.  Use either spaces or tabs for indentation, not both.

Your description lines must not exceed 79 characters. If a line is exceeding
this number, cut it to fit in two or more lines.

medusa.src: W: invalid-license GPL
The value of the License tag was not recognized.  Known values are: "GPL+",
"GPL+ or Artistic", "GPLv1",
"GPLv2 or Artistic", "GPLv2+ or Artistic", "GPLv2", "GPLv2 with exceptions",
"GPLv2+", "GPLv2+ with exceptions", "GPLv3", "GPLv3 with exceptions",
"GPLv3+", "GPLv3+ with exceptions"

When you will fix these errors, I will check rest of the requirements.

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




More information about the Fedora-package-review mailing list