[Bug 1009742] Review Request: perl-Term-Menu - print an interactive menu
https://bugzilla.redhat.com/show_bug.cgi?id=1009742
Ralf Corsepius changed:
What|Removed |Added
CC||rc040...@freenet.de
--- Comment #1 from Ralf Corsepius ---
* Fails to build in mock.
Please add:
BuildRequires: perl(Test::More)
* With the BR: above added, the testsuite skips a test.
Adding
BuildRequires: perl(Test::Expect)
causes the skipped test to be run.
* The License: field needs to be
License:GPL+ or Artistic
* Unless you intent to support EPEL5,
Group: ...
...
BuildRoot: %{_tmppath}/%{name}-%{version}-build
...
rm -rf $RPM_BUILD_ROOT
can all be removed.
* Macro usage is inconsistent:
%__perl and %{__perl} are both being used.
Please only use one of these (I recommend using %{__perl})
--
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=rpvzcA8n9C&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1009752] Review Request: perl-Term-Menus - Create powerful menus
https://bugzilla.redhat.com/show_bug.cgi?id=1009752 Chris 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=jUIwOOfw9B&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1009742] Review Request: perl-Term-Menu - print an interactive menu
https://bugzilla.redhat.com/show_bug.cgi?id=1009742 Chris 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=q2iiyorwyb&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1009754] Review Request: perl-Term-RawInput - Replacement for in perl scripts
https://bugzilla.redhat.com/show_bug.cgi?id=1009754 Chris 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=GirqaL42nG&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1009754] New: Review Request: perl-Term-RawInput - Replacement for in perl scripts
https://bugzilla.redhat.com/show_bug.cgi?id=1009754 Bug ID: 1009754 Summary: Review Request: perl-Term-RawInput - Replacement for in perl scripts Product: Fedora Version: rawhide Component: Package Review Severity: medium Assignee: nob...@fedoraproject.org Reporter: chjoh...@gmail.com QA Contact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Spec URL: https://www.dropbox.com/s/zqologv0gy5ksic/perl-Term-RawInput.spec SRPM URL: https://www.dropbox.com/s/ddkzioplfj4x4no/perl-Term-RawInput-1.15-1.fc18.src.rpm Description: A simple drop-in replacement for in scripts with the additional ability to capture and return the non-standard keys like 'End', 'Escape', 'Insert', etc. Fedora Account System Username: cjohnston -- 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=RxXYp5teW5&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1009752] New: Review Request: perl-Term-Menus - Create powerful menus
https://bugzilla.redhat.com/show_bug.cgi?id=1009752 Bug ID: 1009752 Summary: Review Request: perl-Term-Menus - Create powerful menus Product: Fedora Version: rawhide Component: Package Review Severity: medium Assignee: nob...@fedoraproject.org Reporter: chjoh...@gmail.com QA Contact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Spec URL: https://www.dropbox.com/s/s8xlbqoyy29m970/perl-Term-Menus.spec SRPM URL: https://www.dropbox.com/s/ih9yokbl866y0p4/perl-Term-Menus-2.45-1.fc18.src.rpm Description: Term::Menus allows you to create powerful Terminal, Console and CMD environment menus. Any perl script used in a Terminal, Console or CMD environment can now include a menu facility that includes sub-menus, forward and backward navigation, single or multiple selection capabilities, dynamic item creation and customized banners. All this power is simple to implement with a straight forward and very intuitive configuration hash structure that mirrors the actual menu architecture needed by the application. A separate configuration file is optional. Term::Menus is cross platform compatible. Fedora Account System Username: cjohnston -- 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=dTV7aj1XGe&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1009750] Review Request: python-falcon - High-performance cloud API framework
https://bugzilla.redhat.com/show_bug.cgi?id=1009750 Jamie Lennox 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=ZKjzjddpHM&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1009750] New: Review Request: python-falcon - High-performance cloud API framework
https://bugzilla.redhat.com/show_bug.cgi?id=1009750 Bug ID: 1009750 Summary: Review Request: python-falcon - High-performance cloud API framework Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: jlen...@redhat.com QA Contact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Spec URL: https://dl.dropboxusercontent.com/u/52259114/python-falcon.spec SRPM URL: https://dl.dropboxusercontent.com/u/52259114/python-falcon-0.1.7-1.fc19.src.rpm Description: Falcon is a high-performance Python framework for building cloud APIs. It encourages the REST architectural style, and tries to do as little as possible while remaining highly effective. It will be a requirement of the marconi openstack component. Fedora Account System Username: jamielennox This is my first package so i am in need of a sponsor. rawhide: https://koji.fedoraproject.org/koji/taskinfo?taskID=5949439 f19: https://koji.fedoraproject.org/koji/taskinfo?taskID=5949421 rpmbuild]$ rpmlint SRPMS/python-falcon-0.1.7-1.fc19.src.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. -- 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=v1fndWDoAT&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 983340] Review Request: fcitx-kkc - Japanese Kana Kanji Engine for Fcitx
https://bugzilla.redhat.com/show_bug.cgi?id=983340
Parag AN(पराग) changed:
What|Removed |Added
CC||package-review@lists.fedora
||project.org
Flags|fedora-review? |fedora-review+
--- Comment #7 from Parag AN(पराग) ---
Suggestions:
1) Can header file be removed as I see no use of it?
2) I am not sure who owns this
%{_libdir}/fcitx/qt
if no one let this package own this also.
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=BCjG3B6bCW&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1009742] Review Request: perl-Term-Menu - print an interactive menu
https://bugzilla.redhat.com/show_bug.cgi?id=1009742 Chris changed: What|Removed |Added URL||http://search.cpan.org/~daz ||jorz/Term-Menu-0.09/lib/Ter ||m/Menu.pm -- 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=b2OjfD4qbQ&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1009742] New: Review Request: perl-Term-Menu - print an interactive menu
https://bugzilla.redhat.com/show_bug.cgi?id=1009742 Bug ID: 1009742 Summary: Review Request: perl-Term-Menu - print an interactive menu Product: Fedora Version: rawhide Component: Package Review Severity: medium Assignee: nob...@fedoraproject.org Reporter: chjoh...@gmail.com QA Contact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Spec URL: https://www.dropbox.com/s/tbujesxtmxknr39/perl-Term-Menu.spec SRPM URL: https://www.dropbox.com/s/qtug6gu6xgn0178/perl-Term-Menu-0.09-1.fc18.src.rpm Description: Simple perl module that makes it easy to print an interactive menu Fedora Account System Username: cjohnston -- 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=ii4D6157zz&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 882711] Review Request: ovirt-host-deploy - oVirt host deploy tool
https://bugzilla.redhat.com/show_bug.cgi?id=882711 Douglas Schilling Landgraf changed: What|Removed |Added CC||yzasl...@redhat.com --- Comment #10 from Douglas Schilling Landgraf --- (In reply to Alon Bar-Lev from comment #9) > [!]: Sources can be downloaded from URI in Source: tag > Note: Could not download Source0 > > This will be fixed soon. Correct. ovirt-host-deploy.src: W: invalid-url Source0: http://resources.ovirt.org/releases/stable/3.3/ovirt-host-deploy-1.1.1.tar.gz HTTP Error 404: Not Found As soon you get this fixed, please update the bugzilla. Also, please check the output of rpmlint: ovirt-host-deploy.noarch: W: dangling-relative-symlink /usr/share/ovirt-host-deploy/interface-3/otopi-plugins/otopi ../../../otopi/plugins/otopi ovirt-host-deploy.noarch: W: hidden-file-or-dir /usr/share/ovirt-host-deploy/interface-3/.bundled ovirt-host-deploy.noarch: E: zero-length /usr/share/ovirt-host-deploy/interface-3/.bundled ovirt-host-deploy.noarch: W: dangling-relative-symlink /usr/share/ovirt-host-deploy/interface-3/otopi ../../../sbin/otopi ovirt-host-deploy.noarch: W: hidden-file-or-dir /usr/share/ovirt-host-deploy/plugins/ovirt-host-deploy/vdsmhooks/hooks.d/.keep ovirt-host-deploy.noarch: E: zero-length /usr/share/ovirt-host-deploy/plugins/ovirt-host-deploy/vdsmhooks/hooks.d/.keep ovirt-host-deploy.noarch: W: hidden-file-or-dir /usr/share/ovirt-host-deploy/plugins/ovirt-host-deploy/vdsmhooks/packages.d/.keep ovirt-host-deploy.noarch: E: zero-length /usr/share/ovirt-host-deploy/plugins/ovirt-host-deploy/vdsmhooks/packages.d/.keep ovirt-host-deploy.noarch: W: dangling-relative-symlink /usr/share/ovirt-host-deploy/interface-3/pythonlib/otopi ../../../../li http://fedoraproject.org/wiki/ParagNemade/CommonRpmlintErrors#zero-length Thanks Douglas -- 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=QpsEvMICXR&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1009703] New: Review Request: RoboJournal - Keep a journal/diary of day-to-day events in your life
https://bugzilla.redhat.com/show_bug.cgi?id=1009703 Bug ID: 1009703 Summary: Review Request: RoboJournal - Keep a journal/diary of day-to-day events in your life Product: Fedora Version: rawhide Component: Package Review Severity: medium Assignee: nob...@fedoraproject.org Reporter: pwiz...@gmail.com QA Contact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org ** FYI: This is my first package and I'm seeking a sponsor. I am the upstream maintainer. ** Spec URL: http://pagewizardwebdesign.com/stuff/pub/fedora/robojournal.spec SRPM URL: http://pagewizardwebdesign.com/stuff/pub/fedora/robojournal-0.4.2-1.fc18.src.rpm Description: RoboJournal is a cross-platform journal/diary tool written in Qt/C++. Right now, RoboJournal only supports MySQL but support for SQLite (and possibly Postgres) will be added in future releases. RoboJournal runs on Windows and Linux. Fedora Account System Username: pwizard Koji F20 Builds: http://koji.fedoraproject.org/koji/taskinfo?taskID=5953085 http://kojipkgs.fedoraproject.org//work/tasks/3088/5953088/robojournal-0.4.2-1.fc20.i686.rpm http://kojipkgs.fedoraproject.org//work/tasks/3087/5953087/robojournal-0.4.2-1.fc20.x86_64.rpm http://kojipkgs.fedoraproject.org//work/tasks/3086/5953086/robojournal-0.4.2-1.fc20.armv7hl.rpm http://kojipkgs.fedoraproject.org//work/tasks/3088/5953088/robojournal-doc-0.4.2-1.fc20.noarch.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=YP9hU1bPsL&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 950281] Review Request: golang - The Go Programming Language
https://bugzilla.redhat.com/show_bug.cgi?id=950281 Adam Miller changed: What|Removed |Added CC||admil...@redhat.com Flags|fedora-cvs+ |fedora-cvs? --- Comment #26 from Adam Miller --- Package Change Request == Package Name: golang New Branches: el6 Owners: maxamillion -- 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=F3hEmwaleD&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 855843] Review Request: mina-ftpserver - A 100% pure Java FTP server
https://bugzilla.redhat.com/show_bug.cgi?id=855843 gil cattaneo changed: What|Removed |Added Alias||mina, ftpserver -- 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=xl2iPgC5Bt&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1009153] Review Request: javolution - Real-time Java library
https://bugzilla.redhat.com/show_bug.cgi?id=1009153
--- Comment #5 from gil cattaneo ---
(In reply to gil cattaneo from comment #3)
> please, remove from spec file these
>
> BuildRequires: javapackages-tools
> BuildRequires: maven
> BuildRequires: maven-compiler-plugin
> BuildRequires: maven-install-plugin
> BuildRequires: maven-javadoc-plugin
> BuildRequires: maven-source-plugin
> BuildRequires: maven-surefire-plugin
>
> all Group fiels- no more required
> all Requires - added by java package tools (XMvn)
>
> fix unowned directory
> add %dir %{_javadir}/%{name}
> in main package
>
> if possible use pom macros
> e.g.
> %pom_remove_plugin javolution:colapi
> %pom_xpath_remove "pom:project/pom:build/pom:extensions[pom:groupId =
> 'org.jvnet.wagon-svn' ]"
> or
> %pom_xpath_remove "pom:project/pom:build/pom:extensions"
>
> fix license field
> License: BSD and Public domain
should be correct as this last one
License: BSD
>
> these are under Public domain license
>
because you must remove all bundled libraries
https://fedoraproject.org/wiki/Packaging:No_Bundled_Libraries
> already available in xml-commons-apis package or in OpenJDK rt.jar
> /var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/
> src/main/java/org/xml/sax/AttributeList.java
> /var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/
> src/main/java/org/xml/sax/DTDHandler.java
> /var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/
> src/main/java/org/xml/sax/DocumentHandler.java
> /var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/
> src/main/java/org/xml/sax/EntityResolver.java
> /var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/
> src/main/java/org/xml/sax/ErrorHandler.java
> /var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/
> src/main/java/org/xml/sax/HandlerBase.java
> /var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/
> src/main/java/org/xml/sax/InputSource.java
> /var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/
> src/main/java/org/xml/sax/Locator.java
> /var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/
> src/main/java/org/xml/sax/Parser.java
> /var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/
> src/main/java/org/xml/sax/SAXException.java
> /var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/
> src/main/java/org/xml/sax/SAXParseException.java
>
--
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=TrNIavNhgP&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1009153] Review Request: javolution - Real-time Java library
https://bugzilla.redhat.com/show_bug.cgi?id=1009153 gil cattaneo changed: What|Removed |Added Status|NEW |ASSIGNED -- 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=CZMfCKokZg&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1009153] Review Request: javolution - Real-time Java library
https://bugzilla.redhat.com/show_bug.cgi?id=1009153 --- Comment #4 from gil cattaneo --- Created attachment 799593 --> https://bugzilla.redhat.com/attachment.cgi?id=799593&action=edit review notes -- 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=ZAjEgaeYTX&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1009153] Review Request: javolution - Real-time Java library
https://bugzilla.redhat.com/show_bug.cgi?id=1009153
gil cattaneo changed:
What|Removed |Added
Assignee|nob...@fedoraproject.org|punto...@libero.it
Flags||fedora-review+
--- Comment #3 from gil cattaneo ---
please, remove from spec file these
BuildRequires: javapackages-tools
BuildRequires: maven
BuildRequires: maven-compiler-plugin
BuildRequires: maven-install-plugin
BuildRequires: maven-javadoc-plugin
BuildRequires: maven-source-plugin
BuildRequires: maven-surefire-plugin
all Group fiels- no more required
all Requires - added by java package tools (XMvn)
fix unowned directory
add %dir %{_javadir}/%{name}
in main package
if possible use pom macros
e.g.
%pom_remove_plugin javolution:colapi
%pom_xpath_remove "pom:project/pom:build/pom:extensions[pom:groupId =
'org.jvnet.wagon-svn' ]"
or
%pom_xpath_remove "pom:project/pom:build/pom:extensions"
fix license field
License: BSD and Public domain
these are under Public domain license
Public domain
-
/var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/src/main/java/org/xml/sax/AttributeList.java
/var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/src/main/java/org/xml/sax/DTDHandler.java
/var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/src/main/java/org/xml/sax/DocumentHandler.java
/var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/src/main/java/org/xml/sax/EntityResolver.java
/var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/src/main/java/org/xml/sax/ErrorHandler.java
/var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/src/main/java/org/xml/sax/HandlerBase.java
/var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/src/main/java/org/xml/sax/InputSource.java
/var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/src/main/java/org/xml/sax/Locator.java
/var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/src/main/java/org/xml/sax/Parser.java
/var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/src/main/java/org/xml/sax/SAXException.java
/var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/src/main/java/org/xml/sax/SAXParseException.java
remove bundled library
already available in xml-commons-apis package or in OpenJDK rt.jar
/var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/src/main/java/org/xml/sax/Attributes.java
/var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/src/main/java/org/xml/sax/ContentHandler.java
/var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/src/main/java/org/xml/sax/SAXNotRecognizedException.java
/var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/src/main/java/org/xml/sax/SAXNotSupportedException.java
/var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/src/main/java/org/xml/sax/XMLFilter.java
/var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/src/main/java/org/xml/sax/XMLReader.java
fix wrong-file-end-of-line-encoding /usr/share/doc/javolution/LICENSE.txt
e.g.
sed -i 's/\r//' LICENSE.txt
not 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=Iqhx9D5NOp&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1009654] Review Request: tempus-fugit - Java classes for temporal testing and concurrency
https://bugzilla.redhat.com/show_bug.cgi?id=1009654 --- Comment #1 from gil cattaneo --- Task info: http://koji.fedoraproject.org/koji/taskinfo?taskID=5952792 -- 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=rWAd0ojpyt&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1009654] Review Request: tempus-fugit - Java classes for temporal testing and concurrency
https://bugzilla.redhat.com/show_bug.cgi?id=1009654 gil cattaneo changed: What|Removed |Added Alias||tempus-fugit -- 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=n2gz25diSw&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1009654] New: Review Request: tempus-fugit - Java classes for temporal testing and concurrency
https://bugzilla.redhat.com/show_bug.cgi?id=1009654 Bug ID: 1009654 Summary: Review Request: tempus-fugit - Java classes for temporal testing and concurrency Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: punto...@libero.it QA Contact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Spec URL: http://gil.fedorapeople.org/tempus-fugit.spec SRPM URL: http://gil.fedorapeople.org/tempus-fugit-1.1-1.fc19.src.rpm Description: Java micro-library for time sensitive testing and concurrency. Fedora Account System Username: gil -- 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=NL2q97yTgS&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 855843] Review Request: mina-ftpserver - A 100% pure Java FTP server
https://bugzilla.redhat.com/show_bug.cgi?id=855843 --- Comment #3 from gil cattaneo --- Spec URL: http://gil.fedorapeople.org/mina-ftpserver.spec SRPM URL: http://gil.fedorapeople.org/mina-ftpserver-1.0.6-1.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=lCiuIYfufL&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 856238] Review Request: scratch - Programming language learning environment for stories, games, music and art
https://bugzilla.redhat.com/show_bug.cgi?id=856238 Trevor Hemsley changed: What|Removed |Added CC||trevor.hems...@ntlworld.com --- Comment #29 from Trevor Hemsley --- Thanks! Have just rebuild this under mock for CentOS 6 and it builds cleanly. Have attached a patch for squeak-vm build for i386 under the other ticket linked above as that would not build. My 8 y.o says thanks too. -- 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=FjXGgQHsAG&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1009153] Review Request: javolution - Real-time Java library
https://bugzilla.redhat.com/show_bug.cgi?id=1009153 --- Comment #2 from Pete MacKinnon --- Please read the comments in the Description. -- 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=JY4cDqAihE&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1009153] Review Request: javolution - Real-time Java library
https://bugzilla.redhat.com/show_bug.cgi?id=1009153 gil cattaneo changed: What|Removed |Added CC||punto...@libero.it --- Comment #1 from gil cattaneo --- hi there is a newer release in https://svn.java.net/svn/javolution~svn/tags/ regards -- 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=syi413Myyh&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1009247] Review Request: minimodem - General-purpose software audio FSK modem
https://bugzilla.redhat.com/show_bug.cgi?id=1009247 Jim changed: What|Removed |Added CC|package-review@lists.fedora | |project.org | Flags||fedora-cvs? --- Comment #3 from Jim --- New Package SCM Request === Package Name: minimodem Short Description: General-purpose software audio FSK modem Owners: kg4sgp Branches: f18 f19 f20 el5 el6 InitialCC: codeblock -- 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=mEj4UPxsTy&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1008856] Review Request: perl-File-ShareDir-ProjectDistDir - Simple set-and-forget using of a '/share' directory in your projects root
https://bugzilla.redhat.com/show_bug.cgi?id=1008856 Petr Pisar changed: What|Removed |Added CC|package-review@lists.fedora | |project.org | Status|NEW |ASSIGNED CC||ppi...@redhat.com Assignee|nob...@fedoraproject.org|ppi...@redhat.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=0Y7DlnZMOZ&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1009446] New: Review Request: open-mtools - Tools for testing IP multicast
https://bugzilla.redhat.com/show_bug.cgi?id=1009446 Bug ID: 1009446 Summary: Review Request: open-mtools - Tools for testing IP multicast Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: ppi...@redhat.com QA Contact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Spec URL: http://ppisar.fedorapeople.org/open-mtools/open-mtools.spec SRPM URL: http://ppisar.fedorapeople.org/open-mtools/open-mtools-1.0-1.fc19.src.rpm Description: This package contains the msend, mdump, and mpong tools to aid in testing multicast networks. Fedora Account System Username: ppisar -- 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=lsfJQ8izNT&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 975312] Review Request: libodb - Common ODB runtime library
https://bugzilla.redhat.com/show_bug.cgi?id=975312 --- Comment #5 from Dave Johansen --- Now that ODB itself has been approved, this package is ready for review. The current files can be found at: Spec URL: http://daveisfera.fedorapeople.org/odb_2.2/specs/libodb.spec SRPM URL: http://daveisfera.fedorapeople.org/odb_2.2/SRPMS/libodb-2.2.3-1.el6.src.rpm The only output from rpmlint is the following: libodb.src: W: spelling-error Summary(en_US) runtime -> run time, run-time, untimely libodb.src: W: spelling-error %description -l en_US runtime -> run time, run-time, untimely But that seems to be a valid spelling ( http://en.wikipedia.org/wiki/Runtime_library ), so I'm guessing that it's actually ok, but is there an "upstream" that I can put in a request for that to be considered a valid spelling? -- 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=4Tk6269wse&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1009178] Review Request: python-sockjs-tornado - SockJS server implementation for Tornado
https://bugzilla.redhat.com/show_bug.cgi?id=1009178 Jon Ciesla changed: What|Removed |Added CC|package-review@lists.fedora | |project.org | Flags|fedora-cvs? |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=uVpZdwKjdQ&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 769919] Review Request: hydra - Very fast network log-on cracker
https://bugzilla.redhat.com/show_bug.cgi?id=769919 Jon Ciesla changed: What|Removed |Added CC|package-review@lists.fedora | |project.org | Flags|fedora-cvs? |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=Af7KhxxppW&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 989775] Review Request: rubygem-omniauth - A generalized Rack framework for multiple-provider authentication
https://bugzilla.redhat.com/show_bug.cgi?id=989775 Axilleas Pipinellis changed: What|Removed |Added CC|package-review@lists.fedora | |project.org | Flags||fedora-cvs? --- Comment #5 from Axilleas Pipinellis --- Sorry for the late reply, thanks for the review :) New Package SCM Request === Package Name: rubygem-omniauth Short Description: A generalized Rack framework for multiple-provider authentication Owners: axilleas Branches: f19 f20 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=jvQHkf9CSD&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1008787] Review Request: ghc-yaml - Support for parsing and rendering YAML documents
https://bugzilla.redhat.com/show_bug.cgi?id=1008787 Jens Petersen changed: What|Removed |Added CC|package-review@lists.fedora | |project.org | Alias||ghc-yaml -- 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=tpf1NXfbuU&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1009375] Review Request: ghc-hslua - Lua language interpreter embedding in Haskell
https://bugzilla.redhat.com/show_bug.cgi?id=1009375 Jens Petersen changed: What|Removed |Added Alias||ghc-hslua -- 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=fQJ0zRIYLh&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1009375] Review Request: ghc-hslua - Lua language interpreter embedding in Haskell
https://bugzilla.redhat.com/show_bug.cgi?id=1009375 Jens Petersen changed: What|Removed |Added CC||haskell-devel@lists.fedorap ||roject.org Blocks||1008396 Whiteboard||Ready -- 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=fUS2atBVEP&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1009375] New: Review Request: ghc-hslua - Lua language interpreter embedding in Haskell
https://bugzilla.redhat.com/show_bug.cgi?id=1009375 Bug ID: 1009375 Summary: Review Request: ghc-hslua - Lua language interpreter embedding in Haskell Product: Fedora Version: rawhide Component: Package Review Assignee: nob...@fedoraproject.org Reporter: peter...@redhat.com QA Contact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Spec URL: http://petersen.fedorapeople.org/reviews/ghc-hslua.spec SRPM URL: http://petersen.fedorapeople.org/reviews/ghc-hslua-0.3.6.1-1.fc19.src.rpm Description: The Scripting.Lua module is a wrapper of Lua language interpreter as described in www.lua.org. -- 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=TgZevOHJLK&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1009375] Review Request: ghc-hslua - Lua language interpreter embedding in Haskell
https://bugzilla.redhat.com/show_bug.cgi?id=1009375 --- Comment #1 from Jens Petersen --- This package built on koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=5949925 -- 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=kXOWKJIY2u&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1008532] Review Request: copr - Cool Other Package Repo
https://bugzilla.redhat.com/show_bug.cgi?id=1008532 Miroslav Suchý changed: What|Removed |Added CC|package-review@lists.fedora | |project.org | --- Comment #3 from Miroslav Suchý --- oops, 1.5 failed to build Updated: Spec URL: http://miroslav.suchy.cz/fedora/copr/copr.spec SRPM URL: http://miroslav.suchy.cz/fedora/copr/copr-1.6-1.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=cuBOIWS9zJ&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 983340] Review Request: fcitx-kkc - Japanese Kana Kanji Engine for Fcitx
https://bugzilla.redhat.com/show_bug.cgi?id=983340 Daiki Ueno changed: What|Removed |Added CC|package-review@lists.fedora | |project.org | Flags|needinfo?(du...@redhat.com) | --- Comment #6 from Daiki Ueno --- Sorry for the long delay. Updated the package: Spec URL: http://ueno.fedorapeople.org/fcitx-kkc/fcitx-kkc.spec SRPM URL: http://ueno.fedorapeople.org/fcitx-kkc/fcitx-kkc-0.1.0-2.fc19.src.rpm (In reply to Parag AN(पराग) from comment #1) > - License should be GPLv3+ Done. > - Directory ownership issues > /usr/share/fcitx/imicon > /usr/share/fcitx/inputmethod > > I can't find who owns these. Since 4.2.8-3, the fcitx package owns those directories: http://pkgs.fedoraproject.org/cgit/fcitx.git/commit/?id=b6284bbc85b3bc754f09f0c8c01fb91abfd41637 > suggestions: > 1) we don't need now > rm -rf $RPM_BUILD_ROOT Done. -- 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=Nipi6g7XyJ&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1009059] Review Request: appdata-tools - Tools for AppData files
https://bugzilla.redhat.com/show_bug.cgi?id=1009059 Richard Hughes changed: What|Removed |Added CC|package-review@lists.fedora | |project.org | --- Comment #2 from Richard Hughes --- (In reply to Pierre-Yves Luyten from comment #1) > (A little surprised with the output on random example : "start tag 'name' > not allowed from section 'application' " isn't name a valid node? Anyway > this point is rather a question / bug report than part of the package review) Already fixed upstream. I'll do a 0.1.1 release next week with all the other bugs that people have also found. Thanks. -- 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=6hma3n8LS8&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 882711] Review Request: ovirt-host-deploy - oVirt host deploy tool
https://bugzilla.redhat.com/show_bug.cgi?id=882711 --- Comment #9 from Alon Bar-Lev --- [!]: Sources can be downloaded from URI in Source: tag Note: Could not download Source0 This will be fixed soon. -- 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=BPU9ZiabYL&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 882711] Review Request: ovirt-host-deploy - oVirt host deploy tool
https://bugzilla.redhat.com/show_bug.cgi?id=882711 --- Comment #8 from Alon Bar-Lev --- Updated artifacts. spec[1] srpm[2] review report[3] Thanks! [1] https://github.com/alonbl/fedora-files/blob/master/ovirt-host-deploy.spec [2] https://github.com/alonbl/fedora-files/blob/master/ovirt-host-deploy-1.1.1-1.fc19.src.rpm [3] https://github.com/alonbl/fedora-files/blob/master/ovirt-host-deploy-review.txt -- 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=0DeBheS9w4&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1008816] Review Request: libkkc-data - Language model data for libkkc
https://bugzilla.redhat.com/show_bug.cgi?id=1008816
Parag AN(पराग) changed:
What|Removed |Added
CC|package-review@lists.fedora |
|project.org |
Flags|fedora-review? |fedora-review+
--- Comment #2 from Parag AN(पराग) ---
Review:
+ package build successfully in mock
- rpmlint on rpms gave
libkkc-data.x86_64: E: no-binary
libkkc-data.x86_64: W: only-non-binary-in-usr-lib
libkkc-data.x86_64: W: no-documentation
libkkc-data-debuginfo.x86_64: E: empty-debuginfo-package
3 packages and 0 specfiles checked; 2 errors, 2 warnings.
+ Source validated to upstream as (sha256sum)
srpm tarball :
9e678755a030043da68e37a4049aa296c296869ff1fb9e6c70026b2541595b99
upstream tarball :
9e678755a030043da68e37a4049aa296c296869ff1fb9e6c70026b2541595b99
+ license is valid and included in COPYING
Suggestions:
1) Add %doc COPYING
also good if you are upstream then add some contents to file README, ChangeLog
and add them in %doc
2) you may want to disable -debuginfo package by adding at the top of spec
# This is just data package that installs data files only
%global debug_package %{nil}
I see this is still valid https://bugzilla.redhat.com/show_bug.cgi?id=906248#c8
so, not to worry for rpmlint warning only-non-binary-in-usr-lib
When you import this package you first need to update libkkc.spec by removing
libkkc-data subpackage and then build this package.
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=EOWXwTo8Fw&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
