[Bug 487148] Review Request: gearmand - A distributed job system

2009-05-24 Thread bugzilla
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=487148





--- Comment #10 from Peter Lemenkov   2009-05-24 05:17:43 
EDT ---
Thanks for the feedback. I'll fix these issues ASAP.

-- 
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 487148] Review Request: gearmand - A distributed job system

2009-05-24 Thread bugzilla
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=487148


Terje Røsten  changed:

   What|Removed |Added

 CC||terje...@phys.ntnu.no




--- Comment #9 from Terje Røsten   2009-05-24 04:24:07 
EDT ---
Hi guys, nice to see gearman in Fedora, however could you please take the time
to
test the package a little bit?

The are several bugs in the init script:

[ -z "${PIDFILE}" ] && pidfile = "/var/run/gearmand/gearmand.pid"

Two bugs here: the directory /var/run/gearmand is not created by the package
and syntax error: no space around the = sign.

More bugs:

gearmand=/usr/bin/gearmand

gearmand is install in /usr/sbin not /usr/bin 

Might be a bug:

# config: /etc/sysconfig/gearmand

this file is not shipped in the package.

-- 
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 487148] Review Request: gearmand - A distributed job system

2009-05-13 Thread bugzilla
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=487148


Ruben Kerkhof  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




-- 
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 487148] Review Request: gearmand - A distributed job system

2009-05-04 Thread bugzilla
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=487148


Kevin Fenzi  changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #8 from Kevin Fenzi   2009-05-04 16:38:30 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 487148] Review Request: gearmand - A distributed job system

2009-05-04 Thread bugzilla
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=487148


Ruben Kerkhof  changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #7 from Ruben Kerkhof   2009-05-04 07:38:40 
EDT ---
New Package CVS Request
===
Package Name: gearmand
Short Description: A distributed job system
Owners: ruben
Branches: F-11 EL-5

-- 
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 487148] Review Request: gearmand - A distributed job system

2009-04-29 Thread bugzilla
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=487148


Peter Lemenkov  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #6 from Peter Lemenkov   2009-04-29 11:58:53 
EDT ---
+ md5 correct

[pe...@sulaco SOURCES]$ md5sum gearmand-0.5.tar.gz*
41946441094bc260c46b1391ae6ef3a4  gearmand-0.5.tar.gz
41946441094bc260c46b1391ae6ef3a4  gearmand-0.5.tar.gz.1
[pe...@sulaco SOURCES]$

+ Issue with warning "missing-lsb-keyword" fixed.

+ Issue with unowned directories fixed

+ Koji scratchbuild
http://koji.fedoraproject.org/koji/taskinfo?taskID=1328821

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 487148] Review Request: gearmand - A distributed job system

2009-04-29 Thread bugzilla
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=487148





--- Comment #5 from Ruben Kerkhof   2009-04-29 10:25:05 
EDT ---
Hi Peter, thanks for the review.
I've fixed all the issues mentioned above, and disabled building with tcmalloc
(from google-perftools-devel) for ppc64.

In the meantime upstream released a new version.

New version here:
SPEC: http://ruben.fedorapeople.org/gearmand.spec
SRPM: http://ruben.fedorapeople.org/gearmand-0.5-1.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 487148] Review Request: gearmand - A distributed job system

2009-04-23 Thread bugzilla
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=487148


Peter Lemenkov  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED




-- 
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 487148] Review Request: gearmand - A distributed job system

2009-04-22 Thread bugzilla
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=487148


Peter Lemenkov  changed:

   What|Removed |Added

 AssignedTo|nob...@fedoraproject.org|lemen...@gmail.com
   Flag||fedora-review?




--- Comment #4 from Peter Lemenkov   2009-04-23 01:08:22 
EDT ---
REVIEW:

+/- rpmlint is not silent:

[pe...@host-12-116 Desktop]$ rpmlint *gearman*
gearmand.i586: W: missing-lsb-keyword Default-Stop in /etc/rc.d/init.d/gearmand
gearmand.i586: E: incoherent-subsys /etc/rc.d/init.d/gearmand gearmand}
4 packages and 0 specfiles checked; 1 errors, 1 warnings.
[pe...@host-12-116 Desktop]$

The first warning may be omitted, hoewer I advice you to suppress it by adding
necessary "# Default-Stop:" into init-script. You don't need even to assign
some values to it.

The second message is false positive and may be simply ignored.

+ The package is named according to the Package Naming Guidelines.
+ The spec file name matches the base package %{name}, in the format
%{name}.spec.
+ The package meets the Packaging Guidelines.
+ The package is licensed with a Fedora approved license and meets the
Licensing Guidelines .
+ The License field in the package spec file matches the actual license.
+ The file, containing the text of the license(s) for the package, is included
in %doc.
+ The spec file is written in American English.
+ The spec file for the package is legible.
+ The sources used to build the package matches the upstream source.

[pe...@sulaco SOURCES]$ md5sum gearmand-0.3.tar.gz*
6acf8b8ca7087a1264ba2a96d58fcc5d  gearmand-0.3.tar.gz
6acf8b8ca7087a1264ba2a96d58fcc5d  gearmand-0.3.tar.gz.1
[pe...@sulaco SOURCES]$ 

+ The package successfully compiles and builds into binary rpms on at least one
primary architecture.

http://koji.fedoraproject.org/koji/taskinfo?taskID=1316249

- The package failed to build on ppc64, due to missing gogle-perftools-devel
for this target. So ppc64 architectures should be listed in the spec in
ExcludeArch, and bug should be filed in bugzilla, describing the reason that
the package does not compile/build/work on that architecture. The bug number
MUST be placed in a comment, next to the corresponding ExcludeArch line.

http://koji.fedoraproject.org/koji/getfile?taskID=1316252&name=root.log
http://cvs.fedoraproject.org/viewvc/rpms/google-perftools/devel/google-perftools.spec?view=markup

- All build dependencies are listed in BuildRequires, but it seems, that there
is one missing "Requires: procps" (usage of pgrep in the init-script). 

+ The subpackage which stores shared library files (not just symlinks) in any
of the dynamic linker's default paths, calls ldconfig in %post and %postun.

- A package must own all directories that it creates. Unfortunately, you missed
%{_includedir}/libgearman/ in the %files section of libgearman-devel
subpackage. Please add it as %dir.

+ The package does not list any file more than once in the spec file's %files
listings.
+ Permissions on files are set properly.
+ The package has a %clean section, which contains rm -rf %{buildroot} (or
$RPM_BUILD_ROOT).
+ The package consistently uses macros.
+ The package contains code, or permissable content.
+ No extremely large documentation files.
+ Anything, the package includes as %doc, does not affect the runtime of the
application.
+ Header files are in a -devel package.
+ No static libraries.
+ The sub-package containing pkgconfig(.pc) files has 'Requires: pkgconfig'.
+ The library files that end in .so (without suffix) are in a -devel package. 
+ devel sub-package requires the base package using a fully versioned
dependency: Requires: %{name} = %{version}-%{release}.
+ The package does NOT contain any .la libtool archives.
+ Not a GUI application.
+ The package does not own files or directories already owned by other
packages.
+ At the beginning of %install, the package runs rm -rf %{buildroot} (or
$RPM_BUILD_ROOT).
+ All filenames in rpm packages are valid UTF-8.

Please fix issues, noted above.

-- 
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 487148] Review Request: gearmand - A distributed job system

2009-04-22 Thread bugzilla
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=487148





--- Comment #3 from Ruben Kerkhof   2009-04-22 17:26:05 
EDT ---
Oops, somehow they disappeared. Please try 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 487148] Review Request: gearmand - A distributed job system

2009-04-21 Thread bugzilla
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=487148


Peter Lemenkov  changed:

   What|Removed |Added

 CC||lemen...@gmail.com




--- Comment #2 from Peter Lemenkov   2009-04-22 01:51:56 
EDT ---
404 while retrieving files.

-- 
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 487148] Review Request: gearmand - A distributed job system

2009-02-25 Thread bugzilla
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=487148





--- Comment #1 from Ruben Kerkhof   2009-02-25 04:44:56 
EDT ---
New version with init script:

Spec URL: http://ruben.fedorapeople.org/gearmand.spec
SRPM URL: http://ruben.fedorapeople.org/gearmand-0.3-2.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