[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 #7 from Mamoru Tasaka <mtasaka(a)ioa.s.u-tokyo.ac.jp> 2008-08-03 12:25:01 EDT ---
(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.
> > * 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
> > * 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
--
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
[Bug 453465] Review Request: transbot - An irc bot designed to translate between languages on separate channels
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=453465
Mamoru Tasaka <mtasaka(a)ioa.s.u-tokyo.ac.jp> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |mtasaka(a)ioa.s.u-tokyo.ac.jp
--- Comment #6 from Mamoru Tasaka <mtasaka(a)ioa.s.u-tokyo.ac.jp> 2008-08-03 12:03:44 EDT ---
For 1.0-3:
* License
- (As you seem to be the upstream developer) unless you mention
in the tarball that this package is licensed under GPLv2+,
we regard the license of this package to be GPL+ (from
section 9)
* Source
- The tarball which can be downloaded from the URL differs from
the one in the tarball.
When you modify the tarball itself, please change the version number
and upload the new one.
* %defattr
- We now recommend %defattr(-,root,root,-)
* General rpmlint issue
--------------------------------------------------------------
transbot.noarch: E: non-executable-script
/usr/lib/python2.5/site-packages/trans/transbot.py 0644
transbot.noarch: E: standard-dir-owned-by-package /usr/share/man/man1
--------------------------------------------------------------
- If the permission of transbot.py is correct (as 0644), then
this script should not have shebang.
- This package must not own /usr/share/man/man1 (owned by filesystem)
--
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
[Bug 445224] Review Request: stapitrace - user space instruction trace
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=445224
Josh Boyer <jwboyer(a)gmail.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|ASSIGNED |CLOSED
Resolution| |RAWHIDE
--- Comment #18 from Josh Boyer <jwboyer(a)gmail.com> 2008-08-03 11:33:50 EDT ---
(In reply to comment #17)
> Created an attachment (id=313294)
--> (https://bugzilla.redhat.com/attachment.cgi?id=313294) [details]
> updates for Roland McGraths upgrade to utrace
>
> Roland McGrath is upgrading utrace in Fedora10. The attached tar file contains
> a src rpm, spec file, and patch file to upgrade Dpiperf/src/stap/usr_itrace.stp
>
> This is based on a preliminary version of Roland's changes and has only had
> limited testing due to the need for uprobes (which also must be adapted for the
> new utrace)
Just handle this in CVS at your leisure, or open a different bug for it. The
review is done and this bug should be closed out.
--
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