Bug 447751

Spec URL: http://lokthare.fedorapeople.org/temp/diary.spec
SRPM URL: http://lokthare.fedorapeople.org/temp/diary-0.3-1.fc9.src.rpm
- Update to the new release

Spec URL: http://lokthare.fedorapeople.org/temp/diary.spec
SRPM URL: http://lokthare.fedorapeople.org/temp/diary-0.3.1-1.fc9.src.rpm

- Update to the new release
- Drop the patch (fixed upstream)

The name of this package seems also too generic to me.

What are the problems with having a generically-named package? Before I named
the project (I'm the developer), I checked and I could find no projects which
use the name "diary", and it hasn't collided with anything so far.

I tried to explain it on:

https://fedoraproject.org/wiki/PackageMaintainers/PackagingTricks#Use_of_common_namespace

Hmmm. I appreciate the point, but it would be an awful lot of hassle to change
the name. That isn't to say it isn't possible, but how important do you think it
is to change the name? No other distributions have raised problems with the
genericness of the name, although Diary isn't too widely packaged yet.

(In reply to comment #6)
> Hmmm. I appreciate the point, but it would be an awful lot of hassle to change
> the name. That isn't to say it isn't possible, but how important do you think it
> is to change the name? 

To me it is important, but you could also find another
reviewer who don't care about it and approve the package.

> No other distributions have raised problems with the
> genericness of the name, although Diary isn't too widely packaged yet.

If so it is the right time to do it, before it is widely
packaged.

I really like to see this package in Fedora because I use it a lot. I'm willing
to do the review but I fully agree with Patrice. I think best is to ask upstream
if he's willing to rename the package to something like gdiary or gnome-diary
(since it uses Gnome's GConv).

There already was a project named gnome-diary [1] but it never released something.

[1] http://sourceforge.net/projects/gnome-diary

I can't really change the name to include "GNOME", since it's not part of the
GNOME desktop (even though it uses the same technologies).

How about "Almanac Diary", with "almanac" as the executable name?

(In reply to comment #9)
> I can't really change the name to include "GNOME", ...

I wasn't talking about you but about upstream.

> ...since it's not part of the
> GNOME desktop (even though it uses the same technologies).

To me GConf2 _is_ Gnome technology. There are a lot of gnome-* applications that
are not part of the official GNOME desktop.

(In reply to comment #10)
> (In reply to comment #9)
> > I can't really change the name to include "GNOME", ...
> 
> I wasn't talking about you but about upstream.

I *am* upstream!

> > ...since it's not part of the
> > GNOME desktop (even though it uses the same technologies).
> 
> To me GConf2 _is_ Gnome technology. There are a lot of gnome-* applications that
> are not part of the official GNOME desktop.

Regardless, I think it would be a bit foolish to choose a name which could be
confused with the gnome-diary project; even though it never released anything, I
don't want to have the possibility of any confusion.

(In reply to comment #11)
> 
> I *am* upstream!

Oops, sorry, I did not read close enough. My bad. :(

You are free to name the program whatever you like. "almanac" as the name of the
binary is still a little general, but I could not find something conflicting.

OK, I'm renaming it to "Almanah Diary", and the executable's now called
"almanah" (note: this isn't "almanac" --- it's the Croatian translation, which
is a lot less generic).

I'll make a 0.4 release in a few days, to give some time for the translations to
catch up.

Created attachment 311333 [details]
German Translation for svn r64

Note: In almanah.ui.h:3 I replaced "Attached Links" with "Attachments" because
I think files and notes are no "links" from a linguistic pov.

(In reply to comment #13)
> OK, I'm renaming it to "Almanah Diary", and the executable's now called
> "almanah" (note: this isn't "almanac" --- it's the Croatian translation, which
> is a lot less generic).

I really like the name, so I already replaced "diary" with it in my translation.

Jean-François, the Gconf scriptlets are not needed because this application
doesn't install a schema (yet?). Gconf is only used to read the value of the
default encryption key.

(In reply to comment #14)
> Created an attachment (id=311333) [edit]
> German Translation for svn r64
> 
> Note: In almanah.ui.h:3 I replaced "Attached Links" with "Attachments" because
> I think files and notes are no "links" from a linguistic pov.

Thanks for that. I've committed it to trunk.

(In reply to comment #16)
> Jean-François, the Gconf scriptlets are not needed because this application
> doesn't install a schema (yet?). Gconf is only used to read the value of the
> default encryption key.

I will fix this in the next package but i will wait for the next release to
build it.

(In reply to comment #18)
> (In reply to comment #16)
> > Jean-François, the Gconf scriptlets are not needed because this application
> > doesn't install a schema (yet?). Gconf is only used to read the value of the
> > default encryption key.
> 
> I will fix this in the next package but i will wait for the next release to
> build it.

0.4.0 is out: http://tecnocode.co.uk/2008/07/10/extended-absence/ and
http://tecnocode.co.uk/downloads/almanah-0.4.0.tar.gz

Spec URL: http://lokthare.fedorapeople.org/temp/almanah.spec
SRPM URL: http://lokthare.fedorapeople.org/temp/almanah-0.4.0-1.fc9.src.rpm
- Change to the new name
- Remove GConf scriplets

Koji build task: http://koji.fedoraproject.org/koji/taskinfo?taskID=711758

Review for da202ef7a75b5d85a22b641eef0f61a2  almanah-0.4.0-1.fc9.src.rpm

FAIL - MUST: rpmlint /var/lib/mock/fedora-rawhide-i386/result/almanah-*
almanah.i386: W: spurious-executable-perm /usr/share/doc/almanah-0.4.0/README
almanah.i386: W: spurious-executable-perm /usr/share/doc/almanah-0.4.0/AUTHORS
almanah.i386: W: spurious-executable-perm /usr/share/doc/almanah-0.4.0/NEWS
almanah-debuginfo.i386: W: spurious-executable-perm
/usr/src/debug/almanah-0.4.0/src/main.c
almanah-debuginfo.i386: W: spurious-executable-perm
/usr/src/debug/almanah-0.4.0/src/main.h
almanah-debuginfo.i386: W: spurious-executable-perm
/usr/src/debug/almanah-0.4.0/src/interface.c
3 packages and 0 specfiles checked; 0 errors, 6 warnings.

You should chown these files in %prep to fix these errors.

OK - MUST: The package is named according to the  Package Naming Guidelines
Ok - MUST: The spec file name matches the base package %{name}, in the format
%{name}.spec
OK - MUST: The package meets the  Packaging Guidelines
OK - MUST: The package is licensed with a Fedora approved license (GPLv3+)
OK - MUST: The License field in the package spec file matches the actual license
OK - MUST: License text from source is included in %doc
OK - MUST: The spec file is written in American English
OK - MUST: The spec file for the package is legible
OK - MUST: The sources used to build the package match the upstream source by
md5 59cdbff0ba8e53d736011608aeb6a8e3
OK - MUST: The package successfully compiles and builds into binary rpms on i386
OK - MUST: All build dependencies are listed in BuildRequires
OK - MUST: The spec file handles locales properly with %find_lang
OK - MUST: The package owns all directories that it creates
OK - MUST: The package does not contain any duplicate files in the %files listing
FAIL - MUST: Permissions on files are not set properly, see above
OK - MUST: The package has a %clean section, which contains rm -rf %{buildroot}
OK - MUST: The package consistently uses macros, as described in the macros
section of Packaging Guidelines
OK - MUST: The package contains code, or permissible content
OK - MUST: Files included in %doc do not affect the runtime of the application.
FIX? - Consider including ChangeLog, although it's not that important
OK - MUST: The package does not contain any .la libtool archives
OK - MUST: The package contains a GUI application and includes a %{name}.desktop
file that file is properly installed with desktop-file-install
FIX - The icon of the desktop file is still diary. To fix this:
  sed -i 's!Icon=diary!Icon=almanah!' data/%{name}.desktop.in
OK - MUST: The packages does not own files or directories already owned by other
packages
OK - MUST: The package runs rm -rf %{buildroot} at the beginning of %install
OK - MUST: All filenames in the package are valid UTF-8
OK - SHOULD: The package builds in mock
OK - SHOULD: The package functions as described. There are some issues with
encrypted databases, but we can troubleshoot them with upstream once this
package is imported and he is back from vacation
OK - SHOULD: Latest version of the package

NEEDSWORK

There is a typo in the summary: A application needs to be "An Application" or
just "Application".

(In reply to comment #21)
>
> You should chown these files in %prep to fix these errors.

Did I just say "chown"??? Of course I meant "chmod" ;)

Spec URL: http://lokthare.fedorapeople.org/temp/almanah.spec
SRPM URL: http://lokthare.fedorapeople.org/temp/almanah-0.4.0-2.fc9.src.rpm
- Fix rpmlint warnings
- Include ChangeLog
- Fix icon name in desktop file

Koji build task: http://koji.fedoraproject.org/koji/taskinfo?taskID=714157

286c425227b1d271f88327cf3b023349  almanah-0.4.0-2.fc9.src.rpm
fixes all issues, so this package is

APPROVED

New Package CVS Request
=======================
Package Name: almanah
Short Description: An application to allow you to keep a diary of your life
Owners: lokthare
Branches: F-9
InitialCC: 
Cvsextras Commits: yes

cvs done.

almanah-0.4.0-2.fc9 has been submitted as an update for Fedora 9

almanah-0.4.0-2.fc9 has been pushed to the Fedora 9 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update almanah'.  You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F9/FEDORA-2008-6390

(In reply to comment #29)
> almanah-0.4.0-2.fc9 has been pushed to the Fedora 9 testing repository.  If
problems still persist, please make note of it in this bug report.

You should push new packages directly to the stable tree, there's no need for it
to languish in testing.

(In reply to comment #30)
> You should push new packages directly to the stable tree, there's no need for it
> to languish in testing.

Ok, i have just pushed it to stable

almanah-0.4.0-2.fc9 has been pushed to the Fedora 9 stable repository.  If problems still persist, please make note of it in this bug report.