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