[Bug 1358041] Review Request: python3-scipy - Scientific Tools for Python

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1358041



--- Comment #5 from Antonio Trande  ---
Package review:

[x] The package must be named according to the Package Naming Guidelines .
[x] The spec file name must match the base package %{name}, in the format
%{name}.spec unless your package has an exemption.
[x] The package must meet the Packaging Guidelines.
[x] The package must be licensed with a Fedora approved license and meet the
Licensing Guidelines.
[x] The License field in the package spec file must match the actual license.
[-] 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 must be included in %license.
[x] The spec file must be written in American English. [5]
[x] The spec file for the package MUST be legible. [6]
[x] The sources used to build the package must match the upstream source, as
provided in the spec URL. Reviewers should use sha256sum for this task as it is
used by the sources file once imported into git. If no upstream URL can be
specified for this package, please see the Source URL Guidelines for how to
deal with this.

[x] The package MUST successfully compile and build into binary rpms on at
least one primary architecture.
[-] If the package does not successfully compile, build or work on an
architecture, then those architectures should be listed in the spec in
ExcludeArch. Each architecture listed in ExcludeArch MUST have a bug filed in
bugzilla, describing the reason that the package does not compile/build/work on
that architecture. The bug number MUST be placed in a comment, next to the
corresponding ExcludeArch line.

[x] All build dependencies must be listed in BuildRequires, except for any that
are listed in the exceptions section of the Packaging Guidelines ; inclusion of
those as BuildRequires is optional. Apply common sense.

[-] The spec file MUST handle locales properly. This is done by using the
%find_lang macro. Using %{_datadir}/locale/* is strictly forbidden.
[-] Every binary RPM package (or subpackage) which stores shared library files
(not just symlinks) in any of the dynamic linker's default paths, must call
ldconfig in %post and %postun.
[x] Packages must NOT bundle copies of system libraries.
[-] If the package is designed to be relocatable, the packager must state this
fact in the request for review, along with the rationalization for relocation
of that specific package. Without this, use of Prefix: /usr is considered a
blocker.

[x] A package must own all directories that it creates. If it does not create a
directory that it uses, then it should require a package which does create that
directory.
[x] A Fedora package must not list a file more than once in the spec file's
%files listings. (Notable exception: license texts in specific situations)
[!] Permissions on files must be set properly. Executables should be set with
executable permissions, for example.
[x] Each package must consistently use macros.
[x] The package must contain code, or permissible content.
[-] Large documentation files must go in a -doc subpackage. (The definition of
large is left up to the packager's best judgement, but is not restricted to
size. Large can refer to either size or quantity)

[-] If a package includes something as %doc, it must not affect the runtime of
the application. To summarize: If it is in %doc, the program must run properly
if it is not present.
[x] Static libraries must be in a -static package.
[-] Development files must be in a -devel package.
   In the vast majority of cases, devel packages must require the base package
using a fully versioned dependency: Requires: %{name}%{?_isa} =
%{version}-%{release}

[-] Packages must NOT contain any .la libtool archives, these must be removed
in the spec if they are built.
[-] Packages containing GUI applications must include a %{name}.desktop file,
and that file must be properly installed with desktop-file-install in the
%install section. If you feel that your packaged GUI application does not need
a .desktop file, you must put a comment in the spec file with your explanation.

[x] Packages must not own files or directories already owned by other packages.
The rule of thumb here is that the first package to be installed should own the
files or directories that other packages may rely upon. This means, for
example, that no package in Fedora should ever share ownership with any of the
files or directories owned by the filesystem or man package. If you feel that
you have a good reason to own a file or directory that another package owns,
then please present that at package review time.

[x] All filenames in rpm packages must be valid UTF-8.

[-] If the source package does not include license text(s) as a separate file
from upstream, the packager SHOULD query upstream to include it.
[-] The description and summary sections in the package spec file should
contain translations for supported Non-English languages, if 

[Bug 1358041] Review Request: python3-scipy - Scientific Tools for Python

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1358041



--- Comment #6 from Antonio Trande  ---
Created attachment 1183062
  --> https://bugzilla.redhat.com/attachment.cgi?id=1183062&action=edit
rpmlint output of python3-scipy

-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1358041] Review Request: python3-scipy - Scientific Tools for Python

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1358041



--- Comment #7 from Antonio Trande  ---
Created attachment 1183063
  --> https://bugzilla.redhat.com/attachment.cgi?id=1183063&action=edit
rpmlint output of python3-scipy-debuginfo

-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1356739] Review Request: zulucrypt - Qt GUI front end to cryptsetup

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1356739



--- Comment #13 from Raphael Groner  ---
(In reply to gil cattaneo from comment #9)
> report this notice
> 
> /*
>  * This routine is part of libc.  We include it nevertheless,
>  * since the libc version has some security flaws.
>  *
>  * TODO: use canonicalize_file_name() when exist in glibc
>  */
> zuluCrypt-5.0.0/zuluCrypt-cli/lib/canonicalize/canonicalize.c
> 
> 
> zuluCrypt-5.0.0/zuluCrypt-cli/lib/canonicalize/canonicalize.h

https://github.com/mhogomchungu/zuluCrypt/issues/41

-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1356739] Review Request: zulucrypt - Qt GUI front end to cryptsetup

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1356739



--- Comment #14 from gil cattaneo  ---
(In reply to Raphael Groner from comment #13)
> (In reply to gil cattaneo from comment #9)
> > report this notice
> > 
> > /*
> >  * This routine is part of libc.  We include it nevertheless,
> >  * since the libc version has some security flaws.
> >  *
> >  * TODO: use canonicalize_file_name() when exist in glibc
> >  */
> > zuluCrypt-5.0.0/zuluCrypt-cli/lib/canonicalize/canonicalize.c
> > 
> > 
> > zuluCrypt-5.0.0/zuluCrypt-cli/lib/canonicalize/canonicalize.h
> 
> https://github.com/mhogomchungu/zuluCrypt/issues/41

perhaps, it would not be enough to add Provides: bundled(glibc) or what is ...
also for this and should update license field

-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1352042] Review Request: perl-Specio - Type constraints and coercions for Perl

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1352042



--- Comment #3 from Fedora Update System  ---
perl-Specio-0.24-2.fc24 has been submitted as an update to Fedora 24.
https://bodhi.fedoraproject.org/updates/FEDORA-2016-81706dac20

-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1352042] Review Request: perl-Specio - Type constraints and coercions for Perl

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1352042

Fedora Update System  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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1359110] Review Request: xreader - Simple document viewer

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1359110



--- Comment #3 from Mario Blättermann  ---
I've fixed the scriptlet paths and some dierectory ownerships. It's somewhat
strange: The caja and nemo packages provides directories for
%{_libdir}/[caja,nemo]/extensions-[2,3].0/ but not for
%{_datadir}/[caja,nemo]/extensions/. Maybe we should add them to those
packages?

New files:
Spec URL: https://mariobl.fedorapeople.org/Review/SPECS/xreader.spec
SRPM URL:
https://mariobl.fedorapeople.org/Review/SRPMS/xreader-1.0.7-2.fc24.src.rpm

-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1354210] Review Request: xviewer - Fast and functional graphics viewer

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1354210

Mario Blättermann  changed:

   What|Removed |Added

 Blocks||1359390




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1359390
[Bug 1359390] Cinnamon X-apps
-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1358699] Review Request: xed - A small and lightweight text editor

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1358699

Mario Blättermann  changed:

   What|Removed |Added

 Blocks||1359390




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1359390
[Bug 1359390] Cinnamon X-apps
-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1359110] Review Request: xreader - Simple document viewer

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1359110

Mario Blättermann  changed:

   What|Removed |Added

 Blocks||1359390




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1359390
[Bug 1359390] Cinnamon X-apps
-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1354210] Review Request: xviewer - Fast and functional graphics viewer

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1354210

Mario Blättermann  changed:

   What|Removed |Added

 Blocks||1359392 (xviewer-plugins)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1359392
[Bug 1359392] Review Request: xviewer-plugins - A collection of plugins for
xviewer
-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1359392] New: Review Request: xviewer-plugins - A collection of plugins for xviewer

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1359392

Bug ID: 1359392
   Summary: Review Request: xviewer-plugins - A collection of
plugins for xviewer
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: mario.blaetterm...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://mariobl.fedorapeople.org/Review/SPECS/xviewer-plugins.spec
SRPM URL:
https://mariobl.fedorapeople.org/Review/SRPMS/xviewer-plugins-1.0.2-1.fc24.src.rpm

Description:
This package contains plugins for additional features in xviewer.

Fedora Account System Username: mariobl

-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1359392] Review Request: xviewer-plugins - A collection of plugins for xviewer

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1359392

Mario Blättermann  changed:

   What|Removed |Added

 Blocks||1359390
 Depends On||1354210 (xviewer)
  Alias||xviewer-plugins




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1354210
[Bug 1354210] Review Request: xviewer - Fast and functional graphics viewer
https://bugzilla.redhat.com/show_bug.cgi?id=1359390
[Bug 1359390] Cinnamon X-apps
-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1332306] Review Request: libcxxabi - Low level support for a standard C++ library

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1332306



--- Comment #3 from Mario Blättermann  ---
%{_libdir}/libc++abi.a is in the -devel package. Shouldn't this go in a
separate -static package?

-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1358699] Review Request: xed - A small and lightweight text editor

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1358699



--- Comment #1 from Mario Blättermann  ---
I've fixed some issues, such as missing scriptlet paths and the license of the
user manual.

Spec URL: https://mariobl.fedorapeople.org/Review/SPECS/xed.spec
SRPM URL:
https://mariobl.fedorapeople.org/Review/SRPMS/xed-1.0.5-3.fc24.src.rpm

-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1358699] Review Request: xed - A small and lightweight text editor

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1358699



--- Comment #2 from Mario Blättermann  ---
New scratch build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=14992112

-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1359397] New: Review Request: libqtelegram-ae - Fork of libqtelegram by Aseman Team

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1359397

Bug ID: 1359397
   Summary: Review Request: libqtelegram-ae - Fork of libqtelegram
by Aseman Team
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: ignate...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
http://copr-dist-git.fedorainfracloud.org/cgit/ignatenkobrain/cutegram/libqtelegram-ae.git/plain/libqtelegram-ae.spec?id=241ba02c8ca7605734f628b20d7732fc4a303f53
SRPM URL:
https://copr-be.cloud.fedoraproject.org/results/ignatenkobrain/cutegram/fedora-rawhide-x86_64/00409656-libqtelegram-ae/libqtelegram-ae-10.0.0-2.fc25.src.rpm
Description: Fork of libqtelegram by Aseman Team.
Fedora Account System Username: ignatenkobrain

-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1359399] Review Request: aseman-qt-tools - Shared tools and functions , used in the aseman's projects

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1359399

Igor Gnatenko  changed:

   What|Removed |Added

  Alias||aseman-qt-tools



-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1359399] New: Review Request: aseman-qt-tools - Shared tools and functions, used in the aseman' s projects

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1359399

Bug ID: 1359399
   Summary: Review Request: aseman-qt-tools - Shared tools and
functions, used in the aseman's projects
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: ignate...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
http://copr-dist-git.fedorainfracloud.org/cgit/ignatenkobrain/cutegram/aseman-qt-tools.git/plain/aseman-qt-tools.spec?id=2894ffaa6043a2f509e4c35da2f8e97c9e3c8ab6
SRPM URL:
https://copr-be.cloud.fedoraproject.org/results/ignatenkobrain/cutegram/fedora-rawhide-x86_64/00410364-aseman-qt-tools/aseman-qt-tools-1.0.0-2.fc25.src.rpm
Description: Shared tools and functions, used in the aseman's projects.
Fedora Account System Username: ignatenkobrain

-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1359400] New: Review Request: telegramqml - Telegram API tools for QtQml and Qml

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1359400

Bug ID: 1359400
   Summary: Review Request: telegramqml - Telegram API tools for
QtQml and Qml
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: ignate...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
http://copr-dist-git.fedorainfracloud.org/cgit/ignatenkobrain/cutegram/telegramqml.git/plain/telegramqml.spec?id=81803045af1bd1ac3a8de997350834b11962840b
SRPM URL:
https://copr-be.cloud.fedoraproject.org/results/ignatenkobrain/cutegram/fedora-rawhide-x86_64/00410323-telegramqml/telegramqml-2.0.0-1.fc25.src.rpm
Description: Telegram API tools for QtQml and Qml.
Fedora Account System Username: ignatenkobrain

-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1359400] Review Request: telegramqml - Telegram API tools for QtQml and Qml

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1359400

Igor Gnatenko  changed:

   What|Removed |Added

 CC||he...@kde.org



--- Comment #1 from Igor Gnatenko  ---
*** Bug 1246762 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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1246762] Review Request: telegramqml - Qt Qml qtelegram wrapper library

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1246762

Igor Gnatenko  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 CC||ignate...@redhat.com
 Resolution|--- |DUPLICATE
Last Closed||2016-07-23 14:28:21



--- Comment #4 from Igor Gnatenko  ---


*** This bug has been marked as a duplicate of bug 1359400 ***

-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1246762] Review Request: telegramqml - Qt Qml qtelegram wrapper library

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1246762

Igor Gnatenko  changed:

   What|Removed |Added

  Alias|telegramqml |



-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1359400] Review Request: telegramqml - Telegram API tools for QtQml and Qml

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1359400

Igor Gnatenko  changed:

   What|Removed |Added

  Alias||telegramqml



-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1359400] Review Request: telegramqml - Telegram API tools for QtQml and Qml

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1359400

Igor Gnatenko  changed:

   What|Removed |Added

 Depends On||1246761 (libqtelegram-ae)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1246761
[Bug 1246761] Review Request: libqtelegram-ae - Qt Telegram API wrapper
library
-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1246761] Review Request: libqtelegram-ae - Qt Telegram API wrapper library

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1246761

Igor Gnatenko  changed:

   What|Removed |Added

 Blocks||1359400 (telegramqml)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1359400
[Bug 1359400] Review Request: telegramqml - Telegram API tools for QtQml
and Qml
-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1359401] Review Request: cutegram - Cutegram is a telegram client by Aseman Land

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1359401

Igor Gnatenko  changed:

   What|Removed |Added

 CC||he...@kde.org



--- Comment #1 from Igor Gnatenko  ---
*** Bug 1246763 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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1359401] New: Review Request: cutegram - Cutegram is a telegram client by Aseman Land

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1359401

Bug ID: 1359401
   Summary: Review Request: cutegram - Cutegram is a telegram
client by Aseman Land
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: ignate...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
http://copr-dist-git.fedorainfracloud.org/cgit/ignatenkobrain/cutegram/cutegram.git/plain/cutegram.spec?id=477d8c4832abccfb753c60c7054099b500e6f100
SRPM URL:
https://copr-be.cloud.fedoraproject.org/results/ignatenkobrain/cutegram/fedora-rawhide-x86_64/00410486-cutegram/cutegram-3.0-0.3git7294861.fc25.src.rpm
Description:
A different telegram client from Aseman team. Cutegram forked from Sigram
by Sialan Labs. Cutegram project are released under the terms of the GPLv3
license.
Fedora Account System Username: ignatenkobrain

-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1246763] Review Request: cutegram - A Qt Qml based Telegram client

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1246763

Igor Gnatenko  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 CC||ignate...@redhat.com
 Resolution|--- |DUPLICATE
  Alias|cutegram|
Last Closed||2016-07-23 14:31:24



--- Comment #5 from Igor Gnatenko  ---


*** This bug has been marked as a duplicate of bug 1359401 ***

-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1359401] Review Request: cutegram - Cutegram is a telegram client by Aseman Land

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1359401

Igor Gnatenko  changed:

   What|Removed |Added

  Alias||cutegram



-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1359400] Review Request: telegramqml - Telegram API tools for QtQml and Qml

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1359400

Igor Gnatenko  changed:

   What|Removed |Added

 Blocks||1359401 (cutegram)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1359401
[Bug 1359401] Review Request: cutegram - Cutegram is a telegram client by
Aseman Land
-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1359399] Review Request: aseman-qt-tools - Shared tools and functions , used in the aseman's projects

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1359399

Igor Gnatenko  changed:

   What|Removed |Added

 Blocks||1359401 (cutegram)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1359401
[Bug 1359401] Review Request: cutegram - Cutegram is a telegram client by
Aseman Land
-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1359401] Review Request: cutegram - Cutegram is a telegram client by Aseman Land

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1359401

Igor Gnatenko  changed:

   What|Removed |Added

 Depends On||1359400 (telegramqml),
   ||1359399 (aseman-qt-tools)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1359399
[Bug 1359399] Review Request: aseman-qt-tools - Shared tools and functions,
used in the aseman's projects
https://bugzilla.redhat.com/show_bug.cgi?id=1359400
[Bug 1359400] Review Request: telegramqml - Telegram API tools for QtQml
and Qml
-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1359402] New: Review Request: coot - crystallographic macromolecular building toolkit (unretire request )

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1359402

Bug ID: 1359402
   Summary: Review Request: coot - crystallographic macromolecular
building toolkit (unretire request)
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: tim.f...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://drive.google.com/open?id=0BxOz1eBnQT3najl1NDBoVkc2ZUk
SRPM URL: https://drive.google.com/open?id=0BxOz1eBnQT3nTHRGcWFCMm5yMzA
Description: This is a request to unretire coot for rawhide/F24 (there was an
error on compilation that needed fixing, it just took me some time to track it
down and add the patch - upstream has also been notified and future releases
include the patch).  Original review here:
https://bugzilla.redhat.com/show_bug.cgi?id=472150

Coot is a toolkit for Macromolecular Crystallography and model-building.  Coot
uses OpenGL, together with a new approach to map contouring and
importing/creation and other modelling and building operations.  It displays
maps and models and allows model manipulations such as idealization, real space
refinement, manual rotation/translation, rigid-body fitting, ligand search,
solvation, mutations, rotamers, Ramachandran plots...

Also see: http://www2.mrc-lmb.cam.ac.uk/personal/pemsley/coot/

Fedora Account System Username: timfenn

-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1353343] Review Request: qmc2 - M.A.M.E. Catalog / Launcher II

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1353343



--- Comment #14 from Fedora Update System  ---
qmc2-0.65-4.fc24 has been pushed to the Fedora 24 stable repository. If
problems still persist, please make note of it in this bug report.

-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1280054] Review Request: nodejs-array-reduce - `[].reduce() ` for old browsers

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1280054

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed|2016-07-01 21:00:14 |2016-07-23 14:49:45



-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1280054] Review Request: nodejs-array-reduce - `[].reduce() ` for old browsers

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1280054



--- Comment #6 from Fedora Update System  ---
nodejs-array-reduce-0.0.0-1.fc24 has been pushed to the Fedora 24 stable
repository. If problems still persist, please make note of it in this bug
report.

-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1279981] Review Request: nodejs-array-filter - Array#filter for older browsers

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1279981

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed|2016-07-03 08:53:29 |2016-07-23 14:49:53



-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1279981] Review Request: nodejs-array-filter - Array#filter for older browsers

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1279981



--- Comment #6 from Fedora Update System  ---
nodejs-array-filter-1.0.0-1.fc24 has been pushed to the Fedora 24 stable
repository. If problems still persist, please make note of it in this bug
report.

-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1273134] Review Request: nodejs-tmatch - acilitate the `t.match() ` method in `tap`

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1273134



--- Comment #10 from Fedora Update System  ---
nodejs-tmatch-2.0.1-1.fc24 has been pushed to the Fedora 24 stable repository.
If problems still persist, please make note of it in this bug report.

-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1273134] Review Request: nodejs-tmatch - acilitate the `t.match() ` method in `tap`

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1273134

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed|2016-07-01 20:13:01 |2016-07-23 14:50:02



-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1359399] Review Request: aseman-qt-tools - Shared tools and functions , used in the aseman's projects

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1359399

Christian Dersch  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||lupi...@mailbox.org
   Assignee|nob...@fedoraproject.org|lupi...@mailbox.org
  Flags||fedora-review?



-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1246761] Review Request: libqtelegram-ae - Qt Telegram API wrapper library

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1246761



--- Comment #17 from Fedora Update System  ---
libqtelegram-ae-10.0.0-2.fc23 has been submitted as an update to Fedora 23.
https://bodhi.fedoraproject.org/updates/FEDORA-2016-b83952c941

-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1359399] Review Request: aseman-qt-tools - Shared tools and functions , used in the aseman's projects

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1359399

Christian Dersch  changed:

   What|Removed |Added

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



--- Comment #1 from Christian Dersch  ---
Looks nice => Approved!

One point you should have a look at: The permissions in debuginfo reported by
rpmlint, maybe permissions of sources are weird.


Package Review
==

Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed


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

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: "Apache (v2.0)", "LGPL (v2.1)", "LGPL (v3 or later)", "GPL (v3
 or later)", "Unknown or generated". 186 files have unknown license.
 Detailed output of licensecheck in /home/review/1359399-aseman-qt-
 tools/licensecheck.txt
[x]: License file installed when any subpackage combination is installed.
[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.
[-]: 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.
[x]: 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]: 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 %license.
[x]: Package requires other packages for directories it uses.
[x]: Package must own all directories that it creates.
[x]: Package does not own files or directories owned by other packages.
[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]: Dist tag is present.
[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 does not use a name that already exists.
[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]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 0 bytes in 0 files.
[x]: Packages must not store files under /srv, /opt or /usr/local

= SHOULD items =

Generic:
[-]: 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.
 Note: No Requires: %{name}%{?_isa} = %{version}-%{release} in aseman-
 qt-tools-debuginfo
[x]: Package functions as described.
[x]: Latest version is packaged.
[x]: Package does not include license text files separate from upstream.
[x]: Patches link to upstream bugs/comments/lists

[Bug 1246761] Review Request: libqtelegram-ae - Qt Telegram API wrapper library

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1246761



--- Comment #18 from Fedora Update System  ---
libqtelegram-ae-10.0.0-2.fc24 has been submitted as an update to Fedora 24.
https://bodhi.fedoraproject.org/updates/FEDORA-2016-f124196a0d

-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1359400] Review Request: telegramqml - Telegram API tools for QtQml and Qml

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1359400

Christian Dersch  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||lupi...@mailbox.org
   Assignee|nob...@fedoraproject.org|lupi...@mailbox.org
  Flags||fedora-review?



-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1270405] Review Request: native_client - Google Native Client Toolchain

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1270405



--- Comment #12 from Zbigniew Jędrzejewski-Szmek  ---
Created attachment 1183147
  --> https://bugzilla.redhat.com/attachment.cgi?id=1183147&action=edit
log from failed build in fedora-24-x86_64 mock

Seems to fail in the same way:

Running: subprocess.check_call(['/usr/bin/arm-linux-gnu-gcc',
'-mcpu=cortex-a9', '-D__arm_nonsfi_linux__', '-I/usr/arm-linux-gnu/include/',
'-O2', '-Wall', '-Werror', '-I../../../..', '-DNACL_LINUX=1', '-DDEFINE_MAIN',
'-c', '../../../src/nonsfi/irt/irt_interfaces.c', '-o',
'../unsandboxed_runtime_arm_linux_install/translator/arm-linux/lib/unsandboxed_irt.o'],
cwd='/builddir/build/BUILD/native_client/toolchain_build/out/unsandboxed_runtime_arm_linux_work')
../../../src/nonsfi/irt/irt_interfaces.c:7:20: fatal error: assert.h: No such
file or directory
 #include 
^
compilation terminated.
Error building unsandboxed_runtime_arm_linux: Command
'['/usr/bin/arm-linux-gnu-gcc', '-mcpu=cortex-a9', '-D__arm_nonsfi_linux__',
'-I/usr/arm-linux-gnu/include/', '-O2', '-Wall', '-Werror', '-I../../../..',
'-DNACL_LINUX=1', '-DDEFINE_MAIN', '-c',
'../../../src/nonsfi/irt/irt_interfaces.c', '-o',
'../unsandboxed_runtime_arm_linux_install/translator/arm-linux/lib/unsandboxed_irt.o']'
returned non-zero exit status 1

-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1359401] Review Request: cutegram - Cutegram is a telegram client by Aseman Land

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1359401



--- Comment #2 from Igor Gnatenko  ---
SPEC:
http://copr-dist-git.fedorainfracloud.org/cgit/ignatenkobrain/cutegram/cutegram.git/plain/cutegram.spec?id=18d4250bdb4d7612bbfc9ae2b762ac5ac89dbde7
SRPM:
https://copr-be.cloud.fedoraproject.org/results/ignatenkobrain/cutegram/fedora-rawhide-x86_64/00410657-cutegram/cutegram-3.0-0.4git7294861.fc25.src.rpm

Added missing Requires

-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1353343] Review Request: qmc2 - M.A.M.E. Catalog / Launcher II

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1353343



--- Comment #15 from Fedora Update System  ---
qmc2-0.65-4.fc23 has been pushed to the Fedora 23 stable repository. If
problems still persist, please make note of it in this bug report.

-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1353343] Review Request: qmc2 - M.A.M.E. Catalog / Launcher II

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1353343

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2016-07-23 16:49:05



-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1356739] Review Request: zulucrypt - Qt GUI front end to cryptsetup

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1356739

Raphael Groner  changed:

   What|Removed |Added

 Attachment|0   |1
#1180782 is||
   obsolete||



-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1356739] Review Request: zulucrypt - Qt GUI front end to cryptsetup

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1356739

Raphael Groner  changed:

   What|Removed |Added

 Whiteboard||NotReady



--- Comment #15 from Raphael Groner  ---
(In reply to gil cattaneo from comment #14)
…
> > https://github.com/mhogomchungu/zuluCrypt/issues/41
> 
> perhaps, it would not be enough to add Provides: bundled(glibc) or what is
> ...
> also for this and should update license field

There's a patch now to use external glibc function. I'll apply it.

(In reply to Raphael Groner from comment #12)
> Created attachment 1180782 [details]
> unbundle lxqtwallet
> 
> Unbundle lxqt-wallet.
> 
> Patch got from upstream.

This patch is incomplete. I wrote an e-mail to upstream to get it working with
'rm -r zuluCrypt-gui/lxqt_wallet'.

(In reply to Raphael Groner from comment #11)
…
> 1. It is currently not possible to use upstream version of tcplay with
> zuluCrypt. Upstream seems to have lost interest in the project and i
> have extended my "personal" copy of the library in incompatible ways.
> For example,upstream version does not have this[2] pull request but my
> copy has it.My copy extended this pull request to add unlocking
> VeraCrypt volumes that uses a PIM value.A bunch of APIs like this[3]
> one are also not in the upstream version. Upstream seems uninterested
> in the project and i have not attempted to offer them upstream because
> of it.

We've to discuss that with the maintainer of the tcplay package. Maybe we can
add the patches and fully unbundle from the zuluCrypt package.

> 2. It should be possible to cleanly unbundle "lxqt_wallet" since
> zuluCrypt only uses lxqt_wallet documented API.Will add a config
> option shortly to build using an external version.

See above.

> 3. If you dont want to have zulu*-cli CLI components as suid-root
> binaries(offers best user experience in my opinion),look at what
> debian[4] is doing to have the second best user experience. With the
> "debian way",a user clicks an icon and they get a polkit prompt to
> root authenticate before they start the application with root's
> privileges but they still wont be able to do everything since the user
> that
> invokes the application is remembered and the application will do only
> what that user is allowed to do.This is done through pkexec[5].

IMHO that should be handled in another package review for zulucrypt-pkexec (or
the like) because it's a new upstream.

-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1359412] New: Review Request: gawkextlib - library providing support functions for gawk extension libraries

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1359412

Bug ID: 1359412
   Summary: Review Request: gawkextlib - library providing support
functions for gawk extension libraries
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Assignee: nob...@fedoraproject.org
  Reporter: ajsch...@alumni.princeton.edu
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: http://sourceforge.net/projects/gawkextlib/files/rpms/gawkextlib.spec
SRPM URL:
http://sourceforge.net/projects/gawkextlib/files/rpms/gawkextlib-1.0.2-1.fc24.src.rpm
Description: gawkextlib is a library providing helpful support functions for
gawk extension libraries.
Fedora Account System Username: ajschorr

This is my first package, and I am seeking a sponsor. I am the upstream
maintainer of the gawkextlib project. This project provides a number of
extension libraries for gawk, including XML, postgresql, lmdb, and select I/O
multiplexing.

The koji build is here:
http://koji.fedoraproject.org/koji/taskinfo?taskID=14994986

-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1359412] Review Request: gawkextlib - library providing support functions for gawk extension libraries

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1359412

Andrew Schorr  changed:

   What|Removed |Added

 Blocks||177841 (FE-NEEDSPONSOR)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=177841
[Bug 177841] Tracker: Review requests from new Fedora packagers who need a
sponsor
-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1359416] New: Review Request: gawk-xml - XML support for gawk

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1359416

Bug ID: 1359416
   Summary: Review Request: gawk-xml - XML support for gawk
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Assignee: nob...@fedoraproject.org
  Reporter: ajsch...@alumni.princeton.edu
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: http://sourceforge.net/projects/gawkextlib/files/rpms/gawk-xml.spec
SRPM URL:
http://sourceforge.net/projects/gawkextlib/files/rpms/gawk-xml-1.0.4-1.fc24.src.rpm
Description: The gawk-xml package contains the gawk XML shared library
extension, as well as the xmlgawk script and some awk include libraries for
enhanced XML processing.
Fedora Account System Username: ajschorr

This is my first package, and I need a sponsor. I am the upstream package
maintainer. This is a shared library gawk extension that uses the expat library
to enable gawk to process XML files.

The koji build is here:
http://koji.fedoraproject.org/koji/taskinfo?taskID=14995194
It currently fails because it requires the gawkextlib package that I just
submitted in bugzilla # 1359412

-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1359416] Review Request: gawk-xml - XML support for gawk

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1359416

Andrew Schorr  changed:

   What|Removed |Added

 Blocks||177841 (FE-NEEDSPONSOR)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=177841
[Bug 177841] Tracker: Review requests from new Fedora packagers who need a
sponsor
-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1352042] Review Request: perl-Specio - Type constraints and coercions for Perl

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1352042

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #4 from Fedora Update System  ---
perl-Specio-0.24-2.fc24 has been pushed to the Fedora 24 testing repository. If
problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2016-81706dac20

-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1352175] Review Request: gap-pkg-float - GAP access to mpfr, mpfi, mpc, fplll and cxsc

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1352175

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #6 from Fedora Update System  ---
gap-pkg-float-0.7.4-1.fc24 has been pushed to the Fedora 24 testing repository.
If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2016-49d0bb5c71

-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1131019] Review Request: lejos-nxj - Run Java on the Lego NXT

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1131019

Omar Berroteran  changed:

   What|Removed |Added

 CC||omarberroteran...@gmail.com



--- Comment #4 from Omar Berroteran  ---
The build fails in RawHide and Fedora 24.

-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1359426] New: Review Request: nodejs-clap - Command line argument parser

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1359426

Bug ID: 1359426
   Summary: Review Request: nodejs-clap - Command line argument
parser
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: jsmith.fed...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://jsmith.fedorapeople.org/Packaging/nodejs-clap/nodejs-clap.spec
SRPM URL:
https://jsmith.fedorapeople.org/Packaging/nodejs-clap/nodejs-clap-1.1.1-1.fc24.src.rpm
Description: Command line argument parser
Fedora Account System Username: jsmith

-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1359426] Review Request: nodejs-clap - Command line argument parser

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1359426

Jared Smith  changed:

   What|Removed |Added

 Blocks||956806 (nodejs-reviews),
   ||1269538 (IoT), 1031982




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=956806
[Bug 956806] Node.js Review Tracker
https://bugzilla.redhat.com/show_bug.cgi?id=1031982
[Bug 1031982] nodejs-tap-6.2.0 is available
https://bugzilla.redhat.com/show_bug.cgi?id=1269538
[Bug 1269538] Tracker for IoT on Fedora
-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1359473] New: Review Request: legendsbrowser - Java-based legends viewer for Dwarf Fortress

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1359473

Bug ID: 1359473
   Summary: Review Request: legendsbrowser - Java-based legends
viewer for Dwarf Fortress
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: rosser@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://www.acm.jhu.edu/~bjr/fedora/dwarffortress/fedora/legendsbrowser.spec
SRPM URL:
https://www.acm.jhu.edu/~bjr/fedora/dwarffortress/fedora/legendsbrowser-1.0.12-1.fc23.src.rpm

Description:
Legends Browser is an multi-platform, open source, java-based legends viewer
for Dwarf Fortress. It works in the browser of your choice and recreates
Legends
Mode, with objects accessible as links. Several statistics and overviews are
added.

Fedora Account System Username: tc01

rpmlint output:

$ rpmlint ../RPMS/noarch/legendsbrowser-1.0.12-1.fc23.noarch.rpm 
legendsbrowser.noarch: W: spelling-error %description -l en_US multi -> mulch,
mufti
legendsbrowser.noarch: W: no-manual-page-for-binary legendsbrowser

$ rpmlint ../RPMS/noarch/legendsbrowser-javadoc-1.0.12-1.fc23.noarch.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

$ rpmlint ../SRPMS/legendsbrowser-1.0.12-1.fc23.src.rpm 
legendsbrowser.src: W: spelling-error %description -l en_US multi -> mulch,
mufti

-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1157255] Review Request: ufoai - UFO: Alien Invasion strategy game

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1157255

Ben Rosser  changed:

   What|Removed |Added

 CC||rosser@gmail.com



--- Comment #17 from Ben Rosser  ---
It looks like the font issue has been resolved-- does that mean this review can
proceed? Are you in need of a reviewer?

-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1176277] Review Request: Shinobi - Meta build system for Ninja

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1176277
Bug 1176277 depends on bug 1176420, which changed state.

Bug 1176420 Summary: RFE: Help should be patched to use right command name
https://bugzilla.redhat.com/show_bug.cgi?id=1176420

   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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1324267] Review Request: python-keyutils - Python3 bindings for the keyutils package

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1324267



--- Comment #2 from Ben Boeckel  ---
Spec URL:
http://mathstuf.fedorapeople.org/packaging/python-keyutils/python-keyutils.spec
SRPM URL:
http://mathstuf.fedorapeople.org/packaging/python-keyutils/python-keyutils-0.3-1.fc25.src.rpm

OK, vhost was broken on that server and admins haven't responded yet.

-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1245022] Review Request: ghc-base-compat - A compatibility layer for base

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1245022



--- Comment #4 from Ben Boeckel  ---
Spec URL:
https://mathstuf.fedorapeople.org/packaging/ghc-base-compat/ghc-base-compat.spec
SRPM URL:
https://mathstuf.fedorapeople.org/packaging/ghc-base-compat/ghc-base-compat-0.9.1-1.fc25.src.rpm

-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1245021] Review Request: ghc-mtl-compat - Backported Control.Monad.Except module from mtl

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1245021



--- Comment #3 from Ben Boeckel  ---
Spec URL:
https://mathstuf.fedorapeople.org/packaging/ghc-mtl-compat/ghc-mtl-compat.spec
SRPM URL:
https://mathstuf.fedorapeople.org/packaging/ghc-mtl-compat/ghc-mtl-compat-0.2.1.3-1.fc25.src.rpm

-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1256057] Review Request: ghc-time-locale-compat - Compatibility of TimeLocale between old-locale and time-1.5

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1256057



--- Comment #5 from Ben Boeckel  ---
Spec URL:
https://mathstuf.fedorapeople.org/packaging/ghc-time-locale-compat/ghc-time-locale-compat.spec
SRPM URL:
https://mathstuf.fedorapeople.org/packaging/ghc-time-locale-compat/ghc-time-locale-compat-0.1.1.3-2.fc25.src.rpm

-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1256091] Review Request: ghc-http-conduit - HTTP client package with conduit interface and HTTPS support

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1256091



--- Comment #3 from Ben Boeckel  ---
Spec URL:
https://mathstuf.fedorapeople.org/packaging/ghc-http-conduit/ghc-http-conduit.spec
SRPM URL:
https://mathstuf.fedorapeople.org/packaging/ghc-http-conduit/ghc-http-conduit-2.1.10-1.fc25.src.rpm

-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1256046] Review Request: eventd - A simple daemon to track remote or local events and do actions

2016-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1256046



--- Comment #14 from Ben Boeckel  ---
Spec URL: https://mathstuf.fedorapeople.org/packaging/eventd/eventd.spec
SRPM URL:
https://mathstuf.fedorapeople.org/packaging/eventd/eventd-0.19.1-1.fc25.src.rpm

-- 
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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org