I've compiled & verified the functionality on devel, and taken an initial look
at the spec file.

I did notice it got confused and hung a few times while forinstance trying to
get the user list.. Also the help menu->contents doesn't do anything, but these
might be upstream issues.

I've noticed the %dir %{_datadir}/mysql-gui/administrator/ with the trailing /,
i didn't think it needed to be there?

Also rpmbuild gave me the following error:
warning: Installed (but unpackaged) file(s) found:
   /usr/share/locale/de/LC_MESSAGES/mysql-gui-common.mo
   /usr/share/locale/ja/LC_MESSAGES/mysql-gui-common.mo
   /usr/share/locale/pt_BR/LC_MESSAGES/mysql-gui-common.mo

which indicates you forgot to include the find_lang macro to pick up the
language files:

%install 
.....
%find_lang mysql-gui-common

and in the mysql-gui-common files part:
%files -n mysql-gui-common -f mysql-gui-common.lang 

rpmlint didn't report any errors for the build packages

I'll try to do a mock build and an official review checklist once the packaging
errors mentioned above are corrected.

One more note, when installing a .desktop file you should run
update-desktop-database:

Requires(post): desktop-file-utils
Requires(postun): desktop-file-utils
...
%post
update-desktop-database &> /dev/null ||:

%postun
update-desktop-database &> /dev/null ||:

See:
http://fedoraproject.org/wiki/ScriptletSnippets#head-d37d740d062d3aa6013aab44a79de88a6c1fe533

Depends on what is included in the .desktop file. If MIME type
assignments are done, update-desktop-database must be run. Else, not.

there is no MIME type in the .desktop  so there is no need to run  
update-desktop-database   
 
fixed the lang  files,  thanks  i missed them in the output and ive removed 
the trailing /  i changed the .desktop file options  to make it show up in 
"System Tools" menu  i feel this is the best place for it.  
  
updated SRPM http://ausil.us/packages/mysql-administrator-1.1.5-2.src.rpm  
updated SPEC http://ausil.us/packages/mysql-administrator.spec 
 
 

Woops my bad about the update-desktop, was confused by the menu item not showing up.

Giving the new package a swirl now

The formal review steps:

MUST review items:
- Builds cleanly on FC5 devel.
- rpmlint has no output / complaints
- Source included matches upsteam source (md5sum)
- Package name meets guidelines
- spec file name is in %{name}.spec format
- Licence (GPL) is fedora extra's compatible & is included in spec
- Spec file is in (american) english
- Does not list buildrequires that are excepted in the package guidelines
- All build dependencies are listed
- Spec file handles locales properly (thru find_lang macro)
- No so files so no ldconfig needed
- Installs propper desktop file, under fedora category, shows up in menu and
with proper icon
- All files have proper permissions (double checked with ls -al `rpm -ql
mysql-administrator`)
- Package is not relocatable
- No duplicate files in %files section
- No missing files in %files section
- Has a proper %clean section with rm -rf $RPM_BUILD_ROOT
- Uses macro's described in PackagingGuidelines
- No entries in %doc that are required for standard program operation
- No header files or static libs, so no need for -devel package
- Nor pkgconfig files, .so files or .la files
- Package contains GUI application, and includes proper desktop file
- Package owns directories properly

Should items:
- Includes upstream licence file (COPYING)
- Build in mock (fc5 devel)
- Could not try build on all architectures, plague will hopefully tell us :-)
- Functionality seems to work (though some problems with user admin with fedora
5's mysql 5.0.18, but that problem exists upsteam too)
- No insane scriplets, or scriplets at all
- Sane require chain (mysql-administrator requires mysql-gui-common package

FE-ACCEPTED in my book (presuming this is not your first package and needing
sponsorship)

FWIW, a couple of notes from an unsuccessful build on CentOS 4.2 (due to too old
gtkmm24 version), most likely applies to FC3 too:

- BuildRequires: gtkmm24-devel >= 2.6 (add version, 2.4.x won't work)
- BuildRequires: gettext (probably, for translations)
- In beginning of %build: export CPPFLAGS="$(pcre-config --cflags)"
  (required for old pcre)
- I *guess* that the explicit mysql and gtkmm24 dependencies might not be needed

Some or all of the above, eg. at least the minimum required gtkmm24-devel
version would be nice to have even if you're targetting FC >= 4.

Thank you for your review.    
   
i made a change to only include the gcc4.1 patch on fedora >4   as i got a gcc   
error on fc4 with the patch applied. everything built ok without the patch on 
fc4.   
  
i am also going to implement the version in requires. as it will help people 
who rebuild extras packages. 
  
Importing    

> mysql-gui-common : This package contains several common libraries and
> data shared among the MySQL GUI Suites in all platforms.

This is odd. The package doesn't contain any libraries at all.
How is "all platforms" supposed to be understood? The common gui
libraries are compiled and linked statically, so they cannot be
shared with any other packages currently. Are the images and Glade
files shared? If so, how?

> Requires: gtkmm24

Redundant, since an automatic dependency on the soname is added by the
rpmbuild backend.

Actually rpm -ql mysql-gui-common tells me it only contains language (.mo) files
and all the png's and a couple of glade files the apps use

So the description is wrong in this case

i pulled the description from the README file in the mysql-gui-common tree.  
  
i have been  trying to work out the best way to build mysql-query-browser  
also.  as it needs some of the files that get built in mysql-gui-common tree   
but not installed  i think i will patch the Makefile to install the neccesary  
files and create a mysql-gui-common-devel package with them.  I will remove  
the redundant Requires  
 
removing the mysql-gui-common package  if i try and start mysql-administrator 
i get  
[dennis@rpclnx001 ~]$ mysql-administrator 
 
(mysql-administrator-bin:4147): libglade-WARNING **: could not find glade file 
'/usr/share/mysql-gui/common/preferences.glade' 
terminate called after throwing an instance of 'MGGladeXML::Error' 
/usr/bin/mysql-administrator: line 19:  4147 Aborted                 $PRG-bin 
 
so it is loading at least one glade file at run time 
 
 
I should change the description to be more appropriate  perhaps: 
 
this package contains  glade files images and translations used by mysql tools 
at runtime