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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

František Dvořák <valtri@xxxxxxxxxx> changed:

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



--- Comment #4 from František Dvořák <valtri@xxxxxxxxxx> ---
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@xxxxxxxxxx> - 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.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review





[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]