Bug 204423
Summary: | Review Request: libstroke - A stroke interface library | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Chitlesh GOORAH <chitlesh> |
Component: | Package Review | Assignee: | Mamoru TASAKA <mtasaka> |
Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Package Reviews List <fedora-package-review> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | hdegoede, mtasaka |
Target Milestone: | --- | Flags: | kevin:
fedora-cvs+
|
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2006-09-01 23:38:20 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: | |||
Bug Depends On: | |||
Bug Blocks: | 163779, 204598 |
Description
Chitlesh GOORAH
2006-08-28 23:49:19 UTC
First review for libstroke: 1. From http://fedoraproject.org/wiki/Packaging/Guidelines : * Licensing + This package says it is distributed under GPL, however, this does not include the copy of GPL license. I won't block this package by this issue, however, it is recommended that you ask for upstream to include GPL documents to this package. * Requires - %package -n libgstroke-devel Conflicts: libstroke-devel What does this mean? Is there any way to remove this conflict sentence? * BuildRequirs - texinfo Is this really required? It seems that makeinfo is not used in rebuilding. - libX11-devel <- required by gtk+-devel * File and Directory Ownership %{_datadir}/libstroke %{_datadir}/libstroke/java %{_datadir}/libstroke/test --- not owned by any packages. 2. From http://fedoraproject.org/wiki/Packaging/ReviewGuidelines = Nothing. 3. Other things I have noticed = Nothing. (In reply to comment #1) > First review for libstroke: > > 1. From http://fedoraproject.org/wiki/Packaging/Guidelines : > > * Licensing > + This package says it is distributed under GPL, however, > this does not include the copy of GPL license. > I won't block this package by this issue, however, it is > recommended that you ask for upstream to include GPL > documents to this package. > I've contacted upstream. > * Requires > - %package -n libgstroke-devel > Conflicts: libstroke-devel > > What does this mean? Is there any way to remove this > conflict sentence? > I don't see how. Updatedb: Spec URL: http://chitlesh.funpic.de/rpm/libstroke.spec SRPM URL: http://chitlesh.funpic.de/rpm/libstroke-0.5.1-7.src.rpm Well: * I just want to know why libgstroke-devel conflicts with libstroke-devel . * %{_datadir}/stroke/ (in main package) Well, no. When writing as this, this (main) package includes the directory %{_datadir}/stroke/ and all files under this directory. If you want to include only the directory, explicit declaration of %dir is needed. There are suprisingly not much difference, only libgstroke-devel is the gnome bindings of libstroke-devel. Updated: Spec URL: http://chitlesh.funpic.de/rpm/libstroke.spec SRPM URL: http://chitlesh.funpic.de/rpm/libstroke-0.5.1-8.src.rpm I'm not sure why there is a conflict -- is that some artifact of your packaging system? (Which I'm not familiar with.) I'll put COPYING in the next release. I haven't made a release for a long time, but I'll do so to resolve that and the conflict issue, if we can figure out exactly what's broken there and if there's something for me to fix on my release. (In reply to comment #4) > There are suprisingly not much difference, only libgstroke-devel is the gnome > bindings of libstroke-devel. Umm.. I still cannot understand why these conflicts each other. ( as Mark says ) * File lists do not make duplicate. * When using libstroke-devel, it requires to write "include <stroke.h>" in source code and to use -lstroke. On the other hand when using libgstroke-devel they changes to "include <gstroke.h>" and -lgstroke . They never overwrite each other. Ok, the "conflict" in the spec was written by the previous packager. I thought it was founded enough. Updated: Spec URL: http://chitlesh.funpic.de/rpm/libstroke.spec SRPM URL: http://chitlesh.funpic.de/rpm/libstroke-0.5.1-9.src.rpm Well: As was in main package %files -n %{name}-devel %{_datadir}/stroke/tests/ %{_datadir}/stroke/tests/stroke_test %{_datadir}/stroke/tests/stroke_test.tcl %files -n javastroke %{_datadir}/stroke/java/ %{_datadir}/stroke/java/*.java These are duplicates. Please check file entries so that * all necessary files and directories are owned. * duplicate files are not listed. Updated: Spec URL: http://chitlesh.funpic.de/rpm/libstroke.spec SRPM URL: http://chitlesh.funpic.de/rpm/libstroke-0.5.1-10.src.rpm (In reply to comment #10) > Updated: > Spec URL: http://chitlesh.funpic.de/rpm/libstroke.spec > SRPM URL: http://chitlesh.funpic.de/rpm/libstroke-0.5.1-10.src.rpm Well, not yet okay. rpm -qlp *0.5.1*rpm | sort says /usr/share/stroke/tests /usr/share/stroke/tests/gnome_stroke_test /usr/share/stroke/tests/gnome_stroke_test /usr/share/stroke/tests/stroke_test /usr/share/stroke/tests/stroke_test.tcl /usr/src/debug/libstroke-0.5.1 ... /usr/share/stroke/tests/gnome_stroke_test is owned by libstroke-devel and libgstroke-devel and... from this package: there is a case that only libgstroke and libgstroke-devel and libstroke are installed: in that case the directory %{_datadir}/stroke/tests/ is not owned by any packages. This is not correct. So: %files -n %{name} %defattr(-,root,root) %doc README COPYRIGHT ChangeLog NEWS AUTHORS TODO CREDITS %dir %{_datadir}/stroke/ %dir %{_datadir}/stroke/tests/ %{_libdir}/libstroke.so.* %files -n %{name}-devel %defattr(-,root,root) %doc doc/standard_strokes* %{_datadir}/stroke/tests/stroke_test* %{_datadir}/aclocal/libstroke.m4 %{_libdir}/libstroke.so %{_includedir}/stroke.h %files -n libgstroke %defattr(-,root,root) %doc README COPYRIGHT ChangeLog NEWS AUTHORS TODO CREDITS %{_libdir}/libgstroke.so.* %files -n libgstroke-devel %defattr(-,root,root) %doc README.libgstroke %{_datadir}/stroke/tests/gnome_stroke_test %{_datadir}/aclocal/libgstroke.m4 %{_libdir}/libgstroke.so %{_includedir}/gstroke.h %files -n javastroke %defattr(-,root,root) %doc javastroke/README %{_datadir}/stroke/java/ is correct (I think. check it). Updated: Spec URL: http://chitlesh.funpic.de/rpm/libstroke.spec SRPM URL: http://chitlesh.funpic.de/rpm/libstroke-0.5.1-11.src.rpm Well, now the files/directories ownership consistency is okay. I checked my mock build and it turned out that automake is unnecessary for BuildRequires as automake and so on is not called in rebuilding and I suceeded in rebuilding this with automake removed from BR. Please remove automake from BuildRequires, which is the only things to be fixed. This package is APPROVED by me. New Package CVS Request ======================= Package Name: libstroke Short Description: A stroke interface library Owners: chitlesh Branches: EL-5 cvs done. |