[Bug 661902] Review Request: moksha - A platform for creating real-time web applications
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=661902 --- Comment #2 from Casey Dahlin 2010-12-10 02:36:41 EST --- # SHOULD: If the source package does not include license text(s) as a separate file from upstream, the packager SHOULD query upstream to include it. [25] N/A # SHOULD: The description and summary sections in the package spec file should contain translations for supported Non-English languages, if available. [26] I'll leave that to you. # SHOULD: The reviewer should test that the package builds in mock. [27] Did. # SHOULD: The package should compile and build into binary rpms on all supported architectures. [28] Its noarch. # SHOULD: The reviewer should test that the package functions as described. A package should not segfault instead of running, for example. What's a good way to test this'un? I'll trust you if you want since you say this package has been around a bit. # SHOULD: If scriptlets are used, those scriptlets must be sane. This is vague, and left up to the reviewers judgement to determine sanity. [29] Is that post section a good example of how to distribute selinux policy? Might want to ask dwalsh about that. # SHOULD: Usually, subpackages other than devel should require the base package using a fully versioned dependency. [21] Looks ok. # SHOULD: The placement of pkgconfig(.pc) files depends on their usecase, and this is usually for development purposes, so should be placed in a -devel pkg. A reasonable exception is that the main pkg itself is a devel tool not installed in a user runtime, e.g. gcc or gdb. [30] N/A # SHOULD: If the package has file dependencies outside of /etc, /bin, /sbin, /usr/bin, or /usr/sbin consider requiring the package which provides the file instead of the file itself. [31] Fine. # SHOULD: your package should contain man pages for binaries/scripts. If it doesn't, work with upstream to add them where they make sense.[32] This would be nice for moksha and moksha-hub. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 661902] Review Request: moksha - A platform for creating real-time web applications
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=661902
--- Comment #1 from Casey Dahlin 2010-12-10 02:31:31 EST
---
# MUST: rpmlint must be run on the source rpm and all binary rpms the build
produces. The output should be posted in the review.
moksha.src: W: spelling-error %description -l en_US jQuery -> j Query, query,
equerry
moksha.src:150: E: files-attr-not-set
moksha.src:151: E: files-attr-not-set
moksha.src:152: E: files-attr-not-set
moksha.src: W: invalid-url Source0: moksha-0.5.0.tar.bz2
moksha.spec:150: E: files-attr-not-set
moksha.spec:151: E: files-attr-not-set
moksha.spec:152: E: files-attr-not-set
moksha.spec: W: invalid-url Source0: moksha-0.5.0.tar.bz2
1 packages and 1 specfiles checked; 6 errors, 3 warnings.
You should fix the files-addr-not-set ones, just need a %defattr directive
after line 148.
The source one should be fixed too. It should be a full URI to the file. I
recommend specifying the file as %{name}-%{version}.tar.gz if upstream's naming
is consistent enough.
You omitted the end of the last sentence in the description. Looks like
# MUST: The package must be named according to the Package Naming Guidelines .
Looks good.
# MUST: The spec file name must match the base package %{name}, in the format
%{name}.spec unless your package has an exemption. [2] .
Yep.
# MUST: The package must meet the Packaging Guidelines .
Yep.
# MUST: The package must be licensed with a Fedora approved license and meet
the Licensing Guidelines .
Yep.
# MUST: The License field in the package spec file must match the actual
license. [3]
You have a file called LICENSE and a file called COPYING. Both the same. Won't
fault it but seems redundant.
# MUST: If (and only if) the source package includes the text of the license(s)
in its own file, then that file, containing the text of the license(s) for the
package must be included in %doc.[4]
You also copy both files to disk. This must be some new thing I haven't heard
about.
# MUST: The spec file must be written in American English. [5]
Yep.
# MUST: The spec file for the package MUST be legible. [6]
Fine.
# MUST: The sources used to build the package must match the upstream source,
as provided in the spec URL. Reviewers should use md5sum for this task. If no
upstream URL can be specified for this package, please see the Source URL
Guidelines for how to deal with this.
Haven't found the actual URL, and as noted above you didn't put it in Source0.
Please enlighten.
# MUST: The package MUST successfully compile and build into binary rpms on at
least one primary architecture. [7]
# MUST: If the package does not successfully compile, build or work on an
architecture, then those architectures should be listed in the spec in
ExcludeArch. Each architecture listed in ExcludeArch MUST have a bug filed in
bugzilla, describing the reason that the package does not compile/build/work on
that architecture. The bug number MUST be placed in a comment, next to the
corresponding ExcludeArch line. [8]
Its a noarch.
# MUST: All build dependencies must be listed in BuildRequires, except for any
that are listed in the exceptions section of the Packaging Guidelines ;
inclusion of those as BuildRequires is optional. Apply common sense.
I built in mock, so I'm assuming this is good.
# MUST: The spec file MUST handle locales properly. This is done by using the
%find_lang macro. Using %{_datadir}/locale/* is strictly forbidden.[9]
N/A
# MUST: Every binary RPM package (or subpackage) which stores shared library
files (not just symlinks) in any of the dynamic linker's default paths, must
call ldconfig in %post and %postun. [10]
N/A
# MUST: Packages must NOT bundle copies of system libraries.[11]'
Fine
# MUST: If the package is designed to be relocatable, the packager must state
this fact in the request for review, along with the rationalization for
relocation of that specific package. Without this, use of Prefix: /usr is
considered a blocker. [12]
Fine.
# MUST: A package must own all directories that it creates. If it does not
create a directory that it uses, then it should require a package which does
create that directory. [13]
Looks good.
# MUST: A Fedora package must not list a file more than once in the spec file's
%files listings. (Notable exception: license texts in specific situations)[14]
Good.
# MUST: Permissions on files must be set properly. Executables should be set
with executable permissions, for example. Every %files section must include a
%defattr(...) line. [15]
See rpmlint output.
# MUST: Each package must consistently use macros. [16]
Good.
# MUST: The package must contain code, or permissable content. [17]
Good.
# MUST: Large documentation files must go in a -doc subpackage. (The definition
of large is left up to the packager's best judgement, but is not restricted to
size. Large can refer to either size or quantity
[Bug 533167] Review Request: emacs-ecb - Emacs Code Browser
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=533167 tux...@gmail.com changed: What|Removed |Added CC||tux...@gmail.com --- Comment #11 from tux...@gmail.com 2010-12-10 02:07:13 EST --- Karel, I built this package ( using rpmbuild ) and it works for me on Fedora 14 - x86_64. I haven't noticed any issues as till now on my system. How can someone locate the issues you just mentioned ( to be fixed in Elisp ) ? Also that the bug status says that package is NotReady in the "Whiteboard" field, so it can't be reviewed. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 630644] Review Request: perl-Bio-SamTools - Bio::SamTools Perl module
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=630644 --- Comment #7 from Petr Pisar 2010-12-10 02:01:12 EST --- It still cannot find the development files because sub find_sam in Build.PL does not expect /usr/include/samtools and /usr/lib64. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 661902] Review Request: moksha - A platform for creating real-time web applications
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=661902 Casey Dahlin changed: What|Removed |Added CC||cdah...@redhat.com Flag||fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 661961] New: Review Request: kyotocabinet - A lightweight database library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: kyotocabinet - A lightweight database library https://bugzilla.redhat.com/show_bug.cgi?id=661961 Summary: Review Request: kyotocabinet - A lightweight database library Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: cdah...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Target Release: --- Spec URL: http://screwyouinc.com/dump/kyotocabinet.spec SRPM URL: http://screwyouinc.com/dump/kyotocabinet-1.2.27-1.fc14.src.rpm Description: Kyoto Cabinet is a library of routines for managing a database. The database is a simple data file containing records, and each record is a pair of a key and a value. Every key and value is an array of bytes with variable length. Both binary data and character strings can be used as a key or a value. There is neither a concept of tables nor of data types. I am preparing to package Kyoto Tycoon as well, but wanted to get this in first. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 658611] Review Request: nuxwdog
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=658611 --- Comment #11 from Ade Lee 2010-12-10 00:51:16 EST --- I've changed the perl module licensing to say "same as perl". Also, fixed a library reference in the perl build (as pointed out by Parag) Thanks Parag! New spec and rpm at: https://fedorahosted.org/released/nuxwdog/nuxwdog-1.0.0-10.fc13.src.rpm https://fedorahosted.org/released/nuxwdog/nuxwdog.spec -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 661832] Review Request: kdevelop-pg-qt - A parser generator
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=661832 --- Comment #3 from Dmitrij S. Kryzhevich 2010-12-10 00:40:51 EST --- One more notice. For Fedora-only, not EPEL, you not need any more: - Buildroot tag.[1] - rm of buildroot in begin of %install. - %clean section.[2] [1] https://fedoraproject.org/wiki/Packaging/Guidelines#BuildRoot_tag [2] https://fedoraproject.org/wiki/Packaging/Guidelines#.25clean -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 661832] Review Request: kdevelop-pg-qt - A parser generator
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=661832
Dmitrij S. Kryzhevich changed:
What|Removed |Added
Status|NEW |ASSIGNED
CC||kr...@land.ru
AssignedTo|nob...@fedoraproject.org|kr...@land.ru
Flag||fedora-review?
--- Comment #2 from Dmitrij S. Kryzhevich 2010-12-10 00:36:08
EST ---
Grate.
review:
Good:
# rpmlint *.rpm
kdevelop-pg-qt.x86_64: W: no-manual-page-for-binary kdev-pg-qt
kdevelop-pg-qt-devel.noarch: W: no-documentation
4 packages and 0 specfiles checked; 0 errors, 2 warnings.
rpmlint output could be ignored (but if somebody would ping upstream about man
page...).
* Package is named according to the Package Naming Guidelines .
* Spec file name must matchs the base package %{name}.
* Package meets the Packaging Guidelines.
Notice: you have in %prep
%setup -q -n kdevelop-pg-qt-%{version}
It is as how %setup -q will run, right?
* The package must be licensed with a Fedora approved license and meet the
Licensing Guidelines .
* File, containing the text of the license(s) for the package is included in
%doc.
* The spec file is written in American English.
* The spec file for the package is legible.
* The sources used to build the package matchs the upstream source with md5
summ 06e25c81cc34a5bddda3091ed3280e71.
* The package successfully compiles and builds into binary rpms on x86_64.
* All build dependencies are listed in BuildRequires.
* There are no locales.
* There are no shared libraries.
* Packages does not bundle copies of system libraries.
Notice. Package use part of astyle project, which is not a library.
* Not a relocatable package.
* Package owns all directories that it creates.
* Package must not list a file more than once in the spec file's %files
listings.
* Permissions on files are set properly.
* Package uses macros consistently.
* The package contains code.
* There are no documentation.
* Everyting included as %doc does not affect the runtime of the application.
* Header files are in a -devel package.
* There are no static libraries.
* Devel package requires the base package in proper way.
* Not a GUI application.
* Package does not own files or directories already owned by other packages.
* All filenames in rpm packages are valid UTF-8.
Should items:
* Package boukds in mock.
* Package function as described.
* cmake file in -devel subpackage.
==
Not good:
- The License field in the package spec file does not match the actual license.
There are files licensed as GPLv.2+.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 652573] Review Request: ghc-zip-archive - Haskell zip-archive library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=652573 --- Comment #8 from Lakshmi Narasimhan 2010-12-09 21:53:32 EST --- I am getting a build error in rawhide. RPM build errors: File listed twice: /usr/share/doc/ghc-zip-archive-0.1.1.6 create archive failed on file /home/narasimhan/rpmbuild/BUILDROOT/ghc-zip-archive-0.1.1.6-3.fc14.x86_64/usr/share/doc/ghc-zip-archive-0.1.1.6/LICENSE: cpio: Bad magic http://koji.fedoraproject.org/koji/taskinfo?taskID=2656250 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 652573] Review Request: ghc-zip-archive - Haskell zip-archive library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=652573 --- Comment #7 from Lakshmi Narasimhan 2010-12-09 21:24:58 EST --- >buildroot setting and cleaning is now deprecated in the Packaging Guidelines. True. I thought if I was patching it, I would add those two lines as well to reduce the rpmlint warnings :). ghc-pkg list zip-archive /usr/lib64/ghc-6.12.3/package.conf.d zip-archive-0.1.1.6 --A small program like import Codec.Archive.Zip main = do putStrLn "hello world" return () --gives me this as a normal user ghc --make test.hs test.hs:1:7: Could not find module `Codec.Archive.Zip': Use -v to see a list of the files searched for. But compiles as root. I can approve the package. Once the build occurs in rawhide, I can help you out in testing the functionality using that small program. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 541462] Review Request: rasmol - Molecular Graphics Visualization Tool
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=541462 Carl Byington changed: What|Removed |Added Flag|needinfo?(c...@five-ten-sg. | |com)| --- Comment #18 from Carl Byington 2010-12-09 21:11:34 EST --- I don't have time to package the four dependencies cqrlib, cvector, neartree and cbflib. I think the cleanest way is to autoconf them, which I did for cqrlib. That same method should work for the other three. http://www.five-ten-sg.com/cqrlib.spec http://www.five-ten-sg.com/CQRlib-1.0.6.tar.gz http://www.five-ten-sg.com/autoconf-files.tar.gz http://www.five-ten-sg.com/CQRlib-1.0.6-1.fc12.src.rpm http://www.five-ten-sg.com/rasmol.spec Once those four are accepted into Fedora, there is not much more to do to get rasmol itself accepted. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 661179] Review Request: cement-devtools - Development Tools for the Cement CLI Application Framework
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=661179 BJ Dierkes changed: What|Removed |Added Depends on|609295 | -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 609295] Review Request: cement - CLI Application Framework for Python
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=609295 BJ Dierkes changed: What|Removed |Added Blocks|661179 | -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 661179] Review Request: cement-devtools - Development Tools for the Cement CLI Application Framework
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=661179 BJ Dierkes changed: What|Removed |Added Status|NEW |CLOSED Resolution||DUPLICATE Last Closed||2010-12-09 20:45:16 --- Comment #2 from BJ Dierkes 2010-12-09 20:45:16 EST --- Closing, devtools is now bundled as part of python-cement at: https://bugzilla.redhat.com/show_bug.cgi?id=609295 *** This bug has been marked as a duplicate of bug 609295 *** -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 609295] Review Request: cement - CLI Application Framework for Python
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=609295 --- Comment #3 from BJ Dierkes 2010-12-09 20:45:16 EST --- *** Bug 661179 has been marked as a duplicate of this bug. *** -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 609295] Review Request: cement - CLI Application Framework for Python
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=609295 --- Comment #2 from BJ Dierkes 2010-12-09 20:44:13 EST --- Updated, upstream now includes -devtools and -test under the primary source. The spec now performs nose testing as well. Also, renamed as python-cement as it is more appropriate. SPEC: http://5dollarwhitebox.org/tmp/python-cement.spec SRPM: http://5dollarwhitebox.org/tmp/python-cement-0.8.14-1.fc14.src.rpm $ rpmlint -i SPECS/python-cement.spec 0 packages and 1 specfiles checked; 0 errors, 0 warnings. $ rpmlint -i RPMS/noarch/python-cement-* python-cement-devtools.noarch: E: zero-length /usr/lib/python2.7/site-packages/cement/paste/templates/cementapp/LICENSE python-cement-devtools.noarch: E: zero-length /usr/lib/python2.7/site-packages/cement/paste/templates/cementplugin/LICENSE 3 packages and 0 specfiles checked; 2 errors, 0 warnings. Note the warning are because the -devtools package includes application templates... the LICENSE and README in those templates are left blank by design. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 609295] Review Request: python-cement - CLI Application Framework for Python
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=609295 BJ Dierkes changed: What|Removed |Added Summary|Review Request: cement -|Review Request: |CLI Application Framework |python-cement - CLI |for Python |Application Framework for ||Python -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 661193] Review Request: man-pages-zh-CN - Chinese man pages
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=661193 Ding-Yi Chen changed: What|Removed |Added Status|NEW |ASSIGNED --- Comment #2 from Ding-Yi Chen 2010-12-09 19:45:35 EST --- Conflicted files removed. Spec URL: http://dchen.fedorapeople.org/files/rpms/man-pages-zh-CN.spec SRPM URL: http://dchen.fedorapeople.org/files/rpms/man-pages-zh-CN-1.5.1-2.el6.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 661902] New: Review Request: moksha - A platform for creating real-time web applications
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: moksha - A platform for creating real-time web applications https://bugzilla.redhat.com/show_bug.cgi?id=661902 Summary: Review Request: moksha - A platform for creating real-time web applications Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: lmac...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Target Release: --- Spec URL: http://lmacken.fedorapeople.org/rpms/moksha.spec SRPM URL: http://lmacken.fedorapeople.org/rpms/moksha-0.5.0-1.fc14.src.rpm Description: Moksha is a platform for creating real-time collaborative web applications. It. provides a set of Python and JavaScript API's that make it simple to create. rich applications that can acquire, manipulate, and visualize data from. external services. It is a unified framework build using the best available. open source technologies such as TurboGears2, jQuery, AMQP, and Orbited. More. information can be found on the Moksha Project Page at. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 661354] Review Request: identicurse - Curses based Status.net client
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=661354
--- Comment #5 from Julian Aloofi 2010-12-09
17:33:37 EST ---
(In reply to comment #4)
> Updated for all above items. Note that rpmlint does spit out warnings on the
> cosmetic changes but I assume it just has not caught up yet :-)
Yup, that's correct, and therefore rpmlint has a couple of things to complain
about, but they're not relevant:
[make...@julians-notebook rpmbuild]$ rpmlint ./SPECS/identicurse.spec
SRPMS/identicurse-0.4.1-2.fc14.src.rpm
./RPMS/noarch/identicurse-0.4.1-2.fc14.noarch.rpm
./SPECS/identicurse.spec: W: no-cleaning-of-buildroot %install
./SPECS/identicurse.spec: W: no-cleaning-of-buildroot %clean
./SPECS/identicurse.spec: W: no-buildroot-tag
./SPECS/identicurse.spec: W: no-%clean-section
identicurse.src: W: no-cleaning-of-buildroot %install
identicurse.src: W: no-cleaning-of-buildroot %clean
identicurse.src: W: no-buildroot-tag
identicurse.src: W: no-%clean-section
identicurse.noarch: W: no-manual-page-for-binary identicurse
2 packages and 1 specfiles checked; 0 errors, 9 warnings.
[x] Package is named according to the Package Naming Guidelines. [1]
[x] Spec file name must match the base package %{name}, in the format
%{name}.spec.
[x] Spec file is legible and written in American English.
[x] Spec file lacks Packager, Vendor, PreReq tags.
[x] Spec uses macros instead of hard-coded directory names.
[x] Package consistently uses macros.
[x] Macros in Summary, %description expandable at SRPM build time.
[x] PreReq is not used.
[x] Requires correct, justified where necessary.
[x] All build dependencies are listed in BuildRequires, except for any that
are listed in the exceptions section of Packaging Guidelines. [2]
[x] The spec file handles locales properly.
[x] Changelog in prescribed format.
Mock builds the package just fine.
I also saw the patch you use is already applied in upstream trunk, awesome.
It wouldn't hurt to list %{_python_sitelib}/%{name}/ instead of just
%{_python_sitelib}/* in %files though. Apart from that it looks all very sane.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 647510] Review Request: premake - A cross-platform build configuration tool
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=647510 Fedora Update System changed: What|Removed |Added Fixed In Version|premake-4.3-2.fc13 |premake-4.3-2.fc14 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 647510] Review Request: premake - A cross-platform build configuration tool
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=647510 --- Comment #18 from Fedora Update System 2010-12-09 17:03:47 EST --- premake-4.3-2.fc14 has been pushed to the Fedora 14 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 659895] Review Request: makedepf90 - Create Makefile dependency list for Fortran source files
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=659895 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #7 from Fedora Update System 2010-12-09 17:00:47 EST --- makedepf90-2.8.8-1.fc14 has been pushed to the Fedora 14 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update makedepf90'. You can provide feedback for this update here: https://admin.fedoraproject.org/updates/makedepf90-2.8.8-1.fc14 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 499992] Review Request: mingw32-webkitgtk - MinGW Windows web content engine library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=42 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #23 from Fedora Update System 2010-12-09 16:59:17 EST --- mingw32-webkitgtk-1.3.6-2.fc14 has been pushed to the Fedora 14 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update mingw32-webkitgtk'. You can provide feedback for this update here: https://admin.fedoraproject.org/updates/mingw32-webkitgtk-1.3.6-2.fc14 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 655379] Review Request: tritonus - Java Sound API Implementation
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=655379 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #18 from Fedora Update System 2010-12-09 17:01:57 EST --- tritonus-0.3.7-0.9.20101108cvs.fc14 has been pushed to the Fedora 14 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update tritonus'. You can provide feedback for this update here: https://admin.fedoraproject.org/updates/tritonus-0.3.7-0.9.20101108cvs.fc14 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 652403] Review Request: rubygem-boxgrinder-build-rpm-based-os-plugin - Files required to build appliances based on RPMs
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=652403 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #14 from Fedora Update System 2010-12-09 16:59:30 EST --- rubygem-boxgrinder-build-rpm-based-os-plugin-0.0.8-3.fc13 has been pushed to the Fedora 13 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update rubygem-boxgrinder-build-rpm-based-os-plugin'. You can provide feedback for this update here: https://admin.fedoraproject.org/updates/rubygem-boxgrinder-build-rpm-based-os-plugin-0.0.8-3.fc13 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 632970] Review Request: rec-applet - An audio recording applet for the GNOME-desktop
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=632970 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||rec-applet-0.2.3-3.fc13 Resolution||ERRATA Last Closed||2010-12-09 16:58:55 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 632970] Review Request: rec-applet - An audio recording applet for the GNOME-desktop
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=632970 Fedora Update System changed: What|Removed |Added Fixed In Version|rec-applet-0.2.3-3.fc13 |rec-applet-0.2.3-3.fc14 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 632970] Review Request: rec-applet - An audio recording applet for the GNOME-desktop
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=632970 --- Comment #20 from Fedora Update System 2010-12-09 17:02:47 EST --- rec-applet-0.2.3-3.fc14 has been pushed to the Fedora 14 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 632970] Review Request: rec-applet - An audio recording applet for the GNOME-desktop
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=632970 --- Comment #19 from Fedora Update System 2010-12-09 16:58:49 EST --- rec-applet-0.2.3-3.fc13 has been pushed to the Fedora 13 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 652412] Review Request: rubygem-boxgrinder-build-local-delivery-plugin - BoxGrinder plugin delivering appliance to local filesystem
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=652412 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #10 from Fedora Update System 2010-12-09 16:58:23 EST --- rubygem-boxgrinder-build-local-delivery-plugin-0.0.5-1.fc14 has been pushed to the Fedora 14 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update rubygem-boxgrinder-build-local-delivery-plugin'. You can provide feedback for this update here: https://admin.fedoraproject.org/updates/rubygem-boxgrinder-build-local-delivery-plugin-0.0.5-1.fc14 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 543718] Review Request: wxmacmolplt - A graphics program for plotting 3-D molecular structures and normal modes
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=543718 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #7 from Fedora Update System 2010-12-09 16:57:12 EST --- wxmacmolplt-7.4.1-1.fc13 has been pushed to the Fedora 13 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update wxmacmolplt'. You can provide feedback for this update here: https://admin.fedoraproject.org/updates/wxmacmolplt-7.4.1-1.fc13 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 652571] Review Request: ghc-digest - cryptographic hashes of bytestrings
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=652571 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #8 from Fedora Update System 2010-12-09 16:59:25 EST --- ghc-digest-0.0.0.8-1.fc13 has been pushed to the Fedora 13 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update ghc-digest'. You can provide feedback for this update here: https://admin.fedoraproject.org/updates/ghc-digest-0.0.0.8-1.fc13 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 657405] Review Request: lbzip2 - fast, multi-threaded bzip2 utility
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=657405 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #8 from Fedora Update System 2010-12-09 16:57:18 EST --- lbzip2-0.23-1.fc14 has been pushed to the Fedora 14 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update lbzip2'. You can provide feedback for this update here: https://admin.fedoraproject.org/updates/lbzip2-0.23-1.fc14 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 647510] Review Request: premake - A cross-platform build configuration tool
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=647510 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||premake-4.3-2.fc13 Resolution||ERRATA Last Closed||2010-12-09 16:57:31 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 647510] Review Request: premake - A cross-platform build configuration tool
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=647510 --- Comment #17 from Fedora Update System 2010-12-09 16:57:26 EST --- premake-4.3-2.fc13 has been pushed to the Fedora 13 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 660095] Review Request: impressive - A program that displays presentation slides
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=660095 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #15 from Fedora Update System 2010-12-09 16:56:40 EST --- impressive-0.10.3-3.fc13 has been pushed to the Fedora 13 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update impressive'. You can provide feedback for this update here: https://admin.fedoraproject.org/updates/impressive-0.10.3-3.fc13 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 652408] Review Request: rubygem-boxgrinder-build-vmware-platform-plugin - BoxGrinder files required to convert appliances to VMware format
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=652408 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #7 from Fedora Update System 2010-12-09 16:56:56 EST --- rubygem-boxgrinder-build-vmware-platform-plugin-0.0.5-1.fc14 has been pushed to the Fedora 14 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update rubygem-boxgrinder-build-vmware-platform-plugin'. You can provide feedback for this update here: https://admin.fedoraproject.org/updates/rubygem-boxgrinder-build-vmware-platform-plugin-0.0.5-1.fc14 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 650717] Review Request: PolicyKit-olpc - OLPC-specific PolicyKit overrides
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=650717 Daniel Drake changed: What|Removed |Added Flag|fedora-cvs+ |fedora-cvs? --- Comment #7 from Daniel Drake 2010-12-09 15:13:54 EST --- Thanks. I will do that. Unfortunately I still can't build this in rawhide or F14. Please could you investigate? 2655752 build (dist-rawhide, /PolicyKit-olpc:e180650d86e0441c8171e0050c19fb3b1b0b11bb): open (x86-03.phx2.fedoraproject.org) -> FAILED: BuildError: package PolicyKit-olpc is blocked for tag dist-f15 2655760 build (dist-f14-updates-candidate, /PolicyKit-olpc:e180650d86e0441c8171e0050c19fb3b1b0b11bb): open (x86-13.phx2.fedoraproject.org) -> FAILED: BuildError: package PolicyKit-olpc is blocked for tag dist-f14-updates-candidate Package Change Request == Package Name: PolicyKit-olpc Unblock all tags F14 and onwards -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 640356] Review Request: clutter-gtk3
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=640356 --- Comment #23 from Colin Walters 2010-12-09 14:05:55 EST --- Possibly we can just switch everything over to GTK3 now. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 661810] Review Request: libcacard - Common Access Card emulation library
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=661810
--- Comment #2 from Hans de Goede 2010-12-09 14:01:58 EST
---
Some initial comments / things to fix (which make doing a full review a bit
hard):
You've the following defattr line in the spec file (multiple times):
%defattr(-,root,root.-)
That should be:
%defattr(-,root,root,-)
The:
%doc COPYING README
Line should be part of the %files for the main package, iow make the main
%files look like this:
%files
%defattr(-,root,root,-)
%doc COPYING README
%{_libdir}/libcacard.so.*
And remove the %doc line further below
In %files devel I see:
%{_libdir}/libcacard.*a
In Fedora we do not ship .la nor .a files, please remove those (see
/etc/rpmdevtools/spectemplate-minimal.spec )
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 661810] Review Request: libcacard - Common Access Card emulation library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=661810 Hans de Goede changed: What|Removed |Added Status|NEW |ASSIGNED CC||hdego...@redhat.com AssignedTo|nob...@fedoraproject.org|hdego...@redhat.com Flag||fedora-review? --- Comment #1 from Hans de Goede 2010-12-09 13:54:35 EST --- I'll review this and I'll sponsor Alon. Alon note you forgot to set the FE-NEEDSPONSOR blocker bug on this bug. No need to do that now as I'll sponsor you. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 661833] Review Request: kdevelop-php - Php language plugin for KDevelop
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=661833 --- Comment #1 from Rex Dieter 2010-12-09 13:50:15 EST --- rpmlint *.rpm x86_64/*.rpm kdevelop-php.x86_64: E: invalid-soname /usr/lib64/libkdev4phpparser.so libkdev4phpparser.so kdevelop-php.x86_64: E: invalid-soname /usr/lib64/libkdev4phpcompletion.so libkdev4phpcompletion.so kdevelop-php.x86_64: E: invalid-soname /usr/lib64/libkdev4phpduchain.so libkdev4phpduchain.so 3 packages and 0 specfiles checked; 3 errors, 0 warnings. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 661833] Review Request: kdevelop-php - Php language plugin for KDevelop
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=661833 Rex Dieter changed: What|Removed |Added Alias||kdevelop-php -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 661832] Review Request: kdevelop-pg-qt - A parser generator
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=661832 Rex Dieter changed: What|Removed |Added Alias||kdevelop-pg-qt -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 661832] Review Request: kdevelop-pg-qt - A parser generator
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=661832 Rex Dieter changed: What|Removed |Added Blocks||656997(kde-reviews) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 661832] Review Request: kdevelop-pg-qt - A parser generator
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=661832 --- Comment #1 from Rex Dieter 2010-12-09 13:45:03 EST --- scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=2655630 rpmlint *.rpm */*.rpm kdevelop-pg-qt-devel.noarch: W: no-documentation kdevelop-pg-qt.x86_64: W: no-manual-page-for-binary kdev-pg-qt 4 packages and 0 specfiles checked; 0 errors, 2 warnings. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 661833] New: Review Request: kdevelop-php - Php language plugin for KDevelop
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: kdevelop-php - Php language plugin for KDevelop https://bugzilla.redhat.com/show_bug.cgi?id=661833 Summary: Review Request: kdevelop-php - Php language plugin for KDevelop Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: rdie...@math.unl.edu QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://rdieter.fedorapeople.org/rpms/kdevelop-php/kdevelop-php.spec SRPM URL: http://rdieter.fedorapeople.org/rpms/kdevelop-php/kdevelop-php-1.1.1-1.fc13.src.rpm Description: Php language plugin for KDevelop -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 661833] Review Request: kdevelop-php - Php language plugin for KDevelop
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=661833 Rex Dieter changed: What|Removed |Added Blocks||656997(kde-reviews) Depends on||661832 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 661832] Review Request: kdevelop-pg-qt - A parser generator
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=661832 Rex Dieter changed: What|Removed |Added Blocks||661833 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 661832] New: Review Request: kdevelop-pg-qt - A parser generator
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: kdevelop-pg-qt - A parser generator https://bugzilla.redhat.com/show_bug.cgi?id=661832 Summary: Review Request: kdevelop-pg-qt - A parser generator Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: rdie...@math.unl.edu QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://rdieter.fedorapeople.org/rpms/kdevelop-php/kdevelop-pg-qt.spec SRPM URL: http://rdieter.fedorapeople.org/rpms/kdevelop-php/kdevelop-pg-qt-0.9.0-1.fc13.src.rpm Description: KDevelop-PG-Qt is a parser generator written in readable source-code and generating readable source-code. Its syntax was inspirated by AntLR. It implements the visitor-pattern and uses the Qt library. That is why it is ideal to be used in Qt-/KDE-based applications like KDevelop. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 647885] Review Request: apache-rat - Apache Release Audit Tool (RAT)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=647885 --- Comment #8 from Orion Poplawski 2010-12-09 13:39:01 EST --- Also: - Drop versioned jar and javadoc - Drop BuildRoot and %%clean -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 647885] Review Request: apache-rat - Apache Release Audit Tool (RAT)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=647885 Orion Poplawski changed: What|Removed |Added Blocks||652183(FE-JAVASIG) --- Comment #7 from Orion Poplawski 2010-12-09 13:34:39 EST --- http://www.cora.nwra.com/~orion/fedora/apache-rat.spec http://www.cora.nwra.com/~orion/fedora/apache-rat-0.8-0.4.20100827.fc14.src.rpm * Mon Nov 1 2010 Orion Poplawski 0.8-0.4.20100827 - Change BR to ant-antunit -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 458714] Review Request: libkate - Libraries to handle the Kate bitstream format
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=458714 Nicolas Chauvet (kwizart) changed: What|Removed |Added Flag|fedora-cvs+ |fedora-cvs? --- Comment #21 from Nicolas Chauvet (kwizart) 2010-12-09 12:23:08 EST --- Package Change Request == Package Name: libkate New Branches: el5 el6 Owners: kwizart -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 461119] Review Request: libtiger - Rendering library for Kate streams using Pango and Cairo
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=461119 Nicolas Chauvet (kwizart) changed: What|Removed |Added Flag|fedora-cvs+ |fedora-cvs? --- Comment #12 from Nicolas Chauvet (kwizart) 2010-12-09 12:23:25 EST --- Package Change Request == Package Name: libtiger New Branches: el5 el6 Owners: kwizart -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 661810] New: Review Request: libcacard - Common Access Card emulation library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: libcacard - Common Access Card emulation library https://bugzilla.redhat.com/show_bug.cgi?id=661810 Summary: Review Request: libcacard - Common Access Card emulation library Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: al...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Target Release: --- Spec URL: http://people.freedesktop.org/~alon/libcacard.spec SRPM URL: http://people.freedesktop.org/~alon/libcacard-0.1.0-2.fc14.src.rpm Description: This library emulates CAC, or Common Access Cards, a specification used for java smart cards used for authentication, carrying personal information, encryption of and signature of documents. The library relies on NSS for it's backend, and provides an API for clients to instantiate cards and applets on the cards, and provide certificates for the CAC applet. It was written for usage by SPICE and Qemu to let a vm access a remote or host physical or emulated card. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 223657] Review Request: PerceptualDiff - An image comparison utility
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=223657 Nicolas Chauvet (kwizart) changed: What|Removed |Added Flag|fedora-cvs+ |fedora-cvs? --- Comment #32 from Nicolas Chauvet (kwizart) 2010-12-09 12:23:59 EST --- Package Change Request == Package Name: PerceptualDiff New Branches: el5 el6 Owners: kwizart -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 427121] Review Request: grib_api - ECMWF encoding/decoding GRIB software
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=427121 Orion Poplawski changed: What|Removed |Added Flag|needinfo?(pertu...@free.fr) | -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 243642] Review Request: schroedinger - Portable libraries for the high quality Dirac video codec
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=243642 Nicolas Chauvet (kwizart) changed: What|Removed |Added Flag|fedora-cvs+ |fedora-cvs? --- Comment #9 from Nicolas Chauvet (kwizart) 2010-12-09 11:34:46 EST --- Package Change Request == Package Name: schroedinger New Branches: el5 el6 Owners: jcollie kwizart -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 626122] Review Request: libqmf - Qt Messaging Framework
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=626122 --- Comment #8 from Rex Dieter 2010-12-09 11:35:30 EST --- We're trying to negotiate the qmf namespace clash with qpid-cpp maintainers (that package currently produces subpkgs named qmf and qmf-devel). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 624146] Review Request: ups debugger - a C/C++ debugger w/ GUI
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=624146 Jason Tibbitts changed: What|Removed |Added Status Whiteboard||StalledSubmitter -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 647602] Review Request: ant-antunit - Provide antunit ant task
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=647602 Orion Poplawski changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE Last Closed||2010-12-09 11:09:35 --- Comment #11 from Orion Poplawski 2010-12-09 11:09:35 EST --- Imported and built. Thanks. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 647885] Review Request: apache-rat - Apache Release Audit Tool (RAT)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=647885 Bug 647885 depends on bug 647602, which changed state. Bug 647602 Summary: Review Request: ant-antunit - Provide antunit ant task https://bugzilla.redhat.com/show_bug.cgi?id=647602 What|Old Value |New Value Status|NEW |ASSIGNED Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 541462] Review Request: rasmol - Molecular Graphics Visualization Tool
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=541462 --- Comment #17 from Takanori MATSUURA 2010-12-09 10:55:06 EST --- I'd like to add GTK2 support for el6 and Fedora 13-15 based on Carl's SRPM because it is reviewed here. However I cannot get spec and SRPM files from www.five-ten-sg.com website. If Carl doesn't have enough time, I can work. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 623606] Review Request: gxneur - GTK front-end for X Neural Switcher
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=623606 Pavel Alexeev (aka Pahan-Hubbitus) changed: What|Removed |Added Alias||gxneur -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 541462] Review Request: rasmol - Molecular Graphics Visualization Tool
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=541462 --- Comment #16 from Chen Lei 2010-12-09 10:21:47 EST --- (In reply to comment #15) > Where is the latest Spec and SRPM files? CBFlib isn't available in Fedora yet. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 490988] Review Request: libvdpau - Wrapper library for the Video Decode and Presentation API
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=490988 Nicolas Chauvet (kwizart) changed: What|Removed |Added Flag|fedora-cvs+ |fedora-cvs? --- Comment #17 from Nicolas Chauvet (kwizart) 2010-12-09 10:26:15 EST --- Package Change Request == Package Name: libvdpau New Branches: el5 el6 Owners: kwizart -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 522245] Review Request: vdpauinfo - Tool to query the capabilities of a VDPAU implementation
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=522245 Nicolas Chauvet (kwizart) changed: What|Removed |Added Flag|fedora-cvs+ |fedora-cvs? --- Comment #7 from Nicolas Chauvet (kwizart) 2010-12-09 10:27:02 EST --- Package Change Request == Package Name: vdpauinfo New Branches: el5 el6 Owners: kwizart -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 652573] Review Request: ghc-zip-archive - Haskell zip-archive library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=652573 Jens Petersen changed: What|Removed |Added Status|NEW |ASSIGNED --- Comment #6 from Jens Petersen 2010-12-09 10:22:21 EST --- (In reply to comment #3) > patch to fix up rpmlint warning buildroot setting and cleaning is now deprecated in the Packaging Guidelines. > added COPYING to the base rpm Thanks - fixing: Spec: http://petersen.fedorapeople.org/ghc-zip-archive/ghc-zip-archive.spec SRPM: http://petersen.fedorapeople.org/ghc-zip-archive/ghc-zip-archive-0.1.1.6-3.fc13.src.rpm (In reply to comment #4) > I am unable to find the exported module ( Codec.Archive.Zip ) via ghci. I was trying to test earlier but had some system (rpm or fs issue) problem. Can you describe the steps you are taking? So, sorry I can't test right now with rawhide but at least the following works for me: $ rpm -q ghc ghc-6.12.1-5.fc13.i686 $ cabal install zip-zrchive : : $ ghci -package zip-archive GHCi, version 6.12.1: http://www.haskell.org/ghc/ :? for help Loading package ghc-prim ... linking ... done. : Loading package zip-archive-0.1.1.6 ... linking ... done. Loading package ffi-1.0 ... linking ... done. Prelude> :module Codec.Archive.Zip Prelude Codec.Archive.Zip> I can certainly look into to it more later, and we should certainly support ghci, but I am not sure if it is a review blocker. (In reply to comment #5) > When I run ghci as root, I can see the exported module but not as a normal > user. As a normal user what is your output of "ghc-pkg list zip-archive"? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 659895] Review Request: makedepf90 - Create Makefile dependency list for Fortran source files
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=659895 --- Comment #5 from Fedora Update System 2010-12-09 09:09:06 EST --- makedepf90-2.8.8-1.fc14 has been submitted as an update for Fedora 14. https://admin.fedoraproject.org/updates/makedepf90-2.8.8-1.fc14 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 660600] Rename Request: pcsc-lite-ccid - Generic USB CCID smart card reader driver
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=660600 Peter Lemenkov changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #4 from Peter Lemenkov 2010-12-09 09:06:42 EST --- Ok, good, I can;'t find any other issues so this package is APPROVED. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 659895] Review Request: makedepf90 - Create Makefile dependency list for Fortran source files
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=659895 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 659895] Review Request: makedepf90 - Create Makefile dependency list for Fortran source files
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=659895 --- Comment #6 from Fedora Update System 2010-12-09 09:09:14 EST --- makedepf90-2.8.8-1.fc13 has been submitted as an update for Fedora 13. https://admin.fedoraproject.org/updates/makedepf90-2.8.8-1.fc13 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 660625] Review Request: techne - A general purpose, programmable physical simulator and renderer
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=660625
--- Comment #3 from Tadej Janež 2010-12-09
10:01:35 EST ---
Hans, thanks for the review!
(In reply to comment #2)
> MUST FIX / needs work:
> - sources match upstream
> I cannot verify this, please ask upstream to put a pre-release / alpha
> / beta tarbal somewhere.
I'm in contact with the upstream author and he'll put a tarball out shortly.
> - owns all directories that it creates
> Currently the package does not own the following dirs:
> %{_libdir}/%{name}
> %{_datadir}/%{name}
> %{_datadir}/%{name}/common
> This can be fixed by replacing (in %files) this:
> %{_libdir}/%{name}/*.so
> %{_libdir}/%{name}/*.bin
> %{_datadir}/%{name}/common/*
> With:
> %{_libdir}/%{name}
> %{_datadir}/%{name}
Thanks for pointing this out, fixed.
> - release field not properly used.
> Since this is a pre release version being used the release field
> should be something like:
> Release: 0.1%{?dist}
> or:
> Release: 0.1.beta1%{?dist}
Fixed.
The updated SPEC and SRPM files are available at:
http://tadej.fedorapeople.org/techne.spec
http://tadej.fedorapeople.org/techne-0.2-0.2.fc13.src.rpm
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 543718] Review Request: wxmacmolplt - A graphics program for plotting 3-D molecular structures and normal modes
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=543718 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 652412] Review Request: rubygem-boxgrinder-build-local-delivery-plugin - BoxGrinder plugin delivering appliance to local filesystem
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=652412 --- Comment #9 from Fedora Update System 2010-12-09 09:22:57 EST --- rubygem-boxgrinder-build-local-delivery-plugin-0.0.5-1.fc13 has been submitted as an update for Fedora 13. https://admin.fedoraproject.org/updates/rubygem-boxgrinder-build-local-delivery-plugin-0.0.5-1.fc13 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 660600] Rename Request: pcsc-lite-ccid - Generic USB CCID smart card reader driver
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=660600 Kalev Lember changed: What|Removed |Added Flag||fedora-cvs? --- Comment #5 from Kalev Lember 2010-12-09 09:21:27 EST --- New Package SCM Request === Package Name: pcsc-lite-ccid Short Description: Generic USB CCID smart card reader driver Owners: rrelyea kalev Branches: InitialCC: A special request for the SCM admin: This is a rename of an existing package, so I would like to keep the history if possible. When you are creating the new git repo, can you keep it initialized in empty state so I can push the existing history from ccid package's master branch? Thanks! -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 543718] Review Request: wxmacmolplt - A graphics program for plotting 3-D molecular structures and normal modes
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=543718 --- Comment #5 from Fedora Update System 2010-12-09 09:22:21 EST --- wxmacmolplt-7.4.1-1.fc14 has been submitted as an update for Fedora 14. https://admin.fedoraproject.org/updates/wxmacmolplt-7.4.1-1.fc14 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 543718] Review Request: wxmacmolplt - A graphics program for plotting 3-D molecular structures and normal modes
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=543718 --- Comment #6 from Fedora Update System 2010-12-09 09:22:29 EST --- wxmacmolplt-7.4.1-1.fc13 has been submitted as an update for Fedora 13. https://admin.fedoraproject.org/updates/wxmacmolplt-7.4.1-1.fc13 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 660600] Rename Request: pcsc-lite-ccid - Generic USB CCID smart card reader driver
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=660600 --- Comment #3 from Kalev Lember 2010-12-09 08:57:13 EST --- Thanks for doing this, Peter. * Thu Dec 09 2010 Kalev Lember - 1.4.0-4 - Install src/openct/LICENSE file as LICENSE.openct in docs (#660600) - Added 'and GPLv2+' to license tag to cover RSA_SecurID (#660600) Spec URL: http://kalev.fedorapeople.org/pcsc-lite-ccid.spec SRPM URL: http://kalev.fedorapeople.org/pcsc-lite-ccid-1.4.0-4.fc15.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 659896] Review Request: cp2k - A molecular dynamics engine capable of classical and Car-Parrinello simulations
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=659896 Dominik 'Rathann' Mierzejewski changed: What|Removed |Added Flag||fedora-cvs? --- Comment #7 from Dominik 'Rathann' Mierzejewski 2010-12-09 08:50:56 EST --- Thanks a lot! New Package SCM Request === Package Name: cp2k Short Description: A molecular dynamics engine capable of classical and Car-Parrinello simulations Owners: rathann Branches: f13 f14 el6 InitialCC: -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 523877] Review Request: CBFlib - crystallography binary format library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=523877 --- Comment #28 from Dmitrij S. Kryzhevich 2010-12-09 08:31:54 EST --- 1. What with License? "GPLv2+ and (GPLv2+ or LGPLv2+)" is rather strange one. May be it should be just "GPLv2+ and LGPLv2+"? 2. For clearity, split "iconv" string into 2, uniting them with "\". 3. Looks like everybody wants it's own copy of md5. Ok. Let it be so. 4. rpmlint claims on shared-lib-calls-exit /usr/lib64/libcbf.so.0.0.0 e...@glibc_2.2.5 Could it be resolved? 5. +1 to -lib subpackage. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 590244] Review Request: pinta - Simple Paint Application
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=590244 Paul Lange changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE Last Closed||2010-12-09 08:15:44 --- Comment #34 from Paul Lange 2010-12-09 08:15:44 EST --- Pinta is now available in devel. Thanks to everyone who helped! -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 523877] Review Request: CBFlib - crystallography binary format library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=523877 Dmitrij S. Kryzhevich changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|nob...@fedoraproject.org|kr...@land.ru Status Whiteboard|BuildFails | Flag||fedora-review? --- Comment #27 from Dmitrij S. Kryzhevich 2010-12-09 08:07:45 EST --- Let me see what I can do with this. Btw, remioved BuildFails as Comment 26 shows it builds. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 652412] Review Request: rubygem-boxgrinder-build-local-delivery-plugin - BoxGrinder plugin delivering appliance to local filesystem
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=652412 --- Comment #8 from Fedora Update System 2010-12-09 08:02:12 EST --- rubygem-boxgrinder-build-local-delivery-plugin-0.0.5-1.fc14 has been submitted as an update for Fedora 14. https://admin.fedoraproject.org/updates/rubygem-boxgrinder-build-local-delivery-plugin-0.0.5-1.fc14 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 652412] Review Request: rubygem-boxgrinder-build-local-delivery-plugin - BoxGrinder plugin delivering appliance to local filesystem
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=652412 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED Bug 652412 depends on bug 652400, which changed state. Bug 652400 Summary: Review Request: rubygem-boxgrinder-build - Creates appliances for various virtual environments https://bugzilla.redhat.com/show_bug.cgi?id=652400 What|Old Value |New Value Status|ASSIGNED|MODIFIED Status|MODIFIED|ON_QA Resolution||ERRATA Status|ON_QA |CLOSED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 652414] Review Request: rubygem-boxgrinder-build-sftp-delivery-plugin - BoxGrinder plugin delivering appliances to SFTP servers
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=652414 Bug 652414 depends on bug 652400, which changed state. Bug 652400 Summary: Review Request: rubygem-boxgrinder-build - Creates appliances for various virtual environments https://bugzilla.redhat.com/show_bug.cgi?id=652400 What|Old Value |New Value Status|ASSIGNED|MODIFIED Status|MODIFIED|ON_QA Resolution||ERRATA Status|ON_QA |CLOSED --- Comment #5 from Marek Goldmann 2010-12-09 07:46:43 EST --- Dependencies should be now in stable. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 652408] Review Request: rubygem-boxgrinder-build-vmware-platform-plugin - BoxGrinder files required to convert appliances to VMware format
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=652408 --- Comment #6 from Fedora Update System 2010-12-09 07:45:30 EST --- rubygem-boxgrinder-build-vmware-platform-plugin-0.0.5-1.fc14 has been submitted as an update for Fedora 14. https://admin.fedoraproject.org/updates/rubygem-boxgrinder-build-vmware-platform-plugin-0.0.5-1.fc14 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 652408] Review Request: rubygem-boxgrinder-build-vmware-platform-plugin - BoxGrinder files required to convert appliances to VMware format
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=652408 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED Bug 652408 depends on bug 652400, which changed state. Bug 652400 Summary: Review Request: rubygem-boxgrinder-build - Creates appliances for various virtual environments https://bugzilla.redhat.com/show_bug.cgi?id=652400 What|Old Value |New Value Status|ASSIGNED|MODIFIED Status|MODIFIED|ON_QA Resolution||ERRATA Status|ON_QA |CLOSED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 652408] Review Request: rubygem-boxgrinder-build-vmware-platform-plugin - BoxGrinder files required to convert appliances to VMware format
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=652408 --- Comment #5 from Fedora Update System 2010-12-09 07:36:02 EST --- rubygem-boxgrinder-build-vmware-platform-plugin-0.0.5-1.fc13 has been submitted as an update for Fedora 13. https://admin.fedoraproject.org/updates/rubygem-boxgrinder-build-vmware-platform-plugin-0.0.5-1.fc13 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 652403] Review Request: rubygem-boxgrinder-build-rpm-based-os-plugin - Files required to build appliances based on RPMs
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=652403 --- Comment #13 from Fedora Update System 2010-12-09 07:19:26 EST --- rubygem-boxgrinder-build-rpm-based-os-plugin-0.0.8-3.fc13 has been submitted as an update for Fedora 13. https://admin.fedoraproject.org/updates/rubygem-boxgrinder-build-rpm-based-os-plugin-0.0.8-3.fc13 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 652403] Review Request: rubygem-boxgrinder-build-rpm-based-os-plugin - Files required to build appliances based on RPMs
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=652403 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED Bug 652403 depends on bug 652400, which changed state. Bug 652400 Summary: Review Request: rubygem-boxgrinder-build - Creates appliances for various virtual environments https://bugzilla.redhat.com/show_bug.cgi?id=652400 What|Old Value |New Value Status|ASSIGNED|MODIFIED Status|MODIFIED|ON_QA Resolution||ERRATA Status|ON_QA |CLOSED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 652403] Review Request: rubygem-boxgrinder-build-rpm-based-os-plugin - Files required to build appliances based on RPMs
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=652403 --- Comment #12 from Fedora Update System 2010-12-09 07:18:49 EST --- rubygem-boxgrinder-build-rpm-based-os-plugin-0.0.8-3.fc14 has been submitted as an update for Fedora 14. https://admin.fedoraproject.org/updates/rubygem-boxgrinder-build-rpm-based-os-plugin-0.0.8-3.fc14 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 661660] New: Review Request: metamorphose2 - A cross platform file and folder mass renamer
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: metamorphose2 - A cross platform file and folder mass renamer https://bugzilla.redhat.com/show_bug.cgi?id=661660 Summary: Review Request: metamorphose2 - A cross platform file and folder mass renamer Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: pin...@pingoured.fr QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://pingou.fedorapeople.org/RPMs/metamorphose2.spec SRPM URL: http://pingou.fedorapeople.org/RPMs/metamorphose2-2.0.7.1-1.fc14.src.rpm Description: A cross platform file and folder mass renamer, allows many different renaming operations in a GUI. Features include search and replace (with RE), insert, numbering, date/time, id3 & EXIF tag read, change length, get all files in sub-dirs, undo/redo, etc. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 541462] Review Request: rasmol - Molecular Graphics Visualization Tool
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=541462 Takanori MATSUURA changed: What|Removed |Added Flag||needinfo?(c...@five-ten-sg. ||com) Bug 541462 depends on bug 545046, which changed state. Bug 545046 Summary: Review Request: CVector - ANSI C API for Dynamic Arrays https://bugzilla.redhat.com/show_bug.cgi?id=545046 What|Old Value |New Value Status|ASSIGNED|MODIFIED Status|MODIFIED|ON_QA Resolution||ERRATA Status|ON_QA |CLOSED Bug 541462 depends on bug 545045, which changed state. Bug 545045 Summary: Review Request: CQRlib - ANSI C API for quaternion arithmetic and rotation https://bugzilla.redhat.com/show_bug.cgi?id=545045 What|Old Value |New Value Status|MODIFIED|ON_QA Resolution||ERRATA Status|ON_QA |CLOSED --- Comment #15 from Takanori MATSUURA 2010-12-09 06:29:42 EST --- Where is the latest Spec and SRPM files? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 652576] Review Request: ghc-texmath - Haskell texmathml library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=652576 Jens Petersen changed: What|Removed |Added Status|NEW |ASSIGNED Flag||fedora-cvs? --- Comment #3 from Jens Petersen 2010-12-09 05:22:48 EST --- Thanks for the review. Good point about syb - I will add it before building for rawhide. New Package SCM Request === Package Name: ghc-texmath Short Description: Haskell texmathml library Owners: petersen Branches: f13 f14 InitialCC: haskell-sig -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 661312] Review Request: mingw32-atkmm - MinGW Windows C++ interface for the ATK library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=661312 --- Comment #2 from Thomas Sailer 2010-12-09 05:54:13 EST --- Thanks for taking this and for the quick response! I've updated the links above with a version incorporating all your changes. I now get 8 rpmlint warnings: mingw32-atkmm.src: W: no-cleaning-of-buildroot %install mingw32-atkmm.src: W: no-cleaning-of-buildroot %clean mingw32-atkmm.src: W: no-buildroot-tag mingw32-atkmm.src: W: no-%clean-section mingw32-atkmm-debuginfo.noarch: E: debuginfo-without-sources mingw32-atkmm.spec: W: no-cleaning-of-buildroot %install mingw32-atkmm.spec: W: no-cleaning-of-buildroot %clean mingw32-atkmm.spec: W: no-buildroot-tag mingw32-atkmm.spec: W: no-%clean-section 3 packages and 1 specfiles checked; 1 errors, 8 warnings. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 661660] Review Request: metamorphose2 - A cross platform file and folder mass renamer
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=661660 Mohamed El Morabity changed: What|Removed |Added Status|NEW |ASSIGNED CC||pikachu.2...@gmail.com AssignedTo|nob...@fedoraproject.org|pikachu.2...@gmail.com --- Comment #1 from Mohamed El Morabity 2010-12-09 05:41:26 EST --- I will review it. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 660600] Rename Request: pcsc-lite-ccid - Generic USB CCID smart card reader driver
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=660600
--- Comment #2 from Peter Lemenkov 2010-12-09 05:30:58 EST
---
REVIEW:
Legend: + = PASSED, - = FAILED, 0 = Not Applicable
- rpmlint is not silent:
work ~: rpmlint ~/Desktop/pcsc-lite-ccid-*
pcsc-lite-ccid.src:21: W: unversioned-explicit-provides pcsc-ifd-handler
^^^ this is exactly what all other pcsc-lite backends(read pcsc-lite-openct )
should contain. So it's not an issue.
pcsc-lite-ccid.x86_64: W: non-conffile-in-etc
/etc/udev/rules.d/85-pcscd_ccid.rules
^^^ this file is not intended to modify by user.
pcsc-lite-ccid.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/pcsc/drivers/ifd-ccid.bundle/Contents/Linux/libccid.so
pcsc-lite-ccid.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/pcsc/drivers/serial/libccidtwin.so
^^^ I suspect that these are plugins, so it's ok.
3 packages and 0 specfiles checked; 0 errors, 4 warnings.
work ~:
+ The package is named according to the Package Naming Guidelines.
+ The spec file name matches the base package %{name}, in the format
%{name}.spec.
+ The package meets the Packaging Guidelines.
+ The package is licensed with a Fedora approved license and meets the
Licensing Guidelines.
- The License field in the package spec file MUST matche the actual license.
The RSA_SecurID_getpasswd utility is licensed under GPLv2+. Parts of src/openct
are licensed under MIT but it absorbs by LGPLv2+ which covers the rest of
src/openct stuff, so you need to include only src/openct/LICENSE to meet the
requirements. The file src/strlcpy.c is ISC licensed.
So, please change License tag to GPLv2+ and LGPLv2+ (probably with brief
explanation whuich parts are covered by GPLv2+) and add src/openct/LICENSE to
docs (rename it to something to avoid confusion - it covers only very limited
set of sources.
I'm not sure what to do with ISC-licensed stuff - the Guidelines explicitly
mention that we need to add license only in case when upstream provides them in
the separate file. On the contrary ISC license explicitly demands us to ship
their licensing terms along with our derived product (Fedora rpm for
pcsc-lite-ccid). This is a very minor issue, so I'd rather just ask upstream to
do something with this in the further ccid releases.
+/- The file, containing the text of the license(s) for the package, is
included in %doc (see my previous comments).
+ The spec file is written in American English.
+ The spec file for the package is legible.
+ The sources used to build the package, match the upstream source, as provided
in the spec URL.
sulaco ~/rpmbuild/SOURCES: sha256sum ccid-1.4.0.tar.bz2*
a0b6e56c54f311c4cb39eef0f362e4e6d388bf815a15532e4524d2be1723b4ff
ccid-1.4.0.tar.bz2
a0b6e56c54f311c4cb39eef0f362e4e6d388bf815a15532e4524d2be1723b4ff
ccid-1.4.0.tar.bz2.1
sulaco ~/rpmbuild/SOURCES:
+ The package successfully compiles and builds into binary rpms on at least one
primary architecture.
+ All build dependencies are listed in BuildRequires.
0 No need to handle locales.
0 No shared library files in some of the dynamic linker's default paths.
+ The package does NOT bundle copies of system libraries.
0 The package is not designed to be relocatable.
+ The package owns all directories that it creates.
+ The package does not list a file more than once in the spec file's %files
listings.
+ Permissions on files are set properly.
+ The package has a %clean section, which contains rm -rf %{buildroot} (or
$RPM_BUILD_ROOT).
+ The package consistently uses macros.
+ The package contains code, or permissible content.
0 No extremely large documentation files.
+ Anything, the package includes as %doc, does not affect the runtime of the
application.
0 No header files.
0 No static libraries.
0 No pkgconfig(.pc) files.
0 No devel sub-package.
+ The package does NOT contain any .la libtool archives.
0 Not a GUI application.
+ The package does not own files or directories already owned by other
packages.
+ At the beginning of %install, the package runs rm -rf %{buildroot} (or
$RPM_BUILD_ROOT).
+ All filenames in rpm packages are valid UTF-8.
Please, apply changes, suggested above, to License tag, and I'll continue.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
