[Bug 448745] Review Request: ext3grep - Recovery tool for ext3 filesystems

2008-06-03 Thread bugzilla
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: ext3grep - Recovery tool for ext3 filesystems


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





--- Additional Comments From [EMAIL PROTECTED]  2008-06-03 03:33 EST ---
ext3grep-0.6.0-1.fc8 has been pushed to the Fedora 8 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, or are watching someone who is.

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


[Bug 448745] Review Request: ext3grep - Recovery tool for ext3 filesystems

2008-06-03 Thread bugzilla
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: ext3grep - Recovery tool for ext3 filesystems


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||CURRENTRELEASE
   Fixed In Version||0.6.0-1.fc9




-- 
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@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 448745] Review Request: ext3grep - Recovery tool for ext3 filesystems

2008-06-03 Thread bugzilla
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: ext3grep - Recovery tool for ext3 filesystems


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





--- Additional Comments From [EMAIL PROTECTED]  2008-06-03 03:30 EST ---
ext3grep-0.6.0-1.fc9 has been pushed to the Fedora 9 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, or are watching someone who is.

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


[Bug 448745] Review Request: ext3grep - Recovery tool for ext3 filesystems

2008-05-31 Thread bugzilla
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: ext3grep - Recovery tool for ext3 filesystems


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





--- Additional Comments From [EMAIL PROTECTED]  2008-05-31 16:12 EST ---
ext3grep-0.6.0-1.fc8 has been submitted as an update for Fedora 8

-- 
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@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 448745] Review Request: ext3grep - Recovery tool for ext3 filesystems

2008-05-31 Thread bugzilla
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: ext3grep - Recovery tool for ext3 filesystems


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





--- Additional Comments From [EMAIL PROTECTED]  2008-05-31 16:14 EST ---
ext3grep-0.6.0-1.fc9 has been submitted as an update for Fedora 9

-- 
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@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 448745] Review Request: ext3grep - Recovery tool for ext3 filesystems

2008-05-30 Thread bugzilla
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: ext3grep - Recovery tool for ext3 filesystems


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-05-30 16:24 EST ---
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, or are watching someone who is.

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


[Bug 448745] Review Request: ext3grep - Recovery tool for ext3 filesystems

2008-05-29 Thread bugzilla
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: ext3grep - Recovery tool for ext3 filesystems


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





--- Additional Comments From [EMAIL PROTECTED]  2008-05-29 02:46 EST ---
(In reply to comment #1)
 Minor (non-blocker):
 * you should package the LICENSE.GPL2 file.

Isn't this (as it is done now) the supposed way?:

%doc NEWS README LICENSE.GPL2

 +1 Approved

Thanks for your 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, or are watching someone who is.

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


[Bug 448745] Review Request: ext3grep - Recovery tool for ext3 filesystems

2008-05-29 Thread bugzilla
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: ext3grep - Recovery tool for ext3 filesystems


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2008-05-29 09:48 EST ---
New Package CVS Request
===
Package Name: ext3grep
Short Description: Recovery tool for ext3 filesystems
Owners: mjakubicek
Branches: F-8 F-9
InitialCC:
Cvsextras Commits: yes


-- 
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@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 448745] Review Request: ext3grep - Recovery tool for ext3 filesystems

2008-05-28 Thread bugzilla
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: ext3grep - Recovery tool for ext3 filesystems


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
 Status|NEW |ASSIGNED
   Flag||fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2008-05-28 19:55 EST ---
MD5Sum:
aabd3b85e5862cddc284a37f300df186  ext3grep-0.6.0.tar.gz

Good:
* Source URL is canonical
* Upstream source tarball verified
* Package name conforms to the Fedora Naming Guidelines
* Group Tag is from the official list
* Valid license tag
* Buildroot has all required elements
* All paths begin with macros
* All necessary BuildRequires listed.
* Make succeeds even when %{_smp_mflags} is defined
* Files have appropriate permissions and owners
* Rpmlint produces no warnings or errors
* Package builds fine in koji.
* Package installs and uninstalls cleanly

Minor (non-blocker):
* you should package the LICENSE.GPL2 file.

+1 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, or are watching someone who is.

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