[Bug 927611] Review Request: pyode - Open-source Python bindings for The Open Dynamics Engine
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=927611
--- Comment #3 from John Morris j...@zultron.com ---
- If you remove execute permissions from ode.so, you won't have to bother
with the '%{?filter_setup:...' business. (Why do you set them in %install?)
To get rid of this rpmlint error:
pyode.x86_64: E: non-standard-executable-perm
/usr/lib64/python2.7/site-packages/ode.so 0775L
The permissions must be correct, and since it's an internally used shared
object, it must be removed from provides.
Hmm, I learned something new. For regular packages containing modules, if the
module perms are set 0644, the filter_provides_in macro ignores them. Not so
with python modules.
- Nitpick: (I was dinged for this on my last pkg) Double spaces before
%description, %prep, %build, etc. make the specfile more readable.
I don't think this is required. The spec is readable enough :)
Sure.
- Those comments about the tests might be more clear if they were prefaced
with a line like this that indicates they're the reason for not running
tests, despite the existence of a 'tests' directory:
#%%check
Added phony check section line.
I'm addressing %check differently in the review.
Stay tuned...
--
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug
https://bugzilla.redhat.com/token.cgi?t=qer9rN90nqa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 950624] Review Request: python-flask-whooshee - Whoosh integration
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=950624 Matej Stuchlik mstuc...@redhat.com changed: What|Removed |Added Flags|fedora-review? | Flags||fedora-review+ --- Comment #4 from Matej Stuchlik mstuc...@redhat.com --- Looks good now, approved. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=5CdwHjjaAUa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 927611] Review Request: pyode - Open-source Python bindings for The Open Dynamics Engine
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=927611
John Morris j...@zultron.com changed:
What|Removed |Added
Status|NEW |ASSIGNED
Assignee|nob...@fedoraproject.org|j...@zultron.com
--- Comment #4 from John Morris j...@zultron.com ---
Created attachment 735771
-- https://bugzilla.redhat.com/attachment.cgi?id=735771action=edit
fedora-review results with reviewer notes
Here's the review. These comments may also be found at the bottom of the
attached file.
Reviewer notes
[!]: If (and only if) the source package includes the text of the license(s)
in its own file, then that file, containing the text of the license(s)
for the package is included in %doc.
Rpmlint: pyode.x86_64: W: no-documentation
- No documentation
%doc should include AUTHORS ChangeLog LICENSE LICENSE-BSD README examples
--
[!]: Requires correct, justified where necessary.
- Need Requires: ode and Pyrex
--
[!]: Package meets the Packaging Guidelines::Python
- Python3 support: Infra is in specfile; does it work, and if so,
should it be turned on?
--
[!]: %check is present and all tests pass.
- Has this package been tested? This %check script fails with a
segfault, as does running the tests after install:
%check
export PYTHONPATH=build/lib.linux-%{_target_cpu}-%{python_version}
%{__python} tests/test_xode.py
--
Rpmlint: pyode.x86_64: W: non-standard-group Unspecified
- Main package missing Group: tag (though python3 package includes
it). I want this to be included in EPEL, where Group: is not
optional.
http://fedoraproject.org/wiki/Packaging:Guidelines#Group_tag
--
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug
https://bugzilla.redhat.com/token.cgi?t=XEDf4fSGy3a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 822329] Review Request: libzen - Shared library for libmediainfo and medianfo
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=822329 --- Comment #31 from Ralf Corsepius rc040...@freenet.de --- (In reply to comment #27) Because when it created it empty. Then fix it. This is an arched package, so debuginfo files are mandatory. In other words something is defective and you (the prospective packager) are supposed to fix it. In case wasn't clear enough: This is a MUSTFIX. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=PH2RLpk502a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 822329] Review Request: libzen - Shared library for libmediainfo and medianfo
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=822329 --- Comment #32 from Vasiliy Glazov vasc...@gmail.com --- OK. I change make install-strip to make install. And create debuginfo subpackage. Fixed. Some wildcards in %files section removed. dos2unix removed. Instead it used sed. ./autogen - is upstream authors script $ cat rpmbuild/BUILD/ZenLib/Project/GNU/Library/autogen #libtoolize if test $(uname) = Darwin ; then #Darwin based Systems like Mac OS X: libtoolize is called glibtoolize. glibtoolize --automake else libtoolize --automake fi #aclocal if test -e /usr/bin/aclocal-1.11 ; then #OpenSolaris: no aclocal aclocal-1.11 elif test -e /usr/bin/aclocal-1.10 ; then aclocal-1.10 else aclocal fi #automake if test -e /usr/bin/automake-1.11 ; then #OpenSolaris: no automake automake-1.11 -a elif test -e /usr/bin/automake-1.10 ; then automake-1.10 -a else automake -a fi autoconf -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=4Lzbs3dCPxa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 822329] Review Request: libzen - Shared library for libmediainfo and medianfo
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=822329
--- Comment #33 from Ralf Corsepius rc040...@freenet.de ---
(In reply to comment #31)
(In reply to comment #27)
Because when it created it empty.
Then fix it. This is an arched package, so debuginfo files are mandatory.
In other words something is defective and you (the prospective packager) are
supposed to fix it.
This is the cause:
%install
pushd Project/GNU/Library
make install-strip DESTDIR=%{buildroot}
popd
Invoke make install instead of make install-strip.
(Background: executables must not be stripped for debuginfo generateion.
Rpm automatically strips them later.
Finally there are more questionable items in your spec:
* autogen is supposed to be run in %prep (not in %build)
* you are passing --disable-static to %configure
= find %{buildroot} -name '*.a' -exec rm -f {} ';'
is superfluous.
* Many source files carry bogus permissions. chmod -x them in %
A visible symptom is rpmlint complaining about files in *debuginfo:
...
libzen-debuginfo.x86_64: W: spurious-executable-perm
/usr/src/debug/ZenLib/Source/ZenLib/Format/Http/Http_Cookies.cpp
libzen-debuginfo.x86_64: E: wrong-script-end-of-line-encoding
/usr/src/debug/ZenLib/Source/ZenLib/Format/Http/Http_Cookies.cpp
libzen-debuginfo.x86_64: W: spurious-executable-perm
/usr/src/debug/ZenLib/Source/ZenLib/ZtringListListF.h
...
* Another MUSTFIX:
libzen-devel.x86_64: E: script-without-shebang /usr/bin/libzen-config
--
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug
https://bugzilla.redhat.com/token.cgi?t=NXmi79YqQLa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 822329] Review Request: libzen - Shared library for libmediainfo and medianfo
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=822329 --- Comment #34 from Ralf Corsepius rc040...@freenet.de --- (In reply to comment #32) OK. I change make install-strip to make install. And create debuginfo subpackage. Fixed. Some wildcards in %files section removed. dos2unix removed. Instead it used sed. ./autogen - is upstream authors script Forget about it. It's crap. Use autoreconf or switch to using patches -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=dM4IU1glcLa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 915005] Review Request: qt5-qttools - Qt5 - QtTool components
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=915005 --- Comment #12 from Gregor Tätzner gre...@freenet.de --- ah, last point: you are installing quite some gui applications (assistant, dbusviewer) but there are no signs of desktop files. I'm rather puzzled that these are not shipped in the tarball. I suppose we have to inject our own, as the suse team did -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=twYqwQRG8xa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 822329] Review Request: libzen - Shared library for libmediainfo and medianfo
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=822329
--- Comment #35 from Vasiliy Glazov vasc...@gmail.com ---
Corrected SPEC: https://raw.github.com/RussianFedora/libzen/master/libzen.spec
Corrected SRPM:
http://koji.russianfedora.ru/koji/getfile?taskID=71532name=libzen-0.4.28-6.fc19.R.src.rpm
Removed find %{buildroot} -name '*.a' -exec rm -f {} ';
And corrected others. Added shebang. Moved ./autogen to %prep. All
spurious-executable-perm and wrong-script-end-of-line-encoding corrected.
But I am not sure what i must write instead of ./autogen, because if I use
autoreconf it causes errors and can't compile. Why we can't use ./autogen?
--
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug
https://bugzilla.redhat.com/token.cgi?t=fognbQ7E2Da=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 822329] Review Request: libzen - Shared library for libmediainfo and medianfo
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=822329 --- Comment #36 from Vasiliy Glazov vasc...@gmail.com --- Spec URL: https://raw.github.com/RussianFedora/libzen/master/libzen.spec SRPM URL: http://koji.russianfedora.ru/koji/getfile?taskID=71532name=libzen-0.4.28-6.fc19.R.src.rpm -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=KaVetMyUFJa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 928061] Review Request: nagstamon - Nagios status monitor for desktop
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=928061 Eduardo Echeverria echevemas...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED Blocks|177841 (FE-NEEDSPONSOR) | Assignee|nob...@fedoraproject.org|echevemas...@gmail.com Flags||fedora-review? -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=cmO1wGbtG3a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 928061] Review Request: nagstamon - Nagios status monitor for desktop
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=928061
Eduardo Echeverria echevemas...@gmail.com changed:
What|Removed |Added
Flags|fedora-review? |
Flags||fedora-review+
--- Comment #20 from Eduardo Echeverria echevemas...@gmail.com ---
Yes, nikita For some time was aware of the review, and was in CC list ;), I
made some comments in the review for another packager, and I think that your
review was acceptable
I'm lifting FE-NEEDSPONSOR, congrats you're sponsored
Package Review
==
Key:
[x] = Pass
[!] = Fail
[-] = Not applicable
[?] = Not evaluated
[ ] = Manual review needed
Issues:
===
- update-desktop-database is invoked when required
Note: desktop file(s) in nagstamon
See: http://fedoraproject.org/wiki/Packaging:ScriptletSnippets#Icon_Cache
this is a false positive, the package don't installs icons in
%{_datadir}/icons
= MUST items =
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]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[-]: Development files must be in a -devel package
[x]: Package requires other packages for directories it uses.
[x]: Package uses nothing in %doc for runtime.
[x]: Package is not known to require ExcludeArch.
[x]: Package complies to the Packaging Guidelines
[x]: License field in the package spec file matches the actual license.
Note: Checking patched sources after %prep for licenses. Licenses found:
GPL (v2 or later) (with incorrect FSF address), GPL, GPL (v2 or
later), Unknown or generated. 4 files have unknown license. Detailed
output of licensecheck in
/home/makerpm/ngapril/928061-nagstamon/licensecheck.txt
[x]: Package consistently uses macro is (instead of hard-coded directory
names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
Provides are present.
[x]: Package must own all directories that it creates.
[x]: Package does not own files or directories owned by other packages.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[-]: Large documentation must go in a -doc subpackage.
Note: Documentation size is 112640 bytes in 3 files.
[x]: All build dependencies are listed in BuildRequires, except for any that
are listed in the exceptions section of Packaging Guidelines.
[x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
beginning of %install.
[x]: Each %files section contains %defattr if rpm 4.4
[x]: Macros in Summary, %description expandable at SRPM build time.
[x]: Package contains desktop file if it is a GUI application.
[x]: Package installs a %{name}.desktop using desktop-file-install if there is
such a file.
[x]: Package does not contain duplicates in %files.
[x]: Permissions on files are set properly.
[-]: Fully versioned dependency in subpackages, if present.
[x]: Spec file lacks Packager, Vendor, PreReq tags.
[x]: If (and only if) the source package includes the text of the license(s)
in its own file, then that file, containing the text of the license(s)
for the package is included in %doc.
[-]: Package use %makeinstall only when make install' ' DESTDIR=... doesn't
work.
[x]: Package is named using only allowed ASCII characters.
[x]: Package do not use a name that already exist
[x]: Package is not relocatable.
[x]: Sources used to build the package match the upstream source, as provided
in the spec URL.
[x]: Spec file name must match the spec package %{name}, in the format
%{name}.spec.
[x]: File names are valid UTF-8.
[x]: Packages must not store files under /srv, /opt or /usr/local
[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).
Python:
[x]: Python eggs must not download any dependencies during the build process.
[x]: A package which is used by another package via an egg interface should
provide egg info.
[x]: Package meets the Packaging Guidelines::Python
[x]: Package contains BR: python2-devel or python3-devel
[x]: Binary eggs must be removed in %prep
= SHOULD items =
Generic:
[-]: If the source package does not include license text(s) as a separate file
from upstream, the
[Bug 952093] New: Review Request: python-wtf-peewee - WTForms integration for peewee models
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=952093 Bug ID: 952093 Summary: Review Request: python-wtf-peewee - WTForms integration for peewee models Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: mstuc...@redhat.com QA Contact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Category: --- Spec URL: http://mstuchli.fedorapeople.org/python-wtf-peewee.spec SRPM URL: http://mstuchli.fedorapeople.org/python-wtf-peewee-0.2.1-1.fc18.src.rpm Description: WTForms integration for peewee models Fedora Account System Username: mstuchli -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=9gJVgRabQ9a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 822329] Review Request: libzen - Shared library for libmediainfo and medianfo
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=822329 --- Comment #37 from Jérôme Martinez jer...@mediaarea.net --- (In reply to comment #35) But I am not sure what i must write instead of ./autogen, because if I use autoreconf it causes errors and can't compile. I think it is OK with: autoreconf -i -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=GPOQpWaXDoa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 822329] Review Request: libzen - Shared library for libmediainfo and medianfo
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=822329 --- Comment #38 from Vasiliy Glazov vasc...@gmail.com --- Spec URL: https://raw.github.com/RussianFedora/libzen/master/libzen.spec SRPM URL: http://koji.russianfedora.ru/koji/getfile?taskID=71538name=libzen-0.4.28-6.fc19.R.src.rpm Thanks, changed ./autogen to autoreconf -i. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=Yy9q59ydJRa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 891171] Review Request: npm - Node.js package manager
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=891171 --- Comment #25 from Fedora Update System upda...@fedoraproject.org --- node-gyp-0.9.5-2.el6,nodejs-abbrev-1.0.4-4.el6,nodejs-ansi-0.1.2-6.el6,nodejs-archy-0.0.2-7.el6,nodejs-async-0.2.6-2.el6,nodejs-aws-sign-0.2.0-2.el6,nodejs-bindings-1.1.0-2.el6,nodejs-block-stream-0.0.6-6.el6,nodejs-boom-0.4.0-2.el6,nodejs-chownr-0.0.1-8.el6,nodejs-combined-stream-0.0.4-2.el6,nodejs-config-chain-1.1.5-2.el6,nodejs-couch-login-0.1.15-4.el6,nodejs-cryptiles-0.2.0-2.el6,nodejs-delayed-stream-0.0.5-4.el6,nodejs-forever-agent-0.2.0-2.el6,nodejs-form-data-0.0.7-2.el6,nodejs-fstream-0.1.22-2.el6,nodejs-fstream-ignore-0.0.6-2.el6,nodejs-fstream-npm-0.1.4-2.el6,nodejs-glob-3.1.21-2.el6,nodejs-graceful-fs-1.2.0-2.el6,nodejs-hawk-0.12.1-2.el6,nodejs-hoek-0.8.1-2.el6,nodejs-inherits-1.0.0-8.el6,nodejs-ini-1.1.0-2.el6,nodejs-init-package-json-0.0.7-5.el6,nodejs-json-stringify-safe-4.0.0-3.el6,nodejs-lockfile-0.3.0-2.el6,nodejs-lru-cache-2.3.0-2.el6,nodejs-mime-1.2.9-2.el6,nodejs-minimatch-0.2.11-2.el6,nodejs-mkdirp-0.3.5-2.el6,nodejs-mute-stream-0.0.3-5.el6,nodejs-node-u! uid-1.4.0-3.el6,nodejs-nopt-2.1.1-2.el6,nodejs-npmconf-0.0.23-2.el6,nodejs-npmlog-0.0.2-4.el6,nodejs-npm-registry-client-0.2.20-2.el6,nodejs-oauth-sign-0.2.0-2.el6,nodejs-once-1.1.1-4.el6,nodejs-opener-1.3.0-6.el6,nodejs-opts-1.2.2-4.el6,nodejs-osenv-0.0.3-4.el6,nodejs-promzard-0.2.0-5.el6,nodejs-proto-list-1.2.2-4.el6,nodejs-qs-0.5.6-2.el6,nodejs-read-1.0.4-6.el6,nodejs-read-installed-0.1.1-2.el6,nodejs-read-package-json-0.3.0-4.el6,nodejs-request-2.16.6-3.el6,nodejs-retry-0.6.0-4.el6,nodejs-rimraf-2.1.4-2.el6,nodejs-semver-1.1.4-2.el6,nodejs-sigmund-1.0.0-4.el6,nodejs-slide-1.1.3-6.el6,nodejs-sntp-0.2.1-2.el6,nodejs-tar-0.1.17-2.el6,nodejs-tunnel-agent-0.2.0-2.el6,nodejs-uid-number-0.0.3-6.el6,nodejs-which-1.0.5-6.el6,npm-1.2.17-4.el6 has been submitted as an update for Fedora EPEL 6. https://admin.fedoraproject.org/updates/node-gyp-0.9.5-2.el6,nodejs-abbrev-1.0.4-4.el6,nodejs-ansi-0.1.2-6.el6,nodejs-archy-0.0.2-7.el6,nodejs-async-0.2.6-2.el6,nodejs-aws-sign-0.2.0-2.el6,nodejs-bindings-1.1.0-2.el6,nodejs-block-stream-0.0.6-6.el6,nodejs-boom-0.4.0-2.el6,nodejs-chownr-0.0.1-8.el6,nodejs-combined-stream-0.0.4-2.el6,nodejs-config-chain-1.1.5-2.el6,nodejs-couch-login-0.1.15-4.el6,nodejs-cryptiles-0.2.0-2.el6,nodejs-delayed-stream-0.0.5-4.el6,nodejs-forever-agent-0.2.0-2.el6,nodejs-form-data-0.0.7-2.el6,nodejs-fstream-0.1.22-2.el6,nodejs-fstream-ignore-0.0.6-2.el6,nodejs-fstream-npm-0.1.4-2.el6,nodejs-glob-3.1.21-2.el6,nodejs-graceful-fs-1.2.0-2.el6,nodejs-hawk-0.12.1-2.el6,nodejs-hoek-0.8.1-2.el6,nodejs-inherits-1.0.0-8.el6,nodejs-ini-1.1.0-2.el6,nodejs-init-package-json-0.0.7-5.el6,nodejs-json-stringify-safe-4.0.0-3.el6,nodejs-lockfile-0.3.0-2.el6,nodejs-lru-cache-2.3.0-2.el6,nodejs-mime-1.2.9-2.el6,nodejs-minimatch-0.2.11-2.el6,nodejs-mkdirp-0.3.5-2.el6,node! js-mute-stream-0.0.3-5.el6,nodejs-node-uuid-1.4.0-3.el6,nodejs-nopt-2.1.1-2.el6,nodejs-npmconf-0.0.23-2.el6,nodejs-npmlog-0.0.2-4.el6,nodejs-npm-registry-client-0.2.20-2.el6,nodejs-oauth-sign-0.2.0-2.el6,nodejs-once-1.1.1-4.el6,nodejs-opener-1.3.0-6.el6,nodejs-opts-1.2.2-4.el6,nodejs-osenv-0.0.3-4.el6,nodejs-promzard-0.2.0-5.el6,nodejs-proto-list-1.2.2-4.el6,nodejs-qs-0.5.6-2.el6,nodejs-read-1.0.4-6.el6,nodejs-read-installed-0.1.1-2.el6,nodejs-read-package-json-0.3.0-4.el6,nodejs-request-2.16.6-3.el6,nodejs-retry-0.6.0-4.el6,nodejs-rimraf-2.1.4-2.el6,nodejs-semver-1.1.4-2.el6,nodejs-sigmund-1.0.0-4.el6,nodejs-slide-1.1.3-6.el6,nodejs-sntp-0.2.1-2.el6,nodejs-tar-0.1.17-2.el6,nodejs-tunnel-agent-0.2.0-2.el6,nodejs-uid-number-0.0.3-6.el6,nodejs-which-1.0.5-6.el6,npm-1.2.17-4.el6 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=JRkruAYBKla=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 913367] Review Request: gpick - Advanced color picker
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=913367 --- Comment #12 from Eugene A. Pivnev ti.eug...@gmail.com --- Ooops... Seems expat-devel in F17 has no pkg-config file: http://kojipkgs.fedoraproject.org//work/tasks/2410/5252410/mock_output.log I'm sorry. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=sOhn0BeW4Ya=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 952093] Review Request: python-wtf-peewee - WTForms integration for peewee models
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=952093 Robert Kuska rku...@redhat.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||rku...@redhat.com --- Comment #1 from Robert Kuska rku...@redhat.com --- I'll take this for a review. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=NUOqoS96ifa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 952093] Review Request: python-wtf-peewee - WTForms integration for peewee models
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=952093 Robert Kuska rku...@redhat.com changed: What|Removed |Added Assignee|nob...@fedoraproject.org|rku...@redhat.com -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=LBpxOeeWF0a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 910146] Review Request: python-SocksipyChain - A Python SOCKS/HTTP Proxy module
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=910146 --- Comment #6 from Miroslav Suchý msu...@redhat.com --- Not sure what you mean, the lastest is still 2.0.12. Yes. Correct. Ignore my previous comment about this one issue. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=sxiElEJxmea=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 910146] Review Request: python-SocksipyChain - A Python SOCKS/HTTP Proxy module
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=910146 Miroslav Suchý msu...@redhat.com changed: What|Removed |Added Flags|fedora-review? | Flags||fedora-review+ --- Comment #7 from Miroslav Suchý msu...@redhat.com --- I downloaded src.rpm from that koji task. But next time it is better to post link to new spec and new src.rpm in bugzilla comment. All issues resolved. APPROVED. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=TvHrBtzAKHa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 951360] Review Request: nodejs-uglify-js - JavaScript parser/compressor/beautifier
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=951360 Stanislav Ochotnicky socho...@redhat.com changed: What|Removed |Added Flags|fedora-review? | Flags||fedora-review+ --- Comment #5 from Stanislav Ochotnicky socho...@redhat.com --- I've sponsored Tomas now, package looks good. I'll be mentoring him in coming weeks. automatic requires look OK, licensing is OK, tests pass...APPROVED -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=cvLTbMbnnsa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 952141] New: Review Request: python-mccabe - McCabe complexity checker
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=952141 Bug ID: 952141 Summary: Review Request: python-mccabe - McCabe complexity checker Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: mc...@redhat.com QA Contact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Category: --- Spec URL: http://mcepl.fedorapeople.org/rpms/python-mccabe.spec SRPM URL: http://mcepl.fedorapeople.org/rpms/python-mccabe-0.2.1-1.el7.src.rpm Description: Ned's script to check McCabe complexity. This module provides a plugin for ``flake8``, the Python code checker. Fedora Account System Username: mcepl -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=C2ErP5Cii4a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 952141] Review Request: python-mccabe - McCabe complexity checker
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=952141 Matěj Cepl mc...@redhat.com changed: What|Removed |Added Assignee|nob...@fedoraproject.org|pin...@pingoured.fr Flags||fedora-review? -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=sXlo1v7OQwa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 951360] Review Request: nodejs-uglify-js - JavaScript parser/compressor/beautifier
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=951360 Tomas Hrcka thr...@redhat.com changed: What|Removed |Added Flags|fedora-review+ | --- Comment #6 from Tomas Hrcka thr...@redhat.com --- New Package SCM Request === Package Name: nodejs-uglify-js Short Description: JavaScript parser/compressor/beautifier Owners: humaton Branches: f18 f19 f20 el6 InitialCC: -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=wbuSowgJg6a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 951004] Review Request: rubygem-afm - Reading Adobe Font Metrics (afm) files
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=951004 Josef Stribny jstri...@redhat.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution|--- |RAWHIDE Last Closed||2013-04-15 05:53:43 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=kvIsWRvsJma=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 883437] Review Request: rubygem-prawn - A fast and nimble PDF generator for Ruby
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=883437 Bug 883437 depends on bug 951004, which changed state. Bug 951004 Summary: Review Request: rubygem-afm - Reading Adobe Font Metrics (afm) files https://bugzilla.redhat.com/show_bug.cgi?id=951004 What|Removed |Added Status|ASSIGNED|CLOSED Resolution|--- |RAWHIDE -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=5iBadECLkka=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 915005] Review Request: qt5-qttools - Qt5 - QtTool components
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=915005 --- Comment #13 from Rex Dieter rdie...@math.unl.edu --- We don't *have* to, but .desktop files would be nice for anything one would want to appear in menus. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=SqNdtIZ341a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 948345] Review Request: mozjs17 - JavaScript interpreter and libraries
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=948345 Colin Walters walt...@redhat.com changed: What|Removed |Added Status|ASSIGNED|MODIFIED --- Comment #20 from Colin Walters walt...@redhat.com --- (In reply to comment #19) Hey, did you forget to commit the two fixes from comment #16 to git? Somehow apparently, yes; probinson fixed one, I just pushed the other. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=2CCxiqz5TIa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 915005] Review Request: qt5-qttools - Qt5 - QtTool components
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=915005 --- Comment #14 from Gregor Tätzner gre...@freenet.de --- (In reply to comment #13) We don't *have* to, but .desktop files would be nice for anything one would want to appear in menus. no, it wouldn't be nice - it's necessary... http://fedoraproject.org/wiki/Packaging:Guidelines#Desktop_files Since we don't present qdbusviewer to the user in qt4 packaging, we can get away with that. But for assistant, a menu entry really has to be provided (for designer and linguist too). Even if there is not much to see yet ;) Now if I think about it, the package splitting feels *very* strange. What are doing all those tools in devel? Imho all binaries should be moved to qttools since the target group are devs anyway. Or even better: move every tool into his own subpkg. But again no MUST, so your choice. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=2AkjFySHsVa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 951360] Review Request: nodejs-uglify-js - JavaScript parser/compressor/beautifier
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=951360 Tomas Hrcka thr...@redhat.com changed: What|Removed |Added Flags||fedora-cvs? -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=l8KiHgpmHja=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 883437] Review Request: rubygem-prawn - A fast and nimble PDF generator for Ruby
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=883437 --- Comment #3 from Josef Stribny jstri...@redhat.com --- afm is done, so lets continue with prawn: - upgraded to 1.0.0.rc2 - adjusted description SPEC: http://data-strzibny.rhcloud.com/rubygem-prawn.spec SRPM: http://data-strzibny.rhcloud.com/rubygem-prawn-1.0.0.rc2-1.fc20.src.rpm Koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=5253021 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=IoLGpWE21ba=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 915005] Review Request: qt5-qttools - Qt5 - QtTool components
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=915005 --- Comment #15 from Rex Dieter rdie...@math.unl.edu --- Fair enough. I'll put adding .desktop files on my todo list. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=3jDF7k56Nqa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 951360] Review Request: nodejs-uglify-js - JavaScript parser/compressor/beautifier
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=951360 Stanislav Ochotnicky socho...@redhat.com changed: What|Removed |Added Flags||fedora-review+ -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=pNcwplu7wHa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 952229] New: Review Request: canl-c++ - EMI Common Authentication library - bindings for C++
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=952229 Bug ID: 952229 Summary: Review Request: canl-c++ - EMI Common Authentication library - bindings for C++ Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: mattias.ell...@fysast.uu.se QA Contact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Category: --- Spec URL: http://www.grid.tsl.uu.se/review/canl-c++.spec SRPM URL: http://www.grid.tsl.uu.se/review/canl-c++-1.0.0-1.fc18.src.rpm Description: This is the C++ part of the EMI caNl -- the Common Authentication Library. Fedora Account System Username: ellert -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=4qLCStZvwla=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 926968] Review Request: php-MiniTemplator - Compact template engine for HTML files
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=926968 --- Comment #1 from Remi Collet fed...@famillecollet.com --- Created attachment 735882 -- https://bugzilla.redhat.com/attachment.cgi?id=735882action=edit review.txt Generated by fedora-review 0.4.0 (660ce56) last change: 2013-01-29 Buildroot used: fedora-rawhide-x86_64 Command line :/usr/bin/fedora-review -b 926968 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=nk9be9cjG9a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 926968] Review Request: php-MiniTemplator - Compact template engine for HTML files
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=926968 Remi Collet fed...@famillecollet.com changed: What|Removed |Added Assignee|nob...@fedoraproject.org|fed...@famillecollet.com --- Comment #2 from Remi Collet fed...@famillecollet.com --- Created attachment 735884 -- https://bugzilla.redhat.com/attachment.cgi?id=735884action=edit phpci.log phpci version 2.15.0. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=3FVOwtQpcha=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 926968] Review Request: php-MiniTemplator - Compact template engine for HTML files
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=926968 Remi Collet fed...@famillecollet.com changed: What|Removed |Added Flags||fedora-review? --- Comment #3 from Remi Collet fed...@famillecollet.com --- First important question : do you really want to maintain this package which seems obviously dead upstream: ~9 year old version... [!]: All build dependencies are listed in BuildRequires, except for any that are listed in the exceptions section of Packaging Guidelines. Note: These BR are not needed: unzip See: http://fedoraproject.org/wiki/Packaging/Guidelines#Exceptions_2 Also use %setup -q -c instead of direct call to unzip [!]: If the source package does not include license text(s) as a separate file from upstream, the packager SHOULD query upstream to include it. Please add link to the request [!]: Packages should try to preserve timestamps of original installed files. use install -p or cp -p instead of cp [!]: SourceX is a working URL. Non-versioned URL, should switch to sourceforge which provides versioned URL. [!]: Rpmlint is run on all installed packages. Easy to fix: wrong-file-end-of-line-encoding -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=iBsU8cfdRva=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 951496] Review Request: gimplensfun - gimp plugin to correct lens distortion
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=951496 --- Comment #1 from Graham White graham_al...@hotmail.com --- Successful Koji Build: http://koji.fedoraproject.org/koji/taskinfo?taskID=5253393 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=XhmG4dp6rua=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 920859] Review Request: gnome-klotski - GNOME Klotski game
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=920859 Jon Ciesla limburg...@gmail.com changed: What|Removed |Added Flags|fedora-cvs? | Flags||fedora-cvs+ -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=41bdjs35Cea=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 920859] Review Request: gnome-klotski - GNOME Klotski game
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=920859 --- Comment #7 from Jon Ciesla limburg...@gmail.com --- Git done (by process-git-requests). -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=iL6MHJRi6Da=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 946856] Review Request: spectrwm - Minimalist tiling window manager written in C
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=946856 --- Comment #12 from Tom spot Callaway tcall...@redhat.com --- You can, if you wish, define a %global macro (or macros) for the sharedlib versioning where you make the symlinks, or, if you're feeling brave, you can patch up the Makefile to make the appropriate symlinks (that's really the more appropriate option and eliminates the chance for a disconnect). -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=cCW9vw9t4fa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 920861] Review Request: gnome-tetravex - GNOME Tetravex game
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=920861 Jon Ciesla limburg...@gmail.com changed: What|Removed |Added Flags|fedora-cvs? | Flags||fedora-cvs+ -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=IPN0Ob63vVa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 920861] Review Request: gnome-tetravex - GNOME Tetravex game
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=920861 --- Comment #5 from Jon Ciesla limburg...@gmail.com --- Git done (by process-git-requests). -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=fiYI1Pu6Sha=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 951360] Review Request: nodejs-uglify-js - JavaScript parser/compressor/beautifier
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=951360 Jon Ciesla limburg...@gmail.com changed: What|Removed |Added Flags|fedora-cvs? | Flags||fedora-cvs+ -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=itNnX8VHnxa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 951360] Review Request: nodejs-uglify-js - JavaScript parser/compressor/beautifier
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=951360 --- Comment #7 from Jon Ciesla limburg...@gmail.com --- Git done (by process-git-requests). -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=qbSGlPk7dla=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 916679] Review Request: perl-ElasticSearch - API for communicating with ElasticSearch
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=916679 Petr Šabata psab...@redhat.com changed: What|Removed |Added Flags|fedora-review? | Flags||fedora-review+ --- Comment #3 from Petr Šabata psab...@redhat.com --- Ok, approving. (...) %changelog +* Sun Apr 14 2013 Emmanuel Seyman emman...@seyman.fr - 0.64-2 +Take into account review comments (#916679) + Looks a bit unusual without the preceding '- ', however I'm not sure if it's required... -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=fB6nq4f3gYa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 883437] Review Request: rubygem-prawn - A fast and nimble PDF generator for Ruby
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=883437 --- Comment #4 from Josef Stribny jstri...@redhat.com --- I fixed versioning according to guidelines [1]. SPEC: http://data-strzibny.rhcloud.com/rubygem-prawn.spec SRPM: http://data-strzibny.rhcloud.com/rubygem-prawn-1.0.0-0.1.rc2.fc20.src.rpm Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=5253789 [1] https://fedoraproject.org/wiki/Packaging:NamingGuidelines#Pre-Release_packages -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=ve897B888Ja=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 909671] Review Request: libexif-gtk3 - Library to display EXIF tags in GTK3 programs
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=909671
Matthieu Saulnier cas...@mail.casperlefantom.net changed:
What|Removed |Added
Summary|Review Request: libexif-gtk |Review Request:
|- Library to display EXIF |libexif-gtk3 - Library to
|tags in GTK programs|display EXIF tags in GTK3
||programs
--- Comment #5 from Matthieu Saulnier cas...@mail.casperlefantom.net ---
%changelog
* Mon Apr 15 2013 Matthieu Saulnier fan...@fedoraproject.org -
0.4.1-0.2.20130415cvs
- New upstream snapshot
- Rename Main package as libexif-gtk3 (old name was libexif-gtk)
- Rename first subpackage as libexif-gtk3-devel (old name was
libexif-gtk-devel)
- Rename second subpackage as libexif-gtk2 (old name was
libexif-gtk-gtk3-libs)
- Rename third subpackage as libexif-gtk2-devel (old name was
libexif-gtk-gtk3-devel)
- Add %%{upname} macro to preserve some upstream paths
New SPEC:http://fantom.fedorapeople.org/libexif-gtk3.spec
New SRPM:
http://fantom.fedorapeople.org/libexif-gtk3-0.4.1-0.2.20130415cvs.fc19.src.rpm
--
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug
https://bugzilla.redhat.com/token.cgi?t=fDnzZ1f7rfa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 952141] Review Request: python-mccabe - McCabe complexity checker
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=952141 --- Comment #1 from Matěj Cepl mc...@redhat.com --- Koji scratch build http://koji.fedoraproject.org/koji/taskinfo?taskID=5253450 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=T7Uy9WDrHwa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 839056] Review Request: python-flake8 - code checking using pep8 and pyflakes
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=839056 Matěj Cepl mc...@redhat.com changed: What|Removed |Added Depends On||952141 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=45l6U8t68ga=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 952141] Review Request: python-mccabe - McCabe complexity checker
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=952141 Matěj Cepl mc...@redhat.com changed: What|Removed |Added Blocks||839056 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=Nh1CwQmwtHa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 950639] Review Request: c-ares19 - A library that performs asynchronous DNS operations
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=950639 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |MODIFIED -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=NbDSfNrfqga=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 750902] Review Request: sleep - Multi-paradigm scripting language for Java
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=750902 --- Comment #14 from Jaromír Cápík jca...@redhat.com --- Hi Orion. We somehow forgot about the name conflict mentioned by Ralf. Could you please modify the package name to jsleep, java-sleep or sleep-java? That would be probably the last modification and the package is good to be approved. Thanks, Jaromir. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=cqduGXtrkYa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 913367] Review Request: gpick - Advanced color picker
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=913367 --- Comment #13 from Luya Tshimbalanga l...@fedoraproject.org --- Here is these updated files Spec URL: http://luya.fedorapeople.org/packages/SPECS/gpick.spec SRPM URL: http://luya.fedorapeople.org/packages/SRPMS/gpick-0.2.4-3.fc18.src.rpm Koji build of f19: http://koji.fedoraproject.org/koji/taskinfo?taskID=5254527 Koji build for f17 (with expat-devel): http://koji.fedoraproject.org/koji/taskinfo?taskID=5254552 Once accepted, f18+ version will use pkg-config. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=xrOxhFOdlba=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 946856] Review Request: spectrwm - Minimalist tiling window manager written in C
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=946856 --- Comment #13 from Lokesh Mandvekar l...@buffalo.edu --- I'll make that change by tonight and update this. Is there anything else I need to consider? -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=zgrOsSUqgEa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 883437] Review Request: rubygem-prawn - A fast and nimble PDF generator for Ruby
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=883437
--- Comment #5 from Miroslav Suchý msu...@redhat.com ---
It seems to me that
%{gem_instdir}/data
is used in
lib/prawn/core.rb:
DATADIR = File.expand_path(File.join(dir, '..', '..', 'data'))
So it should be in main package and not in -doc.
--
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug
https://bugzilla.redhat.com/token.cgi?t=xhrBYtmlNba=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 928061] Review Request: nagstamon - Nagios status monitor for desktop
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=928061 Nikita Klimov n...@jaur.su changed: What|Removed |Added Flags||fedora-cvs? --- Comment #21 from Nikita Klimov n...@jaur.su --- New Package SCM Request === Package Name: nagstamon Short Description: Nagios status monitor for desktop Owners: jaur echevemaster Branches: f18 f19 InitialCC: -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=mGSAVXXc5da=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 877651] Review Request: sagemath - A free open-source mathematics software system
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=877651 --- Comment #65 from José Matos jama...@fc.up.pt --- Created attachment 735989 -- https://bugzilla.redhat.com/attachment.cgi?id=735989action=edit Crash report of sage I took the last srpm and I have used mock to rebuild it for Fedora 19. Installing the resulting rpms, via yum install sagemath worked. As soon as I start sagemath from the menu I get this crash with the corresponding crash report. My purpose here is just to test sagemath under a normal scenario. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=gPmS1qathHa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 951777] Review Request: python-pygal - A python svg graph plotting library
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=951777 --- Comment #3 from Ralph Bean rb...@redhat.com --- Asked about license clarification here - https://github.com/Kozea/pygal/pull/32 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=B0kDFz0GsAa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 951775] Review Request: python-cairosvg - A Simple SVG Converter for Cairo
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=951775 --- Comment #3 from Ralph Bean rb...@redhat.com --- I asked about license clarification here - https://github.com/Kozea/CairoSVG/pull/8 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=cs2SIW1ylXa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 952343] New: Review Request: python-svg-path - SVG path objects and parser
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=952343 Bug ID: 952343 Summary: Review Request: python-svg-path - SVG path objects and parser Product: Fedora Version: rawhide Component: Package Review Severity: unspecified Priority: unspecified Assignee: nob...@fedoraproject.org Reporter: rb...@redhat.com QA Contact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Category: --- Spec URL: http://ralph.fedorapeople.org//python-svg-path.spec SRPM URL: http://ralph.fedorapeople.org//python-svg-path-1.0b1-1.fc18.src.rpm Description: svg.path is a collection of objects that implement the different path commands in SVG, and a parser for SVG path definitions. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=VxX2ewddYRa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 952343] Review Request: python-svg-path - SVG path objects and parser
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=952343 --- Comment #1 from Ralph Bean rb...@redhat.com --- This package built on koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=5255408 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=hEsDNjacQOa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 952141] Review Request: python-mccabe - McCabe complexity checker
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=952141 --- Comment #2 from Pierre-YvesChibon pin...@pingoured.fr --- * package meets naming guidelines * package meets packaging guidelines * spec is clean * license MIT is fine * source matches upstream: 5a3f3fa6a4bad126c887dab682f5 * no missing BR * no unnecessary BR * no locales * not relocatable * owns all directories that it creates * no duplicate files * permissions ok * macro use consistent * code, not content * no need for -docs * nothing in %doc affects runtime * no need for .desktop file Please fix the changelog. I still have to test it and I'll approve it. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=emL3k6w7w5a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 952347] New: Review Request: ovirt-image-uploader - Image Uploader tool for oVirt Engine
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=952347 Bug ID: 952347 Summary: Review Request: ovirt-image-uploader - Image Uploader tool for oVirt Engine Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: knese...@redhat.com QA Contact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Category: --- Spec URL: http://knesenko.fedorapeople.org/ovirt-image-uploader.spec.in SRPM URL: http://knesenko.fedorapeople.org/ovirt-image-uploader-3.2.1-1.fc17.src.rpm Description: Image Uploader tool for oVirt Engine Fedora Account System Username: knesenko -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=SIAoGHF1iGa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 952347] Review Request: ovirt-image-uploader - Image Uploader tool for oVirt Engine
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=952347 --- Comment #1 from Kiril Nesenko knese...@redhat.com --- Spec URL: http://knesenko.fedorapeople.org/ovirt-image-uploader.spec SRPM URL: http://knesenko.fedorapeople.org/ovirt-image-uploader-3.2.1-1.fc17.src.rpm Description: Image Uploader tool for oVirt Engine Fedora Account System Username: knesenko -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=Y8yRlI4lsVa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 952351] New: Review Request: ovirt-iso-uploader - ISO Uploader tool for oVirt Engine
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=952351 Bug ID: 952351 Summary: Review Request: ovirt-iso-uploader - ISO Uploader tool for oVirt Engine Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: knese...@redhat.com QA Contact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Category: --- Spec URL: http://knesenko.fedorapeople.org/ovirt-iso-uploader.spec SRPM URL: http://knesenko.fedorapeople.org/ovirt-iso-uploader-3.2.1-1.fc17.src.rpm Description: ISO Uploader tool for oVirt Engine Fedora Account System Username: knesenko -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=NOPtBUhYhja=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 952351] Review Request: ovirt-iso-uploader - ISO Uploader tool for oVirt Engine
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=952351 Kiril Nesenko knese...@redhat.com changed: What|Removed |Added Blocks||177841 (FE-NEEDSPONSOR) -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=ub29Tm7lifa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 952347] Review Request: ovirt-image-uploader - Image Uploader tool for oVirt Engine
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=952347 Kiril Nesenko knese...@redhat.com changed: What|Removed |Added Blocks||177841 (FE-NEEDSPONSOR) -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=KTVc2kH8iwa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 952354] New: Review Request: ovirt-log-collector - Log Collector tool for oVirt Engine
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=952354 Bug ID: 952354 Summary: Review Request: ovirt-log-collector - Log Collector tool for oVirt Engine Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: knese...@redhat.com QA Contact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Category: --- Spec URL: http://knesenko.fedorapeople.org/ovirt-log-collector.spec SRPM URL: http://knesenko.fedorapeople.org/ovirt-log-collector-3.2.1-1.fc17.src.rpm Description: Log Collector tool for oVirt Engine Fedora Account System Username: knesenko -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=3rsErrgJ7Da=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 952355] New: Review Request: hovercraft - Makes impress.js presentations from reStructuredText
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=952355 Bug ID: 952355 Summary: Review Request: hovercraft - Makes impress.js presentations from reStructuredText Product: Fedora Version: rawhide Component: Package Review Severity: unspecified Priority: unspecified Assignee: nob...@fedoraproject.org Reporter: rb...@redhat.com QA Contact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Category: --- Spec URL: http://ralph.fedorapeople.org//hovercraft.spec SRPM URL: http://ralph.fedorapeople.org//hovercraft-1.1-1.fc18.src.rpm Description: The merge of convenience and cool! Hovercraft is a tool to make impress.js presentations from reStructuredText. - Write your presentations in a text markup language. No slow, limiting GUI, no annoying HTML! - Pan, rotate and zoom in 3D, with automatic repositioning of slides! - A presenter console with notes and slide previews! - The slide show generated is in HTML, so you only need a web browser to show it. - Easy sharing, as it can be put up on a website for anyone to see! -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=ptPokSJn3ga=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 952354] Review Request: ovirt-log-collector - Log Collector tool for oVirt Engine
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=952354 Kiril Nesenko knese...@redhat.com changed: What|Removed |Added Blocks||177841 (FE-NEEDSPONSOR) -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=HzppJCqwkya=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 952355] Review Request: hovercraft - Makes impress.js presentations from reStructuredText
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=952355 Ralph Bean rb...@redhat.com changed: What|Removed |Added Depends On||951711, 952343 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=jF46RUV22oa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 952343] Review Request: python-svg-path - SVG path objects and parser
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=952343 Ralph Bean rb...@redhat.com changed: What|Removed |Added Blocks||952355 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=UJFJrRcU1fa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 951711] Review Request: impressjs - Javascript presentation framework
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=951711 Ralph Bean rb...@redhat.com changed: What|Removed |Added Blocks||952355 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=ahDI77m66da=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 928061] Review Request: nagstamon - Nagios status monitor for desktop
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=928061 Jon Ciesla limburg...@gmail.com changed: What|Removed |Added Flags|fedora-cvs? | Flags||fedora-cvs+ -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=TH4y2lVgzFa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 928061] Review Request: nagstamon - Nagios status monitor for desktop
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=928061 --- Comment #22 from Jon Ciesla limburg...@gmail.com --- Git done (by process-git-requests). -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=L02Avfdy3na=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 947453] Review Request: perl-ExtUtils-BuildRC - A reader for Build.PL configuration files
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=947453 Paul Howarth p...@city-fan.org changed: What|Removed |Added Blocks|947455 | -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=2qBfwgsbhua=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 947455] Review Request: perl-Module-Build-Tiny - A tiny replacement for Module::Build
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=947455 Paul Howarth p...@city-fan.org changed: What|Removed |Added Depends On|947453 | --- Comment #2 from Paul Howarth p...@city-fan.org --- Upstream update: http://www.city-fan.org/~paul/extras/perl-Module-Build-Tiny/perl-Module-Build-Tiny-0.017-1.fc20.src.rpm Spec URL unchanged. No longer depends on perl-ExtUtils-BuildRC. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=MeHnTdPoVza=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 947453] Review Request: perl-ExtUtils-BuildRC - A reader for Build.PL configuration files
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=947453 Paul Howarth p...@city-fan.org changed: What|Removed |Added Status|NEW |CLOSED Resolution|--- |NOTABUG Last Closed||2013-04-15 17:04:08 --- Comment #2 from Paul Howarth p...@city-fan.org --- Upstream has deprecated this module and it's no longer required by Module::Build::Tiny. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=5RbnAQHwM7a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 952141] Review Request: python-mccabe - McCabe complexity checker
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=952141 --- Comment #3 from Matěj Cepl mc...@redhat.com --- %changelog updated. http://mcepl.fedorapeople.org/rpms/python-mccabe-0.2.1-2.el7.src.rpm -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=M2nXILIdpra=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 928061] Review Request: nagstamon - Nagios status monitor for desktop
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=928061 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=v2L0QQswsNa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 928061] Review Request: nagstamon - Nagios status monitor for desktop
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=928061 --- Comment #23 from Fedora Update System upda...@fedoraproject.org --- nagstamon-0.9.9-7.fc18 has been submitted as an update for Fedora 18. https://admin.fedoraproject.org/updates/nagstamon-0.9.9-7.fc18 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=f3MXtTIuP3a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 904843] Review Request: acpica-tools - ACPICA tools for the development and debug of ACPI tables
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=904843 --- Comment #26 from Al Stone a...@redhat.com --- Re comment #25 on Exclude/Exclusive Arch: the package definitely works on x86* platforms and on ARM; the others are secondaries and will likely need help from the maintainers there should they want this package at all. Hence, I'm leaving out any exclusions. A new upstream version became available. Successful koji scratch build may be found here: http://koji.fedoraproject.org/koji/taskinfo?taskID=5256169 Updated versions can also be found here: Spec URL: http://fedorapeople.org/~ahs3/acpica-tools.spec SRPM URL: http://fedorapeople.org/~ahs3/acpica-tools-20130328-1.fc18.src.rpm -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=QrmWmSuCjHa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 877651] Review Request: sagemath - A free open-source mathematics software system
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=877651 --- Comment #66 from Paulo Andrade paulo.cesar.pereira.de.andr...@gmail.com --- (In reply to comment #65) Hopefully it should be some simple issue, as I do not see any problems, but I am currently only testing in f18 and rawhide. Created attachment 735989 [details] Crash report of sage I took the last srpm and I have used mock to rebuild it for Fedora 19. Installing the resulting rpms, via yum install sagemath worked. As soon as I start sagemath from the menu I get this crash with the corresponding crash report. Does the same problem happen if starting from an already open terminal? Works for me opening from kde menu. My purpose here is just to test sagemath under a normal scenario. Maybe there is some dependency problem with the subpackages. What does rpm -qa | grep sagemath show? -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=bt6bJCfZZFa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 877651] Review Request: sagemath - A free open-source mathematics software system
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=877651 --- Comment #67 from Paulo Andrade paulo.cesar.pereira.de.andr...@gmail.com --- Actually, there may be a problem right now, updating my test environment, as it may very likely be that it works with ipython-0.13.1 but not ipython-0.13.2. Will know it shortly. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=dkyxj5VO5ha=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 904843] Review Request: acpica-tools - ACPICA tools for the development and debug of ACPI tables
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=904843 Al Stone a...@redhat.com changed: What|Removed |Added Flags||fedora-review? -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=Gio847mWjha=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 891232] nodejs-ronn - markdown to roff/html converter
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=891232 --- Comment #8 from T.C. Hollingsworth tchollingswo...@gmail.com --- New Package SCM Request === Package Name: nodejs-ronn Short Description: Markdown to roff/html converter Owners: patches Branches: f19 f18 el6 InitialCC: -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=wsvYuY55uOa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 891232] nodejs-ronn - markdown to roff/html converter
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=891232 T.C. Hollingsworth tchollingswo...@gmail.com changed: What|Removed |Added Flags||fedora-cvs? -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=39mkPrKNSXa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 750902] Review Request: sleep - Multi-paradigm scripting language for Java
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=750902 --- Comment #15 from Orion Poplawski or...@cora.nwra.com --- Well, in comment #8 you indicated you were fine with the name. Personally I don't see a conflict here, but I can go with java-sleep if you insist. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=LpXl2cpSbta=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 882559] Review Request: mate-file-manager-image-converter - MATE Desktop file manager image converter plugin
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=882559 Dan Mashal dan.mas...@gmail.com changed: What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA Last Closed||2013-04-15 18:59:10 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=jsshXteOLga=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 894585] Review Request: coin-or-CoinUtils - Coin-or Utilities
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=894585 Jerry James loganje...@gmail.com changed: What|Removed |Added Flags|fedora-review? | Flags||fedora-review+ --- Comment #5 from Jerry James loganje...@gmail.com --- OK, that looks good. This package is APPROVED. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=V71N8AzOS8a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 914798] Review Request: jvgs - Minimalistic platform game loosely based on xkcd webcomic
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=914798 --- Comment #5 from Jerry James loganje...@gmail.com --- (In reply to comment #4) The WTFPL comes from the home page http://jvgs.sourceforge.net/ as well as the README in https://github.com/jaspervdj/jvgs But you are right, I did not full proof check it. It appears the license was changed after the last tarball upload to sourceforge. I changed to use the license in the tarball, and not the one in the main page neither in github sources. Oh, okay, I see it. That's confusing. Regarding Wesnoth packages: I believe it may not be reasonable to use. Probaly would need to requires wesnoth and symlink .ogg files from wesnoth data dir, and still, the files are likely very small. Example: $ du -h BUILD/jvgs-0.5-src/resources/hedgehog/hedgehog.ogg 12K BUILD/jvgs-0.5-src/resources/hedgehog/hedgehog.ogg and there are like only 3 or 4 of those. OK, that's fine. I just thought the question should be asked. It is not under %_datadir/icons but in %_datadir/pixmaps. I have at least one other package that installs an icon in %_datadir/pixmaps. Should I move the icon to %_datadir/icons and add the scriptlets? Oh. Hmmm. I don't know the answer to that question. Just leave it alone for now, and I'll try to find out. I will try to confirm what I believe to be the case, but I think this is a specially edited/built font just for the game, to have the hand written effect. That's my opinion, too. I don't think you need to do anything special for that one font file. I made a new package for the sake of correcting what I could, but I will email the author to try to clarify the license issues. Great! I'll wait for word on the license situation, then. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=oPJobBtngua=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 946856] Review Request: spectrwm - Minimalist tiling window manager written in C
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=946856 --- Comment #14 from Lokesh Mandvekar l...@buffalo.edu --- Created attachment 736096 -- https://bugzilla.redhat.com/attachment.cgi?id=736096action=edit installs a config file to /etc and removes scrotwm symlink -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=LO2PoSXAQVa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 946856] Review Request: spectrwm - Minimalist tiling window manager written in C
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=946856 --- Comment #15 from Lokesh Mandvekar l...@buffalo.edu --- Created attachment 736097 -- https://bugzilla.redhat.com/attachment.cgi?id=736097action=edit -Wl,-soname for libswmhack.so.0 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=rYkhugNLVha=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 946856] Review Request: spectrwm - Minimalist tiling window manager written in C
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=946856 --- Comment #16 from Lokesh Mandvekar l...@buffalo.edu --- Created attachment 736098 -- https://bugzilla.redhat.com/attachment.cgi?id=736098action=edit shlib symlinks handled in Makefile -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=lMAXd4Jk9Da=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 946856] Review Request: spectrwm - Minimalist tiling window manager written in C
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=946856
--- Comment #17 from Lokesh Mandvekar l...@buffalo.edu ---
Alright,
new spec file: https://github.com/lsm5/rpmbuild/blob/master/SPECS/spectrwm.spec
koji url: http://koji.fedoraproject.org/koji/taskinfo?taskID=5256488
rpmlint outputs are same as in Comment 11.
Avoiding hardcoded versions for shlibs meant I had to write shlibs in %files
like so:
%{_libdir}/libswmhack.so.*
Is this valid? I guess it would go against Rahul's comment to explicitly
mention all files.
All patches are now attached.
--
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug
https://bugzilla.redhat.com/token.cgi?t=FlFlSYjgJTa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 915902] Review Request: qt5-qtscript - Qt5 - QtScript component
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=915902 Bug 915902 depends on bug 878188, which changed state. Bug 878188 Summary: Review Request: qt5-qtbase - Qt5 - QtBase components https://bugzilla.redhat.com/show_bug.cgi?id=878188 What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=kWPt3Kx248a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 915005] Review Request: qt5-qttools - Qt5 - QtTool components
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=915005 Bug 915005 depends on bug 878188, which changed state. Bug 878188 Summary: Review Request: qt5-qtbase - Qt5 - QtBase components https://bugzilla.redhat.com/show_bug.cgi?id=878188 What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=cMDn9lq9hsa=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 878188] Review Request: qt5-qtbase - Qt5 - QtBase components
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=878188 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA Last Closed||2013-04-15 19:54:43 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=eHBHvz2Fdza=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
