Description: This is a high-level library for facilitating the creation of audio/video non-linear editors. SRPM: http://fedorapeople.org/~limb/review/gst-editing-services/gst-editing-services-1.1.90-1.fc20.20131015git71a6d75.src.rpm SPEC: http://fedorapeople.org/~limb/review/gst-editing-services/gst-editing-services.spec
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
1. Fixed. 2. When I can get it to build. 3. Fixed. 4. Fixed. 5. Fixed. 6. Fixed. 7. Fixed. 8. Either works, but changed to your way. SRPM: http://fedorapeople.org/~limb/review/gst-editing-services/gst-editing-services-1.1.90-2.fc20.20131015git71a6d75.src.rpm SPEC: http://fedorapeople.org/~limb/review/gst-editing-services/gst-editing-services.spec Now if I can just figure out how to build the Python bindings, which pitivi needs.
Jon, If I do an $ rpmbuild --rebuild gst-editing-services 1.1.90-2.fc20.20131015git71a6d75.src.rpm I get the below error (I've copied the output from the %doc target onwards). It looks like there are two necessary files that remain unpackaged. This is on Fedora 20. I too am looking for a new pitivi package. Executing(%doc): /bin/sh -e /var/tmp/rpm-tmp.rJWPmj + umask 022 + cd /home/aidan/rpmbuild/BUILD + cd gst-editing-services-71a6d75 + DOCDIR=/home/aidan/rpmbuild/BUILDROOT/gst-editing-services-1.1.90-2.fc20.20131015git71a6d75.x86_64/usr/share/doc/gst-editing-services + export DOCDIR + /usr/bin/mkdir -p /home/aidan/rpmbuild/BUILDROOT/gst-editing-services-1.1.90-2.fc20.20131015git71a6d75.x86_64/usr/share/doc/gst-editing-services + cp -pr ChangeLog /home/aidan/rpmbuild/BUILDROOT/gst-editing-services-1.1.90-2.fc20.20131015git71a6d75.x86_64/usr/share/doc/gst-editing-services + cp -pr COPYING COPYING.LIB /home/aidan/rpmbuild/BUILDROOT/gst-editing-services-1.1.90-2.fc20.20131015git71a6d75.x86_64/usr/share/doc/gst-editing-services + cp -pr README /home/aidan/rpmbuild/BUILDROOT/gst-editing-services-1.1.90-2.fc20.20131015git71a6d75.x86_64/usr/share/doc/gst-editing-services + cp -pr RELEASE /home/aidan/rpmbuild/BUILDROOT/gst-editing-services-1.1.90-2.fc20.20131015git71a6d75.x86_64/usr/share/doc/gst-editing-services + cp -pr NEWS /home/aidan/rpmbuild/BUILDROOT/gst-editing-services-1.1.90-2.fc20.20131015git71a6d75.x86_64/usr/share/doc/gst-editing-services + cp -pr AUTHORS /home/aidan/rpmbuild/BUILDROOT/gst-editing-services-1.1.90-2.fc20.20131015git71a6d75.x86_64/usr/share/doc/gst-editing-services + exit 0 Provides: gst-editing-services = 1.1.90-2.fc20.20131015git71a6d75 gst-editing-services(x86-64) = 1.1.90-2.fc20.20131015git71a6d75 libges-1.0.so.0()(64bit) Requires(interp): /sbin/ldconfig /sbin/ldconfig Requires(rpmlib): rpmlib(CompressedFileNames) <= 3.0.4-1 rpmlib(FileDigests) <= 4.6.0-1 rpmlib(PayloadFilesHavePrefix) <= 4.0-1 Requires(post): /sbin/ldconfig Requires(postun): /sbin/ldconfig Requires: libc.so.6()(64bit) libc.so.6(GLIBC_2.2.5)(64bit) libc.so.6(GLIBC_2.4)(64bit) libges-1.0.so.0()(64bit) libgio-2.0.so.0()(64bit) libglib-2.0.so.0()(64bit) libgobject-2.0.so.0()(64bit) libgstbase-1.0.so.0()(64bit) libgstcontroller-1.0.so.0()(64bit) libgstpbutils-1.0.so.0()(64bit) libgstreamer-1.0.so.0()(64bit) libgstvideo-1.0.so.0()(64bit) libpthread.so.0()(64bit) libpthread.so.0(GLIBC_2.2.5)(64bit) libxml2.so.2()(64bit) libxml2.so.2(LIBXML2_2.4.30)(64bit) rtld(GNU_HASH) Processing files: gst-editing-services-devel-1.1.90-2.fc20.20131015git71a6d75.x86_64 Executing(%doc): /bin/sh -e /var/tmp/rpm-tmp.LrZDSR + umask 022 + cd /home/aidan/rpmbuild/BUILD + cd gst-editing-services-71a6d75 + DOCDIR=/home/aidan/rpmbuild/BUILDROOT/gst-editing-services-1.1.90-2.fc20.20131015git71a6d75.x86_64/usr/share/doc/gst-editing-services-devel + export DOCDIR + /usr/bin/mkdir -p /home/aidan/rpmbuild/BUILDROOT/gst-editing-services-1.1.90-2.fc20.20131015git71a6d75.x86_64/usr/share/doc/gst-editing-services-devel + cp -pr docs/ /home/aidan/rpmbuild/BUILDROOT/gst-editing-services-1.1.90-2.fc20.20131015git71a6d75.x86_64/usr/share/doc/gst-editing-services-devel + exit 0 Provides: gst-editing-services-devel = 1.1.90-2.fc20.20131015git71a6d75 gst-editing-services-devel(x86-64) = 1.1.90-2.fc20.20131015git71a6d75 pkgconfig(gst-editing-services-1.0) = 1.1.90.1 Requires(rpmlib): rpmlib(CompressedFileNames) <= 3.0.4-1 rpmlib(FileDigests) <= 4.6.0-1 rpmlib(PayloadFilesHavePrefix) <= 4.0-1 Requires: /usr/bin/pkg-config libges-1.0.so.0()(64bit) pkgconfig(gstreamer-1.0) pkgconfig(gstreamer-base-1.0) pkgconfig(gstreamer-controller-1.0) pkgconfig(gstreamer-pbutils-1.0) Processing files: gst-editing-services-debuginfo-1.1.90-2.fc20.20131015git71a6d75.x86_64 Provides: gst-editing-services-debuginfo = 1.1.90-2.fc20.20131015git71a6d75 gst-editing-services-debuginfo(x86-64) = 1.1.90-2.fc20.20131015git71a6d75 Requires(rpmlib): rpmlib(FileDigests) <= 4.6.0-1 rpmlib(PayloadFilesHavePrefix) <= 4.0-1 rpmlib(CompressedFileNames) <= 3.0.4-1 Checking for unpackaged file(s): /usr/lib/rpm/check-files /home/aidan/rpmbuild/BUILDROOT/gst-editing-services-1.1.90-2.fc20.20131015git71a6d75.x86_64 error: Installed (but unpackaged) file(s) found: /usr/lib64/girepository-1.0/GES-1.0.typelib /usr/share/gir-1.0/GES-1.0.gir
Hello Aidan, I can't reproduce the case you mentioned above on rawhide machine, but I think I found these are related: checking for gobject-introspection... no checking for gtkdoc-check... no checking for gtkdoc-rebase... no checking for gtkdoc-mkpdf... no checking whether to build gtk-doc documentation... no -------- And Jon, the build process is not expanded so please add V=1 to make.
Any chance we can ok this package soon. It'd be nice to get a good video editor into F20.
Jon, would you like to checkout the latest snapshot and submit a new one? Thanks.
Any updates on this one? I really want the new pitivi in fedora asap.
I'll try to post something new today.
SRPM: http://fedorapeople.org/~limb/review/gst-editing-services/gst-editing-services-1.1.90-3.fc20.20131206git289b04f.src.rpm SPEC: http://fedorapeople.org/~limb/review/gst-editing-services/gst-editing-services.spec Needs gtreamer1-python, with which it works. I found an active review after making my own, so if someone wanted to do that review that would be great. They need a sponsor, if no one else is willing I could do it.
%{_datadir}/gir-1.0/GES-1.0.gir should be included in -devel package. You can choose to fix it later.
Jon, please scratch a new git version from VCS and paste the new SRPM here, upstream has fixed something important. Thanks.
Just some thoughts from an outsider (feel free to ignore, sorry for the noise): - The latest version entry in the change does not match the version (gst-editing-services-1.1.90-3.fc20.20131206git289b04f.src.rpm) - personally I find it highly useful to include a "fetch-sources" script which can be used to recreate the source archives. I modified Jon's srpm so that the sources can be created directly from git without calling "autogen.sh" before: http://fschwarz.fedorapeople.org/2014/gst-editing-services-1.1.90-4.fc20.20140102git14a2f59.src.rpm If these changes don't seem useful to you, just ignore this post. I don't have any intention to interfere with your packaging Jon or to "take over" this review request in any way.
SRPM: http://fedorapeople.org/~limb/review/gst-editing-services/gst-editing-services-1.1.90-4.fc20.20131206git289b04f.src.rpm SPEC: http://fedorapeople.org/~limb/review/gst-editing-services/gst-editing-services.spec Addressed Christopher's issues, thinking about Felix's.
Package Review ============== Legend: [x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated ===== MUST items ===== C/C++: [x]: Package does not contain kernel modules. [x]: Package contains no static executables. [x]: Header files in -devel subpackage, if present. [x]: Package does not contain any libtool archives (.la) [x]: Rpath absent or only used for internal libs. [x]: Development (unversioned) .so files in -devel subpackage, if present. Generic: [x]: Package is licensed with an open-source compatible license and meets other legal requirements as defined in the legal section of Packaging Guidelines. [x]: License field in the package spec file matches the actual license. Note: Checking patched sources after %prep for licenses. Licenses found: "LGPL (v2 or later)", "GPL (v2 or later)", "GPL (v3 or later)", "Unknown or generated", "LGPL (v2 or later) (with incorrect FSF address)", "LGPL (v3 or later)", "GPL (v2.1 or later) (with incorrect FSF address) LGPL (v2.1 or later) (with incorrect FSF address)". 10 files have unknown license. Detailed output of licensecheck: GPL (v2 or later) ----------------- gst-editing-services-289b04f/common/coverage/coverage-report-entry.pl gst-editing-services-289b04f/common/coverage/coverage-report.pl gst-editing-services-289b04f/common/coverage/coverage-report.xsl gst-editing-services-289b04f/ltmain.sh GPL (v2.1 or later) (with incorrect FSF address) LGPL (v2.1 or later) (with incorrect FSF address) -------------------------------------------------------------------------------------------------- gst-editing-services-289b04f/bindings/python/gi/__init__.py gst-editing-services-289b04f/bindings/python/gi/overrides/GES.py gst-editing-services-289b04f/bindings/python/gi/overrides/__init__.py GPL (v3 or later) ----------------- gst-editing-services-289b04f/ges/ges-timeline-element.c gst-editing-services-289b04f/ges/ges-timeline-element.h LGPL (v2 or later) ------------------ gst-editing-services-289b04f/bindings/python/examples/simple.py gst-editing-services-289b04f/docs/design/gstencodebin.h gst-editing-services-289b04f/docs/design/gstprofile.h gst-editing-services-289b04f/ges/ges-audio-source.c gst-editing-services-289b04f/ges/ges-audio-source.h gst-editing-services-289b04f/ges/ges-audio-test-source.c gst-editing-services-289b04f/ges/ges-audio-test-source.h gst-editing-services-289b04f/ges/ges-audio-track.c gst-editing-services-289b04f/ges/ges-audio-track.h gst-editing-services-289b04f/ges/ges-audio-transition.c gst-editing-services-289b04f/ges/ges-audio-transition.h gst-editing-services-289b04f/ges/ges-audio-uri-source.c gst-editing-services-289b04f/ges/ges-audio-uri-source.h gst-editing-services-289b04f/ges/ges-base-effect-clip.c gst-editing-services-289b04f/ges/ges-base-effect-clip.h gst-editing-services-289b04f/ges/ges-base-effect.c gst-editing-services-289b04f/ges/ges-base-effect.h gst-editing-services-289b04f/ges/ges-base-transition-clip.c gst-editing-services-289b04f/ges/ges-base-transition-clip.h gst-editing-services-289b04f/ges/ges-clip.c gst-editing-services-289b04f/ges/ges-clip.h gst-editing-services-289b04f/ges/ges-container.c gst-editing-services-289b04f/ges/ges-container.h gst-editing-services-289b04f/ges/ges-effect-clip.c gst-editing-services-289b04f/ges/ges-effect-clip.h gst-editing-services-289b04f/ges/ges-effect.c gst-editing-services-289b04f/ges/ges-effect.h gst-editing-services-289b04f/ges/ges-enums.c gst-editing-services-289b04f/ges/ges-enums.h gst-editing-services-289b04f/ges/ges-formatter.c gst-editing-services-289b04f/ges/ges-formatter.h gst-editing-services-289b04f/ges/ges-gerror.h gst-editing-services-289b04f/ges/ges-group.c gst-editing-services-289b04f/ges/ges-image-source.c gst-editing-services-289b04f/ges/ges-image-source.h gst-editing-services-289b04f/ges/ges-internal.h gst-editing-services-289b04f/ges/ges-layer.c gst-editing-services-289b04f/ges/ges-layer.h gst-editing-services-289b04f/ges/ges-operation-clip.c gst-editing-services-289b04f/ges/ges-operation-clip.h gst-editing-services-289b04f/ges/ges-operation.c gst-editing-services-289b04f/ges/ges-operation.h gst-editing-services-289b04f/ges/ges-overlay-clip.c gst-editing-services-289b04f/ges/ges-overlay-clip.h gst-editing-services-289b04f/ges/ges-pipeline.c gst-editing-services-289b04f/ges/ges-pipeline.h gst-editing-services-289b04f/ges/ges-pitivi-formatter.c gst-editing-services-289b04f/ges/ges-pitivi-formatter.h gst-editing-services-289b04f/ges/ges-screenshot.c gst-editing-services-289b04f/ges/ges-screenshot.h gst-editing-services-289b04f/ges/ges-source-clip.c gst-editing-services-289b04f/ges/ges-source-clip.h gst-editing-services-289b04f/ges/ges-source.c gst-editing-services-289b04f/ges/ges-source.h gst-editing-services-289b04f/ges/ges-test-clip.c gst-editing-services-289b04f/ges/ges-test-clip.h gst-editing-services-289b04f/ges/ges-text-overlay-clip.c gst-editing-services-289b04f/ges/ges-text-overlay-clip.h gst-editing-services-289b04f/ges/ges-text-overlay.c gst-editing-services-289b04f/ges/ges-text-overlay.h gst-editing-services-289b04f/ges/ges-timeline.c gst-editing-services-289b04f/ges/ges-timeline.h gst-editing-services-289b04f/ges/ges-title-clip.c gst-editing-services-289b04f/ges/ges-title-clip.h gst-editing-services-289b04f/ges/ges-title-source.c gst-editing-services-289b04f/ges/ges-title-source.h gst-editing-services-289b04f/ges/ges-track-element.c gst-editing-services-289b04f/ges/ges-track-element.h gst-editing-services-289b04f/ges/ges-track.c gst-editing-services-289b04f/ges/ges-track.h gst-editing-services-289b04f/ges/ges-transition-clip.c gst-editing-services-289b04f/ges/ges-transition-clip.h gst-editing-services-289b04f/ges/ges-transition.c gst-editing-services-289b04f/ges/ges-transition.h gst-editing-services-289b04f/ges/ges-types.h gst-editing-services-289b04f/ges/ges-uri-clip.c gst-editing-services-289b04f/ges/ges-uri-clip.h gst-editing-services-289b04f/ges/ges-utils.c gst-editing-services-289b04f/ges/ges-utils.h gst-editing-services-289b04f/ges/ges-video-source.c gst-editing-services-289b04f/ges/ges-video-source.h gst-editing-services-289b04f/ges/ges-video-test-source.c gst-editing-services-289b04f/ges/ges-video-test-source.h gst-editing-services-289b04f/ges/ges-video-track.c gst-editing-services-289b04f/ges/ges-video-track.h gst-editing-services-289b04f/ges/ges-video-transition.c gst-editing-services-289b04f/ges/ges-video-transition.h gst-editing-services-289b04f/ges/ges-video-uri-source.c gst-editing-services-289b04f/ges/ges-video-uri-source.h gst-editing-services-289b04f/ges/ges.c gst-editing-services-289b04f/ges/ges.h gst-editing-services-289b04f/ges/gstframepositionner.c gst-editing-services-289b04f/ges/gstframepositionner.h gst-editing-services-289b04f/tests/check/ges/backgroundsource.c gst-editing-services-289b04f/tests/check/ges/basic.c gst-editing-services-289b04f/tests/check/ges/clip.c gst-editing-services-289b04f/tests/check/ges/effects.c gst-editing-services-289b04f/tests/check/ges/group.c gst-editing-services-289b04f/tests/check/ges/integration.c gst-editing-services-289b04f/tests/check/ges/layer.c gst-editing-services-289b04f/tests/check/ges/overlays.c gst-editing-services-289b04f/tests/check/ges/test-utils.c gst-editing-services-289b04f/tests/check/ges/test-utils.h gst-editing-services-289b04f/tests/check/ges/text_properties.c gst-editing-services-289b04f/tests/check/ges/timelineedition.c gst-editing-services-289b04f/tests/check/ges/titles.c gst-editing-services-289b04f/tests/check/ges/transition.c gst-editing-services-289b04f/tests/check/ges/uriclip.c gst-editing-services-289b04f/tests/examples/concatenate.c gst-editing-services-289b04f/tests/examples/ges-ui.c gst-editing-services-289b04f/tests/examples/overlays.c gst-editing-services-289b04f/tests/examples/simple1.c gst-editing-services-289b04f/tests/examples/test1.c gst-editing-services-289b04f/tests/examples/test2.c gst-editing-services-289b04f/tests/examples/test3.c gst-editing-services-289b04f/tests/examples/test4.c gst-editing-services-289b04f/tests/examples/text_properties.c gst-editing-services-289b04f/tests/examples/thumbnails.c gst-editing-services-289b04f/tests/examples/transition.c gst-editing-services-289b04f/tools/ges-launch.c LGPL (v2 or later) (with incorrect FSF address) ----------------------------------------------- gst-editing-services-289b04f/ges/ges-asset.c gst-editing-services-289b04f/ges/ges-asset.h gst-editing-services-289b04f/ges/ges-base-xml-formatter.c gst-editing-services-289b04f/ges/ges-base-xml-formatter.h gst-editing-services-289b04f/ges/ges-clip-asset.c gst-editing-services-289b04f/ges/ges-clip-asset.h gst-editing-services-289b04f/ges/ges-effect-asset.c gst-editing-services-289b04f/ges/ges-extractable.c gst-editing-services-289b04f/ges/ges-extractable.h gst-editing-services-289b04f/ges/ges-group.h gst-editing-services-289b04f/ges/ges-meta-container.h gst-editing-services-289b04f/ges/ges-project.c gst-editing-services-289b04f/ges/ges-project.h gst-editing-services-289b04f/ges/ges-track-element-asset.c gst-editing-services-289b04f/ges/ges-track-element-asset.h gst-editing-services-289b04f/ges/ges-uri-asset.c gst-editing-services-289b04f/ges/ges-uri-asset.h gst-editing-services-289b04f/ges/ges-xml-formatter.c gst-editing-services-289b04f/ges/ges-xml-formatter.h gst-editing-services-289b04f/tests/benchmarks/timeline.c gst-editing-services-289b04f/tests/check/ges/asset.c gst-editing-services-289b04f/tests/check/ges/project.c gst-editing-services-289b04f/tests/examples/assets.c LGPL (v3 or later) ------------------ gst-editing-services-289b04f/ges/ges-auto-transition.c gst-editing-services-289b04f/ges/ges-auto-transition.h gst-editing-services-289b04f/ges/ges-effect-asset.h gst-editing-services-289b04f/ges/ges-smart-adder.c gst-editing-services-289b04f/ges/ges-smart-adder.h gst-editing-services-289b04f/ges/ges-smart-video-mixer.c gst-editing-services-289b04f/ges/ges-smart-video-mixer.h gst-editing-services-289b04f/tests/check/ges/mixers.c Unknown or generated -------------------- gst-editing-services-289b04f/autogen.sh gst-editing-services-289b04f/autoregen.sh gst-editing-services-289b04f/common/c-to-xml.py gst-editing-services-289b04f/common/gen-changelog.py gst-editing-services-289b04f/common/gst-autogen.sh gst-editing-services-289b04f/common/mangle-tmpl.py gst-editing-services-289b04f/common/plugins.xsl gst-editing-services-289b04f/common/scangobj-merge.py gst-editing-services-289b04f/config.h gst-editing-services-289b04f/ges/ges-meta-container.c [x]: License file installed when any subpackage combination is installed. [x]: If the package is under multiple licenses, the licensing breakdown must be documented in the spec. [x]: Package must own all directories that it creates. [x]: %build honors applicable compiler flags or justifies otherwise. [x]: Package contains no bundled libraries without FPC exception. [x]: Changelog in prescribed format. [x]: Sources contain only permissible code or content. [-]: Package contains desktop file if it is a GUI application. [x]: Development files must be in a -devel package [x]: Package uses nothing in %doc for runtime. [x]: Package consistently uses macros (instead of hard-coded directory names). [x]: Package is named according to the Package Naming Guidelines. [x]: Package does not generate any conflict. [x]: Package obeys FHS, except libexecdir and /usr/target. [-]: If the package is a rename of another package, proper Obsoletes and Provides are present. [x]: Requires correct, justified where necessary. [x]: Spec file is legible and written in American English. [-]: Package contains systemd file(s) if in need. [x]: Useful -debuginfo package or justification otherwise. [x]: Package is not known to require an ExcludeArch tag. [x]: Large documentation must go in a -doc subpackage. Large could be size (~1MB) or number of files. Note: Documentation size is 829440 bytes in 33 files. [x]: Package complies to the Packaging Guidelines [x]: Package successfully compiles and builds into binary rpms on at least one supported primary architecture. [x]: Package installs properly. [x]: Rpmlint is run on all rpms the build produces. Note: There are rpmlint messages (see attachment). [x]: If (and only if) the source package includes the text of the license(s) in its own file, then that file, containing the text of the license(s) for the package is included in %doc. [x]: Package requires other packages for directories it uses. [x]: Package does not own files or directories owned by other packages. [x]: All build dependencies are listed in BuildRequires, except for any that are listed in the exceptions section of Packaging Guidelines. [x]: Package uses either %{buildroot} or $RPM_BUILD_ROOT [x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the beginning of %install. [x]: Macros in Summary, %description expandable at SRPM build time. [x]: Package does not contain duplicates in %files. [x]: Permissions on files are set properly. [x]: Package use %makeinstall only when make install' ' DESTDIR=... doesn't work. [x]: Package is named using only allowed ASCII characters. [x]: Package do not use a name that already exist [x]: Package is not relocatable. [x]: Sources used to build the package match the upstream source, as provided in the spec URL. [x]: Spec file name must match the spec package %{name}, in the format %{name}.spec. [x]: File names are valid UTF-8. [x]: Packages must not store files under /srv, /opt or /usr/local ===== SHOULD items ===== Generic: [x]: If the source package does not include license text(s) as a separate file from upstream, the packager SHOULD query upstream to include it. [x]: Final provides and requires are sane (see attachments). [x]: Fully versioned dependency in subpackages if applicable. [x]: Package functions as described. [x]: Latest version is packaged. [x]: Package does not include license text files separate from upstream. [x]: Scriptlets must be sane, if used. [-]: SourceX tarball generation or download is documented. Note: Package contains tarball without URL, check comments [x]: Description and summary sections in the package spec file contains translations for supported Non-English languages, if available. [x]: Package should compile and build into binary rpms on all supported architectures. [-]: %check is present and all tests pass. [x]: Packages should try to preserve timestamps of original installed files. [x]: Packager, Vendor, PreReq, Copyright tags should not be in spec file [x]: Reviewer should test that the package builds in mock. [x]: Buildroot is not present [x]: Package has no %clean section with rm -rf %{buildroot} (or $RPM_BUILD_ROOT) [x]: Dist tag is present (not strictly required in GL). [x]: No file requires outside of /etc, /bin, /sbin, /usr/bin, /usr/sbin. [x]: Uses parallel make %{?_smp_mflags} macro. [x]: The placement of pkgconfig(.pc) files are correct. [x]: SourceX is a working URL. [x]: Spec use %global instead of %define unless justified. ===== EXTRA items ===== Generic: [x]: Large data in /usr/share should live in a noarch subpackage if package is arched. Note: Arch-ed rpms have a total of 1372160 bytes in /usr/share [x]: Rpmlint is run on all installed packages. Note: There are rpmlint messages (see attachment). [x]: Package should not use obsolete m4 macros Rpmlint ------- Checking: gst-editing-services-1.1.90-4.fc21.20131206git289b04f.i686.rpm gst-editing-services-devel-1.1.90-4.fc21.20131206git289b04f.i686.rpm gst-editing-services-1.1.90-4.fc21.20131206git289b04f.src.rpm gst-editing-services.i686: W: spelling-error Summary(en_US) Gstreamer -> Streamer, G streamer, Steamer gst-editing-services.i686: W: no-manual-page-for-binary ges-launch-1.0 gst-editing-services.src: W: spelling-error Summary(en_US) Gstreamer -> Streamer, G streamer, Steamer gst-editing-services.src: W: invalid-url Source0: gst-editing-services-289b04f.tar.gz 3 packages and 0 specfiles checked; 0 errors, 4 warnings. Rpmlint (installed packages) ---------------------------- # rpmlint gst-editing-services gst-editing-services-devel gst-editing-services.i686: W: spelling-error Summary(en_US) Gstreamer -> Streamer, G streamer, Steamer gst-editing-services.i686: W: no-manual-page-for-binary ges-launch-1.0 2 packages and 0 specfiles checked; 0 errors, 2 warnings. # echo 'rpmlint-done:' Requires -------- gst-editing-services (rpmlib, GLIBC filtered): /sbin/ldconfig libc.so.6 libges-1.0.so.0 libgio-2.0.so.0 libglib-2.0.so.0 libgobject-2.0.so.0 libgstbase-1.0.so.0 libgstcontroller-1.0.so.0 libgstpbutils-1.0.so.0 libgstreamer-1.0.so.0 libgstvideo-1.0.so.0 libpthread.so.0 libxml2.so.2 libxml2.so.2(LIBXML2_2.4.30) python-gstreamer1 rtld(GNU_HASH) gst-editing-services-devel (rpmlib, GLIBC filtered): /usr/bin/pkg-config gst-editing-services(x86-32) libges-1.0.so.0 pkgconfig(gstreamer-1.0) pkgconfig(gstreamer-base-1.0) pkgconfig(gstreamer-controller-1.0) pkgconfig(gstreamer-pbutils-1.0) Provides -------- gst-editing-services: gst-editing-services gst-editing-services(x86-32) libges-1.0.so.0 gst-editing-services-devel: gst-editing-services-devel gst-editing-services-devel(x86-32) pkgconfig(gst-editing-services-1.0) Generated by fedora-review 0.5.1 (bb9bf27) last change: 2013-12-13 Command line :/usr/bin/fedora-review -rvn gst-editing-services-1.1.90-4.fc20.20131206git289b04f.src.rpm Buildroot used: fedora-rawhide-i386 Active plugins: Generic, Shell-api, C/C++ Disabled plugins: Java, Python, fonts, SugarActivity, Ocaml, Perl, Haskell, R, PHP, Ruby Disabled flags: EXARCH, EPEL5, BATCH, DISTTAG ---------------------------- NOTE: 1. Please snapshot again from git repo before SCM import. 2. Please notify upstream about the incorrect FSF address. PACKAGE APPROVED.
New Package SCM Request ======================= Package Name: gst-editiing-services Short Description: Gstreamer editing services Owners: limb Branches: f20 InitialCC: Thanks for the review, will do!
Git done (by process-git-requests).
gst-editing-services-1.1.90-5.fc20.20140123git105d901,pitivi-0.92-1.fc20 has been submitted as an update for Fedora 20. https://admin.fedoraproject.org/updates/gst-editing-services-1.1.90-5.fc20.20140123git105d901,pitivi-0.92-1.fc20
Package gst-editing-services-1.1.90-5.fc20.20140123git105d901, pitivi-0.92-1.fc20: * should fix your issue, * was pushed to the Fedora 20 testing repository, * should be available at your local mirror within two days. Update it with: # su -c 'yum update --enablerepo=updates-testing gst-editing-services-1.1.90-5.fc20.20140123git105d901 pitivi-0.92-1.fc20' as soon as you are able to. Please go to the following url: https://admin.fedoraproject.org/updates/FEDORA-2014-1454/gst-editing-services-1.1.90-5.fc20.20140123git105d901,pitivi-0.92-1.fc20 then log in and leave karma (feedback).
gst-editing-services-1.1.90-5.fc20.20140123git105d901, pitivi-0.92-1.fc20 has been pushed to the Fedora 20 stable repository. If problems still persist, please make note of it in this bug report.