[Bug 1043290] Review Request: engrampa - MATE Desktop file archiver

2014-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1043290
Bug 1043290 depends on bug 1038773, which changed state.

Bug 1038773 Summary: Review Request: caja - File manager for MATE desktop
https://bugzilla.redhat.com/show_bug.cgi?id=1038773

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |ERRATA



-- 
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 1043290] Review Request: engrampa - MATE Desktop file archiver

2014-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1043290

Wolfgang Ulbrich  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |ERRATA
Last Closed|2013-12-29 03:54:07 |2014-09-28 16:00:58



-- 
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 1043290] Review Request: engrampa - MATE Desktop file archiver

2014-07-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1043290

Kevin Fenzi  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 1043290] Review Request: engrampa - MATE Desktop file archiver

2014-07-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1043290



--- Comment #7 from Kevin Fenzi  ---
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 1043290] Review Request: engrampa - MATE Desktop file archiver

2014-07-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1043290

Wolfgang Ulbrich  changed:

   What|Removed |Added

 Status|CLOSED  |ASSIGNED
 Resolution|ERRATA  |---
  Flags|fedora-cvs+ |fedora-cvs?
   Keywords||Reopened



--- Comment #6 from Wolfgang Ulbrich  ---
Package Change Request
==
Package Name: engrampa
New Branches: f20
Owners: raveit65 vicodan
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 1043290] Review Request: engrampa - MATE Desktop file archiver

2014-07-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1043290
Bug 1043290 depends on bug 1038773, which changed state.

Bug 1038773 Summary: Review Request: caja - File manager for MATE desktop
https://bugzilla.redhat.com/show_bug.cgi?id=1038773

   What|Removed |Added

 Status|CLOSED  |ASSIGNED
 Resolution|ERRATA  |---



-- 
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 1043290] Review Request: engrampa - MATE Desktop file archiver

2014-04-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1043290

Wolfgang Ulbrich  changed:

   What|Removed |Added

 Blocks||1086149




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1086149
[Bug 1086149] MATE Desktop 1.8
-- 
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 1043290] Review Request: engrampa - MATE Desktop file archiver

2013-12-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1043290

Wolfgang Ulbrich  changed:

   What|Removed |Added

 Status|POST|CLOSED
 Resolution|--- |ERRATA
Last Closed||2013-12-29 03:54:07



-- 
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 1043290] Review Request: engrampa - MATE Desktop file archiver

2013-12-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1043290

Bug 1043290 depends on bug 1038773, which changed state.

Bug 1038773 Summary: Review Request: caja - File manager for MATE desktop
https://bugzilla.redhat.com/show_bug.cgi?id=1038773

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |ERRATA



-- 
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 1043290] Review Request: engrampa - MATE Desktop file archiver

2013-12-16 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1043290

Jon Ciesla  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 1043290] Review Request: engrampa - MATE Desktop file archiver

2013-12-16 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1043290



--- Comment #5 from Jon Ciesla  ---
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 1043290] Review Request: engrampa - MATE Desktop file archiver

2013-12-16 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1043290

Wolfgang Ulbrich  changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #4 from Wolfgang Ulbrich  ---
Thanks Björn for review the package.

New Package SCM Request
===
Package Name: engrampa
Short Description: MATE Desktop file archiver
Owners: raveit65 vicodan
Branches: 
InitialCC: 

Note: branch is empty because other branches than rawhide will use old package
name until releases are EOL.

-- 
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 1043290] Review Request: engrampa - MATE Desktop file archiver

2013-12-16 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1043290

Björn "besser82" Esser  changed:

   What|Removed |Added

 Status|ASSIGNED|POST
  Flags|fedora-review?  |fedora-review+



--- Comment #3 from Björn "besser82" Esser  ---
Package has one issue, but no blocker.  You can fix during import.  :)


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]: Development (unversioned) .so files in -devel subpackage, if present.
 Note: Unversioned so-files in private %_libdir subdirectory (see
 attachment). Verify they are not in ld path.

 ---> all is fine here.

[x]: Package does not contain any libtool archives (.la)
[x]: Rpath absent or only used for internal libs.

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)", "Unknown or generated". 5
 files have unknown license. Detailed output of licensecheck in
 /home/besser82/shared/fedora/review/1043290-engrampa/licensecheck.txt
[x]: If the package is under multiple licenses, the licensing breakdown must
 be documented in the spec.
[x]: Package requires other packages for directories it uses.
 Note: No known owner of /usr/share/help/nl, /usr/share/help/vi,
 /usr/share/help/uk, /usr/share/help/ja

 ---> false positive, those dirs should be owned by yelp ---> BUG!

[x]: Package must own all directories that it creates.
 Note: Directories without known owners: /usr/share/help/es,
 /usr/share/help/fi, /usr/share/help/en_GB, /usr/share/help/sv,
 /usr/share/help/cs, /usr/share/help, /usr/share/help/de,
 /usr/share/help/vi, /usr/share/help/uk, /usr/share/help/bg,
 /usr/share/help/oc, /usr/share/help/it, /usr/share/help/ca,
 /usr/share/help/ru, /usr/share/help/zh_CN, /usr/share/help/fr,
 /usr/share/help/ja, /usr/share/help/C, /usr/share/help/nl,
 /usr/share/help/el

 ---> those dirs should be owned by `yelp`, but aren't.  File a bug
  against `yelp` and complain about the missing ownerships, please.

[x]: Package does not own files or directories owned by other packages.
 Note: Dirs in package are owned also by:
 /usr/share/engrampa/icons/hicolor(mate-file-archiver),
 /usr/share/engrampa(mate-file-archiver),
 /usr/share/engrampa/icons/hicolor/24x24/actions(mate-file-archiver),
 /usr/share/engrampa/icons/hicolor/16x16/actions(mate-file-archiver),
 /usr/share/engrampa/ui(mate-file-archiver), /usr/share/engrampa/icons
 (mate-file-archiver), /usr/libexec/engrampa(mate-file-archiver),
 /usr/share/engrampa/icons/hicolor/24x24(mate-file-archiver),
 /usr/share/engrampa/icons/hicolor/16x16(mate-file-archiver)

 ---> the above named pkg will be obsoleted by this one.

[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.
[-]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: glib-compile-schemas is run in %postun and %posttrans if package has
 *.gschema.xml files.
 Note: gschema file(s) in engrampa

 ---> scriptlets are sane.

[x]: The spec file handles locales properly.
[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.
[x]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.

 ---> please limit them to be present until a specific distro-release
  like until F23 or so.

[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]: update-desktop-database is invoked in %post and %postun if package
 contains desktop file(s) with a MimeType: entry.
 Note: desktop file(s) with MimeType entry in engrampa

 ---> scriptlets are sane.

[x]: gtk-update-icon-cache is invoked in %postun and %posttrans if package
 contains icons.
 Note: icons in engrampa

 ---> scriptlets are sane.

[x]: Useful -debuginfo package or justification otherwise.
[x]: Package is not known to require an ExcludeArch tag.
[-]: Large documentation must go 

[Bug 1043290] Review Request: engrampa - MATE Desktop file archiver

2013-12-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1043290

Björn "besser82" Esser  changed:

   What|Removed |Added

  Alias||engrampa



-- 
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 1043290] Review Request: engrampa - MATE Desktop file archiver

2013-12-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1043290

Björn "besser82" Esser  changed:

   What|Removed |Added

 Depends On||1038773 (caja)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1038773
[Bug 1038773] Review Request: caja - File manager for MATE desktop
-- 
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 1043290] Review Request: engrampa - MATE Desktop file archiver

2013-12-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1043290

Björn "besser82" Esser  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||bjoern.es...@gmail.com
   Assignee|nob...@fedoraproject.org|bjoern.es...@gmail.com
  Flags||fedora-review?



--- Comment #2 from Björn "besser82" Esser  ---
mine  ;)

-- 
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 1043290] Review Request: engrampa - MATE Desktop file archiver

2013-12-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1043290



--- Comment #1 from Wolfgang Ulbrich  ---
This is a re-review request for a package rename.
Old packackage name is mate-file-archiver.
Note, this affect only rawhide, f18/19/20 will use mate-window-manager-1.6.x
until there EOL.

-- 
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