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

Christopher Meng <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
                 CC|                            |[email protected]
           Assignee|[email protected]    |[email protected]
              Flags|                            |fedora-review?



--- Comment #1 from Christopher Meng <[email protected]> ---
First from SPEC:

1. TOP line is empty, you should remove it.

2. When can we switch to 

http://cgit.freedesktop.org/gstreamer/gst-editing-services/snapshot/gst-editing-services-1.1.90.tar.gz

?

3. Explicit arch requires(isa macro missing):

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

4. I would describe devel package as:

%description       devel
This package contains libraries and header files for
developing applications that use %{name}.

5. autoreconf -if

-->

autoreconf -fiv (v for verbose)

6. rpm -E %configure shows that --prefix=/usr is included, do we need it again?

7. Cleanup rm -rf $RPM_BUILD_ROOT in %install

8. Hint:

find $RPM_BUILD_ROOT -name '*.la' | xargs rm -f

-->

find $RPM_BUILD_ROOT -name '*.la' -delete

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

Reply via email to