[Bug 491712] Review Request: ocaml-mlgmpidl - OCaml interface to GMP and MPFR libraries

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

 



Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Alan Dunn <amdunn@xxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|needinfo?(amdunn@xxxxxxxxx) |




--- Comment #3 from Alan Dunn <amdunn@xxxxxxxxx>  2009-03-25 08:00:56 EDT ---
Should be fixed now in files at the same location (though in this case, unlike
in the example, %{my_ocaml_lib_dir}/*.a should be excluded from main and put
into devel in not just opt build due to libgmp_caml.a)

(In reply to comment #2)
> + rpmlint output
> 
> rpmlint output all looks fine, and the things it notices
> can be ignored.
> 
> + package name satisfies the packaging naming guidelines
> + specfile name matches the package base name
> + package should satisfy packaging guidelines
> + license meets guidelines and is acceptable to Fedora
> + license matches the actual package license
>   LGPLv2 (not +)
> + %doc includes license file
> + spec file written in American English
> + spec file is legible
> + upstream sources match sources in the srpm
>   63ec244511e58bd1cbf5513dc7aaad8e 169186
> + package successfully builds on at least one architecture
> n/a ExcludeArch bugs filed
> + BuildRequires list all build dependencies
> n/a %find_lang instead of %{_datadir}/locale/*
> + binary RPM with shared library files must call ldconfig in %post and %postun
> + does not use Prefix: /usr
> - package owns all directories it creates
> 
>   Package should own %{my_ocaml_lib_dir}.
>   Then %files should add %exclude lines for everything in -devel.
>   See the example:
> http://fedoraproject.org/w/uploads/5/5c/Packaging_OCaml_ocaml-foolib.spec
> 
> + no duplicate files in %files
> + %defattr line
> + %clean contains rm -rf $RPM_BUILD_ROOT
> + consistent use of macros
> + package must contain code or permissible content
> + large documentation files should go in -doc subpackage
> + files marked %doc should not affect package
> n/a header files should be in -devel
> n/a static libraries should be in -static
> n/a packages containing pkgconfig (.pc) files need 'Requires: pkgconfig'
> n/a libfoo.so must go in -devel
> + -devel must require the fully versioned base
> n/a packages should not contain libtool .la files
> n/a packages containing GUI apps must include %{name}.desktop file
> + packages must not own files or directories owned by other packages
> + %install must start with rm -rf %{buildroot} etc.
> + filenames must be valid UTF-8
> 
> Optional:
> 
> n/a if there is no license file, packager should query upstream
> n/a translations of description and summary for non-English languages, if
> available
> + reviewer should build the package in mock
> + the package should build into binary RPMs on all supported architectures
> - review should test the package functions as described
> n/a scriptlets should be sane
> n/a pkgconfig files should go in -devel
> + shouldn't have file dependencies outside /etc /bin /sbin /usr/bin or
> /usr/sbin
> 
> ------------
> 
> Please fix the directory ownership issue.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

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