[Bug 1062757] Review Request: glite-lb-logger-msg - Plugin for sending L&B notifications to messaging infrastructure

bugzilla at redhat.com bugzilla at redhat.com
Fri Sep 26 08:10:54 UTC 2014


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

František Dvořák <valtri at civ.zcu.cz> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |perl-devel at lists.fedoraproj
                   |                            |ect.org



--- Comment #4 from František Dvořák <valtri at civ.zcu.cz> ---
Spec URL:
http://scientific.zcu.cz/fedora/glite-lb-logger-msg-1.2.13-2/glite-lb-logger-msg.spec
SRPM URL:
http://scientific.zcu.cz/fedora/glite-lb-logger-msg-1.2.13-2/glite-lb-logger-msg-1.2.13-2.fc22.src.rpm

%changelog
* Fri Sep 26 2014 František Dvořák <valtri at civ.zcu.cz> - 1.2.13-2
- Added link to upstream man pages patch

(In reply to Matěj Cepl from comment #3)
> 
> Issues:
> =======

> 
> [!]: Patches link to upstream bugs/comments/lists or are otherwise justified.
>     Has been Patch1 filed upstream? Could you please refer URL of the
>     upstream ticket in the SPEC file?

You're right, fixed
(https://github.com/CESNET/glite-lb/commit/b92e20cc3a0c4088c74a7ce4c2b8e20dd7807a16).

> [!]: Development files must be in a -devel package
>     I believe %{_libdir}/activemq_cpp_plugin.so should go to -devel
>     (also, there are no header files, are they?)

This is the plugin and non-versioned link is used at runtime for loading it
dynamically. It is built by libtool, which handles automatically the
versioning.

> [!]: %build honors applicable compiler flags or justifies otherwise.
>     Why do you call ./configure with explicit perl? ./configure has
>     correct shebang line and it is executable so it works without it.
> 

There were a recommendation to not using absolute paths when invoking perl
(https://bugzilla.redhat.com/show_bug.cgi?id=892597#c3):

"Don't use absolute paths in your spec.
Call perl simply 'perl' (preferred) or use the %{__perl} macro."

(CCing also perl-devel list)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=q30tBvjUNp&a=cc_unsubscribe


More information about the perl-devel mailing list