Some minor issues :
#001 ChangeLog should be among the %doc

#002 add timestamps to your make
make INSTALL="%{__install} -c -p" install DESTDIR=$RPM_BUILD_ROOT

And for your scriplets:

you shouldn't be using update-desktop-database &> /dev/null ||:
http://fedoraproject.org/wiki/Packaging/ScriptletSnippets?highlight=%28Scriptlets%29#head-de6770dd9867fcd085a73a4700f6bcd0d10294ef

but 
%{_bindir}/gtk-update-icon-cache --quiet %{_datadir}/%{name}/pixmaps/ || :

* %{_datadir}/%{name}/ is not included!
http://fedoraproject.org/wiki/Packaging/FrequentlyMadeMistakes

* Desktop file category Application has never been official and should
be removed, especially when desktop-file-install (or -validate) complains
about it.

> %{_bindir}/gtk-update-icon-cache --quiet %{_datadir}/%{name}/pixmaps/ || :

Certainly not.

New SPEC file : http://glive.tuxfamily.org/fedora/gfa/gfa.spec
New SRPM file : http://glive.tuxfamily.org/fedora/gfa/gfa-0.4.1-2.src.rpm

* Desktop file
  - Category
-------------------------------------------------------------
	--add-category X-Fedora				\
-------------------------------------------------------------
This category is deprecated and should be removed.

New release

SPEC : http://glive.tuxfamily.org/fedora/gfa/gfa.spec
SRPMS : http://glive.tuxfamily.org/fedora/gfa/gfa-0.4.1-3.src.rpm

MUST Items:

- MUST: rpmlint's output is clean
- MUST: The package is named according to the Package Naming Guidelines.
- MUST: The spec file name matches the base package %{name}
- MUST: The package meets the Packaging Guidelines.
- MUST: The package is licensed (GPL) with an open-source compatible license and
meet other legal requirements as defined in the legal section of Packaging
Guidelines.
- MUST: The License field in the package spec file matches the actual license.
- MUST: the source package includes the text of the license(s) in its own file,
then that file, containing the text of the license(s) for the package is
included in %doc.
- MUST: The spec file must be written in American English.
- MUST: The spec file for the package is be legible. 
- MUST: The sources used to build the package must matches the upstream source,
as provided in the spec URL.
- MUST: The package successfully compiles and builds into binary rpms on at
least i386.
- MUST: All build dependencies is listed in BuildRequires.
- MUST: The spec file handles locales properly.
- MUST: If the package does not contain shared library files located in the
dynamic linker's default paths
- MUST: the package is not designed to be relocatable
- MUST: the package owns all directories that it creates.
- MUST: the package does not contain any duplicate files in the %files listing.
- MUST: Permissions on files are set properly.
- MUST: The package has a %clean section, which contains rm -rf %{buildroot} (or
$RPM_BUILD_ROOT).
- MUST: The package consistently uses macros, as described in the macros section
of Packaging Guidelines.
- MUST: The package contains code, or permissable content. This is described in
detail in the code vs. content section of Packaging Guidelines.
- MUST: There are no Large documentation files
- MUST: %doc does not affect the runtime of the application. To summarize: If it
is in %doc, the program must run properly if it is not present.
- MUST: There are no Header files or static libraries 
- MUST: The package does not contain library files with a suffix 
- MUST: Package does NOT contain any .la libtool archives
- MUST: Package containing GUI applications includes a %{name}.desktop file, and
that file must be properly installed with desktop-file-install in the %install
section.
- MUST: Package does not own files or directories already owned by other packages. 

SHOULD Items:

 - SHOULD: The source package does include license text(s) as COPYING
 - SHOULD: mock builds succcessfully in i386.
 - SHOULD: The reviewer tested that the package functions as described. A
package should not segfault instead of running, for example.
 - SHOULD: No subpackages present.

APPROVED

Follow http://fedoraproject.org/wiki/CVSAdminProcedure for the CVS Request
procedure.

New Package CVS Request
=======================
Package Name: gfa
Short Description: gfa is a small and fast address book 
Owners: splinux25
Branches: FC-6
InitialCC: splinux25

A quick comment:
* sourceURL
  - For source0, please check 
    http://fedoraproject.org/wiki/Packaging/SourceURL
* http://fedoraproject.org/wiki/Packaging/ScriptletSnippets
  - This desktop file does not contain mimetype and
    update-desktop-database is not needed.
* desktop file category
  - Category "Application" "X-Fedora" are deprecated and should
    be removed.

> InitialCC: splinux25

Please don't list the same address in InitialCC as owners list.

Fixed, rebuilt and imported ;-)

I checked -4, however none of the issue I addressed are
not fixed.

Oops.. I meant "none of the issues are fixed"...

(In reply to comment #10)
> A quick comment:
> * sourceURL
>   - For source0, please check 
>     http://fedoraproject.org/wiki/Packaging/SourceURL


> * http://fedoraproject.org/wiki/Packaging/ScriptletSnippets
>   - This desktop file does not contain mimetype and
>     update-desktop-database is not needed.

These 2 issues haven't been addressed yet. Damien, please proceed as necessary
ASAP and close this bug.

Ping ?

From what I can see the two items mentioned in comment #15 are fixed in CVS now. 
Can this bug be closed?