[Bug 1190361] Review Request: clutter-gst3 - GStreamer integration library for Clutter

2015-03-05 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1190361

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||clutter-gst3-3.0.4-1.fc21
 Resolution|--- |ERRATA
Last Closed||2015-03-05 07:35:11



--- Comment #11 from Fedora Update System upda...@fedoraproject.org ---
clutter-gst3-3.0.4-1.fc21 has been pushed to the Fedora 21 stable repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1190361] Review Request: clutter-gst3 - GStreamer integration library for Clutter

2015-02-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1190361

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #10 from Fedora Update System upda...@fedoraproject.org ---
clutter-gst3-3.0.4-1.fc21 has been pushed to the Fedora 21 testing repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1190361] Review Request: clutter-gst3 - GStreamer integration library for Clutter

2015-02-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1190361



--- Comment #8 from Patrick Uiterwijk puiterw...@redhat.com ---
Git done (by process-git-requests).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1190361] Review Request: clutter-gst3 - GStreamer integration library for Clutter

2015-02-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1190361

Patrick Uiterwijk puiterw...@redhat.com changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1190361] Review Request: clutter-gst3 - GStreamer integration library for Clutter

2015-02-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1190361

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1190361] Review Request: clutter-gst3 - GStreamer integration library for Clutter

2015-02-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1190361



--- Comment #9 from Fedora Update System upda...@fedoraproject.org ---
clutter-gst3-3.0.4-1.fc21 has been submitted as an update for Fedora 21.
https://admin.fedoraproject.org/updates/clutter-gst3-3.0.4-1.fc21

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1190361] Review Request: clutter-gst3 - GStreamer integration library for Clutter

2015-02-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1190361



--- Comment #7 from David King amigad...@amigadave.com ---
I don't see why it shouldn't be packaged for F21, especially as F22 won't be
out for quite a while still. As a developer, it would be nice to play with this
on the stable branch (and it should not impact other packages, as nothing
depends on it).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1190361] Review Request: clutter-gst3 - GStreamer integration library for Clutter

2015-02-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1190361



--- Comment #4 from David King amigad...@amigadave.com ---
Thanks for the quick review! I fixed the tabs-vs-spaces warning in-place.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1190361] Review Request: clutter-gst3 - GStreamer integration library for Clutter

2015-02-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1190361

Kalev Lember kalevlem...@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Assignee|nob...@fedoraproject.org|kalevlem...@gmail.com
  Flags||fedora-review+



--- Comment #3 from Kalev Lember kalevlem...@gmail.com ---
Fedora review clutter-gst3-3.0.4-1.fc22.src.rpm 2015-02-22

+ OK
! needs attention

+ rpmlint warnings are harmless and can be ignored
+ The package is named according to Fedora packaging guidelines
+ The spec file name matches the base package name.
+ The package meets the Packaging Guidelines
+ The package is licensed with a Fedora approved license and meets the
  Licensing Guidelines.
+ The license field in the spec file matches the actual license
+ The license text (COPYING) is included in %license 
+ Spec file is written in American English
+ Spec file is legible
+ Upstream sources match the sources in the srpm
  0b4283e6ccf5ee5d9f5fb635230fc714  clutter-gst-3.0.4.tar.xz
  0b4283e6ccf5ee5d9f5fb635230fc714  Download/clutter-gst-3.0.4.tar.xz
+ The package builds in koji
n/a ExcludeArch bugs filed
+ BuildRequires look sane
n/a locale handling
+ ldconfig in %post and %postun
+ Package does not bundle copies of system libraries
n/a Package isn't relocatable
+ Package owns all the directories it creates
+ No duplicate files in %files
+ Permissions are properly set
+ Consistent use of macros
+ The package must contain code or permissible content
n/a Large documentation files should go in -doc subpackage
+ Files marked %doc should not affect the runtime of application
n/a Static libraries should be in -static
+ Development files should be in -devel
+ -devel must require the fully versioned base
+ Packages should not contain libtool .la files
n/a Proper .desktop file handling
+ Doesn't own files or directories already owned by other packages
+ Filenames are valid UTF-8

Looks good to me.

APPROVED

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1190361] Review Request: clutter-gst3 - GStreamer integration library for Clutter

2015-02-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1190361



--- Comment #2 from Kalev Lember kalevlem...@gmail.com ---
$ rpmlint clutter-gst3-3.0.4-1.fc22.src.rpm \
  clutter-gst3 \
  clutter-gst3-debuginfo \
  clutter-gst3-devel
clutter-gst3.src: W: spelling-error Summary(en_US) GStreamer - G Streamer,
Streamer, Steamer
clutter-gst3.src:4: W: mixed-use-of-spaces-and-tabs (spaces: line 1, tab: line
4)
clutter-gst3.x86_64: W: spelling-error Summary(en_US) GStreamer - G Streamer,
Streamer, Steamer
clutter-gst3.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libclutter-gst-3.0.so.0.4.0 /lib64/libcogl-path.so.20
clutter-gst3.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libclutter-gst-3.0.so.0.4.0 /lib64/libatk-1.0.so.0
clutter-gst3.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libclutter-gst-3.0.so.0.4.0 /lib64/libcogl-pango.so.20
clutter-gst3.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libclutter-gst-3.0.so.0.4.0 /lib64/libjson-glib-1.0.so.0
clutter-gst3.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libclutter-gst-3.0.so.0.4.0 /lib64/libgdk-3.so.0
clutter-gst3.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libclutter-gst-3.0.so.0.4.0 /lib64/libpangocairo-1.0.so.0
clutter-gst3.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libclutter-gst-3.0.so.0.4.0 /lib64/libpango-1.0.so.0
clutter-gst3.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libclutter-gst-3.0.so.0.4.0 /lib64/libcairo-gobject.so.2
clutter-gst3.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libclutter-gst-3.0.so.0.4.0 /lib64/libcairo.so.2
clutter-gst3.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libclutter-gst-3.0.so.0.4.0 /lib64/libwayland-cursor.so.0
clutter-gst3.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libclutter-gst-3.0.so.0.4.0 /lib64/libxkbcommon.so.0
clutter-gst3.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libclutter-gst-3.0.so.0.4.0 /lib64/libXi.so.6
clutter-gst3.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libclutter-gst-3.0.so.0.4.0 /lib64/libgmodule-2.0.so.0
clutter-gst3.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libclutter-gst-3.0.so.0.4.0 /lib64/libwayland-egl.so.1
clutter-gst3.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libclutter-gst-3.0.so.0.4.0 /lib64/libwayland-client.so.0
clutter-gst3.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libclutter-gst-3.0.so.0.4.0 /lib64/libgbm.so.1
clutter-gst3.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libclutter-gst-3.0.so.0.4.0 /lib64/libdrm.so.2
clutter-gst3.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libclutter-gst-3.0.so.0.4.0 /lib64/libwayland-server.so.0
clutter-gst3.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libclutter-gst-3.0.so.0.4.0 /lib64/libEGL.so.1
clutter-gst3.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libclutter-gst-3.0.so.0.4.0 /lib64/libXext.so.6
clutter-gst3.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libclutter-gst-3.0.so.0.4.0 /lib64/libXdamage.so.1
clutter-gst3.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libclutter-gst-3.0.so.0.4.0 /lib64/libXfixes.so.3
clutter-gst3.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libclutter-gst-3.0.so.0.4.0 /lib64/libXcomposite.so.1
clutter-gst3.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libclutter-gst-3.0.so.0.4.0 /lib64/libXrandr.so.2
clutter-gst3.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libclutter-gst-3.0.so.0.4.0 /lib64/libgstbase-1.0.so.0
clutter-gst3.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libclutter-gst-3.0.so.0.4.0 /lib64/libpthread.so.0
clutter-gst3-devel.x86_64: W: spelling-error %description -l en_US gst - gs,
gt, st
clutter-gst3-devel.x86_64: W: only-non-binary-in-usr-lib
4 packages and 0 specfiles checked; 0 errors, 31 warnings.

Would be nice to fix up the mixed-use-of-spaces-and-tabs warning to reduce the
rpmlint spew a little bit.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1190361] Review Request: clutter-gst3 - GStreamer integration library for Clutter

2015-02-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1190361

David King amigad...@amigadave.com changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #5 from David King amigad...@amigadave.com ---
New Package SCM Request
===
Package Name: clutter-gst3
Short Description: GStreamer integration library for Clutter
Upstream URL: https://developer.gnome.org/clutter-gst/stable/
Owners: amigadave
Branches: f21 f22
InitialCC:

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1190361] Review Request: clutter-gst3 - GStreamer integration library for Clutter

2015-02-22 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1190361



--- Comment #6 from Kalev Lember kalevlem...@gmail.com ---
Shouldn't it be F22+ ? Nothing in GNOME 3.14 that's in F21 used clutter-gst
3.0.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1190361] Review Request: clutter-gst3 - GStreamer integration library for Clutter

2015-02-21 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1190361

David King amigad...@amigadave.com changed:

   What|Removed |Added

 CC||kalevlem...@gmail.com



--- Comment #1 from David King amigad...@amigadave.com ---
*** Bug 1194995 has been marked as a duplicate of this bug. ***

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review