[Bug 858076] Review Request: mingw-qt5-qtscript - Qt5 for Windows - QtScript component

[bugzilla at redhat.com]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=858076

greg.hellings at gmail.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |greg.hellings at gmail.com
           Assignee|nobody at fedoraproject.org    |greg.hellings at gmail.com
              Flags|                            |fedora-review+

--- Comment #3 from greg.hellings at gmail.com ---
Taking for review

$ rpmlint mingw-qt5-qtscript.spec 
mingw-qt5-qtscript.spec:26: W: macro-in-comment %{qt_module}
mingw-qt5-qtscript.spec:26: W: macro-in-comment %{snapshot_rev}
mingw-qt5-qtscript.spec:26: W: macro-in-comment %{qt_module}
mingw-qt5-qtscript.spec:26: W: macro-in-comment %{snapshot_rev}
0 packages and 1 specfiles checked; 0 errors, 4 warnings.

$ rpmlint mingw-qt5-qtscript-5.0.1-1.fc19.src.rpm 
mingw-qt5-qtscript.src: E: unknown-key GPG#9b393ae8
mingw-qt5-qtscript.src:26: W: macro-in-comment %{qt_module}
mingw-qt5-qtscript.src:26: W: macro-in-comment %{snapshot_rev}
mingw-qt5-qtscript.src:26: W: macro-in-comment %{qt_module}
mingw-qt5-qtscript.src:26: W: macro-in-comment %{snapshot_rev}
1 packages and 0 specfiles checked; 1 errors, 4 warnings.

$ rpmlint results_mingw-qt5-qtscript/5.0.1/1.fc18/mingw*.rpm
mingw32-qt5-qtscript.noarch: W: no-documentation
mingw32-qt5-qtscript.noarch: E: zero-length
/usr/i686-w64-mingw32/sys-root/mingw/lib/pkgconfig/Qt5ScriptTools.pc
mingw32-qt5-qtscript.noarch: E: zero-length
/usr/i686-w64-mingw32/sys-root/mingw/lib/pkgconfig/Qt5Script.pc
mingw32-qt5-qtscript-debuginfo.noarch: E: debuginfo-without-sources
mingw64-qt5-qtscript.noarch: W: no-documentation
mingw64-qt5-qtscript.noarch: E: zero-length
/usr/x86_64-w64-mingw32/sys-root/mingw/lib/pkgconfig/Qt5Script.pc
mingw64-qt5-qtscript.noarch: E: zero-length
/usr/x86_64-w64-mingw32/sys-root/mingw/lib/pkgconfig/Qt5ScriptTools.pc
mingw64-qt5-qtscript-debuginfo.noarch: E: debuginfo-without-sources
mingw-qt5-qtscript.src:26: W: macro-in-comment %{qt_module}
mingw-qt5-qtscript.src:26: W: macro-in-comment %{snapshot_rev}
mingw-qt5-qtscript.src:26: W: macro-in-comment %{qt_module}
mingw-qt5-qtscript.src:26: W: macro-in-comment %{snapshot_rev}
5 packages and 0 specfiles checked; 6 errors, 6 warnings.

The 0-length PC files were discussed over in qtjsbackend and are an artifact of
a problem with the Qt5 out-of-stream building process. Until the issue is
resolved in qtbase, there's nothing to be done.

$ rpm -q --requires mingw32-qt5-qtscript
rpmlib(FileDigests) <= 4.6.0-1
rpmlib(PayloadFilesHavePrefix) <= 4.0-1
rpmlib(CompressedFileNames) <= 3.0.4-1
mingw32-crt
mingw32-filesystem >= 83
mingw32(kernel32.dll)
mingw32(libgcc_s_sjlj-1.dll)
mingw32(libstdc++-6.dll)
mingw32(msvcrt.dll)
mingw32(qt5core.dll)
mingw32(qt5gui.dll)
mingw32(qt5script.dll)
mingw32(qt5widgets.dll)
mingw32(winmm.dll)
rpmlib(PayloadIsXz) <= 5.2-1

$ rpm -q --requires mingw64-qt5-qtscript
rpmlib(FileDigests) <= 4.6.0-1
rpmlib(PayloadFilesHavePrefix) <= 4.0-1
rpmlib(CompressedFileNames) <= 3.0.4-1
mingw64-crt
mingw64-filesystem >= 83
mingw64(kernel32.dll)
mingw64(libstdc++-6.dll)
mingw64(msvcrt.dll)
mingw64(qt5core.dll)
mingw64(qt5gui.dll)
mingw64(qt5script.dll)
mingw64(qt5widgets.dll)
mingw64(winmm.dll)
rpmlib(PayloadIsXz) <= 5.2-1

There is a difference in the requires of libgcc_s_sjlj-1.dll, which seems
relatively common in MinGW projects and seems harmless so far.

$ rpm -q --provides mingw32-qt5-qtscript
mingw32-qt5-qtscript = 5.0.1-1.fc18
mingw32(qt5script.dll)
mingw32(qt5scripttools.dll)

$ rpm -q --provides mingw64-qt5-qtscript
mingw64-qt5-qtscript = 5.0.1-1.fc18
mingw64(qt5script.dll)
mingw64(qt5scripttools.dll)

$ wget --quiet
http://releases.qt-project.org/qt5/5.0.1/submodules_tar/qtscript-opensource-src-5.0.1.tar.xz
-O - | md5sum
349691431c3cc1b03e2571bac7c6c852  -

$ md5sum qtscript-opensource-src-5.0.1.tar.xz 
349691431c3cc1b03e2571bac7c6c852  qtscript-opensource-src-5.0.1.tar.xz

+ OK
! Needs to be looked into
/ Not applicable

[+] Compliant with generic Fedora Packaging Guidelines
[+] Source package name is prefixed with 'mingw-'
[+] Spec file starts with %{?mingw_package_header}
[+] BuildRequires: mingw32-filesystem >= 95 is in the .spec file
[+] BuildRequires: mingw64-filesystem >= 95 is in the .spec file
[+] Spec file contains %package sections for both mingw32 and mingw64 packages
[+] Binary mingw32 and mingw64 packages are noarch
[+] Spec file contains %{?mingw_debug_package} after the %description section
[+] Uses one of the macros %mingw_configure, %mingw_cmake, %mingw_cmake_kde4
    or %mingw_qmake_qt5 to configure the package
[+] Uses the macro %mingw_make to build the package
[+] Uses the macro %mingw_make to install the package
[+] If package contains translations, the %mingw_find_lang macro must be used
[+] No binary package named mingw-$pkgname is generated
[+] Libtool .la files are not bundled
[+] .def files are not bundled
[+] Man pages which duplicate native package are not bundled
[+] Info files which duplicate native package are not bundled
[+] Provides of the binary mingw32 and mingw64 packages are equal
[+] Requires of the binary mingw32 and mingw64 packages are equal

Approved.

-- 
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=P38zJ9RqeS&a=cc_unsubscribe