[Bug 621898] Review Request: libwbxml - Library and tools to parse, encode and handle WBXML documents

[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=621898

--- Comment #4 from Petr Pisar <ppisar@xxxxxxxxxx> 2010-08-09 08:04:37 EDT ---
(In reply to comment #3)
> Some suggestions:
> 1.Provides:       wbxml2 = %{version}-%{release} is not useful since no rpm is
> actually depends wbxml2 explicitly.
> 
But it does not breaks anything. More ever think about third-party
repositories.

In addition, pure theoretically, there is a direct dependency:

$ repoquery --requires wbxml2-devel | grep wbxml2
libwbxml2.so.0
wbxml2 = 0.9.2-16.fc12
libwbxml2.so.0()(64bit)
wbxml2 = 0.9.2-16.fc12

One could say if wbxml2-devel is installed, wbxml2 is installed too. However
who knows how much broken system can an user have.

> 2.
> BuildRequires:  cmake, expat-devel, perl
> 
> Requires:      libxml2-devel, pkgconfig
> 
> rpmbuild will add pkgconfig as a dependency automatically[1],

What about EPEL-5? It's not supported currently, but I'd like to be prepared
for future possibilities.

> perl is on the exception list of Buildrequires[2]
> [2]http://fedoraproject.org/wiki/PackagingGuidelines#Exceptions_2
>
It's not listed.

> 3.
> 
> %build
> # Upstream does not support in-source-directory building
> SRCDIR="$PWD"
> %define builddir ../build
> rm -rf %{builddir}
> mkdir %{builddir}
> cd %{builddir}
> %cmake "$SRCDIR"
> make %{?_smp_mflags}
> 
> ->
> 
>  %build
>  mkdir -p %{_target_platform}
>  pushd %{_target_platform}
>  %{cmake} ..
>  popd
> 
>  make %{?_smp_mflags} -C %{_target_platform}
> 
Why popd and then make -C if one can do make without -C and then popd?

> 4.
> 
> %files devel
> %defattr(-,root,root,-)
> %doc AUTHORS BUGS ChangeLog COPYING GNU-LGPL NEWS README References THANKS TODO
> 
> You should not try to add duplicate docs to -devel, rpmlint warnings can be
> safely ignored.
>
Ok. I will remove them but COPYING and GNU-LGPL because it's new rule that
every package must deliver license copy if some exists in sources.

-- 
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.
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review


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