Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 221349 - Review Request: doodle - tool to quickly search the documents on a computer
Summary: Review Request: doodle - tool to quickly search the documents on a computer
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Mamoru TASAKA
QA Contact: Fedora Package Reviews List
URL:
Whiteboard:
Depends On: 214087
Blocks: FE-ACCEPT
TreeView+ depends on / blocked
 
Reported: 2007-01-03 23:47 UTC by Karol Trzcionka
Modified: 2007-11-30 22:11 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2007-01-07 13:08:27 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
Mock build log of doodle-0.6.6-1 (deleted)
2007-01-04 18:18 UTC, Mamoru TASAKA
no flags Details

Description Karol Trzcionka 2007-01-03 23:47:13 UTC
Spec URL: http://karlik.nonlogic.org/doodle/doodle.spec
SRPM URL: http://karlik.nonlogic.org/doodle/doodle-0.6.6-1.src.rpm
Description: 
Doodle is a tool to quickly search the documents on a computer. Doodle builds
an index using meta-data contained in the documents and allows fast searches on
the resulting database. Doodle uses libextractor to support obtaining meta-data
from various file-formats. The database used by doodle is a suffix tree,
resulting in fast lookups. Doodle supports approximate searches.

Two things to reviewers:
1. I had to comment _smp_mflags, because when this flag is ON, the rpm does not build. I sent mail to author of this program.
2. To build doodle need libextractor-devel. This is not in fedora-extras, but the Review request is bug #214087

I am not sponsored.

Comment 1 Mamoru TASAKA 2007-01-04 18:02:19 UTC
Well, actually I requested doodle. Then:

Your package does not provide doodled (i.e. rebuilding your
srpm does not create /usr/bin/doodled). The homepage (1) says
that doodled requires fam, which is replaced by gamin.
The spec file provided by dag.wieers.com (2) seems that
doodled can be used with the help of gamin, so please
modify your spec file to use gamin so that doodled can be used.

Also, accroding to "Hints for system administrators" on the homepage
doodled can be run as a daemon. In this case, providing 
initscript service scripts (usually under /etc/rc.d/init.d )
is recommended. Would you provide it?

1. http://gnunet.org/doodle/
2. http://dag.wieers.com/packages/doodle/doodle.spec

Comment 2 Mamoru TASAKA 2007-01-04 18:18:52 UTC
Created attachment 144826 [details]
Mock build log of doodle-0.6.6-1

Mockbuild log of doodle-0.6.6-1 on FC-devel i386.
You can see that /usr/bin/doodled is not installed.

Comment 3 Karol Trzcionka 2007-01-04 22:29:52 UTC
(In reply to comment #1)
> Well, actually I requested doodle. Then:
> 
> Your package does not provide doodled (i.e. rebuilding your
> srpm does not create /usr/bin/doodled). The homepage (1) says
> that doodled requires fam, which is replaced by gamin.
> The spec file provided by dag.wieers.com (2) seems that
> doodled can be used with the help of gamin, so please
> modify your spec file to use gamin so that doodled can be used.
> 
Fixed
> Also, accroding to "Hints for system administrators" on the homepage
> doodled can be run as a daemon. In this case, providing 
> initscript service scripts (usually under /etc/rc.d/init.d )
> is recommended. Would you provide it?
> 
> 1. http://gnunet.org/doodle/
> 2. http://dag.wieers.com/packages/doodle/doodle.spec
The initscript is a problem. The doodled need argument which directory it
monitors and the doodled can be run more then once in the same time. The doodled
in initscript can be run with argument i.e. "/home", but the progam is limited
to monitoring less than 1024 directories for changes, so the "/home" sometimes
can be too big.
I think it must be discussed. 

New files:
Spec URL: http://karlik.nonlogic.org/doodle/doodle.spec
SRPM URL: http://karlik.nonlogic.org/doodle/doodle-0.6.6-2.src.rpm


Comment 4 Mamoru TASAKA 2007-01-05 18:08:45 UTC
I will review this later.

Comment 5 Mamoru TASAKA 2007-01-06 07:28:20 UTC
Well,
= now mockbuild for conditional dependency is okay
= License is "actually" GPL
= Other things are all okay.
= Your other review requests seems good from a
  very quick check.

----------------------------------------------------------
   This package (doodle) is APPROVED by me.
----------------------------------------------------------

Please step forward according to
http://fedoraproject.org/wiki/Extras/Contributors .
At a stage, I should receive a main which notices
that you need a sponsor. Then I will SPONSOR YOU.

Comment 6 Mamoru TASAKA 2007-01-06 12:59:50 UTC
Oh.. I just noticed that you already requested for
sponsorship.

Now I am sponsoring you and you should receive a mail.

Comment 7 Karol Trzcionka 2007-01-07 13:08:27 UTC
(In reply to comment #6)
> Oh.. I just noticed that you already requested for
> sponsorship.
> 
> Now I am sponsoring you and you should receive a mail.
Thank You for sponsorship. The package was built correctly on buildsys (all
arch, devel), so I can close this request.


Note You need to log in before you can comment on or make changes to this bug.