[Bug 655601] Review Request: jing-trang - Schema validation and conversion based on RELAX NG
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=655601 Paul Howarth p...@city-fan.org changed: What|Removed |Added CC||p...@city-fan.org --- Comment #20 from Paul Howarth p...@city-fan.org 2011-01-04 04:22:58 EST --- How come there's no build of this in Rawhide? -- 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 655601] Review Request: jing-trang - Schema validation and conversion based on RELAX NG
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=655601 --- Comment #21 from Ville Skyttä ville.sky...@iki.fi 2011-01-04 12:37:14 EST --- I thought that the rawhide compose would pull it automatically in from previous release updates like it tends to do if a package hasn't been built for rawhide and there's a newer version in previous released updates, but perhaps it doesn't work the same way for new packages. Anyway, it's built for rawhide now as well. -- 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 655601] Review Request: jing-trang - Schema validation and conversion based on RELAX NG
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=655601 --- Comment #22 from Paul Howarth p...@city-fan.org 2011-01-04 12:42:26 EST --- I think inheritance was deliberately broken between F14 and Rawhide for something related to deltarpm creation and the new xz file format in Rawhide. -- 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 655601] Review Request: jing-trang - Schema validation and conversion based on RELAX NG
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=655601 --- Comment #23 from Ville Skyttä ville.sky...@iki.fi 2011-01-04 13:39:11 EST --- I see. I've just posted to the devel list asking for clarification. -- 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 655601] Review Request: jing-trang - Schema validation and conversion based on RELAX NG
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=655601 Alexander Kurtakov akurt...@redhat.com changed: What|Removed |Added Blocks|652183(FE-JAVASIG) | -- 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 655601] Review Request: jing-trang - Schema validation and conversion based on RELAX NG
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=655601 --- Comment #19 from Fedora Update System upda...@fedoraproject.org 2010-12-12 15:03:08 EST --- jing-trang-2009-7.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 655601] Review Request: jing-trang - Schema validation and conversion based on RELAX NG
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=655601 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #18 from Fedora Update System upda...@fedoraproject.org 2010-12-04 19:40:20 EST --- jing-trang-2009-7.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 jing-trang'. You can provide feedback for this update here: https://admin.fedoraproject.org/updates/jing-trang-2009-7.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 655601] Review Request: jing-trang - Schema validation and conversion based on RELAX NG
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=655601 Stanislav Ochotnicky socho...@redhat.com changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #14 from Stanislav Ochotnicky socho...@redhat.com 2010-12-03 07:15:54 EST --- Package seems good now, builds fine and is all in all very nice :-) 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 655601] Review Request: jing-trang - Schema validation and conversion based on RELAX NG
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=655601 Ville Skyttä ville.sky...@iki.fi changed: What|Removed |Added Flag||fedora-cvs? --- Comment #15 from Ville Skyttä ville.sky...@iki.fi 2010-12-03 12:06:30 EST --- New Package SCM Request === Package Name: jing-trang Short Description: Schema validation and conversion based on RELAX NG Owners: scop Branches: f14 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 655601] Review Request: jing-trang - Schema validation and conversion based on RELAX NG
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=655601 --- Comment #16 from Jason Tibbitts ti...@math.uh.edu 2010-12-03 15:40:08 EST --- Git done (by process-git-requests). -- 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 655601] Review Request: jing-trang - Schema validation and conversion based on RELAX NG
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=655601 --- Comment #17 from Fedora Update System upda...@fedoraproject.org 2010-12-03 17:20:17 EST --- jing-trang-2009-7.fc14 has been submitted as an update for Fedora 14. https://admin.fedoraproject.org/updates/jing-trang-2009-7.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 655601] Review Request: jing-trang - Schema validation and conversion based on RELAX NG
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=655601 Fedora Update System upda...@fedoraproject.org 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 655601] Review Request: jing-trang - Schema validation and conversion based on RELAX NG
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=655601 --- Comment #13 from Ville Skyttä ville.sky...@iki.fi 2010-12-02 11:40:00 EST --- http://scop.fedorapeople.org/packages/jing-trang.spec http://scop.fedorapeople.org/packages/jing-trang-2009-7.fc14.src.rpm http://koji.fedoraproject.org/koji/taskinfo?taskID=2639853 - Put Xalan instead of Saxon in build path (regression in -6). - Build with OpenJDK. Comment 12 was correct, but it's not enough to fix the build with GCJ, there are other issues. This one's built with OpenJDK, I don't plan to spend time to get the build working with GCJ. -- 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 655601] Review Request: jing-trang - Schema validation and conversion based on RELAX NG
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=655601 --- Comment #11 from Stanislav Ochotnicky socho...@redhat.com 2010-12-01 08:10:00 EST --- Hmm, the build fails in my local mock and even koji now: http://koji.fedoraproject.org/koji/taskinfo?taskID=2636658 -- 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 655601] Review Request: jing-trang - Schema validation and conversion based on RELAX NG
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=655601 --- Comment #12 from Ville Skyttä ville.sky...@iki.fi 2010-12-01 18:18:31 EST --- Hm, it builds fine for F-13 and F-14 mock here, but IIRC it ends up being built with openjdk for me and not gcj like the koji log indicates. There's a leftover buglet from my Saxon tries which I suppose may cause this; export CLASSPATH=$(build-classpath saxon) should be replaced by export CLASSPATH=$(build-classpath xalan-j2) at start of %build. Can't test right now, but probably can tomorrow. -- 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 655601] Review Request: jing-trang - Schema validation and conversion based on RELAX NG
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=655601 --- Comment #8 from Stanislav Ochotnicky socho...@redhat.com 2010-11-30 04:27:14 EST --- (In reply to comment #7) 1. (optional) jing-trang-prepare-tarball.sh from srpm can be removed from srpm All the packages I've worked with include it in the source rpm. It doesn't cost anything to have it there, and the upside is that people can actually verify the contents of the srpm as well as roll updates just with the information in it without having to hunt missing bits from elsewhere. Sure, no problem. 2. simplify documentation installation by using %doc macro I intentionally didn't do that initially because for some reason I remembered that the special %doc macro would do a cp --parents and would thus include unwanted subdirs below the doc dirs. But that's incorrect, simplified in -5. Good 3. remove %clean section and rm -rf from %install section These (and the Group tags) are intentionally left there because I want the package to be cleanly buildable as is for example on EL-5 (assuming dependencies are available). They don't cause any problems, and I can easily live with them around. I though that might be the case. OK, no problem then. I must say I am impressed with the patch and bugs you filed upstream to fix the build system a bit. Great work. Thanks, and thanks for the review which got quite a bit bigger than what you initially signed up for. Next revision with the %doc stuff simplified: http://scop.fedorapeople.org/packages/jing-trang.spec http://scop.fedorapeople.org/packages/jing-trang-2009-5.fc14.src.rpm Looking into this now. Some things I plan to look into whether they'd make sense and be feasible to implement in the future for this package, let me know if you'd like them or some of them done during the review, or have any opinions about them, in no particular order: - Try to port the missing test suite generation bits to some other XSLT processor than the old saxon, and if successful, run the whole test suite. I think the whole building process for Fedora and other distributions would be better off with no XSLT generation at all...but maybe I am wrong, since I just had a glimpse at what you had to do to make this work. - Install dtdinst's schemas and XSLs as non-%doc and register them with the system XML catalog. Yes this should happen too. You probably know how to do it, but just in case...you can look at log4j spec file which installs log4j dtds - Review jing's %docs; isorelax.copying.txt and xerces.copying.txt could/should maybe be dropped because they're not shipped in this package (and copying.html maybe modified not to link to them) and maybe datatype-sample.jar should be dropped as well. Good idea, also asking upstream about it might help. -- 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 655601] Review Request: jing-trang - Schema validation and conversion based on RELAX NG
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=655601 --- Comment #9 from Stanislav Ochotnicky socho...@redhat.com 2010-11-30 05:20:45 EST --- Ok, sorry about not noticing it right away, but javadoc sub-package doesn't include license. There was recent change in packaging guidelines (~2-3 months ago) that requires all independent packages to include license. Since your javadoc doesn't Require either of the other sub-packages it has to include license. Fix that and you have my +. -- 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 655601] Review Request: jing-trang - Schema validation and conversion based on RELAX NG
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=655601 --- Comment #6 from Stanislav Ochotnicky socho...@redhat.com 2010-11-29 08:33:07 EST --- Package Review == Key: - = N/A x = Check ! = Problem ? = Not evaluated === REQUIRED ITEMS === [x] Rpmlint output: dtdinst.noarch: W: no-manual-page-for-binary dtdinst jing.noarch: W: spelling-error Summary(en_US) validator - invalidator, validation, validate jing.noarch: W: spelling-error %description -l en_US validator - invalidator, validation, validate jing.noarch: W: dangling-symlink /usr/share/doc/jing-2009-4.fc15/doc/api /usr/share/javadoc/jing jing.noarch: W: no-manual-page-for-binary jing jing-trang.src: W: strange-permission jing-trang-prepare-tarball.sh 0755L jing-trang.src: W: invalid-url Source0: jing-trang-2009.tar.xz trang.noarch: W: spelling-error Summary(en_US) Multi - Mulch, Mufti trang.noarch: W: no-manual-page-for-binary trang 5 packages and 0 specfiles checked; 0 errors, 9 warnings. The jing-trang-prepare-tarball.sh doesn't have to be included in the srpm since it already includes the tarball. Generally we just include it in repository close to spec file [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] Package meets the Packaging Guidelines[2]. [x] Package successfully compiles and builds into binary rpms. [!] Buildroot definition is not present According to new guidelines Buildroot definitions are to be omitted (automatically provided by recent rpmbuild) [x] Package is licensed with an open-source compatible license and meets other legal requirements as defined in the legal section of Packaging Guidelines[3,4]. [x] License field in the package spec file matches the actual license. License type: BSD [x] If (and only if) the source package includes the text of the license(s) in its own file, then that file, containing the text of the license(s) for the package is included in %doc. But why do all the (from my POV) unnecessary directory creating in %{_docdir} instead of using %doc macro in %files section. No need to do all that copying during %install Something like this can do the trick: -- %doc readme.html doc/ sample/ -- [x] All independent sub-packages have license of their own [x] Spec file is legible and written in American English. [x] Sources used to build the package matches the upstream source, as provided in the spec URL. [x] All build dependencies are listed in BuildRequires, except for any that are listed in the exceptions section of Packaging Guidelines[5]. [x] Package must own all directories that it creates. [x] Package requires other packages for directories it uses. [x] Package does not contain duplicates in %files. [x] Permissions on files are set properly. [!] Package does NOT have a %clean section which contains rm -rf %{buildroot} (or $RPM_BUILD_ROOT). (not needed anymore) %clean and rm -rf at the beginning of %install is not needed and can be omitted [x] Package consistently uses macros (no %{buildroot} and $RPM_BUILD_ROOT mixing) [x] Package contains code, or permissable content. [x] Fully versioned dependency in subpackages, if present. [-] Package contains a properly installed %{name}.desktop file if it is a GUI application. [x] Package does not own files or directories owned by other packages. [x] Javadoc documentation files are generated and included in -javadoc subpackage [x] Javadocs are placed in %{_javadocdir}/%{name} (no -%{version} symlinks) [x] Packages have proper BuildRequires/Requires on jpackage-utils [x] Javadoc subpackages have Require: jpackage-utils Not really but through a dependency.. [-] Package uses %global not %define [x] If package uses tarball from VCS include comment how to re-create that tarball (svn export URL, git clone URL, ...) [x] If source tarball includes bundled jar/class files these need to be removed prior to building [x] All filenames in rpm packages must be valid UTF-8. [x] Jar files are installed to %{_javadir}/%{name}.jar (see [6] for details) [-] If package contains pom.xml files install it (including depmaps) even when building with ant [-] pom files has correct add_to_maven_depmap call which resolves to the pom file (use JPP. and JPP- correctly) === Other suggestions === [x] If possible use upstream build method (maven/ant/javac) [x] Avoid having BuildRequires on exact NVR unless necessary [x] Package has BuildArch: noarch (if possible) [x] Latest version is packaged. [x] Reviewer should test that the package builds in mock. Tested on: fedora-rawhide-x86_64 === Issues === 1. (optional) jing-trang-prepare-tarball.sh from srpm can be removed from srpm 2. simplify documentation installation by using %doc macro 3. remove %clean section and rm -rf from %install section Other than these three things,
[Bug 655601] Review Request: jing-trang - Schema validation and conversion based on RELAX NG
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=655601 --- Comment #7 from Ville Skyttä ville.sky...@iki.fi 2010-11-29 13:41:13 EST --- 1. (optional) jing-trang-prepare-tarball.sh from srpm can be removed from srpm All the packages I've worked with include it in the source rpm. It doesn't cost anything to have it there, and the upside is that people can actually verify the contents of the srpm as well as roll updates just with the information in it without having to hunt missing bits from elsewhere. 2. simplify documentation installation by using %doc macro I intentionally didn't do that initially because for some reason I remembered that the special %doc macro would do a cp --parents and would thus include unwanted subdirs below the doc dirs. But that's incorrect, simplified in -5. 3. remove %clean section and rm -rf from %install section These (and the Group tags) are intentionally left there because I want the package to be cleanly buildable as is for example on EL-5 (assuming dependencies are available). They don't cause any problems, and I can easily live with them around. I must say I am impressed with the patch and bugs you filed upstream to fix the build system a bit. Great work. Thanks, and thanks for the review which got quite a bit bigger than what you initially signed up for. Next revision with the %doc stuff simplified: http://scop.fedorapeople.org/packages/jing-trang.spec http://scop.fedorapeople.org/packages/jing-trang-2009-5.fc14.src.rpm Some things I plan to look into whether they'd make sense and be feasible to implement in the future for this package, let me know if you'd like them or some of them done during the review, or have any opinions about them, in no particular order: - Try to port the missing test suite generation bits to some other XSLT processor than the old saxon, and if successful, run the whole test suite. - Install dtdinst's schemas and XSLs as non-%doc and register them with the system XML catalog. - Review jing's %docs; isorelax.copying.txt and xerces.copying.txt could/should maybe be dropped because they're not shipped in this package (and copying.html maybe modified not to link to them) and maybe datatype-sample.jar should be dropped as well. -- 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 655601] Review Request: jing-trang - Schema validation and conversion based on RELAX NG
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=655601 Ville Skyttä ville.sky...@iki.fi changed: What|Removed |Added Summary|Review Request: jing - |Review Request: jing-trang |RELAX NG validator in Java |- Schema validation and ||conversion based on RELAX ||NG --- Comment #4 from Ville Skyttä ville.sky...@iki.fi 2010-11-28 10:02:03 EST --- http://scop.fedorapeople.org/packages/jing-trang.spec http://scop.fedorapeople.org/packages/jing-trang-2009-4.fc13.src.rpm It took some effort to get this built from upstream svn checkout, but here goes. Unfortunately not all of the test suite could be done because (generating) it has dependencies to an old version of saxon which we don't have. All rpmlint messages are ignorable: dtdinst.noarch: W: no-manual-page-for-binary dtdinst jing-trang.src: W: strange-permission jing-trang-prepare-tarball.sh 0755L jing-trang.src: W: invalid-url Source0: jing-trang-2009.tar.xz jing.noarch: W: spelling-error Summary(en_US) validator - invalidator, validation, validate jing.noarch: W: spelling-error %description -l en_US validator - invalidator, validation, validate jing.noarch: W: dangling-symlink /usr/share/doc/jing-2009-4.fc14/doc/api /usr/share/javadoc/jing jing.noarch: W: no-manual-page-for-binary jing trang.noarch: W: spelling-error Summary(en_US) Multi - Mulch, Mufti trang.noarch: W: no-manual-page-for-binary trang 5 packages and 0 specfiles checked; 0 errors, 9 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 655601] Review Request: jing-trang - Schema validation and conversion based on RELAX NG
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=655601 --- Comment #5 from Ville Skyttä ville.sky...@iki.fi 2010-11-28 10:03:34 EST --- *** Bug 655581 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