[Bug 1749383] Review Request: mesos - Apache Mesos Cluster Manager

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

 



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



--- Comment #13 from Javi Roman <jroman.espinar@xxxxxxxxx> ---
Spec URL:
https://src.fedoraproject.org/fork/jromanes/rpms/mesos/raw/f30/f/mesos.spec 
SRPM URL:
https://github.com/javiroman/mesos-rpm/raw/master/mesos-1.8.1-1.fc30.src.rpm 
Koji build: https://koji.fedoraproject.org/koji/taskinfo?taskID=37824906 

Here the new release candidate for reviewing. Many thanks for the help!

Important changes from the last package:

- Python2 bits removed

The upstream is in the way of reworking the code base towards Python3
(https://issues.apache.org/jira/browse/MESOS-8882). The Python bindings, utils
and new cli based on Python3 will be added further. The Python2 code is
considered deprecated in this current release, so can be removed for sanity.

- Supackages

The base package Mesos only has one sub-package: mesos-devel. This package
contains the development libraries for creating frameworks (Java) and source
code examples, *.so libraries and so on.

Some important issues:

- Shared library versioning

This package is shipped with libmesos library with apparently no versioning
numbers. Rationale: According with Program-Library-HOWTO/shared-libraries.html
when a new version of a library is binary-incompatible with the old one the
soname needs to change. In Apache Mesos libmesos is always incompatible with
other Apache Mesos versions. So the upstream enter the complete package version
into the SONAME:

objdump -p /lib64/libmesos-1.8.1.so | grep SONAME
  SONAME               libmesos-1.8.1.so

- Bundled bits

Provides: bundled(concurrentqueue) = 7b69a8f
Provides: bundled(csi-spec) = 1.1.0

The last one, Container Storage Interface is not a library, is only a Protocol
Buffer specification.

Google Test is not bundled in this package, only is used for building.

Nvidia GPU deployment kit nvml.h header file is not bundled in this package, is
only used for building time. Regarding the license, I guess is right to block
FE-Legal.

- Rpmlint Errors:

mesos.x86_64: E: missing-call-to-chdir-with-chroot /usr/lib64/libmesos-1.8.1.so

I don’t know the meaning of this. 

mesos.x86_64: E: postin-without-ldconfig /usr/lib64/libmesos-1.8.1.so
mesos.x86_64: E: postun-without-ldconfig /usr/lib64/libmesos-1.8.1.so

According with this
https://fedoraproject.org/wiki/Changes/Removing_ldconfig_scriptlets we don’t
need ldconfig, however rpmlint is raising this errors.

-- 
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
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/package-review@xxxxxxxxxxxxxxxxxxxxxxx




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite Conditions]     [KDE Users]

  Powered by Linux