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

Erik van Pienbroek <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
              Flags|fedora-review?              |fedora-review+

--- Comment #7 from Erik van Pienbroek <[email protected]> ---
$ rpmlint mingw-libosinfo.spec 
0 packages and 1 specfiles checked; 0 errors, 0 warnings.

$ rpmlint mingw-libosinfo-0.1.2-2.fc17.src.rpm 
mingw-libosinfo.src: W: spelling-error Summary(en_US) virtualization ->
visualization, actualization, vitalization
mingw-libosinfo.src: W: spelling-error %description -l en_US virtualization ->
visualization, actualization, vitalization
mingw-libosinfo.src: W: spelling-error %description -l en_US hypervisor ->
hyper visor, hyper-visor, supervisory
1 packages and 0 specfiles checked; 0 errors, 3 warnings.

$ rpmlint mingw32-libosinfo-0.1.2-2.fc17.noarch.rpm
mingw64-libosinfo-0.1.2-2.fc17.noarch.rpm 
mingw32-libosinfo.noarch: W: manpage-not-compressed gz
/usr/i686-w64-mingw32/sys-root/mingw/share/man/man1/osinfo-detect.1
mingw32-libosinfo.noarch: W: manpage-not-compressed gz
/usr/i686-w64-mingw32/sys-root/mingw/share/man/man1/osinfo-query.1
mingw32-libosinfo.noarch: W: manpage-not-compressed gz
/usr/i686-w64-mingw32/sys-root/mingw/share/man/man1/osinfo-db-validate.1
mingw32-libosinfo.noarch: W: spelling-error Summary(en_US) virtualization ->
visualization, actualization, vitalization
mingw32-libosinfo.noarch: W: spelling-error %description -l en_US
virtualization -> visualization, actualization, vitalization
mingw32-libosinfo.noarch: W: spelling-error %description -l en_US hypervisor ->
hyper visor, hyper-visor, supervisory
mingw64-libosinfo.noarch: W: manpage-not-compressed gz
/usr/x86_64-w64-mingw32/sys-root/mingw/share/man/man1/osinfo-query.1
mingw64-libosinfo.noarch: W: manpage-not-compressed gz
/usr/x86_64-w64-mingw32/sys-root/mingw/share/man/man1/osinfo-db-validate.1
mingw64-libosinfo.noarch: W: manpage-not-compressed gz
/usr/x86_64-w64-mingw32/sys-root/mingw/share/man/man1/osinfo-detect.1
mingw64-libosinfo.noarch: W: spelling-error Summary(en_US) virtualization ->
visualization, actualization, vitalization
mingw64-libosinfo.noarch: W: spelling-error %description -l en_US
virtualization -> visualization, actualization, vitalization
mingw64-libosinfo.noarch: W: spelling-error %description -l en_US hypervisor ->
hyper visor, hyper-visor, supervisory
2 packages and 0 specfiles checked; 0 errors, 12 warnings.


$ rpm --query --requires mingw32-libosinfo
mingw32(kernel32.dll)  
mingw32(libgio-2.0-0.dll)  
mingw32(libglib-2.0-0.dll)  
mingw32(libgobject-2.0-0.dll)  
mingw32(libosinfo-1.0-0.dll)  
mingw32(libxml2-2.dll)  
mingw32(msvcrt.dll)  
mingw32-crt  
mingw32-filesystem >= 83
pkgconfig  
rpmlib(CompressedFileNames) <= 3.0.4-1
rpmlib(PayloadFilesHavePrefix) <= 4.0-1
rpmlib(PayloadIsXz) <= 5.2-1

$ rpm --query --requires mingw64-libosinfo
mingw64(kernel32.dll)  
mingw64(libgio-2.0-0.dll)  
mingw64(libglib-2.0-0.dll)  
mingw64(libgobject-2.0-0.dll)  
mingw64(libosinfo-1.0-0.dll)  
mingw64(libxml2-2.dll)  
mingw64(msvcrt.dll)  
mingw64-crt  
mingw64-filesystem >= 83
pkgconfig  
rpmlib(CompressedFileNames) <= 3.0.4-1
rpmlib(PayloadFilesHavePrefix) <= 4.0-1
rpmlib(PayloadIsXz) <= 5.2-1


$ rpm --query --provides mingw32-libosinfo
mingw32(libosinfo-1.0-0.dll)  
mingw32-libosinfo = 0.1.2-2.fc17

$ rpm --query --provides mingw64-libosinfo
mingw64(libosinfo-1.0-0.dll)  
mingw64-libosinfo = 0.1.2-2.fc17


$ wget --quiet
https://fedorahosted.org/releases/l/i/libosinfo/libosinfo-0.1.2.tar.gz -O - |
md5sum
f4bd25987a1576e7408d3635da6fb342  -
$ md5sum libosinfo-0.1.2.tar.gz 
f4bd25987a1576e7408d3635da6fb342  libosinfo-0.1.2.tar.gz


+ 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, or %mingw_cmake_kde4
    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


Your package currently also bundles libtool .la files.
These aren't needed any more these days and can be removed.

The man pages are also bundled with the native Fedora libosinfo package.
Because of this they don't need to be bundled with the mingw package.

The remaining rpmlint warnings can be ignored

Please remove the .la files and man pages before importing this package in
Fedora


=====================================================
 The package mingw-libosinfo is APPROVED by epienbro
=====================================================

-- 
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
mingw mailing list
[email protected]
https://admin.fedoraproject.org/mailman/listinfo/mingw

Reply via email to