[Bug 1287794] Review Request: archaius - Library for configuration management API

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1287794



--- Comment #17 from Fedora Update System  ---
archaius-0.7.3-3.fc23 has been submitted as an update to Fedora 23.
https://bodhi.fedoraproject.org/updates/FEDORA-2015-3ae4cc39c5

-- 
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 1287794] Review Request: archaius - Library for configuration management API

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1287794

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

[Bug 1288100] Review Request: console-image-viewer - Terminal image viewer

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1288100



--- Comment #9 from Fedora Update System  ---
console-image-viewer-1.2-1.fc22 has been submitted as an update to Fedora 22.
https://bodhi.fedoraproject.org/updates/FEDORA-2015-cb16dbf919

-- 
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 1288100] Review Request: console-image-viewer - Terminal image viewer

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1288100



--- Comment #8 from Fedora Update System  ---
console-image-viewer-1.2-1.fc23 has been submitted as an update to Fedora 23.
https://bodhi.fedoraproject.org/updates/FEDORA-2015-11f979ec71

-- 
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 1273188] Review Request: u2f-hidraw-policy - Udev rule to allow desktop access to HIDRAW U2F tokens

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1273188



--- Comment #26 from Andrew Morton  ---
Wrapping up my end of things...

Andy confirmed that my gnubby key is an old one which doesn't fully implement
the U2F HID spec.   Here's his fix (which worked for me) (thanks)



Like this, but double-check the vendor and product.

# /etc/udev/rules.d/70-gnubby.rules

ACTION!="add|change", GOTO="gnubby_end"

ATTRS{idVendor}=="1050", ATTRS{idProduct}=="0211", ENV{ID_SECURITY_TOKEN}="1"

LABEL="gnubby_end"

-- 
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 1240116] Review Request: python-music21 - A toolkit for computational musicology

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1240116



--- Comment #22 from Upstream Release Monitoring 
 ---
zbyszek's scratch build of python-music21-2.1.2-2.fc24.src.rpm for rawhide
completed http://koji.fedoraproject.org/koji/taskinfo?taskID=12064308

-- 
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 1240116] Review Request: python-music21 - A toolkit for computational musicology

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1240116



--- Comment #21 from Zbigniew Jędrzejewski-Szmek  ---
Thanks for taking on the review!

(In reply to William Moreno from comment #20)
> Package Review
> ==
> 
> 1. The doc and common subpackage both ship the license text, I will recomend
> to include the license text in the common packages and include a weak
> dependencie in doc:
> 
> Suggest: python-music21-common
I don't think this is allowed. The license file MUST be installed with any
subcombination of packages.

> Also use weak dependencies in python2 and 3 subpackage to suggest the doc
> subpackage.
Good idea, done.

> 2. The common subpackage include the python2.7 files than are linked to the
> python3 subpackage. As part of the move of Fedora to python3 by default I
> will recomend to split the common subpackage to ship the python3.5 files.
> Currently the common subpackage will requires python2 as the owner of the
> /usr/lib/python2.7/site-packages/ directory so python3-music21 will require
> python2 to install python-music21-common.
I moved them.

> There are many warning about symlinks in both packages, also please check
> this issue. 
I think that the symlinks are fine. rpmlint does not understand symlinks.

> The best aproach to this common content is to place it in /usr/share but
> this will require some mayor patching to the code and this will require to
> work with upstream.
I could move the files in common to /usr/share and provide symlinks from
both subpackages. I don't know if this would be better.

> 3. Include in python2 and 3 %files
> 
> %exclude %{python2_sitelib}/music21/LICENSE
> The license file is in the common subpackage.

Done.

> 3. Consider to include the test directory in both packages into a devel
> subpackage (python2-name-devel and python3-name-devel), most user will not
> run the test locally and this will let then get a smaller package to
> download.
>
> 4. Also both packages have a demos directory than do not look to be necesary
> in run time and can be packaged as subpackages, you can use weak depencies
> to suggest the user to install the demos subpackages.
I don't think splitting it up like this makes sense. It is essentially
a user program (using python as its interface), not a library that is
installed as a dependency for other things. Demos can be used as documentation
and are useful when using the package. They are not too big too.

> 5. Package is not known to require an ExcludeArch tag. Builds fails in
> secondary arches, but looks like there is a issue with the packaging root.
Hm, I'll try to rebuild.

> RPMLint:
> 1: python2-music21.noarch: E: explicit-lib-dependency python-matplotlib Not
> need work, it is a rpmlint issue
I actually submitted a PR for that, but it doesn't seem like it will  be
merged:
https://github.com/rpm-software-management/rpmlint/pull/58

> 2: python3-music21.noarch: W: dangling-symlink
> /usr/lib/python3.4/site-packages/music21/corpus/cpebach
> /usr/lib/python2.7/site-packages/music21/corpus/cpebach If you change the
> common to python3 files this warning will be fixed but you will need to
> check if the link in python2 works fine.

> 3: python3-music21.noarch: E: non-executable-script
> /usr/lib/python3.4/site-packages/music21/alpha/trecento/exceldiff.py 644
> Sedout the shebang of this files 
Done.

> 4: python-music21-common.noarch: E: incorrect-fsf-address
> /usr/lib/python2.7/site-packages/music21/corpus/josquin/laPlusDesPlus.abc
> Request upstream to fix it or propose a patch to update it.
https://github.com/cuthbertLab/music21/pull/122

> 5: python-music21-doc.noarch: W: wrong-file-end-of-line-encoding
> /usr/share/doc/python-music21/html/_sources/tutorials/advancedGraphing.txt
> See: https://fedoraproject.org/wiki/Packaging_tricks#Remove_DOS_line_endings
Done.

I need to update to 2.2.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://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1176595] Review Request: hypre - High performance matrix preconditioners

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1176595

Zbigniew Jędrzejewski-Szmek  changed:

   What|Removed |Added

 CC||zbys...@in.waw.pl



--- Comment #16 from Zbigniew Jędrzejewski-Szmek  ---
Use %autosetup please with so many patches.

A single Requires/Provides per line.

%doc is used for some license files.

.so files would usually be named like libHYPRE.so.2.10.1 not
libHYPRE-2.10.1.so. With the current naming, the number is part of the file
name. I don't really know what the effect of this is. It will probably break
linking against the library... Do you have any programs using the headers and
libraries in -devel?

It seem that at least it breaks automatic MPI provides/requires generation:
$ rpm -qp -R results/hypre-openmpi-2.10.1-1.fc24.x86_64.rpm
openmpi(x86-64)
rpmlib(CompressedFileNames) <= 3.0.4-1
rpmlib(FileDigests) <= 4.6.0-1
rpmlib(PayloadFilesHavePrefix) <= 4.0-1
rpmlib(PayloadIsXz) <= 5.2-1

as you can see, there are no requirements on libraries.

-- 
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 1179484] Review Request: python-cram - Simple testing framework for command line applications

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1179484

Zbigniew Jędrzejewski-Szmek  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||zbys...@in.waw.pl
   Assignee|nob...@fedoraproject.org|zbys...@in.waw.pl



--- Comment #11 from Zbigniew Jędrzejewski-Szmek  ---
- Dist tag is not present → add %{?dist} to Release.

You should convert to the new python macros %py2_build, %py2_install,
%py3_build, %py3_install, and build python2- and python3- subpackages
[https://fedoraproject.org/wiki/Packaging:Python#Example_common_spec_file].

The guidelines say you should provide both cram-2.7 and cram-2, and cram-3.5
and cram-3
[https://fedoraproject.org/wiki/Packaging:Python#Executables_in_.2Fusr.2Fbin].

%{py3dir} is also deprecated. Use a subdirectory instead.

python3-cram.noarch: W: summary-ended-with-dot C Utilities and processors for
the Python Imaging Libary.

The python 2 version of the script should use /usr/bin/python2 (i.e.
%{__python2}), not /usr/bin/python in the header.

-- 
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 1181317] Review Request: python-itools - The itools library

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1181317



--- Comment #6 from Zbigniew Jędrzejewski-Szmek  ---
Hm, I lost track of this ticket. I can review it, but there are some things to
fix:

Python packaging guidelines have been nicely simplified. You should now use
%py2_build and %py2_install macros, see
https://fedoraproject.org/wiki/Packaging:Python.

Summary is useless, 'cause it just duplicates the name. Please succinctly
summarize what the package does. Also the %description is a bit unclear: it has
something with various formats, but what does the package actually *do*?

Use %version in %prep.

-- 
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 1185665] Review Request: muttprint - pretty printing of mails for mutt (and other mailers)

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1185665

Zbigniew Jędrzejewski-Szmek  changed:

   What|Removed |Added

 CC||zbys...@in.waw.pl



--- Comment #4 from Zbigniew Jędrzejewski-Szmek  ---
Ping?

-- 
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 1268083] Review Request: python-maxminddb - Reader for the MaxMind DB format

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1268083

Zbigniew Jędrzejewski-Szmek  changed:

   What|Removed |Added

 CC||zbys...@in.waw.pl



--- Comment #1 from Zbigniew Jędrzejewski-Szmek  ---
What is the minddb format? Can you add an explanation to the description?

What does it mean that the C extension is optional? If it is always built in
the Fedora package, for Fedora users it is not "optional", and this part of the
description can be removed.

You can use %{summary} to avoid duplicating the summary.

Egginfo stuff doesn't have to be deleted, unless it contains binary eggs.

"do not buil" → typo.

What does it mean that the library does not build in Fedora? Is it missing some
deps?

-- 
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 1282640] Review Request: lastpass-cli - Command line interface to LastPass.com

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1282640

Zbigniew Jędrzejewski-Szmek  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||zbys...@in.waw.pl
   Assignee|nob...@fedoraproject.org|zbys...@in.waw.pl
  Flags||fedora-review?



--- Comment #3 from Zbigniew Jędrzejewski-Szmek  ---
You can probably replace the make line with %make_build.

...and the two make install lines with %make_install install-docs.

rm -rf $RPM_BUILD_ROOT is not needed and is not recommended
[http://fedoraproject.org/wiki/Packaging:Guidelines#BuildRoot_tag].

%defattr is not needed
[http://fedoraproject.org/wiki/Packaging:Guidelines#File_Permissions].

Use %{_mandir} for the path, and do not hardcode the compression:
/usr/share/man/man1/lpass.1.gz → %{_mandir}/man1/lpass.1.*

The changelog has an empty entry at the bottom. Either fill it in or remove.

-- 
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 1282893] Review Request: quantum-espresso - A suite for electronic-structure calculations and materials modeling

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1282893

Zbigniew Jędrzejewski-Szmek  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||zbys...@in.waw.pl
   Assignee|nob...@fedoraproject.org|zbys...@in.waw.pl
  Flags||fedora-review?



--- Comment #2 from Zbigniew Jędrzejewski-Szmek  ---
Hm, what does this do?:

%if 0%{?el6}
%global mpich mpich
%global mpich_load %_mpich_load
%global mpich_unload %_mpich_unload
%else
%global mpich mpich
%global mpich_load %_mpich_load
%global mpich_unload %_mpich_unload
%endif


Requires are normally written with one =. I think == might be misinterpreted.

You can replace the calls to `basename ${f}` with ${f%/*}, makes things
slightly shorter and more readable.

Please use pushd/popd instead of cd/cd -. This is recommended because the path
is printed, which makes logs easier to understand.

%defattr(-,root,root,-) is not needed.

Use %{!?_licensedir:%global license %doc} and then you don't need conditionals
in %files.

Can you comment on the reason for multiple source tarballs? Are they always
released together?

-- 
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 1288453] Review Request: python-CommonMark - Python parser for the CommonMark Markdown spec

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1288453



--- Comment #3 from Zbigniew Jędrzejewski-Szmek  ---
The point of the -doc subpackage is to avoid duplication to save disk space. It
also saves user confusion, because the don't have to look at python2-*-doc
first, and then at python3-*-doc first, and realize that the documentation is
identical. Files in python2-CommonMark-doc and python2-CommonMark-doc *are*
identical, there should be just one -doc package.

You can avoid defining %sum. Just put the summary in the first Summary, and
then use %summary in subsequent subpackages. The summary for -doc should be
different though.

Please use %{__python3} to refer to /usr/bin/python3 everywhere in the spec
file.

You probably don't need to use sed to fix the shebang line. Just move
%py2_install below %py3_install.

PYTHONPATH=$(pwd):$PYTHONPATH → PYTHONPATH=$(pwd)
PYTHONPATH is normally unset, so you don't need to preserve the previous one.

-- 
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 1287193] Review Request: letsencrypt - A free, automated certificate authority

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1287193

Zbigniew Jędrzejewski-Szmek  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #19 from Zbigniew Jędrzejewski-Szmek  ---
- license is OK
- license file is present, %license is used
- latest version
- uses the new python packaging template
- requires and provides look ok
- no scriptlets
- builds, install, runs fine
- rpmlint:

letsencrypt.noarch: W: spurious-executable-perm
/usr/share/man/man1/letsencrypt-renewer.1.gz
letsencrypt.noarch: W: spurious-executable-perm
/usr/share/man/man1/letsencrypt.1.gz
Please remove executable bit.

letsencrypt-doc.noarch: W: spurious-executable-perm
/usr/share/man/man7/letsencrypt.7.gz
There's no need for this file to be in doc subpackage too.

letsencrypt.noarch: E: non-readable /etc/letsencrypt 0
This dir is %ghosted, seems to be a bug in rpmlint.

letsencrypt.src:60: W: unversioned-explicit-provides bundled(jquery)
letsencrypt.src:61: W: unversioned-explicit-provides bundled(underscore)
letsencrypt.src:62: W: unversioned-explicit-provides bundled(inconsolata-fonts)
letsencrypt.src:63: W: unversioned-explicit-provides bundled(lato-fonts)
letsencrypt.src:64: W: unversioned-explicit-provides bundled(robotoslab-fonts)
You should add versions... Probably matters the most for jquery and underscore,
the rest can be ignored.

letsencrypt-doc.noarch: W: dangling-symlink
/usr/share/doc/letsencrypt-doc/html/_static/fonts/fontawesome-webfont.eot
/usr/share/fonts/fontawesome/fontawesome-webfont.eot
letsencrypt-doc.noarch: W: dangling-symlink
/usr/share/doc/letsencrypt-doc/html/_static/fonts/fontawesome-webfont.svg
/usr/share/fonts/fontawesome/fontawesome-webfont.svg
letsencrypt-doc.noarch: W: dangling-symlink
/usr/share/doc/letsencrypt-doc/html/_static/fonts/fontawesome-webfont.ttf
/usr/share/fonts/fontawesome/fontawesome-webfont.ttf
letsencrypt-doc.noarch: W: dangling-symlink
/usr/share/doc/letsencrypt-doc/html/_static/fonts/fontawesome-webfont.woff
/usr/share/fonts/fontawesome/fontawesome-webfont.woff
Those are actually fine when installed.

python2-letsencrypt.noarch: W: no-documentation
python2-letsencrypt.noarch: W: pem-certificate
/usr/lib/python2.7/site-packages/letsencrypt/tests/testdata/cert-san.pem
python2-letsencrypt.noarch: W: pem-certificate
/usr/lib/python2.7/site-packages/letsencrypt/tests/testdata/cert.pem
python2-letsencrypt.noarch: W: pem-certificate
/usr/lib/python2.7/site-packages/letsencrypt/tests/testdata/dsa_cert.pem
python2-letsencrypt.noarch: W: pem-certificate
/usr/lib/python2.7/site-packages/letsencrypt/tests/testdata/matching_cert.pem
4 packages and 0 specfiles checked; 1 errors, 17 warnings.

Those are obviously used for testing, so that's all fine.

Please fix up the small issues pointed out above when uploading. Package is
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 1190269] Review Request: openstack-barbican - Secrets as a Service

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1190269

Alan Pevec  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED
   Fixed In Version||openstack-barbican-1.0.0-1.
   ||el7



--- Comment #31 from Alan Pevec  ---
Changes to replace uwsgi with gunicorn:
https://github.com/openstack-packages/barbican/commit/343314d29da79cc67d81a3221cc4c8596607962f

-- 
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 1269844] Review Request: jcuber - CUBE reader for Java

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1269844



--- Comment #1 from Orion Poplawski  ---
CubeReader is a GUI application so you need to require java, not java headless. 

Also, you need to install a wrapper script to run the application, see
https://fedorahosted.org/released/javapackages/doc/#_generating_application_shell_scripts

-- 
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 1284255] Review Request: wayland-protocols - Wayland protocols that adds functionality not available in the Wayland core protocol

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1284255



--- Comment #5 from Kalev Lember  ---
Fedora review wayland-protocols-1.0-1.fc24.src.rpm 2015-12-04

$ rpmlint wayland-protocols-1.0-1.fc24.src.rpm \
  wayland-protocols-devel
wayland-protocols.src: E: summary-too-long C Wayland protocols that adds
functionality not available in the Wayland core protocol
2 packages and 0 specfiles checked; 1 errors, 0 warnings.

+ OK
! needs attention

! rpmlint warnings:

  Would be good to fix up the summary-tool-long warning. I'd suggest "Wayland
protocol files" as the new summary.

+ 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
  1ee04fc828cffaf278fdc684a13981eb  wayland-protocols-1.0.tar.xz
  1ee04fc828cffaf278fdc684a13981eb  Download/wayland-protocols-1.0.tar.xz
+ The package builds in koji
n/a ExcludeArch bugs filed
+ BuildRequires look sane

  We don't usually specify 'make' explicitly, but it's fine to have it too if
you want to list that.

n/a locale handling
n/a 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
n/a -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


Some other nitpicks I noticed:

> Requires:   pkgconfig

rpmbuild autogenerates a dep on /usr/bin/pkg-config and there's no need to
manually list "Requires: pkgconfig"

> %make_build

Like Jonas says above, this isn't really needed for this package as it's not
building anything.


Another thing I noticed is that now that you are only building a
wayland-protocols-devel and there's no wayland-protocols package, all the
nicely written summaries are essentially not used in the actual user visible
package -- the -devel subpackage only has a copy-pasted one line blurb. It's
also somewhat misleading to say "Development files for %{name}" because there's
no '%{name}' package.


None of these are blocking issues for the review, but please feel free to
address them before importing.

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 1284255] Review Request: wayland-protocols - Wayland protocols that adds functionality not available in the Wayland core protocol

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1284255

Kalev Lember  changed:

   What|Removed |Added

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

[Bug 1269844] Review Request: jcuber - CUBE reader for Java

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1269844

Orion Poplawski  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||or...@cora.nwra.com
   Assignee|nob...@fedoraproject.org|or...@cora.nwra.com



-- 
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 1281532] Review Request: Pagila - Example database for PostgreSQL

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1281532

Pavel Raiskup  changed:

   What|Removed |Added

 CC||pin...@pingoured.fr
  Flags||needinfo?(pingou@pingoured.
   ||fr)



--- Comment #26 from Pavel Raiskup  ---
Huh :), I'm just curious why scratch build of 'example.spec' resulted into
update of this bug.  Except for my nick I doubt those spec files have
anything in common.

-- 
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 1273877] Review Request: nodejs-path-is-absolute - Node.js 0.12 path.isAbsolute() ponyfill

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1273877



--- Comment #4 from Jared Smith  ---
Sorry for the delay -- I've been traveling this week and haven't been as
attentive to packages as I usually am.  I've pushed this to the master branch
and am building for rawhide now.

-- 
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 769958] Review Request: eqp - Automated theorem prover for first-order equational logic

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=769958

John C Peterson  changed:

   What|Removed |Added

  Flags|needinfo?(j...@eskimo.com)   |



--- Comment #10 from John C Peterson  ---
Hello James,

The short answer is no.

As I noted in my post dated 2012-11-01, I did get into contact with the
organization at Argonne National Laboratory (ANL), where the (now deceased) EQP
author was employed when the original work was performed.

There wasn't any formal procedure at ANL for the licensing or distribution of
software when the code was first released. Consequently, the EQP software is
not even in their own database of ANL developed software. They pretty much just
gave me pointers to the "license" that is shown on the project home page (links
below). I can forward those emails from ANL to anyone who is interested. 

http://www.mcs.anl.gov/research/projects/AR/eqp/
http://www.cs.unm.edu/~mccune/eqp/

If that "license" is not adequate, then the matter should be considered closed.
My feeling is that it's highly unlikely ANL will be inclined to provide any
further clarification above and beyond that.

-- 
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 1284255] Review Request: wayland-protocols - Wayland protocols that adds functionality not available in the Wayland core protocol

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1284255



--- Comment #4 from Upstream Release Monitoring 
 ---
kalev's scratch build of wayland-protocols-1.0-1.fc24.src.rpm for f24 completed
http://koji.fedoraproject.org/koji/taskinfo?taskID=12058568

-- 
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 1284255] Review Request: wayland-protocols - Wayland protocols that adds functionality not available in the Wayland core protocol

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1284255

Kalev Lember  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||klem...@redhat.com
   Assignee|nob...@fedoraproject.org|klem...@redhat.com



--- Comment #3 from Kalev Lember  ---
Taking for 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://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1281532] Review Request: Pagila - Example database for PostgreSQL

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1281532



--- Comment #25 from Upstream Release Monitoring 
 ---
praiskup's scratch build of example-0.0-1.fc23.src.rpm for f23 failed
http://koji.fedoraproject.org/koji/taskinfo?taskID=12058344

-- 
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 1284945] Review Request: nodejs-jison-lex - Lexical analyzer generator used by jison

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1284945



--- Comment #2 from Jared Smith  ---
Not sure why it's failing for you in mock -- I've run it multiple times in mock
myself (against rawhide), and it's running perfectly fine.  Mind trying again?

-- 
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 1287846] Review Request: python-lib389 - python module to access the 389 Directory Server

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1287846



--- Comment #22 from Antonio Trande  ---
(In reply to mreynolds from comment #21)
> (In reply to Antonio Trande from comment #20)
> > 
> > See line #481.
> 
> Ah, I had python-krbV listed in my setup.py file.  Do I need to also remove
> pytest?
> 

None package should be downloaded during package building (see
http://fedoraproject.org/wiki/Packaging:Python#Reviewer_checklist).

Even because download wouldn't permitted in koji.

-- 
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 1288610] Review Request: python-lazy-object-proxy - A fast and thorough lazy object proxy

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1288610



--- Comment #6 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/python-lazy-object-proxy

-- 
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 1285471] Review Request: nodejs-spdx-exceptions - A list of SPDX standard license exceptions

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1285471



--- Comment #3 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/nodejs-spdx-exceptions

-- 
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 1286006] Review Request: logisim - Educational tool for simulating digital logic circuits

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1286006

gil cattaneo  changed:

   What|Removed |Added

 Blocks||652183 (FE-JAVASIG)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=652183
[Bug 652183] Java SIG tracker 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

[Bug 1288651] Review Request: OpenIGTLink

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1288651

Dmitry Mikhirev  changed:

   What|Removed |Added

 Blocks||177841 (FE-NEEDSPONSOR)



--- Comment #1 from Dmitry Mikhirev  ---
I'm sorry, forgot to paste URLs:

Spec URL:
http://copr-dist-git.fedorainfracloud.org/cgit/bizdelnick/neuro/OpenIGTLink.git/plain/OpenIGTLink.spec?id=dcd4b2f058890e46f74a31d47661e4ddeecb97b3
SRPM URL:
https://copr-be.cloud.fedoraproject.org/results/bizdelnick/neuro/fedora-rawhide-x86_64/00143768-OpenIGTLink/OpenIGTLink-0-0.1.20151015gitccb2438.fc24.src.rpm


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

[Bug 1288651] New: Review Request: OpenIGTLink

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1288651

Bug ID: 1288651
   Summary: Review Request: OpenIGTLink
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Assignee: nob...@fedoraproject.org
  Reporter: mikhi...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
SRPM URL:

Description:

The OpenIGTLink Library is a reference implementation of OpenIGTLink protocol.
It is required to build 3D-Slicer that I'm going to package.

OpenIGTLink is a network communication protocol specifically designed and
developed for research on image-guided and computer-assisted interventions. It
provides a standardized mechanism for communications among computers and
devices in operating rooms (OR) for a wide variety of image-guided therapy
(IGT) applications. Examples of such applications include:

- Stereotactic surgical guidance using optical position sensor and medical
image visualization software
- Intraoperative image guidance using real-time MRI and medical image
visualization software
- Robot-assisted interventions using robotic devices and surgical planning
software

OpenIGTLink is a set of messaging formats and rules (protocol) used for data
exchange on a local area network (LAN).

-- 
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 1288649] Review Request: cmark - cmark is the C reference implementation of CommonMark

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1288649

John Dulaney  changed:

   What|Removed |Added

Summary|Review Request:  -|implementation of
   ||CommonMark



-- 
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 1270317] Review Request: lz4-java - LZ4 compression for Java

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1270317

gil cattaneo  changed:

   What|Removed |Added

 Depends On||1282063




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1282063
[Bug 1282063] Review Request: xxhsum - Extremely fast hash algorithm
-- 
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 1288649] New: Review Request: -

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1288649

Bug ID: 1288649
   Summary: Review Request:  - 
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: jdula...@fedoraproject.org
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: http://rpms.jdulaney.com/review/cmark.spec
SRPM URL: http://rpms.jdulaney.com/review/cmark-0.22.0-1.fc24.src.rpm
Description: 
cmark is the C reference implementation of CommonMark,
a rationalized version of Markdown syntax with a spec.
(For the JavaScript reference implementation, see commonmark.js.)

It provides a shared library (libcmark) with functions for
parsing CommonMark documents to an abstract syntax tree
(AST), manipulating the AST, and rendering the document to
HTML, groff man, LaTeX, CommonMark, or an XML representation
of the AST. It also provides a command-line program (cmark) for
parsing and rendering CommonMark documents.
Fedora Account System Username:  jdulaney

-- 
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 1282063] Review Request: xxhsum - Extremely fast hash algorithm

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1282063

gil cattaneo  changed:

   What|Removed |Added

 Blocks||1270317




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1270317
[Bug 1270317] Review Request: lz4-java - LZ4 compression for Java
-- 
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 1286002] Package Review: jfontchooser - Swing-based java component for font selection

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1286002

gil cattaneo  changed:

   What|Removed |Added

 Blocks||652183 (FE-JAVASIG)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=652183
[Bug 652183] Java SIG tracker 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

[Bug 1269844] Review Request: jcuber - CUBE reader for Java

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1269844

gil cattaneo  changed:

   What|Removed |Added

 Blocks||652183 (FE-JAVASIG)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=652183
[Bug 652183] Java SIG tracker 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

[Bug 1269813] Review Request: golang-github-dgrijalva-jwt-go - Golang implementation of JSON Web Tokens (JWT)

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1269813



--- Comment #15 from Fedora Update System  ---
golang-github-dgrijalva-jwt-go-2.2.0-1.fc22 has been pushed to the Fedora 22
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://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1157179] Review Request: log4j-jsonevent-layout - Layout for log4j that generates logstash json_event formatted data

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1157179

gil cattaneo  changed:

   What|Removed |Added

 Blocks||652183 (FE-JAVASIG)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=652183
[Bug 652183] Java SIG tracker 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

[Bug 1190735] Review Request: mycila-pom - Mycila parent POM

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1190735

gil cattaneo  changed:

   What|Removed |Added

 Blocks||652183 (FE-JAVASIG)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=652183
[Bug 652183] Java SIG tracker 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

[Bug 1149176] Review Request: vdsm-jsonrpc-java - jsonrpc communication lib for ovirt

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1149176

gil cattaneo  changed:

   What|Removed |Added

 Blocks||652183 (FE-JAVASIG)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=652183
[Bug 652183] Java SIG tracker 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

[Bug 1284155] Review Request: python-rpdb - A wrapper around pdb allowing remote debugging

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1284155



--- Comment #15 from Fedora Update System  ---
python-rpdb-0.1.5-2.fc22 has been pushed to the Fedora 22 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://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1110070] Review Request: vert.x - A lightweight, high-performance, polyglot, event-driven application platform for JVM

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1110070

gil cattaneo  changed:

   What|Removed |Added

 Blocks||652183 (FE-JAVASIG)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=652183
[Bug 652183] Java SIG tracker 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

[Bug 1283190] Review Request: nodejs-is-my-json-valid - A JSONSchema validator that uses code generation to be extremely fast

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1283190



--- Comment #11 from Fedora Update System  ---
nodejs-is-my-json-valid-2.12.3-1.fc22 has been pushed to the Fedora 22 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://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1256353] Review Request: python-editor - Programmatically open an editor, capture the result.

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1256353



--- Comment #22 from Fedora Update System  ---
python-alembic-0.8.3-3.fc22, python-editor-0.4-4.fc22 has been pushed to the
Fedora 22 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://admin.fedoraproject.org/mailman/listinfo/package-review

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

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1131019

gil cattaneo  changed:

   What|Removed |Added

 Blocks||652183 (FE-JAVASIG)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=652183
[Bug 652183] Java SIG tracker 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

[Bug 1286001] Review Request: java-colors - Color picker for Java applications

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1286001

gil cattaneo  changed:

   What|Removed |Added

 Blocks||652183 (FE-JAVASIG)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=652183
[Bug 652183] Java SIG tracker 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

[Bug 1288610] Review Request: python-lazy-object-proxy - A fast and thorough lazy object proxy

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1288610



--- Comment #5 from Brian Lane  ---
Spec URL:
https://bcl.fedorapeople.org/python-lazy-object-proxy/python-lazy-object-proxy.spec
SRPM URL:
https://bcl.fedorapeople.org/python-lazy-object-proxy/python-lazy-object-proxy-1.2.1-1.fc24.src.rpm
Description: A fast and thorough lazy object proxy
Fedora Account System Username: bcl

Updated with %attr

-- 
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 1288645] New: Review Request: teem

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1288645

Bug ID: 1288645
   Summary: Review Request: teem
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Assignee: nob...@fedoraproject.org
  Reporter: mikhi...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
http://copr-dist-git.fedorainfracloud.org/cgit/bizdelnick/neuro/teem.git/plain/teem.spec?id=c4062dcf9a54cb7f3d50e3a5c5aaf047dc2557fe
SRPM URL:
https://copr-be.cloud.fedoraproject.org/results/bizdelnick/neuro/fedora-rawhide-x86_64/00143767-teem/teem-1.11.0-1.fc24.src.rpm

Description:

Teem is a coordinated group of libraries for representing, processing, and
visualizing scientific raster data. Teem includes command-line tools that
permit the library functions to be quickly applied to files and streams,
without having to write any code. The most important and useful libraries in
Teem are:

- Nrrd (and the unu command-line tool on top of it) supports a range of
operations for transforming N-dimensional raster data (resample, crop, slice,
project, histogram, etc.), as well as the NRRD file format for storing arrays
and their meta-information.
- Gage: fast convolution-based measurements at arbitrary point locations in
volume datasets (scalar, vector, tensor, etc.)
- Mite: a multi-threaded ray-casting volume render with transfer functions
based on any quantity Gage can measure
- Ten: for estimating, processing, and visualizing diffusion tensor fields,
including fiber tractography methods.

This library is required to build 3D-Slicer that I'm going to 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://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1288645] Review Request: teem

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1288645

Dmitry Mikhirev  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://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1282063] Review Request: xxhsum - Extremely fast hash algorithm

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1282063

gil cattaneo  changed:

   What|Removed |Added

 CC||punto...@libero.it



--- Comment #1 from gil cattaneo  ---
HI
could you build libxxhash? is a BuildRequires for
https://bugzilla.redhat.com/show_bug.cgi?id=1270317

can you add this part in your spec file?

# Use ${LIB_INSTALL_DIR} instead of hardcoded lib in cmake file
sed -i 's|DESTINATION lib|DESTINATION ${LIB_INSTALL_DIR}|'
cmake_unofficial/CMakeLists.txt

%build

CFLAGS="${CFLAGS:-%optflags}"
#CPPFLAGS="CPPFLAGS"
export CFLAGS
(
 cd cmake_unofficial
 %{__cmake} -DCMAKE_INSTALL_PREFIX:PATH=%{_prefix}
-DLIB_INSTALL_DIR:PATH=%{_libdir}
)



%install
make install DESTDIR=%{buildroot} -C cmake_unofficial
find %{buildroot} -name '*.a' -delete
find %{buildroot} -name '*.la' -delete

Thanks in advance

-- 
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 1288643] Review Request: dlib

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1288643

Dmitry Mikhirev  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://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1288643] New: Review Request: dlib

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1288643

Bug ID: 1288643
   Summary: Review Request: dlib
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Assignee: nob...@fedoraproject.org
  Reporter: mikhi...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
http://copr-dist-git.fedorainfracloud.org/cgit/bizdelnick/neuro/dlib.git/plain/dlib.spec
SRPM URL:
https://copr-be.cloud.fedoraproject.org/results/bizdelnick/neuro/fedora-rawhide-x86_64/00144844-dlib/dlib-18.18-1.fc24.src.rpm

Description:

Dlib is a modern C++ toolkit containing machine learning algorithms and tools
for creating complex software in C++ to solve real world problems. It is open
source software and licensed under the Boost Software License.

-- 
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 1281532] Review Request: Pagila - Example database for PostgreSQL

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1281532



--- Comment #24 from Upstream Release Monitoring 
 ---
praiskup's scratch build of example-0.0-1.fc23.src.rpm for f23 failed
http://koji.fedoraproject.org/koji/taskinfo?taskID=12057711

-- 
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 1270355] Review Request: nacl-binutils - A GNU collection of binary utilities

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1270355



--- Comment #6 from Tom "spot" Callaway  ---
I've added /usr/x86_64-nacl ownership.

I think chromium broke on the softlinks. Would rather just keep it as is, where
I know it works.

This toolchain (nacl-binutils, nacl-gcc, nacl-newlib) is only ever used to make
x86_64 and i686 builds. I'm not entirely sure if it will work on i686 systems,
but it definitely isn't useful anywhere else. We disable nacl/pnacl on i686
right now. For these packages, ExclusiveArch is correct, because we never
care/need for it to build anywhere else, and pnacl/chromium wouldn't use it
even if we did.

The other nacl toolchain (nacl-arm-$foo) is only ever used to do arm builds,
but because of how pnacl works, it needs this to be built for x86_64 (and i686
and arm in theory). Since we're disabling i686 nacl/pnacl right now and I
haven't got enough good liquor to work on chromium ARM yet, ExclusiveArch is
correct there too.

I think I can figure out how to build the nacl/pnacl stack for arm without
nacl, but thats a big TODO (Tomas Popela added some fixes to pull in the ffmpeg
arm bits, so thats a big part out of the way).

Chromium upstream doesn't support ppc/s390 as far as I can see, so there's no
sense in going through the normal process for that.

I've updated the comment in nacl-binutils to provide a more accurate summary of
the situation.

New SRPM:
https://spot.fedorapeople.org/nacl-binutils-2.24-4.git1d8592c.fc23.src.rpm
New SPEC: https://spot.fedorapeople.org/nacl-binutils.spec

-- 
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 1279786] Review Request: python-dbf - Pure python package for reading/writing dBase, FoxPro, and Visual FoxPro .dbf

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1279786



--- Comment #5 from Fedora Update System  ---
python-dbf-0.96.005-1.fc23 has been submitted as an update to Fedora 23.
https://bodhi.fedoraproject.org/updates/FEDORA-2015-ba1f02a453

-- 
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 1277505] Review Request: jssc - Java Simple Serial Connector

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1277505



--- Comment #16 from gil cattaneo  ---
Damian javapackages-tools-doc package provides
https://fedorahosted.org/released/javapackages/doc

-- 
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 1162011] Review Request: qname - This is the javax.transaction's spec file

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1162011

gil cattaneo  changed:

   What|Removed |Added

 CC||punto...@libero.it



--- Comment #2 from gil cattaneo  ---
The spec file should be cleanup.
Please, see https://bugzilla.redhat.com/show_bug.cgi?id=1157302#c1
and (how wrote Quan Xu) BuildRequires: java-1.7.0-openjdk no more exist in
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://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 804125] Review Request: rdkit - A toolkit for cheminformatics and machine learning

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=804125



--- Comment #40 from Gianluca Sforna  ---
Updated package and spec, the tests are now fixed:

https://giallu.fedorapeople.org/rdkit-2015.09.2-2.fc23.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://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1279786] Review Request: python-dbf - Pure python package for reading/writing dBase, FoxPro, and Visual FoxPro .dbf

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1279786



--- Comment #4 from Fedora Update System  ---
python-dbf-0.96.005-1.fc22 has been submitted as an update to Fedora 22.
https://bodhi.fedoraproject.org/updates/FEDORA-2015-4a867ad4e4

-- 
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 1279786] Review Request: python-dbf - Pure python package for reading/writing dBase, FoxPro, and Visual FoxPro .dbf

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1279786

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

[Bug 1162000] Review Request:jonathan-core - Distributed Object Platform (DOP) written entirely in Java

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1162000

gil cattaneo  changed:

   What|Removed |Added

 CC||punto...@libero.it



--- Comment #1 from gil cattaneo  ---
The spec file should be cleanup.
Please, see https://bugzilla.redhat.com/show_bug.cgi?id=1157302#c1

-- 
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 1277505] Review Request: jssc - Java Simple Serial Connector

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1277505



--- Comment #15 from gil cattaneo  ---
Other trick
Remove
BuildRequires:jpackage-utils
BuildRequires:maven-local

Requires:java-headless
Requires:jpackage-utils

and add 
BuildRequires:javapackages-local

because jpackage-utils is an alias of javapackages-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://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1144865] Review Request: glite-lb-doc - gLite Logging and Bookkeeping documentation

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1144865



--- Comment #12 from Upstream Release Monitoring 
 ---
pbrobinson's scratch build of
glite-px-proxyrenewal?#4338269af302c9682fd1b8dd2852e8f05589b8e7 for
epel7-archbootstrap and
git://pkgs.fedoraproject.org/glite-px-proxyrenewal?#4338269af302c9682fd1b8dd2852e8f05589b8e7
failed http://koji.fedoraproject.org/koji/taskinfo?taskID=12056238

-- 
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 1144865] Review Request: glite-lb-doc - gLite Logging and Bookkeeping documentation

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1144865



--- Comment #11 from Upstream Release Monitoring 
 ---
pbrobinson's scratch build of
glite-lb-utils?#a5f7629036a2c87266412d58a2365dd4d9236f30 for
epel7-archbootstrap and
git://pkgs.fedoraproject.org/glite-lb-utils?#a5f7629036a2c87266412d58a2365dd4d9236f30
failed http://koji.fedoraproject.org/koji/taskinfo?taskID=12056233

-- 
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 1277505] Review Request: jssc - Java Simple Serial Connector

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1277505



--- Comment #14 from gil cattaneo  ---
(In reply to Damian Wrobel from comment #13)
> (In reply to gil cattaneo from comment #10)
> > instead of maven-local you should use javapackages-local
> > 
> Gil, thanks for your hints, I started to look at them and compare with
> packaging guidelines [1] and I found that e.g. [1] clearly states the
> following:
> 
> "Java packages MUST BuildRequire their respective build system:
> 
> BuildRequires: maven-local for packages built with Maven"
> 
> and there is no single reference that javapackages-local can be used as a
> substitute for maven-local in this case.

Only if the package provide a pom.xml file and use that for build the package
in this case you use only java-devel. javapackages-local is enough

> So is there any specification on top of it which overwrites this or I
> misunderstood or missed something?
> 
> [1]. https://fedoraproject.org/wiki/Packaging:Java#BuildRequires_and_Requires
see also
https://fedorahosted.org/released/javapackages/doc/#_generic_java_builds
regards

-- 
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 1144865] Review Request: glite-lb-doc - gLite Logging and Bookkeeping documentation

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1144865



--- Comment #10 from Upstream Release Monitoring 
 ---
pbrobinson's scratch build of
glite-lb-logger?#5d888cd26b591dd49ddcd86339543ad5a7682c98 for
epel7-archbootstrap and
git://pkgs.fedoraproject.org/glite-lb-logger?#5d888cd26b591dd49ddcd86339543ad5a7682c98
failed http://koji.fedoraproject.org/koji/taskinfo?taskID=12056221

-- 
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 1144865] Review Request: glite-lb-doc - gLite Logging and Bookkeeping documentation

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1144865



--- Comment #9 from Upstream Release Monitoring 
 ---
pbrobinson's scratch build of
glite-lb-server?#0b7eec393509758e03e4d2d437cb7f2a5f6136e6 for
epel7-archbootstrap and
git://pkgs.fedoraproject.org/glite-lb-server?#0b7eec393509758e03e4d2d437cb7f2a5f6136e6
failed http://koji.fedoraproject.org/koji/taskinfo?taskID=12056226

-- 
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 1144865] Review Request: glite-lb-doc - gLite Logging and Bookkeeping documentation

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1144865



--- Comment #7 from Upstream Release Monitoring 
 ---
pbrobinson's scratch build of
glite-lb-client?#ab3f7850e46d640d3844321d6a0bdde00e9d34f0 for
epel7-archbootstrap and
git://pkgs.fedoraproject.org/glite-lb-client?#ab3f7850e46d640d3844321d6a0bdde00e9d34f0
failed http://koji.fedoraproject.org/koji/taskinfo?taskID=12056199

-- 
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 1144865] Review Request: glite-lb-doc - gLite Logging and Bookkeeping documentation

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1144865



--- Comment #8 from Upstream Release Monitoring 
 ---
pbrobinson's scratch build of
glite-lb-common?#15e85324e0a340ac46100a3ae2af1c3572693837 for
epel7-archbootstrap and
git://pkgs.fedoraproject.org/glite-lb-common?#15e85324e0a340ac46100a3ae2af1c3572693837
failed http://koji.fedoraproject.org/koji/taskinfo?taskID=12056204

-- 
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 1159703] Review Request: Java persistence API implementation

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1159703

gil cattaneo  changed:

   What|Removed |Added

 CC||punto...@libero.it



--- Comment #1 from gil cattaneo  ---
Package already available:
http://pkgs.fedoraproject.org/cgit/geronimo-jpa.git/
Please, close 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

[Bug 1159691] Review Request: Database statement interceptor for Java

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1159691

gil cattaneo  changed:

   What|Removed |Added

 CC||punto...@libero.it



--- Comment #1 from gil cattaneo  ---
The spec file should be cleanup.
Please, see https://bugzilla.redhat.com/show_bug.cgi?id=1157302#c1

-- 
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 1277505] Review Request: jssc - Java Simple Serial Connector

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1277505



--- Comment #13 from Damian Wrobel  ---
(In reply to gil cattaneo from comment #10)
> instead of maven-local you should use javapackages-local
> 
Gil, thanks for your hints, I started to look at them and compare with
packaging guidelines [1] and I found that e.g. [1] clearly states the
following:

"Java packages MUST BuildRequire their respective build system:

BuildRequires: maven-local for packages built with Maven"

and there is no single reference that javapackages-local can be used as a
substitute for maven-local in this case.

So is there any specification on top of it which overwrites this or I
misunderstood or missed something?

[1]. https://fedoraproject.org/wiki/Packaging:Java#BuildRequires_and_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://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1159688] Review Request: monolog - API for monitoring and logging

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1159688

gil cattaneo  changed:

   What|Removed |Added

 Blocks||652183 (FE-JAVASIG)
 CC||punto...@libero.it



--- Comment #5 from gil cattaneo  ---
The spec file should be cleanup.
Please, see https://bugzilla.redhat.com/show_bug.cgi?id=1157302#c1


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=652183
[Bug 652183] Java SIG tracker 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

[Bug 1159687] Review Request: This is the javax.transaction's spec file

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1159687

gil cattaneo  changed:

   What|Removed |Added

 CC||punto...@libero.it



--- Comment #3 from gil cattaneo  ---
The spec file should be cleanup.
Please, see https://bugzilla.redhat.com/show_bug.cgi?id=1157302#c1
and BuildRequires: java-1.7.0-openjdk no more exist in 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://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1159223] Review Request: ow2-jta-1.1-spec - "ow2-jta-1.1-spec" spec file

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1159223

gil cattaneo  changed:

   What|Removed |Added

 CC||punto...@libero.it



--- Comment #3 from gil cattaneo  ---
The spec file should be cleanup.
Please, see https://bugzilla.redhat.com/show_bug.cgi?id=1157302#c1
and BuildRequires: java-1.7.0-openjdk no more exist in 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://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1159220] Review Request: A Java Open Transaction Manager

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1159220

gil cattaneo  changed:

   What|Removed |Added

 CC||punto...@libero.it



--- Comment #1 from gil cattaneo  ---
The spec file should be cleanup.
Please, see https://bugzilla.redhat.com/show_bug.cgi?id=1157302#c1

-- 
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 1159217] Review Request: J2EE(tm) Connector Architecture

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1159217

gil cattaneo  changed:

   What|Removed |Added

 CC||punto...@libero.it



--- Comment #1 from gil cattaneo  ---
Please, see:
https://fedoraproject.org/wiki/Packaging:Java#Pre-built_dependencies
https://fedoraproject.org/wiki/Packaging:Guidelines#No_inclusion_of_pre-built_binaries_or_libraries

-- 
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 1157302] Review Request: oldkilim - A generic configuration framework for Java

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1157302

gil cattaneo  changed:

   What|Removed |Added

 CC||punto...@libero.it



--- Comment #1 from gil cattaneo  ---
The spec file should be cleanup, please remove:

Group:Development/Libraries/Java
#Vendor:JPackage Project
#Distribution:JPackage

BuildRoot:  %{_tmppath}/%{name}-%{version}-%{release}-buildroot

Group: Development/Documentation

rm -rf $RPM_BUILD_ROOT in %install section

%clean
rm -rf $RPM_BUILD_ROOT

%post javadoc

%postun javadoc

%defattr(0644,root,root,0755)

* jar and javadoc must be without version, e.g.

# jars
install -d -m 755 $RPM_BUILD_ROOT%{_javadir}
install -pm 644 distrib/kilim.jar $RPM_BUILD_ROOT%{_javadir}/%{name}.jar
install -pm 644 distrib/kilim-tools.jar
$RPM_BUILD_ROOT%{_javadir}/%{name}-tools.jar

# javadoc
install -p -d -m 755 $RPM_BUILD_ROOT%{_javadocdir}/%{name}
cp -pr doc/apis/* $RPM_BUILD_ROOT%{_javadocdir}/%{name}

%files javadoc
%{_javadocdir}/%{name}

-- 
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 1279786] Review Request: python-dbf - Pure python package for reading/writing dBase, FoxPro, and Visual FoxPro .dbf

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1279786



--- Comment #3 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/python-dbf

-- 
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 1288231] Review Request: nodejs-postgres-array - Parse postgres array columns

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1288231



--- Comment #3 from John Dulaney  ---
Okay, I'll rerun tomorrow, unless you have a pressing need for it to be done
today.

-- 
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 1159226] Review Request: JavaBeans Activation Framework

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1159226

gil cattaneo  changed:

   What|Removed |Added

 CC||punto...@libero.it



--- Comment #1 from gil cattaneo  ---
The JAF apis are part of JRE (since JAVA >= 1.6). Please, close 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

[Bug 1288231] Review Request: nodejs-postgres-array - Parse postgres array columns

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1288231



--- Comment #2 from Tom Hughes  ---
The nodejs-ap package was only reviewed this morning, so it's it rawhide but
won't have reach most of the mirrors 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://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1279786] Review Request: python-dbf - Pure python package for reading/writing dBase, FoxPro, and Visual FoxPro .dbf

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1279786

Julien Enselme  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED



--- Comment #2 from Julien Enselme  ---
Thanks for the review. I'll correct the line ending of the README and contact
upstream for the license before importing the package into pkgdb.

-- 
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 1288231] Review Request: nodejs-postgres-array - Parse postgres array columns

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1288231

John Dulaney  changed:

   What|Removed |Added

 Whiteboard||NEEDSWORK



--- Comment #1 from John Dulaney  ---
mock failed with:

DEBUG util.py:399:  No matching package to install: 'npm(ap)'
DEBUG util.py:399:  Not all dependencies satisfied
DEBUG util.py:399:  Error: Some packages could not be found.
DEBUG util.py:521:  Child return code was: 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://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1288453] Review Request: python-CommonMark - Python parser for the CommonMark Markdown spec

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1288453



--- Comment #2 from Julien Enselme  ---
SPEC: http://dl.jujens.eu/SPECS/python-CommonMark.spec
SRPM: http://dl.jujens.eu/SRPMS/python-CommonMark-0.5.4-2.fc23.src.rpm


* Fri Dec 4 2015 Julien Enselme  - 0.5.4-2
- Correct shebang of cmark.py (/usr/bin/python2 -> /usr/bin/python3)
- Add doc packages.

-- 
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 1288231] Review Request: nodejs-postgres-array - Parse postgres array columns

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1288231

John Dulaney  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||jdula...@fedoraproject.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://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1288610] Review Request: python-lazy-object-proxy - A fast and thorough lazy object proxy

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1288610



--- Comment #4 from David Shea  ---
For the ones I left blank because I did a half-assed job of copying and pasting
the second time after bugzilla ate my first comment:

- no, there's no kernel modules
- or static executables
- unversioned .so files in non-standard locations is how Python binary modules
work
- and the package follows all of the directives within the 26 pages of the
Fedora Packaging Guidelines plus the 7 pages of the Python appendix plus all of
the other linked pages, probably.

-- 
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 1159226] Review Request: JavaBeans Activation Framework

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1159226

gil cattaneo  changed:

   What|Removed |Added

 Blocks||177841 (FE-NEEDSPONSOR)
 Blocks||652183 (FE-JAVASIG)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=177841
[Bug 177841] Tracker: Review requests from new Fedora packagers who need a
sponsor
https://bugzilla.redhat.com/show_bug.cgi?id=652183
[Bug 652183] Java SIG tracker 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

[Bug 1162011] Review Request: qname - This is the javax.transaction's spec file

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1162011

gil cattaneo  changed:

   What|Removed |Added

 Blocks||177841 (FE-NEEDSPONSOR)
 Blocks||652183 (FE-JAVASIG)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=177841
[Bug 177841] Tracker: Review requests from new Fedora packagers who need a
sponsor
https://bugzilla.redhat.com/show_bug.cgi?id=652183
[Bug 652183] Java SIG tracker 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

[Bug 1159703] Review Request: Java persistence API implementation

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1159703

gil cattaneo  changed:

   What|Removed |Added

 Blocks||177841 (FE-NEEDSPONSOR)
 Blocks||652183 (FE-JAVASIG)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=177841
[Bug 177841] Tracker: Review requests from new Fedora packagers who need a
sponsor
https://bugzilla.redhat.com/show_bug.cgi?id=652183
[Bug 652183] Java SIG tracker 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

[Bug 1157302] Review Request: oldkilim - A generic configuration framework for Java

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1157302

gil cattaneo  changed:

   What|Removed |Added

 Blocks||177841 (FE-NEEDSPONSOR)
 Blocks||652183 (FE-JAVASIG)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=177841
[Bug 177841] Tracker: Review requests from new Fedora packagers who need a
sponsor
https://bugzilla.redhat.com/show_bug.cgi?id=652183
[Bug 652183] Java SIG tracker 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

[Bug 1159691] Review Request: Database statement interceptor for Java

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1159691

gil cattaneo  changed:

   What|Removed |Added

 Blocks||177841 (FE-NEEDSPONSOR)
 Blocks||652183 (FE-JAVASIG)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=177841
[Bug 177841] Tracker: Review requests from new Fedora packagers who need a
sponsor
https://bugzilla.redhat.com/show_bug.cgi?id=652183
[Bug 652183] Java SIG tracker 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

[Bug 1162000] Review Request:jonathan-core - Distributed Object Platform (DOP) written entirely in Java

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1162000

gil cattaneo  changed:

   What|Removed |Added

 Blocks||177841 (FE-NEEDSPONSOR)
 Blocks||652183 (FE-JAVASIG)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=177841
[Bug 177841] Tracker: Review requests from new Fedora packagers who need a
sponsor
https://bugzilla.redhat.com/show_bug.cgi?id=652183
[Bug 652183] Java SIG tracker 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

[Bug 1159223] Review Request: ow2-jta-1.1-spec - "ow2-jta-1.1-spec" spec file

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1159223

gil cattaneo  changed:

   What|Removed |Added

 Blocks||177841 (FE-NEEDSPONSOR)
 Blocks||652183 (FE-JAVASIG)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=177841
[Bug 177841] Tracker: Review requests from new Fedora packagers who need a
sponsor
https://bugzilla.redhat.com/show_bug.cgi?id=652183
[Bug 652183] Java SIG tracker 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

[Bug 1159220] Review Request: A Java Open Transaction Manager

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1159220

gil cattaneo  changed:

   What|Removed |Added

 Blocks||177841 (FE-NEEDSPONSOR)
 Blocks||652183 (FE-JAVASIG)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=177841
[Bug 177841] Tracker: Review requests from new Fedora packagers who need a
sponsor
https://bugzilla.redhat.com/show_bug.cgi?id=652183
[Bug 652183] Java SIG tracker 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

[Bug 1159217] Review Request: J2EE(tm) Connector Architecture

2015-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1159217

gil cattaneo  changed:

   What|Removed |Added

 Blocks||177841 (FE-NEEDSPONSOR)
 Blocks||652183 (FE-JAVASIG)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=177841
[Bug 177841] Tracker: Review requests from new Fedora packagers who need a
sponsor
https://bugzilla.redhat.com/show_bug.cgi?id=652183
[Bug 652183] Java SIG tracker 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

  1   2   3   4   >