[Bug 1895567] Review Request: usd - 3D VFX pipeline interchange file format

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

 



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

Ben Beasley <code@xxxxxxxxxxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
              Flags|                            |needinfo?(luya_tfz@thefinal
                   |                            |zone.net)



--- Comment #19 from Ben Beasley <code@xxxxxxxxxxxxxxxxxx> ---
This is getting much, much closer. There are still a few remaining things that
need to be looked at.

Package Review
==============

Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated


Issues:
=======
- Header files in -devel subpackage, if present.
  Note: usd : /usr/share/usd/examples/include/pxr/imaging/hdTiny/mesh.h usd
  : /usr/share/usd/examples/include/pxr/imaging/hdTiny/renderDelegate.h usd
  : /usr/share/usd/examples/include/pxr/imaging/hdTiny/renderPass.h usd :
  /usr/share/usd/examples/include/pxr/imaging/hdTiny/rendererPlugin.h
  See: https://docs.fedoraproject.org/en-US/packaging-
  guidelines/#_devel_packages

  This is spurious; these are documentation. If you ever manage to build the
  documentation, you will need a -doc subpackage, and they should go in that.
  For now, they are fine where they are.

- Development (unversioned) .so files in -devel subpackage, if present.
  Note: Unversioned so-files directly in %_libdir.
  See: https://docs.fedoraproject.org/en-US/packaging-
  guidelines/#_devel_packages

  You have patched the CMake files to set an so-version, and offered the change
  upstream—great! But your change only works when PXR_BUILD_MONOLITHIC is set,
so
  you need to set it.

  Then you can do something like:

  and

    %files libs
    […]
    %{_libdir}/lib%{name}_ms.so.%{libmajor}
    […]

  and

    %files devel
    […]
    %{_libdir}/lib%{name}_ms.so
    […]

  Interestingly, this also fixes the cmake install path, so you can remove
  these lines:

    # Relocate cmake folder
    mkdir -p %{buildroot}%{_libdir}/cmake
    mv %{buildroot}%{_prefix}/cmake/* %{buildroot}%{_libdir}/cmake
    rm -rf %{buildroot}%{_prefix}/cmake

  Note that the monolithic build means there is only one shared library file,
  for better or for worse.

- The License field appears correct, but you must break down which files are
  covered by which licenses in a spec file comment or packaged auxiliary
  license file, as mandated in
 
https://docs.fedoraproject.org/en-US/packaging-guidelines/LicensingGuidelines/#_multiple_licensing_scenarios.

  You can use “licensecheck -r” to help you look for files with various
licenses.

- The -devel package should Require: cmake-filesystem for %{_libdir}/cmake.

- The package fails to own the following directories:

    /usr/lib64/usd/hdx, /usr/lib64/usd/ndr, /usr/lib64/usd/plugin,
    /usr/lib64/usd/usd/resources, /usr/lib64/usd/usd,
    /usr/lib64/usd/usd/resources/usd, /usr/lib64/usd/hd, /usr/lib64/usd/hdSt,
    /usr/lib64/usd/hgiGL, /usr/lib64/usd/hio, /usr/lib64/usd/ar,
    /usr/lib64/usd/glf

  The easiest way to fix this would be to change

    %dir %{_libdir}/%{name}
    %{_libdir}/%{name}/sdf
    %{_libdir}/%{name}/plugInfo.json
    %{_libdir}/%{name}/{ar,glf,hd,hdSt,hdx,hgiGL,hio,ndr}/*
    %{_libdir}/%{name}/plugin/*
    %{_libdir}/%{name}/%{name}/resources/generatedSchema.usda
    %{_libdir}/%{name}/%{name}/resources/plugInfo.json
    %{_libdir}/%{name}/%{name}/resources/usd/schema.usda
    %{_libdir}/%{name}/%{name}{Geom,Hydra,Imaging,ImagingGL,Lux,Media,Render}/
    %{_libdir}/%{name}/%{name}{Ri,RiImaging,Shade,Skel,SkelImaging,UI}/
    %{_libdir}/%{name}/%{name}{Vol,VolImaging}/

  to

    %{_libdir}/%{name}
    %exclude %{_libdir}/%{name}/%{name}/resources/codegenTemplates

- Version 21.05 is available. Please update.

- The error you reported building the Python components was because uic-qt5
  defaults to C++ output. You can fix it by adding, at the beginning of %build:

    cat > uic-wrapper <<'EOF'
    #!/bin/sh
    exec uic-qt5 -g python "$@"
    EOF
    chmod +x uic-wrapper

  and then, in the arguments to %cmake, changing PYSIDEUICBINARY to:

    -DPYSIDEUICBINARY:PATH=${PWD}/uic-wrapper

  You can link https://github.com/PixarAnimationStudios/USD/issues/1419, which
  is the most relevant issue. I added this workaround in a comment there, while
  requesting direct support for uic-qt5 as the proper solution.

  Now you have the problem that the package is installed under
  %{python3_sitelib} even though it has compiled extensions. The ugly but
  simple solution is to add this after %cmake_install:

    mkdir -p %{buildroot}%{python3_sitearch}
    mv %{buildroot}%{python3_sitelib}/* %{buildroot}%{python3_sitearch}

  Worse, some files are installed under a flagrantly wrong path:

    mv %{buildroot}%{prefix}/lib/python/pxr/*.* \
        %{buildroot}%{python3_sitearch}/pxr/
    mv %{buildroot}%{prefix}/lib/python/pxr/Usdviewq/* \
        %{buildroot}%{python3_sitearch}/pxr/Usdviewq/

  Instead of all those “mv” commands in %install, you might prefer instead to
  patch the build system with sed in %prep; I have not taken the time to figure
  out the right way to do this.

  See the ancient issue https://github.com/PixarAnimationStudios/USD/issues/30.

- It turns out upstream also bundles a couple of fonts. You can unbundle these
  by, in %prep:

    rm -rvf pxr/usdImaging/usdviewq/fonts/*
    ln -s %{_datadir}/fonts/google-roboto pxr/usdImaging/usdviewq/fonts/Roboto
    ln -s %{_datadir}/fonts/google-roboto-mono \
        pxr/usdImaging/usdviewq/fonts/Roboto_Mono

  and add:

    Requires:       font(roboto)
    Requires:       font(robotoblack)
    Requires:       font(robotolight)
    Requires:       font(robotomono)

  to the Python subpackage. See
 
https://docs.fedoraproject.org/en-US/packaging-guidelines/#_avoid_bundling_of_fonts_in_other_packages.

- Once I got the Python components building, I tried:

    python3 -m pxr.Usdviewq

  and got (on Fedora 33):

      File "/usr/lib64/python3.9/runpy.py", line 188, in _run_module_as_main
        mod_name, mod_spec, code = _get_module_details(mod_name, _Error)
      File "/usr/lib64/python3.9/runpy.py", line 147, in _get_module_details
        return _get_module_details(pkg_main_name, error)
      File "/usr/lib64/python3.9/runpy.py", line 111, in _get_module_details
        __import__(pkg_name)
      File "/usr/lib64/python3.9/site-packages/pxr/Usdviewq/__init__.py", line
30, in <module>
        from .common import Timer
      File
"/usr/lib64/python3.9/site-packages/shiboken2/files.dir/shibokensupport/__feature__.py",
line 142, in _import
        return original_import(name, *args, **kwargs)
      File "/usr/lib64/python3.9/site-packages/pxr/Usdviewq/common.py", line
28, in <module>
        from pxr import Ar, Tf, Sdf, Kind, Usd, UsdGeom, UsdShade
      File
"/usr/lib64/python3.9/site-packages/shiboken2/files.dir/shibokensupport/__feature__.py",
line 142, in _import
        return original_import(name, *args, **kwargs)
      File "/usr/lib64/python3.9/site-packages/pxr/Ar/__init__.py", line 24, in
<module>
        from . import _ar
      File
"/usr/lib64/python3.9/site-packages/shiboken2/files.dir/shibokensupport/__feature__.py",
line 142, in _import
        return original_import(name, *args, **kwargs)
    ImportError: /lib64/libjemalloc.so.2: cannot allocate memory in static TLS
block

  so I tried putting

    BuildRequires:  pkgconfig(jemalloc)

  and

    -DPXR_MALLOC_LIBRARY="%{_libdir}/libjemalloc.so" \

  behind a disabled build conditional, and got this instead:

    /usr/bin/python3: No module named pxr.Usdviewq.__main__; 'pxr.Usdviewq' is
a package and cannot be directly executed

  OK, I guess this isn’t how to start usdviewq, but the import is working, so
  it looks like not using jemalloc is the way to go. Maybe there is some other
  way to fix this, but this way works—maybe at some performance cost, or maybe
  not.

- It turns out that with Python enabled, you get quite a few more binaries in
  the main “usd” package:

    /usr/bin/sdfdump:        ELF 64-bit LSB pie executable, x86-64, version 1
(SYSV), dynamically linked, interpreter /lib64/ld-linux-x86-64.so.2,
BuildID[sha1]=d5c4551cc2fc937008004724d7dabeab13f92be2, for GNU/Linux 3.2.0,
stripped
    /usr/bin/sdffilter:      ELF 64-bit LSB pie executable, x86-64, version 1
(SYSV), dynamically linked, interpreter /lib64/ld-linux-x86-64.so.2,
BuildID[sha1]=f9c69423f6eea1aa506e58c95f99db631cfe78ec, for GNU/Linux 3.2.0,
stripped
    /usr/bin/testusdview:    Python script, ASCII text executable
    /usr/bin/usdcat:         Python script, ASCII text executable
    /usr/bin/usdchecker:     Python script, ASCII text executable
    /usr/bin/usddiff:        Python script, ASCII text executable
    /usr/bin/usddumpcrate:   Python script, ASCII text executable
    /usr/bin/usdedit:        Python script, ASCII text executable
    /usr/bin/usdGenSchema:   Python script, ASCII text executable
    /usr/bin/usdrecord:      Python script, ASCII text executable
    /usr/bin/usdresolve:     Python script, ASCII text executable
    /usr/bin/usdstitch:      Python script, ASCII text executable
    /usr/bin/usdstitchclips: Python script, ASCII text executable
    /usr/bin/usdtree:        Python script, ASCII text executable
    /usr/bin/usdview:        Python script, ASCII text executable
    /usr/bin/usdzip:         Python script, ASCII text executable

  Hey, look, there’s the usdview launcher. Anyway, it looks like you now need
  to either put most of these in the python3-usd subpackage, or (probably
  better) you need to add

    Requires:       python3-%{name}%{?_isa} = %{version}-%{release}

  to the main “usd” package.

- Since the importable Python package is actually called “pxr”, you should add

    %py_provides python3-pxr

  per
 
https://docs.fedoraproject.org/en-US/packaging-guidelines/Python/#_the_py_provides_macro
  to get the right virtual provides. Or, you could rename the Python subpackage
  to match the Python import, and the %py_provides would be unnecessary.

- I tried usdview on the file top.geom.usd and had the window appear for a
  couple of seconds, with a glitchy, transparent hole where the OpenGL render
  should be. Then it crashed:

    QSocketNotifier: Can only be used with threads started with QThread
    State file not found, a new one will be created.
    Traceback (most recent call last):
      File "/usr/lib64/python3.9/site-packages/pxr/Usdviewq/stageView.py", line
1659, in paintGL
        renderer = self._getRenderer()
      File "/usr/lib64/python3.9/site-packages/pxr/Usdviewq/stageView.py", line
970, in _getRenderer
        self._renderer = UsdImagingGL.Engine()
    pxr.Tf.ErrorException: 
        Error in 'pxrInternal_v0_21__pxrReserved__::GlfContextCaps::_LoadCaps'
at line 162 in file
/builddir/build/BUILD/USD-21.02/pxr/imaging/glf/contextCaps.cpp : 'Failed
verification: ' GlfGLContext::GetCurrentGLContext()->IsValid() ''
        Error in
'pxrInternal_v0_21__pxrReserved__::{anonymous}::_IsHydraEnabled' at line 109 in
file /builddir/build/BUILD/USD-21.02/pxr/usdImaging/usdImagingGL/engine.cpp :
'OpenGL context required, using reference renderer'
    /usr/include/c++/10/bits/stl_vector.h:1063: std::vector<_Tp,
_Alloc>::const_reference std::vector<_Tp, _Alloc>::operator[](std::vector<_Tp,
_Alloc>::size_type) const [with _Tp = float; _Alloc = std::allocator<float>;
std::vector<_Tp, _Alloc>::const_reference = const float&; std::vector<_Tp,
_Alloc>::size_type = long unsigned int]: Assertion '__builtin_expect(__n <
this->size(), true)' failed.
    Aborted (core dumped)

  Is this what happens for you, once you fix the Python build?

  It looks similar to https://github.com/PixarAnimationStudios/USD/issues/1467
  and https://github.com/PixarAnimationStudios/USD/issues/764, but not
  identical. It may be something you can only report upstream. It would be nice
  not to have such a glaring bug right out of the gate, but I wouldn’t consider
  it a blocker for the package given how much other functionality it offers.

- Since usdview is a desktop application, you will need to write a downstream
  desktop file (which you can offer upstream):
  https://docs.fedoraproject.org/en-US/packaging-guidelines/#_desktop_files.

  AppStream XML files,
  https://docs.fedoraproject.org/en-US/packaging-guidelines/AppData/, are good
  too, and are a SHOULD, but I have found they are very hard to properly create
  downstream since they need to have screenshots and a description under an
  ultra-permissive license such as CC0. The .desktop file, though, is
  necessary.

- I think

    %global test OFF

  should be a build conditional:

    %bcond_with test

  and then you can replace

    -DPXR_BUILD_TESTS=%{test} \

  with

    -DPXR_BUILD_TESTS=%{?with_test:ON}%{!?with_test:OFF} \

- I would still like to see the tests runnable, but I am going to go ahead and
  provide this review while I look into what that would require. I intend to
  offer suggestions here if I can figure out why they did not work for you.

Notes (a change is not required)
================================

- This is not needed if you are only targeting F33 and later:

    %undefine       __cmake_in_source_build

  (Since F32 is almost EOL, why bother with it?)

- You could drop the license file from the main package “usd” since it depends
  on “usd-libs”, which does have the license file.

- You have correctly justified the ExclusiveArch in a comment. After the
  package is imported to dist-git, you must file Bugzilla tickets blocking the
  tracker bugs for all unsupported architectures, as required by
 
https://docs.fedoraproject.org/en-US/packaging-guidelines/#_architecture_build_failures.

- Instead of

    # Fix all Python shebangs recursively in .
    pathfix.py -pni "%{__python3} %{py3_shbang_opts}" .

  it would be simpler to write

    %py3_shebang_fix .

- I think the pyproject-rpm-macros will not help with these Python bindings
  after all, since the build is integrated into the overall CMake build.

- See https://github.com/PixarAnimationStudios/USD/issues/1259 for feedback
  from an openSUSE packager dealing with some of the same issues.

===== MUST items =====

C/C++:
[x]: Package does not contain kernel modules.
[x]: Package contains no static executables.
[x]: If your application is a C or C++ application you must list a
     BuildRequires against gcc, gcc-c++ or clang.
[x]: Package does not contain any libtool archives (.la)
[x]: Rpath absent or only used for internal libs.

Generic:
[x]: Package is licensed with an open-source compatible license and meets
     other legal requirements as defined in the legal section of Packaging
     Guidelines.
[x]: License field in the package spec file matches the actual license.
     Note: Checking patched sources after %prep for licenses. Licenses
     found: "Unknown or generated", "Khronos License Expat License Apache
     License 2.0", "Apache License 2.0", "Expat License", "BSD 3-clause
     "New" or "Revised" License", "Apache License 2.0 [generated file]",
     "Khronos License", "Apache License 2.0 GNU General Public License v3.0
     or later", "BSD 2-clause "Simplified" License", "the Unlicense Expat
     License", "*No copyright* Apache License 2.0". 2550 files have unknown
     license. Detailed output of licensecheck in
     /home/reviewer/1895567-usd/licensecheck.txt
[x]: License file installed when any subpackage combination is installed.
[!]: If the package is under multiple licenses, the licensing breakdown
     must be documented in the spec.
[!]: Package requires other packages for directories it uses.
     Note: No known owner of /usr/lib64/usd/plugin, /usr/lib64/usd/hd,
     /usr/lib64/usd/hgiGL, /usr/lib64/usd/hdx, /usr/lib64/usd/ndr,
     /usr/lib64/usd/usd/resources/usd, /usr/lib64/usd/usd/resources,
     /usr/lib64/usd/usd, /usr/lib64/usd/hdSt, /usr/lib64/usd/hio,
     /usr/lib64/usd/ar, /usr/lib64/usd/glf

     The -devel package should Require: cmake-filesystem for %{_libdir}/cmake.

[!]: Package must own all directories that it creates.
     Note: Directories without known owners: /usr/lib64/usd/hdx,
     /usr/lib64/usd/ndr, /usr/lib64/usd/plugin,
     /usr/lib64/usd/usd/resources, /usr/lib64/usd/usd,
     /usr/lib64/usd/usd/resources/usd, /usr/lib64/usd/hd,
     /usr/lib64/usd/hdSt, /usr/lib64/usd/hgiGL, /usr/lib64/usd/hio,
     /usr/lib64/usd/ar, /usr/lib64/usd/glf
[x]: %build honors applicable compiler flags or justifies otherwise.
[x]: Package contains no bundled libraries without FPC exception.

     This point in the fedora-review tool output is obsolete; there are now
     very specific rules for bundling libraries
     (https://docs.fedoraproject.org/en-US/packaging-guidelines/#bundling), no
     FPC exception required. As far as I can tell, you have followed them for
     all bundled libraries.

[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[-]: Package contains desktop file if it is a GUI application.

     (This will apply if you get the Python part working.)

[!]: Development files must be in a -devel package

     This will be correct after fixing the shared library versioning as
     described in the Issues section, above.

[x]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory
     names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
     Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: Useful -debuginfo package or justification otherwise.
[-]: Package is not known to require an ExcludeArch tag.

     Package is correctly ExclusiveArch. Must file Bugzilla tickets after the
     package is imported to dist-git, as required by
    
https://docs.fedoraproject.org/en-US/packaging-guidelines/#_architecture_build_failures.

[-]: Large documentation must go in a -doc subpackage. Large could be size
     (~1MB) or number of files.
     Note: Documentation size is 245760 bytes in 7 files.

     A -doc subpackage will be required if you manage to build the
documentation.

[x]: Package complies to the Packaging Guidelines

     (except as otherwise noted)

[x]: Package successfully compiles and builds into binary rpms on at least
     one supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
     Note: There are rpmlint messages (see attachment).
[x]: If (and only if) the source package includes the text of the
     license(s) in its own file, then that file, containing the text of the
     license(s) for the package is included in %license.
[x]: Package does not own files or directories owned by other packages.
[x]: Package uses either %{buildroot} or $RPM_BUILD_ROOT
[x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
     beginning of %install.
[x]: Macros in Summary, %description expandable at SRPM build time.
[x]: Dist tag is present.
[x]: Package does not contain duplicates in %files.
[x]: Permissions on files are set properly.
[x]: Package must not depend on deprecated() packages.
[x]: Package use %makeinstall only when make install DESTDIR=... doesn't
     work.
[x]: Package is named using only allowed ASCII characters.
[x]: Package does not use a name that already exists.
[x]: Package is not relocatable.
[x]: Sources used to build the package match the upstream source, as
     provided in the spec URL.
[x]: Spec file name must match the spec package %{name}, in the format
     %{name}.spec.
[x]: File names are valid UTF-8.
[x]: Packages must not store files under /srv, /opt or /usr/local

===== SHOULD items =====

Generic:
[-]: If the source package does not include license text(s) as a separate
     file from upstream, the packager SHOULD query upstream to include it.
[x]: Final provides and requires are sane (see attachments).
[x]: Fully versioned dependency in subpackages if applicable.
     Note: No Requires: %{name}%{?_isa} = %{version}-%{release} in usd-libs
     , usd-devel
[?]: Package functions as described.
[!]: Latest version is packaged.

     Version 21.05 is available. Please update.

[x]: Package does not include license text files separate from upstream.
[x]: Patches link to upstream bugs/comments/lists or are otherwise
     justified.
[-]: Sources are verified with gpgverify first in %prep if upstream
     publishes signatures.
     Note: gpgverify is not used.
[-]: Description and summary sections in the package spec file contains
     translations for supported Non-English languages, if available.
[!]: %check is present and all tests pass.

     TODO: look into this.

[x]: Packages should try to preserve timestamps of original installed
     files.
[x]: Reviewer should test that the package builds in mock.
[x]: Buildroot is not present
[x]: Package has no %clean section with rm -rf %{buildroot} (or
     $RPM_BUILD_ROOT)
[x]: No file requires outside of /etc, /bin, /sbin, /usr/bin, /usr/sbin.
[x]: Packager, Vendor, PreReq, Copyright tags should not be in spec file
[x]: Sources can be downloaded from URI in Source: tag
[x]: SourceX is a working URL.
[x]: Package should compile and build into binary rpms on all supported
     architectures.
[x]: Spec use %global instead of %define unless justified.

===== EXTRA items =====

Generic:
[x]: Rpmlint is run on debuginfo package(s).
     Note: No rpmlint messages.
[x]: Rpmlint is run on all installed packages.
     Note: There are rpmlint messages (see attachment).
[x]: Large data in /usr/share should live in a noarch subpackage if package
     is arched.
[x]: Spec file according to URL is the same as in SRPM.


Rpmlint
-------
Checking: usd-21.02-4.fc35.x86_64.rpm
          usd-libs-21.02-4.fc35.x86_64.rpm
          usd-devel-21.02-4.fc35.x86_64.rpm
          usd-debuginfo-21.02-4.fc35.x86_64.rpm
          usd-debugsource-21.02-4.fc35.x86_64.rpm
          usd-21.02-4.fc35.src.rpm
usd.x86_64: W: devel-file-in-non-devel-package
/usr/share/usd/examples/include/pxr/imaging/hdTiny/mesh.h
usd.x86_64: W: devel-file-in-non-devel-package
/usr/share/usd/examples/include/pxr/imaging/hdTiny/renderDelegate.h
usd.x86_64: W: devel-file-in-non-devel-package
/usr/share/usd/examples/include/pxr/imaging/hdTiny/renderPass.h
usd.x86_64: W: devel-file-in-non-devel-package
/usr/share/usd/examples/include/pxr/imaging/hdTiny/rendererPlugin.h
usd.x86_64: W: no-manual-page-for-binary sdfdump
usd.x86_64: W: no-manual-page-for-binary sdffilter
usd-libs.x86_64: E: invalid-soname /usr/lib64/libar.so libar.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libarch.so libarch.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libcameraUtil.so libcameraUtil.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libgarch.so libgarch.so
usd-libs.x86_64: W: shared-lib-calls-exit /usr/lib64/libgarch.so
exit@GLIBC_2.2.5
usd-libs.x86_64: E: invalid-soname /usr/lib64/libgf.so libgf.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libglf.so libglf.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libhd.so libhd.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libhdSt.so libhdSt.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libhdx.so libhdx.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libhf.so libhf.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libhgi.so libhgi.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libhgiGL.so libhgiGL.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libhgiInterop.so libhgiInterop.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libhio.so libhio.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libjs.so libjs.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libkind.so libkind.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libndr.so libndr.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libpcp.so libpcp.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libplug.so libplug.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libpxOsd.so libpxOsd.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libsdf.so libsdf.so
usd-libs.x86_64: W: shared-lib-calls-exit /usr/lib64/libsdf.so exit@GLIBC_2.2.5
usd-libs.x86_64: E: invalid-soname /usr/lib64/libsdr.so libsdr.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libtf.so libtf.so
usd-libs.x86_64: W: shared-lib-calls-exit /usr/lib64/libtf.so exit@GLIBC_2.2.5
usd-libs.x86_64: W: shared-lib-calls-exit /usr/lib64/libtf.so _exit@GLIBC_2.2.5
usd-libs.x86_64: E: invalid-soname /usr/lib64/libtrace.so libtrace.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libusd.so libusd.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libusdAppUtils.so
libusdAppUtils.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libusdGeom.so libusdGeom.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libusdHydra.so libusdHydra.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libusdImaging.so libusdImaging.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libusdImagingGL.so
libusdImagingGL.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libusdLux.so libusdLux.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libusdMedia.so libusdMedia.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libusdRender.so libusdRender.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libusdRi.so libusdRi.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libusdRiImaging.so
libusdRiImaging.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libusdShade.so libusdShade.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libusdSkel.so libusdSkel.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libusdSkelImaging.so
libusdSkelImaging.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libusdUI.so libusdUI.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libusdUtils.so libusdUtils.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libusdVol.so libusdVol.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libusdVolImaging.so
libusdVolImaging.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libvt.so libvt.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libwork.so libwork.so
usd-devel.x86_64: W: only-non-binary-in-usr-lib
6 packages and 0 specfiles checked; 44 errors, 11 warnings.




Rpmlint (debuginfo)
-------------------
Checking: usd-debuginfo-21.02-4.fc35.x86_64.rpm
          usd-libs-debuginfo-21.02-4.fc35.x86_64.rpm
2 packages and 0 specfiles checked; 0 errors, 0 warnings.





Rpmlint (installed packages)
----------------------------
usd-devel.x86_64: W: only-non-binary-in-usr-lib
usd-libs.x86_64: E: invalid-soname /usr/lib64/libar.so libar.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libarch.so libarch.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libcameraUtil.so libcameraUtil.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libgarch.so libgarch.so
usd-libs.x86_64: W: shared-lib-calls-exit /usr/lib64/libgarch.so
exit@GLIBC_2.2.5
usd-libs.x86_64: E: invalid-soname /usr/lib64/libgf.so libgf.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libglf.so libglf.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libhd.so libhd.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libhdSt.so libhdSt.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libhdx.so libhdx.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libhf.so libhf.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libhgi.so libhgi.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libhgiGL.so libhgiGL.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libhgiInterop.so libhgiInterop.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libhio.so libhio.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libjs.so libjs.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libkind.so libkind.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libndr.so libndr.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libpcp.so libpcp.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libplug.so libplug.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libpxOsd.so libpxOsd.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libsdf.so libsdf.so
usd-libs.x86_64: W: shared-lib-calls-exit /usr/lib64/libsdf.so exit@GLIBC_2.2.5
usd-libs.x86_64: E: invalid-soname /usr/lib64/libsdr.so libsdr.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libtf.so libtf.so
usd-libs.x86_64: W: shared-lib-calls-exit /usr/lib64/libtf.so exit@GLIBC_2.2.5
usd-libs.x86_64: W: shared-lib-calls-exit /usr/lib64/libtf.so _exit@GLIBC_2.2.5
usd-libs.x86_64: E: invalid-soname /usr/lib64/libtrace.so libtrace.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libusd.so libusd.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libusdAppUtils.so
libusdAppUtils.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libusdGeom.so libusdGeom.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libusdHydra.so libusdHydra.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libusdImaging.so libusdImaging.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libusdImagingGL.so
libusdImagingGL.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libusdLux.so libusdLux.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libusdMedia.so libusdMedia.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libusdRender.so libusdRender.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libusdRi.so libusdRi.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libusdRiImaging.so
libusdRiImaging.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libusdShade.so libusdShade.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libusdSkel.so libusdSkel.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libusdSkelImaging.so
libusdSkelImaging.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libusdUI.so libusdUI.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libusdUtils.so libusdUtils.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libusdVol.so libusdVol.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libusdVolImaging.so
libusdVolImaging.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libvt.so libvt.so
usd-libs.x86_64: E: invalid-soname /usr/lib64/libwork.so libwork.so
usd.x86_64: W: devel-file-in-non-devel-package
/usr/share/usd/examples/include/pxr/imaging/hdTiny/mesh.h
usd.x86_64: W: devel-file-in-non-devel-package
/usr/share/usd/examples/include/pxr/imaging/hdTiny/renderDelegate.h
usd.x86_64: W: devel-file-in-non-devel-package
/usr/share/usd/examples/include/pxr/imaging/hdTiny/renderPass.h
usd.x86_64: W: devel-file-in-non-devel-package
/usr/share/usd/examples/include/pxr/imaging/hdTiny/rendererPlugin.h
usd.x86_64: W: no-manual-page-for-binary sdfdump
usd.x86_64: W: no-manual-page-for-binary sdffilter
6 packages and 0 specfiles checked; 44 errors, 11 warnings.



Unversioned so-files
--------------------
usd-libs: /usr/lib64/libar.so
usd-libs: /usr/lib64/libarch.so
usd-libs: /usr/lib64/libcameraUtil.so
usd-libs: /usr/lib64/libgarch.so
usd-libs: /usr/lib64/libgf.so
usd-libs: /usr/lib64/libglf.so
usd-libs: /usr/lib64/libhd.so
usd-libs: /usr/lib64/libhdSt.so
usd-libs: /usr/lib64/libhdx.so
usd-libs: /usr/lib64/libhf.so
usd-libs: /usr/lib64/libhgi.so
usd-libs: /usr/lib64/libhgiGL.so
usd-libs: /usr/lib64/libhgiInterop.so
usd-libs: /usr/lib64/libhio.so
usd-libs: /usr/lib64/libjs.so
usd-libs: /usr/lib64/libkind.so
usd-libs: /usr/lib64/libndr.so
usd-libs: /usr/lib64/libpcp.so
usd-libs: /usr/lib64/libplug.so
usd-libs: /usr/lib64/libpxOsd.so
usd-libs: /usr/lib64/libsdf.so
usd-libs: /usr/lib64/libsdr.so
usd-libs: /usr/lib64/libtf.so
usd-libs: /usr/lib64/libtrace.so
usd-libs: /usr/lib64/libusd.so
usd-libs: /usr/lib64/libusdAppUtils.so
usd-libs: /usr/lib64/libusdGeom.so
usd-libs: /usr/lib64/libusdHydra.so
usd-libs: /usr/lib64/libusdImaging.so
usd-libs: /usr/lib64/libusdImagingGL.so
usd-libs: /usr/lib64/libusdLux.so
usd-libs: /usr/lib64/libusdMedia.so
usd-libs: /usr/lib64/libusdRender.so
usd-libs: /usr/lib64/libusdRi.so
usd-libs: /usr/lib64/libusdRiImaging.so
usd-libs: /usr/lib64/libusdShade.so
usd-libs: /usr/lib64/libusdSkel.so
usd-libs: /usr/lib64/libusdSkelImaging.so
usd-libs: /usr/lib64/libusdUI.so
usd-libs: /usr/lib64/libusdUtils.so
usd-libs: /usr/lib64/libusdVol.so
usd-libs: /usr/lib64/libusdVolImaging.so
usd-libs: /usr/lib64/libvt.so
usd-libs: /usr/lib64/libwork.so
usd-libs: /usr/lib64/usd/plugin/hdEmbree.so
usd-libs: /usr/lib64/usd/plugin/hdStorm.so
usd-libs: /usr/lib64/usd/plugin/hioOiio.so
usd-libs: /usr/lib64/usd/plugin/sdrGlslfx.so
usd-libs: /usr/lib64/usd/plugin/sdrOsl.so
usd-libs: /usr/lib64/usd/plugin/usdAbc.so
usd-libs: /usr/lib64/usd/plugin/usdShaders.so

Source checksums
----------------
https://github.com/PixarAnimationStudios/usd/archive/v21.02/usd-21.02.tar.gz :
  CHECKSUM(SHA256) this package     :
97bbae5d86d6e4db357645db183be03afe28fe024b242f27bd4f62b4e2eade53
  CHECKSUM(SHA256) upstream package :
97bbae5d86d6e4db357645db183be03afe28fe024b242f27bd4f62b4e2eade53


Requires
--------
usd (rpmlib, GLIBC filtered):
    libarch.so()(64bit)
    libboost_program_options.so.1.75.0()(64bit)
    libc.so.6()(64bit)
    libgcc_s.so.1()(64bit)
    libgcc_s.so.1(GCC_3.0)(64bit)
    libgcc_s.so.1(GCC_3.3.1)(64bit)
    libsdf.so()(64bit)
    libstdc++.so.6()(64bit)
    libstdc++.so.6(CXXABI_1.3)(64bit)
    libstdc++.so.6(CXXABI_1.3.5)(64bit)
    libstdc++.so.6(CXXABI_1.3.9)(64bit)
    libtf.so()(64bit)
    libvt.so()(64bit)
    rtld(GNU_HASH)
    usd-libs(x86-64)

usd-libs (rpmlib, GLIBC filtered):
    ld-linux-x86-64.so.2()(64bit)
    libAlembic.so.1.8()(64bit)
    libGL.so.1()(64bit)
    libHalf-2_5.so.25()(64bit)
    libOpenColorIO.so.1()(64bit)
    libOpenImageIO.so.2.2()(64bit)
    libPtex.so.2.3()(64bit)
    libX11.so.6()(64bit)
    libar.so()(64bit)
    libarch.so()(64bit)
    libc.so.6()(64bit)
    libcameraUtil.so()(64bit)
    libdl.so.2()(64bit)
    libembree3.so.3()(64bit)
    libgarch.so()(64bit)
    libgcc_s.so.1()(64bit)
    libgcc_s.so.1(GCC_3.0)(64bit)
    libgcc_s.so.1(GCC_3.3)(64bit)
    libgcc_s.so.1(GCC_3.3.1)(64bit)
    libgf.so()(64bit)
    libglf.so()(64bit)
    libhd.so()(64bit)
    libhdSt.so()(64bit)
    libhdx.so()(64bit)
    libhf.so()(64bit)
    libhgi.so()(64bit)
    libhgiGL.so()(64bit)
    libhgiInterop.so()(64bit)
    libhio.so()(64bit)
    libjemalloc.so.2()(64bit)
    libjs.so()(64bit)
    libkind.so()(64bit)
    libm.so.6()(64bit)
    libndr.so()(64bit)
    libopenvdb.so.8.0()(64bit)
    libosdCPU.so.3.4.4()(64bit)
    libosdGPU.so.3.4.4()(64bit)
    liboslquery.so.1.11()(64bit)
    libpcp.so()(64bit)
    libplug.so()(64bit)
    libpthread.so.0()(64bit)
    libpxOsd.so()(64bit)
    libsdf.so()(64bit)
    libsdr.so()(64bit)
    libstdc++.so.6()(64bit)
    libstdc++.so.6(CXXABI_1.3)(64bit)
    libstdc++.so.6(CXXABI_1.3.1)(64bit)
    libstdc++.so.6(CXXABI_1.3.11)(64bit)
    libstdc++.so.6(CXXABI_1.3.5)(64bit)
    libstdc++.so.6(CXXABI_1.3.7)(64bit)
    libstdc++.so.6(CXXABI_1.3.8)(64bit)
    libstdc++.so.6(CXXABI_1.3.9)(64bit)
    libtbb.so.2()(64bit)
    libtf.so()(64bit)
    libtrace.so()(64bit)
    libusd.so()(64bit)
    libusdGeom.so()(64bit)
    libusdImaging.so()(64bit)
    libusdImagingGL.so()(64bit)
    libusdLux.so()(64bit)
    libusdShade.so()(64bit)
    libusdSkel.so()(64bit)
    libusdVol.so()(64bit)
    libvt.so()(64bit)
    libwork.so()(64bit)
    rtld(GNU_HASH)

usd-devel (rpmlib, GLIBC filtered):
    cmake-filesystem(x86-64)
    usd-libs(x86-64)

usd-debuginfo (rpmlib, GLIBC filtered):

usd-debugsource (rpmlib, GLIBC filtered):



Provides
--------
usd:
    bundled(SPIRV-Reflect)
    bundled(VulkanMemoryAllocator)
    bundled(double-conversion)
    bundled(ilmbase)
    bundled(lz4)
    bundled(pugixml)
    bundled(rapidjson)
    bundled(stb_image)
    bundled(stb_image_resize)
    bundled(stb_image_write)
    usd
    usd(x86-64)

usd-libs:
    libar.so()(64bit)
    libarch.so()(64bit)
    libcameraUtil.so()(64bit)
    libgarch.so()(64bit)
    libgf.so()(64bit)
    libglf.so()(64bit)
    libhd.so()(64bit)
    libhdSt.so()(64bit)
    libhdx.so()(64bit)
    libhf.so()(64bit)
    libhgi.so()(64bit)
    libhgiGL.so()(64bit)
    libhgiInterop.so()(64bit)
    libhio.so()(64bit)
    libjs.so()(64bit)
    libkind.so()(64bit)
    libndr.so()(64bit)
    libpcp.so()(64bit)
    libplug.so()(64bit)
    libpxOsd.so()(64bit)
    libsdf.so()(64bit)
    libsdr.so()(64bit)
    libtf.so()(64bit)
    libtrace.so()(64bit)
    libusd.so()(64bit)
    libusdAppUtils.so()(64bit)
    libusdGeom.so()(64bit)
    libusdHydra.so()(64bit)
    libusdImaging.so()(64bit)
    libusdImagingGL.so()(64bit)
    libusdLux.so()(64bit)
    libusdMedia.so()(64bit)
    libusdRender.so()(64bit)
    libusdRi.so()(64bit)
    libusdRiImaging.so()(64bit)
    libusdShade.so()(64bit)
    libusdSkel.so()(64bit)
    libusdSkelImaging.so()(64bit)
    libusdUI.so()(64bit)
    libusdUtils.so()(64bit)
    libusdVol.so()(64bit)
    libusdVolImaging.so()(64bit)
    libvt.so()(64bit)
    libwork.so()(64bit)
    usd-libs
    usd-libs(x86-64)

usd-devel:
    cmake(pxr)
    usd-devel
    usd-devel(x86-64)

usd-debuginfo:
    debuginfo(build-id)
    usd-debuginfo
    usd-debuginfo(x86-64)

usd-debugsource:
    usd-debugsource
    usd-debugsource(x86-64)



Generated by fedora-review 0.7.6 (b083f91) last change: 2020-11-10
Command line :/usr/bin/fedora-review -b 1895567
Buildroot used: fedora-rawhide-x86_64
Active plugins: C/C++, Generic, Shell-api
Disabled plugins: PHP, fonts, Perl, Haskell, Python, SugarActivity, Java, R,
Ocaml, Ruby
Disabled flags: EPEL6, EPEL7, DISTTAG, BATCH, EXARCH


-- 
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
Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure





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

  Powered by Linux