[Bug 523199] Review Request: ncrack - high-speed network authentication cracking tool
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=523199 --- Comment #32 from Fedora Update System upda...@fedoraproject.org 2009-10-15 18:32:34 EDT --- ncrack-0.01-0.7.ALPHA.fc11 has been pushed to the Fedora 11 stable repository. If problems still persist, please make note of it in this bug report. -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 523199] Review Request: ncrack - high-speed network authentication cracking tool
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=523199 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||0.01-0.7.ALPHA.fc11 Resolution||ERRATA -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 523199] Review Request: ncrack - high-speed network authentication cracking tool
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=523199 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ASSIGNED|ON_QA --- Comment #31 from Fedora Update System upda...@fedoraproject.org 2009-10-06 23:11:10 EDT --- ncrack-0.01-0.7.ALPHA.fc11 has been pushed to the Fedora 11 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update ncrack'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F11/FEDORA-2009-10297 -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 523199] Review Request: ncrack - high-speed network authentication cracking tool
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=523199 --- Comment #30 from Fedora Update System upda...@fedoraproject.org 2009-10-05 10:10:33 EDT --- ncrack-0.01-0.6.ALPHA.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/ncrack-0.01-0.6.ALPHA.fc11 -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 523199] Review Request: ncrack - high-speed network authentication cracking tool
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=523199 Kevin Fenzi ke...@tummy.com changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #29 from Kevin Fenzi ke...@tummy.com 2009-10-03 17:24:01 EDT --- cvs done. -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 523199] Review Request: ncrack - high-speed network authentication cracking tool
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=523199 Steve Milner smil...@redhat.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #28 from Steve Milner smil...@redhat.com 2009-10-01 09:57:20 EDT --- New Package CVS Request === Package Name: ncrack Short Description: a high-speed network authentication cracking tool Owners: smilner Branches: F-11 F-12 EL-4 EL-5 InitialCC: -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 523199] Review Request: ncrack - high-speed network authentication cracking tool
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=523199 Martin Gieseking martin.giesek...@uos.de changed: What|Removed |Added Flag|fedora-review- |fedora-review+ --- Comment #27 from Martin Gieseking martin.giesek...@uos.de 2009-09-30 12:34:50 EDT --- The latest package release is identical to revision 4, so the review in comment #11 is still valid. I just had a look into the package and everything seems to be fine now. Since the default location of file ncrack-services turned out to be OK, I dare to approve the package again. -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 523199] Review Request: ncrack - high-speed network authentication cracking tool
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=523199 --- Comment #22 from Steve Milner smil...@redhat.com 2009-09-29 13:15:22 EDT --- Someone *could* set it up to run periodically. It's usage is quite similar to nmap though ... [st...@arc steve]$ rpm -qc nmap [st...@arc steve]$ rpm -ql nmap | grep service /usr/share/nmap/nmap-service-probes /usr/share/nmap/nmap-services [st...@arc steve]$ -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 523199] Review Request: ncrack - high-speed network authentication cracking tool
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=523199 --- Comment #23 from Toshio Ernie Kuratomi a.bad...@gmail.com 2009-09-29 14:30:56 EDT --- (In reply to comment #22) Someone *could* set it up to run periodically. This is the part that needs to be evaluated. While this:: It's usage is quite similar to nmap though ... [st...@arc steve]$ rpm -qc nmap [st...@arc steve]$ rpm -ql nmap | grep service /usr/share/nmap/nmap-service-probes /usr/share/nmap/nmap-services [st...@arc steve]$ means that nmap potentially needs to move those files. My position would be -- the system administrator should not have to configure things in /root/ in order to setup a service. Hhat's what /etc/ is for. However, if running this as a service is very esoteric (in most cases this is run at the command line by a user when they're evaluating the security of a network. Very rarely does a system admin want to drop it in a cron job or otherwise use it to constantly monitor the security of their network) then it's okay to leave things in /usr/share/ and have the users put the configuration to run the program in their home directories. In no case should the system administrator change files in /usr/ in order to configure the service. Does that make sense? -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 523199] Review Request: ncrack - high-speed network authentication cracking tool
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=523199 --- Comment #24 from Steve Milner smil...@redhat.com 2009-09-29 14:59:14 EDT --- Yes, it does. I think in this case it's OK as, in the few cases where an admin would want to run ncrack (or nmap) via cron they can use the command line switch to specify what config they want to use (or use the environment variable). They could place the file in /etc/ where system configs should be. In the rest of the (by far most common) cases they can use the data file, or override it with the same command line switches or environment variables. -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 523199] Review Request: ncrack - high-speed network authentication cracking tool
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=523199 --- Comment #25 from Toshio Ernie Kuratomi a.bad...@gmail.com 2009-09-29 15:45:23 EDT --- Excellent. I have no objection to that explanation. -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 523199] Review Request: ncrack - high-speed network authentication cracking tool
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=523199 --- Comment #26 from Steve Milner smil...@redhat.com 2009-09-29 16:12:49 EDT --- e7433325e4df7a89f7509a5101af8c7ef459b166 ncrack.spec 76c3345d0e1b921d9cbcd4d9c79c4a3abe37c5ae ncrack-0.01-0.6.ALPHA.fc11.src.rpm Spec URL: http://stevemilner.org/images/ncrack.spec SRPM URL: http://stevemilner.org/images/ncrack-0.01-0.6.ALPHA.fc11.src.rpm -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 523199] Review Request: ncrack - high-speed network authentication cracking tool
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=523199 --- Comment #20 from Steve Milner smil...@redhat.com 2009-09-28 14:19:06 EDT --- After speaking with the developers on the list it sounds like they don't recommend modifying ncrack-services as it is more of a data file ... From Fyodor: I suppose people could modify it, but that is not recommmended. If they want to use a custom services file, it is usually better for them to put it in ~/.ncrack or specify the location with the --datadir command-line option or the NCRACKDIR environmental variable. You can read the full thread at http://seclists.org/nmap-dev/2009/q3/1059.html. -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 523199] Review Request: ncrack - high-speed network authentication cracking tool
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=523199 --- Comment #21 from Toshio Ernie Kuratomi a.bad...@gmail.com 2009-09-28 14:58:21 EDT --- Yep. If people are meant to override those defaults by specifying their own versions in a ~/.ncrack/ file then /usr/share is fine. I don't know anything about the specifics of ncrack, though -- is it meant to be run by a user or setup to be run periodically by the system administrator? If it's run ad hoc by a user then configuration in a ~/.ncrack file is fine. If it's run on a schedule then it's very nice for system administrators to know that the configuration of services is all in /etc. Note that many programs which are used as both services and invoked ad hoc (or even ones that are purely ad hoc, for instance, mutt) do store files like this in /etc for the system admin to configure while leaving the user to edit dot files in their home directories when they want to customize their ad hoc usage. -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 523199] Review Request: ncrack - high-speed network authentication cracking tool
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=523199 --- Comment #18 from Steve Milner smil...@redhat.com 2009-09-23 13:41:40 EDT --- Excellent! I'll make these changes, test, update this BZ and notify upstream of the patch. Thanks Martin! -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 523199] Review Request: ncrack - high-speed network authentication cracking tool
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=523199 --- Comment #19 from Steve Milner smil...@redhat.com 2009-09-23 16:32:09 EDT --- eaca5f1b0ad2d776341cc7af8dfac834e8df7163 ncrack.spec 4575eee604f1c6ff01ee2cd03b5ee6e570b30a8a ncrack-0.01-0.5.ALPHA.fc11.src.rpm Spec URL: http://stevemilner.org/images/ncrack.spec SRPM URL: http://stevemilner.org/images/ncrack-0.01-0.5.ALPHA.fc11.src.rpm -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 523199] Review Request: ncrack - high-speed network authentication cracking tool
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=523199 Martin Gieseking martin.giesek...@uos.de changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #14 from Martin Gieseking martin.giesek...@uos.de 2009-09-22 12:09:52 EDT --- According to the replies I got from fedora-devel it's not necessary to move the file to /etc, so we can finish here. The package is APPROVED. -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 523199] Review Request: ncrack - high-speed network authentication cracking tool
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=523199
Toshio Ernie Kuratomi a.bad...@gmail.com changed:
What|Removed |Added
CC||a.bad...@gmail.com
Flag|fedora-review+ |fedora-review-
--- Comment #15 from Toshio Ernie Kuratomi a.bad...@gmail.com 2009-09-22
14:03:07 EDT ---
Setting UNAPPROVED for now.
Config files go in %{_sysconfdir} (/etc on Fedora). This is part of the FHS
and there is a Fedora Guideline that we follow the FHS.
The question in this package is whether this file is a config file or a data
file. In your (Martin) last email you gave an example of when the local system
admin might want to modify this file to suit the local site/machine. If this
is a valid use then this is a config file. So it needs to be installed
somewhere below /etc and the file marked %config or %config(noreplace) in the
rpm spec file.
For reference, the message with the example
https://www.redhat.com/archives/fedora-devel-list/2009-September/msg00887.html
Sorry people didn't make clear the real factors in deciding whether this needs
to get moved to /etc.
--
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.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 523199] Review Request: ncrack - high-speed network authentication cracking tool
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=523199 --- Comment #16 from Martin Gieseking martin.giesek...@uos.de 2009-09-22 14:22:15 EDT --- OK, thanks for the clarification. It shouldn't be too complicated to adapt the sources. I'll have a look at it. -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 523199] Review Request: ncrack - high-speed network authentication cracking tool
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=523199
--- Comment #17 from Martin Gieseking martin.giesek...@uos.de 2009-09-22
16:43:52 EDT ---
Created an attachment (id=362136)
-- (https://bugzilla.redhat.com/attachment.cgi?id=362136)
extends ncrack to look in /etc for configuration files
I've attached a patch that extends the ncrack sources to also look in /etc for
config files. Steve, you should ask upstream to apply it to the sources or add
another mechanism that allows relocating file ncrack-services.
The spec file must also be modified:
- in the %setup section, apply the patch (with option -p1)
- add the following to %install:
mkdir $RPM_BUILD_ROOT%{_sysconfdir}
mv $RPM_BUILD_ROOT%{_datadir}/%{name}/ncrack-services
$RPM_BUILD_ROOT%{_sysconfdir}
- add %config %{_sysconfdir}/ncrack-services to the %files section
This is just a suggestion. If necessary, feel free to modify it.
Toshio, is it required to put the config file in a subfolder of /etc, or can we
place it directly in /etc?
--
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.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 523199] Review Request: ncrack - high-speed network authentication cracking tool
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=523199 --- Comment #10 from Steve Milner smil...@redhat.com 2009-09-21 12:07:18 EDT --- a674c6c495ad24411bd2f809d300b23c2e3ead84 ncrack-0.01-0.4.ALPHA.fc11.src.rpm 943102e58844847163cd5c574d555fcde316d906 ncrack.spec Spec URL: http://stevemilner.org/images/ncrack.spec SRPM URL: http://stevemilner.org/images/ncrack-0.01-0.4.ALPHA.fc11.src.rpm -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 523199] Review Request: ncrack - high-speed network authentication cracking tool
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=523199
Martin Gieseking martin.giesek...@uos.de changed:
What|Removed |Added
Status|NEW |ASSIGNED
AssignedTo|nob...@fedoraproject.org|martin.giesek...@uos.de
Flag||fedora-review?
--- Comment #11 from Martin Gieseking martin.giesek...@uos.de 2009-09-21
14:46:28 EDT ---
The package looks fine now. However, I'm not sure about file ncrack-services
that is placed in /usr/share/ncrack by default. It looks as it could be a
candidate for /etc because it's a kind of configuration file. I'm going to ask
on fedora-devel.
rpmlint /var/lib/mock/fedora-11-x86_64/result/ncrack-*
3 packages and 0 specfiles checked; 0 errors, 0 warnings.
-
keys used in following checklist:
[+] OK
[.] OK, not applicable
[X] needs work
-
[+] MUST: The package must be named according to the Package Naming Guidelines.
[+] MUST: The spec file name must match the base package %{name}.
[X] MUST: The package must meet the Packaging Guidelines.
- not sure whether /usr/share/ncrack/ncrack-services should go to /etc
because it's a kind of admin configuration file
[+] MUST: The package must be licensed with a Fedora approved license.
- according to source headers GPLv2 with exceptions
[+] MUST: The License field in the package spec file must match the actual
license.
[+] MUST: File containing the text of the license(s) for the package must be
included in %doc.
[+] MUST: The spec file must be written in American English.
[+] MUST: The spec file for the package MUST be legible.
[+] MUST: The sources used to build the package must match the upstream source,
as provided in the spec URL.
$ sha1sum ncrack-0.01ALPHA.tar.gz*
797f3274cfab33091330df6425c251c979a8bfb8 ncrack-0.01ALPHA.tar.gz
797f3274cfab33091330df6425c251c979a8bfb8 ncrack-0.01ALPHA.tar.gz.1
[+] MUST: The package MUST successfully compile and build into binary rpms on
at least one primary architecture.
koji scratch build:
https://koji.fedoraproject.org/koji/taskinfo?taskID=1695351
[.] MUST: If the package does not successfully compile,...
- package builds for all targets
[+] MUST: All build dependencies must be listed in BuildRequires.
[.] MUST: The spec file MUST handle locales properly. This is done by using the
%find_lang macro. Using %{_datadir}/locale/* is strictly forbidden.
- no locales
[.] MUST: Packages containing shared library files must call ldconfig in %post
and %postun.
- no shared libs
[.] MUST: If the package is designed to be relocatable, ...
- not relocatable
[+] MUST: A package must own all directories that it creates.
[+] MUST: A Fedora package must not list a file more than once in the spec
file's %files listings.
[+] MUST: Permissions on files must be set properly.
[+] MUST: Each package must have a %clean section, which contains rm -rf
%{buildroot}.
[+] MUST: Each package must consistently use macros.
[+] MUST: The package must contain code, or permissable content.
[.] MUST: Large documentation files must go in a -doc subpackage.
- no large docs
[+] MUST: If a package includes something as %doc, it must not affect the
runtime of the application.
[.] MUST: Header files must be in a -devel package.
- no header files
[.] MUST: Static libraries must be in a -static package.
- no static libs
[.] MUST: Packages containing pkgconfig(.pc) files must 'Requires: pkgconfig'
- no pkgconfig files
[.] MUST: .so files must go in a -devel package.
- no shared libs
[.] MUST: devel packages must require the base package using a fully versioned
dependency: Requires: %{name} = %{version}-%{release}
- no devel package
[.] MUST: Packages must NOT contain any .la libtool archives, these must be
removed in the spec if they are built.
- no .la files
[.] MUST: Packages containing GUI applications must include a %{name}.desktop
file,...
- no GUI
[+] MUST: Packages must not own files or directories already owned by other
packages.
[+] MUST: At the beginning of %install, each package MUST run rm -rf
%{buildroot} (or $RPM_BUILD_ROOT).
[+] MUST: All filenames in rpm packages must be valid UTF-8.
[+] SHOULD: The reviewer should test that the package builds in mock.
- builds in mock
[+] SHOULD: The package should compile and build into binary rpms on all
supported architectures.
- builds in koji
[+] SHOULD: The reviewer should test that the package functions as described.
- made a short test, seems to work properly
[.] SHOULD: If scriptlets are used, those scriptlets must be sane.
- no scriptlets
[.] SHOULD: Usually,
[Bug 523199] Review Request: ncrack - high-speed network authentication cracking tool
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=523199 --- Comment #12 from Steve Milner smil...@redhat.com 2009-09-21 15:33:33 EDT --- Thanks for the review! It could go in /etc/ ... but the man page has this: --datadir directoryname (Specify custom Ncrack data file location) . Ncrack needs a file called ncrack-services to load a lookup-table of supported services/ports. This file shouldn´t be changed, unless you know what you are doing (e.g extending Ncrack for additional modules). If you think I should move (or symlink) into sysconfdir I'll make the change :-). I'm also fixing the man page which has a wrong path in it (it states /usr/local/share instead of /usr/share/). -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 523199] Review Request: ncrack - high-speed network authentication cracking tool
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=523199 --- Comment #13 from Martin Gieseking martin.giesek...@uos.de 2009-09-21 15:45:47 EDT --- (In reply to comment #12) Thanks for the review! You're welcome. It could go in /etc/ ... but the man page has this: ... Yes, I read the paragraph of the man page. I was just wondering whether there is some kind of Fedora policy that forces us to move the file to /etc. If there's no such demand, I can directly approve the package. Otherwise some patching is required first. :) -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 523199] Review Request: ncrack - high-speed network authentication cracking tool
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=523199 --- Comment #9 from Martin Gieseking martin.giesek...@uos.de 2009-09-17 07:51:37 EDT --- $ rpmlint /var/lib/mock/fedora-11-i386/result/ncrack-* ncrack-debuginfo.i586: E: empty-debuginfo-package 3 packages and 0 specfiles checked; 1 errors, 0 warnings. The debuginfo package is empty because the debugging information is stripped too early. To fix this issue, add sed -i '/\$(STRIP)/d' Makefile.in to the %prep section. -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 523199] Review Request: ncrack - high-speed network authentication cracking tool
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=523199 --- Comment #6 from Steve Milner smil...@redhat.com 2009-09-16 10:55:35 EDT --- Making the changes and rebuilding now ... -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 523199] Review Request: ncrack - high-speed network authentication cracking tool
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=523199 --- Comment #7 from Steve Milner smil...@redhat.com 2009-09-16 11:10:05 EDT --- $ sha1sum ncrack-0.01-0.3.ALPHA.fc11.src.rpm ncrack.spec 5c70bc79bb891904b0f4ebb27164afd92651e573 ncrack-0.01-0.3.ALPHA.fc11.src.rpm 927ab719e67daddf67ddffb0ba8e28fb73c09a6f ncrack.spec Spec URL: http://stevemilner.org/images/ncrack.spec SRPM URL: http://stevemilner.org/images/ncrack-0.01-0.3.ALPHA.fc11.src.rpm -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 523199] Review Request: ncrack - high-speed network authentication cracking tool
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=523199
Till Maas opensou...@till.name changed:
What|Removed |Added
CC||opensou...@till.name
--- Comment #8 from Till Maas opensou...@till.name 2009-09-16 17:08:44 EDT
---
This
%{_mandir}/man1/ncrack.1.gz
should be
%{_mandir}/man1/ncrack.1*
because the compression of the manpage depends on the rpm configuration.
%description contains an extra space as the last character of the last line,
this is not needed.
Feel free to update this just when some other issues are discovered.
--
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.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 523199] Review Request: ncrack - high-speed network authentication cracking tool
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=523199 --- Comment #2 from Steve Milner smil...@redhat.com 2009-09-15 14:39:09 EDT --- Updated: $ sha1sum ncrack-0.01-1.ALPHA.fc11.src.rpm ncrack.spec 1cb7b4ea93d56bf23d6f5e00e3bb0984ab266e3b ncrack-0.01-1.ALPHA.fc11.src.rpm 9689d5cab38d70fb18b9811e44a96e6fbeb37965 ncrack.spec Spec URL: http://stevemilner.org/images/ncrack.spec SRPM URL: http://stevemilner.org/images/ncrack-0.01-1.ALPHA.fc11.src.rpm -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 523199] Review Request: ncrack - high-speed network authentication cracking tool
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=523199 --- Comment #3 from Martin Gieseking martin.giesek...@uos.de 2009-09-15 15:38:18 EDT --- Could you provide links to the updated files (release 2)? The URLs given above point to the initial ones. -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 523199] Review Request: ncrack - high-speed network authentication cracking tool
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=523199 --- Comment #4 from Steve Milner smil...@redhat.com 2009-09-15 16:04:03 EDT --- $ sha1sum ncrack.spec ncrack-0.01-2.ALPHA.fc11.src.rpm 6c7acea31b496bfa4d997f4439f41fb669605565 ncrack.spec e308b3e578998808879e89867c167ea8f85b6fbb ncrack-0.01-2.ALPHA.fc11.src.rpm Spec URL: http://stevemilner.org/images/ncrack.spec SRPM URL: http://stevemilner.org/images/ncrack-0.01-2.ALPHA.fc11.src.rpm -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 523199] Review Request: ncrack - high-speed network authentication cracking tool
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=523199
--- Comment #5 from Martin Gieseking martin.giesek...@uos.de 2009-09-15
17:49:14 EDT ---
* since this is a pre-release, the revision tag must be 0.X.ALPHA%{?dist} where
X is the revision number to increment (see
http://fedoraproject.org/wiki/Packaging/NamingGuidelines#NonNumericRelease)
* as far as I can see, Requires: openssl is not necessary
* remove COPYING.OpenSSL from %doc because it's not part of the package
(openssl is bundled with the tarball for Windows builds)
* the changelog entries should contain information about the things that have
been changed, so that the file history can be reproduced, e.g.
- updated license tag to GPLv2 with restrictions
- preserve timestamp of file COPYING.OpenSSH
instead of
- Updated for package review
:)
--
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.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 523199] Review Request: ncrack - high-speed network authentication cracking tool
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=523199 Steve Milner smil...@redhat.com changed: What|Removed |Added Summary|Review Request: main |Review Request: ncrack - |package name here - short |high-speed network |summary here |authentication cracking ||tool -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 523199] Review Request: ncrack - high-speed network authentication cracking tool
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=523199 Martin Gieseking martin.giesek...@uos.de changed: What|Removed |Added CC||martin.giesek...@uos.de --- Comment #1 from Martin Gieseking martin.giesek...@uos.de 2009-09-14 14:43:43 EDT --- - according to the source file headers the license is GPLv2 only but with exceptions (the tag must probably be GPLv2 with exceptions) - You should preserve the timestamp of COPYING.OpenSSH. Also, replace ASCII by ISO-8859-1: iconv -f ISO-8859-1 -t UTF8 COPYING.OpenSSH -o COPYING.OpenSSH.new touch -r COPYING.OpenSSH COPYING.OpenSSH.new mv COPYING.OpenSSH.new COPYING.OpenSSH -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
