Note: This is a beta release of Red Hat Bugzilla 5.0. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Also email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback here.
Bug 230806 - Review Request: tasks - To-do list
Summary: Review Request: tasks - To-do list
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Mamoru TASAKA
QA Contact: Fedora Package Reviews List
: 232797 (view as bug list)
Depends On:
TreeView+ depends on / blocked
Reported: 2007-03-02 23:33 UTC by Dan Young
Modified: 2007-11-30 22:11 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2007-04-20 18:00:49 UTC
mtasaka: fedora-review+
wtogami: fedora-cvs+

Attachments (Terms of Use)

Description Dan Young 2007-03-02 23:33:24 UTC
Spec URL:
Tasks is a simple to do list application that uses libecal, part of Embedded
Evolution Data Server.

Upstream URL:

This is my first package. I am seeking a sponsor.

Comment 1 Michał Bentkowski 2007-03-03 18:08:01 UTC
** We no longer add X-Fedora Category in desktop files. Also, you ought to 
remove Application category:
fedora-tasks.desktop: warning: The 'Application' category is not defined by the 
desktop entry specification.  Please use one of "AudioVideo", "Audio", "Video", 
"Development", "Education", "Game", "Graphics", "Network", "Office", 
"Settings", "System", "Utility" instead

If you can't see this error on your own system, install the newest dekstop-file-
validate version from rawhide.

** Why do you use -n %{name}-%{version} parameter to setup? This is RPM default 

Comment 2 Dan Young 2007-03-06 19:03:08 UTC
Spec URL:

I made the suggested changes:
* Removed X-Fedora category
* Fixed .desktop file (upstream is aware of this now)
* Changed setup macro to just "%setup -q"

There was also a new upstream release in the interim.

Comment 3 Michał Bentkowski 2007-03-07 17:42:31 UTC
** You don't need to patch desktop file against such simple thing as removing a 
category. Have a look at examples of usage of desktop-file-install at

** Why do you do update-desktop-database?

** Read carefully to 
know how to you %find_lang properly. In your spec file %find_lang is useless 
and you don't take advantage of its output.

Comment 4 Dan Young 2007-03-07 21:12:22 UTC
Spec URL:

I dropped the .desktop patch and am just using --remove-category for the GTK,
Application, and ProjectManagement categories. As I said before, upstream is
aware of this now, so expect it will be fixed in their .desktop file in the next

Dropped the spurious update-desktop-database. Think that snuck in on a cut-n-paste.

Fixed up the %find_lang usage. Thanks for the pointers.

Comment 5 Dan Young 2007-03-07 21:46:43 UTC
Spec URL:

Went and read the Desktop Menu spec from fd.o and realized I only need to drop
the "Application" category.

Comment 7 Dan Young 2007-03-20 21:26:18 UTC
Spec URL:

Fixed ownership of %{_datadir}/tasks

Comment 8 Mamoru TASAKA 2007-04-09 15:01:59 UTC
Well, for 0.4-2:

* Timestamps
  - Please keep timestamps on files which are not
    created or modified during build stage as much as
    possible (for this package, png/svg/xml files).

    The following method works for this package.
make install DESTDIR=$RPM_BUILD_ROOT INSTALL="%{__install} -p"

* Duplicate entry
  - mockbuild says:
+ /bin/mkdir -p /var/tmp/tasks-0.4-2.fc7-root-mockbuild/usr/share/doc/tasks-0.4
+ exit 0
warning: File listed twice: /usr/share/tasks/tasks-ui.xml
Requires(interp): /bin/sh /bin/sh
Requires(rpmlib): rpmlib(CompressedFileNames) <= 3.0.4-1
rpmlib(PayloadFilesHavePrefix) <= 4.0-1
    Note: the entry 
    (where foo is a directory) means the directory foo/ itself and
    all the files/directories/etc under foo/, while
%dir foo/
    means the directory foo/ only.

NOTE: Before being sponsored:

This package will be accepted with another few work. 
But before I accept this package, someone (I am a candidate) 
must sponsor you.

Once you are sponsored, you have the right to review other 
submitters' review requests and approve the packages formally. 
For this reason, the person who want to be sponsored (like you) 
are required to "show that you have an understanding 
of the process and of the packaging guidelines" as is described
on :

Usually there are two ways to show this.
A. submit other review requests with enough quality.
B. Do a "pre-review" of other person's review request
   (at the time you are not sponsored, you cannot do
   a formal review)

When you have submitted a new review request or have pre-reviewed other 
person's review request, please write the bug number on this bug report 
so that I can check your comments or review request.

Fedora Extras package review requests which are waiting for someone to
review can be checked on:
NOTE: FE-NEW blockers are now not complete.

Review guidelines are described mainly on:

Comment 9 Dan Young 2007-04-11 19:53:36 UTC
Spec URL:

- Preserve timestamps on install
- Fix /usr/share/tasks/tasks-ui.xml files listing twice
- Patch Help -> About -> License to fix name

Thanks for the feedback. I'll continue to polish this one, am looking at some
other candidates to package, and will try to do some pre-review on other packages.

The license typo I patched out has been fixed in upstream SVN, but not yet
released. I'll drop the patch when that happens.

Comment 10 Dan Young 2007-04-13 22:56:23 UTC
Pre-review notes on remind:

Proposed patch to stop stripping binaries, generate non-empty debuginfo.

Comment 11 Mamoru TASAKA 2007-04-13 23:49:47 UTC

* tasks-0.4-3 is okay
* I have not checked remind (bug 235802) in detail,
  however your patch is actually needed.

  This package (tasks) is APPROVED by me.

I will sponsor you. Please follow according to

Comment 12 Dan Young 2007-04-14 04:43:20 UTC
Thank you for the review and sponsorship.

I've applied for my account and will notify you when I'm in the cvsextras and
fedorabugs groups.

Comment 13 Dan Young 2007-04-17 01:35:58 UTC
New Package CVS Request
Package Name: tasks
Short Description: To-do list
Branches: FC-5 FC-6

Comment 14 Mamoru TASAKA 2007-04-17 03:07:19 UTC
(Removing NEEDSPONSOR. I am sponsoring)

Comment 15 Dan Young 2007-04-20 18:00:49 UTC
Sent 0.4-4 to builders. Built successfully; they are now in needsign state.
Closing with NEXTRELEASE.

Comment 16 Mamoru TASAKA 2007-04-20 18:05:28 UTC
Seems good.
If you have something again, feel free to let me know.

Comment 17 Mamoru TASAKA 2007-04-26 07:05:37 UTC
*** Bug 232797 has been marked as a duplicate of this bug. ***

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