August 2008 - package-review - Fedora Mailing-Lists

[Bug 453520] Review Request: libUnihan - C library for Unihan character database in 5NF

by Red Hat Bugzilla

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=453520 Ding-Yi Chen <dchen(a)redhat.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Customer Facing| |--- --- Comment #4 from Ding-Yi Chen <dchen(a)redhat.com> 2008-08-03 20:51:38 EDT --- New SPEC and SRPM for libUnihan 0.4.0 are at: SPEC: http://downloads.sourceforge.net/libunihan/libUnihan.spec?use_mirror=voxe... SRPM: http://downloads.sourceforge.net/libunihan/libUnihan-0.4.0-0.fc9.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.

15 years, 9 months

1
0
0 / 0

[Bug 453519] Review Request: libUnihan-data - The Unihan character database in 5NF

by Red Hat Bugzilla

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=453519 --- Comment #6 from Ding-Yi Chen <dchen(a)redhat.com> 2008-08-03 20:48:34 EDT --- BTW, the new SPEC and SRPM is available at: SPEC: http://downloads.sourceforge.net/libunihan/UnihanDb.spec?use_mirror=voxel... SRPM: http://downloads.sourceforge.net/libunihan/UnihanDb-5.1-2.fc9.src.rpm?use... Regards. -- 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.

15 years, 9 months

1
0
0 / 0

[Bug 453519] Review Request: libUnihan-data - The Unihan character database in 5NF

by Red Hat Bugzilla

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=453519 Ding-Yi Chen <dchen(a)redhat.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Customer Facing| |--- --- Comment #5 from Ding-Yi Chen <dchen(a)redhat.com> 2008-08-03 20:46:15 EDT --- I've changed the name of this package from libUnihan-data to UnihanDb. The reason I change the name is to aviod the end-user confusion. To me, the name "libUnihan-data" implies that it is optional and does not need to be install by default. However, database file of UnihanDb is compulsory for the whole package to work. In addition, it has its own versioning from Unicode upstream, so it should be a separate package. UnihanDd now BuildDepends and Depends on libUnihan, as it is generate by libUnihan. -- 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.

15 years, 9 months

1
0
0 / 0

[Bug 455555] Review Request: libhocr - A Hebrew optical character recognition library

by Red Hat Bugzilla

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=455555 --- Comment #8 from Oron Peled <oron(a)actcom.co.il> 2008-08-03 17:31:15 EDT --- (In reply to comment #7) > (In reply to comment #6) > > (In reply to comment #5) > > > ** Requires > > > * Requires for -devel subpackage > > > --------------------------------------------------------------- > > > This means that libhocr-devel subpackage must have "Requires: gtk2-devel" > > > > > > B. %_libdir/pkgconfig/libhocr.pc contains: > > > --------------------------------------------------------------- > > > Libs: -lhocr -lhspell -lz -ltiff -lm > > > --------------------------------------------------------------- > > > This means that libhocr-devel must have > > > "Requires: hspell-devel zlib-devel libtiff-devel" (because of > > > -lhspell -lz -ltiff) > > > However all these 3 linkage seem unneeded, because none of the > > > header files in libhocr-devel needs header files from > > > fftw-devel, hspell-devel, libtiff-devel. > > > I guess "@ld_hspell@ @ld_tiff@" in libhocr.pc.in should be removed. > > > > Their inclusion in the .pc file is correct since they are used > > by the applications, either explicitly (tiffio.h and fftw3.h) or > > implicitly (hspell via gtkspell). > > I guess currently I don't understand what you want to say correctly. > libhocr-devel itself does not need tiffio.h or fftw3.h. Sorry, my previous reply wasn't clear enough: 1. You are correct that fftw-devel and libtiff-devel should not be required by libhocr, since only libhocr *implementation* uses them. (fixed, they are only mentioned in BuildRequires). 2. However, I think that we still need them listed in the .pc files for other (future) applications that may want to link against libhocr. Such applications would probably have in their build something like: libhocr_LIBS=`pkgconfig --libs libhocr` So the linker would add '-lfftw -ltiff' 3. TODO: libz is also mentioned in libhocr.pc, but I want to fix it first with upstream, since the author forgot to check for it in its configure.ac (actually, it's a dependency of hspell which don't come with pkgconfig file of its own). 4. TODO: hspell should be removed from libhocr.pc (IMO) since its not linked explicitly, but rather loaded during runtime by gtkspell. I'll discuss it with upstream as well. 5. I added "Requires: hspell" to the gtk subpackage because rpmbuild would not find this dependency automatically (as mentioned it is detected by gtkspell during runtime). > > > * Timestamps > > > - To keep timestamps on installed files, please consider to use > > > ---------------------------------------------------------------- > > > make install \ > > > INSTALL="install -p" \ > > > DESTDIR=%{buildroot} \ > > > ..... > > > ----------------------------------------------------------------- > > > This method usually works for based Makefiles based on recent > > > autotools > > > > Hmmm... first time I see this consideration. It seems logical > > but are these any guideline about it? > > Although in PackagingDrafts, I usually recommend this: > https://fedoraproject.org/wiki/PackagingDrafts/MultilibTricks#Timestamps Fixed. [added the link to my bookmarks, as it does not appear in https://fedoraproject.org/wiki/PackagingDrafts]. > > > * Desktop files > > > - Category "X-Fedora" is deprecated and should be removed. > > > > Done. I must have got its addition from some obsolete guide > > somewhere but I cannot find any reference to it now, nevermind. > > Perhaps this is difficult to find: > http://www.redhat.com/archives/fedora-extras-list/2006-October/msg00725.html Thanks, I also noticed we are blocked on rawhide by bug 457502 So now libhocr can be used as a debugging tool 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.

15 years, 9 months

1
0
0 / 0

[Bug 436568] Review Request: supybot - Cross-platform IRC bot written in Python

by Red Hat Bugzilla

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=436568 --- Comment #40 from Ricky Zhou <rickyz(a)cmu.edu> 2008-08-03 16:33:13 EDT --- Package Change Request ====================== Package Name: supybot New Branches: EL-4 EL-5 -- 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.

15 years, 9 months

1
0
0 / 0

[Bug 454940] Review Request: rcssbase - Base libraries for Robocup Soccer Server (2D)

by Red Hat Bugzilla

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=454940 --- Comment #11 from Hedayat Vatankhah <hedayat(a)grad.com> 2008-08-03 15:52:41 EDT --- Updated package: SPEC:http://www.assembla.com/spaces/hedayat/documents/dZnKuQtSOr3An4ab7jn... SRPM: http://www.assembla.com/spaces/hedayat/documents/c-GdeKyzqr3yzEab7jnrAJ/d... -- 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.

15 years, 9 months

1
0
0 / 0

[Bug 432195] Review Request: disk-manager - Disk manager is a simple filesystem configurator

by Red Hat Bugzilla

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=432195 Pavel Alexeev <pahan(a)hubbitus.spb.su> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |pahan(a)hubbitus.spb.su --- Comment #15 from Pavel Alexeev <pahan(a)hubbitus.spb.su> 2008-08-03 15:03:10 EDT --- http://liviopl.fedorapeople.org/disk-manager/disk-manager.spec is 404 error: Not Found. -- 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.

15 years, 9 months

1
0
0 / 0

[Bug 454462] Review Request: ksplice - Patching a Linux kernel without reboot

by Red Hat Bugzilla

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=454462 --- Comment #14 from Jochen Schmitt <jochen(a)herr-schmitt.de> 2008-08-03 14:40:36 EDT --- I have add some enhancements to the kspliceprep schript which I have renamed to fedora-ksplice-prepare. The ksplace-extras subpackage is renamed into fedora-ksplice to signal that the content is fedora specific. The uploaded stuff may be find at: Spec URL: http://www.herr-schmitt.de/pub/ksplice/ksplice.spec SRPM URL: http://www.herr-schmitt.de/pub/ksplice/ksplice-0.8.7-8.fc9.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.

15 years, 9 months

1
0
0 / 0

[Bug 450371] Review Request: pokegen - Strategy/RPG game engine

by Red Hat Bugzilla

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=450371 --- Comment #9 from Mamoru Tasaka <mtasaka(a)ioa.s.u-tokyo.ac.jp> 2008-08-03 13:39:24 EDT --- I just tried to rebuild your latest srpm but it failed as below. http://koji.fedoraproject.org/koji/taskinfo?taskID=755698 Note: Please make build.log more verbose. The output like: ---------------------------------------------------------- Building CXX object pokemod/CMakeFiles/pokemod.dir/MapTrainerTeamMember.o [ 9%] Building CXX object pokemod/CMakeFiles/pokemod.dir/MapWarp.o ---------------------------------------------------------- is not useful. We want to check if Fedora compiler flags are correcly honored, for example. -- 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.

15 years, 9 months

1
0
0 / 0

[Bug 454207] Review Request: Terminator - Multiple terminals in one window

by Red Hat Bugzilla

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=454207 --- Comment #23 from Mamoru Tasaka <mtasaka(a)ioa.s.u-tokyo.ac.jp> 2008-08-03 12:59:28 EDT --- To Wolfy: It seems that Mathieu has another review requests, bug 456774 and bug 456783 . To Mathieu: Please post to this bug entry that you have already submitted some other review requests. This is needed for sponsoring process. -- 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.

15 years, 9 months

1
0
0 / 0

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

package-review August 2008