Output from rpmlint below:

[trondd@localhost SPECS]$ rpmlint ../RPMS/i386/gedit-plugins-2.15.5-1.i386.rpm 
W: gedit-plugins non-conffile-in-etc
/etc/gconf/schemas/gedit-show-tabbar-plugin.schemas
[trondd@localhost SPECS]$ 

Skimming over the spec...

> %{_sysconfdir}/gconf/schemas/gedit-show-tabbar-plugin.schemas

Scriptlets for gconftool install/uninstall of these schemas
is missing: http://fedoraproject.org/wiki/Packaging/ScriptletSnippets

> %{_datadir}/locale/*

Use the %find_lang macro appropriately instead of including the
files like this. This is a MUST item according to the guidelines.

> %doc README INSTALL NEWS AUTHORS COPYING

The 'INSTALL' file most likely is the standard "Basic Installation"
document which is irrelevant to RPM package users.

> Summary:        Gedit-plugins is a collection of plugins for gedit

Repeating the package name in the summary is commonly considered
bad taste. Better and short to the point:

Summary: Plug-ins for gEdit

Or:

Summary: plug-in collection for gEdit

Or like the summary of this ticket. :)

Summary: Collection of plugins for gedit

(In reply to comment #2)
> Skimming over the spec...
> 
> > %{_sysconfdir}/gconf/schemas/gedit-show-tabbar-plugin.schemas
> 
> Scriptlets for gconftool install/uninstall of these schemas
> is missing: http://fedoraproject.org/wiki/Packaging/ScriptletSnippets
> 

Fixed.

> > %{_datadir}/locale/*
> 
> Use the %find_lang macro appropriately instead of including the
> files like this. This is a MUST item according to the guidelines.
> 

Fixed.

> > %doc README INSTALL NEWS AUTHORS COPYING
> 
> The 'INSTALL' file most likely is the standard "Basic Installation"
> document which is irrelevant to RPM package users.
> 

INSTALL file removed.

> > Summary:        Gedit-plugins is a collection of plugins for gedit
> 
> Repeating the package name in the summary is commonly considered
> bad taste. Better and short to the point:
> 
> Summary: Plug-ins for gEdit
> 
> Or:
> 
> Summary: plug-in collection for gEdit

Fixed.

Thanks for the quick reply. I have uploaded the new and improved versions to the
same location as previously (ftp://open-gnss.org/pub/fedora)

* I've waited for new gedit in Rawhide. This breaks the package
finally:

checking for GEDIT... configure: error: Package requirements (
        glib-2.0 >= 2.8.0
        gtk+-2.0 >= 2.8.0
        gtksourceview-1.0 >= 1.2.0
        gedit-2.14 >= 2.15.7
        libglade-2.0 >= 2.4.0
        gnome-vfs-2.0 >= 2.13.4
) were not met:

No package 'gedit-2.14' found


* Much newer gedit-plugins tarballs are available upstream.

* Build on FC-6 fails, too, and disables python-vte plugin. This
is because rpmbuild unsets $DISPLAY, which is needed by the
configure check:

$ unset DISPLAY
$ python -c "import vte"
Traceback (most recent call last):
  File "<string>", line 1, in ?
ImportError: could not import gtk
$ python -c "import gtk"
Traceback (most recent call last):
  File "<string>", line 1, in ?
  File "/usr/lib/python2.4/site-packages/gtk-2.0/gtk/__init__.py", line 76, in ?
    _init()
  File "/usr/lib/python2.4/site-packages/gtk-2.0/gtk/__init__.py", line 64, in _init
    _gtk.init_check()
RuntimeError: could not open display


* Some minor requests:

Please really drop the package name from the Summary. When the
package name is repeated in the summary, this looks very ugly 
wherever %name and %summary are displayed next to eachother, e.g.
repoview pages and other forms of package listings.


> BuildRequires:  perl-XML-Parser

BuildRequires: perl(XML::Parser)

Please prefer Perl module names over package names, as they are
less likely to break when a module is moved to a different package.


> killall -HUP gconfd-2 || :

In the scriptlets is not needed for FC5 and newer.

(In reply to comment #5)
> * I've waited for new gedit in Rawhide. This breaks the package
> finally:
> 
> checking for GEDIT... configure: error: Package requirements (
>         glib-2.0 >= 2.8.0
>         gtk+-2.0 >= 2.8.0
>         gtksourceview-1.0 >= 1.2.0
>         gedit-2.14 >= 2.15.7
>         libglade-2.0 >= 2.4.0
>         gnome-vfs-2.0 >= 2.13.4
> ) were not met:
> 
> No package 'gedit-2.14' found
> 
> 
> * Much newer gedit-plugins tarballs are available upstream.

But FC6 only has 2.15.9, and gedit-plugins/2.15.5 is the latest version that
compiles with that version of gedit. I will initally create a version for FC6,
and then upgrade it to the latest version in rawhide.

> * Build on FC-6 fails, too, and disables python-vte plugin. This
> is because rpmbuild unsets $DISPLAY, which is needed by the
> configure check:
> 
> $ unset DISPLAY
> $ python -c "import vte"
> Traceback (most recent call last):
>   File "<string>", line 1, in ?
> ImportError: could not import gtk
> $ python -c "import gtk"
> Traceback (most recent call last):
>   File "<string>", line 1, in ?
>   File "/usr/lib/python2.4/site-packages/gtk-2.0/gtk/__init__.py", line 76, in ?
>     _init()
>   File "/usr/lib/python2.4/site-packages/gtk-2.0/gtk/__init__.py", line 64, in
_init
>     _gtk.init_check()
> RuntimeError: could not open display

This is currently the biggest show stopper. I'll look into it and see what I can
come up with.

> * Some minor requests:
> 
> Please really drop the package name from the Summary. When the
> package name is repeated in the summary, this looks very ugly 
> wherever %name and %summary are displayed next to eachother, e.g.
> repoview pages and other forms of package listings.

FIXED.

> > BuildRequires:  perl-XML-Parser
> 
> BuildRequires: perl(XML::Parser)
> 
> Please prefer Perl module names over package names, as they are
> less likely to break when a module is moved to a different package.

FIXED.
 
> > killall -HUP gconfd-2 || :
> 
> In the scriptlets is not needed for FC5 and newer.

FIXED.

Sorry for the delay; I will release a new version very soon.

New srpm and spec file at ftp://open-gnss.org/pub/fedora/gedit-plugins

> -%configure --disable-schemas-install
> +%configure --disable-schemas-install DISPLAY=":0.0"

This is completely untested, right? I have strong doubts that it is
acceptable, since it will fail for X-less machines, like build servers,
for example.

(In reply to comment #8)
> > -%configure --disable-schemas-install
> > +%configure --disable-schemas-install DISPLAY=":0.0"
> 
> This is completely untested, right? I have strong doubts that it is
> acceptable, since it will fail for X-less machines, like build servers,
> for example.

It works on my computer, but I haven't tested on another machine. Any ideas how
to solve this in another way?

Created attachment 151197 [details]
spec patch

It builds for you also in run-level 3 or with X disabled?

Anyway...
Here's a spec patch that alters the check for the Python vte module
and adds a %check section to verify that the terminal.py file is
built/included. Alternatively, you could list the module file in
%files explicitly -- though that would not make clear it's supposed
to be a check ;).

I've updated the spec file to the latest version of gedit-plugins, so now it
build  just fine on rawhide. I also applied the patch that you submitted, and
removed the old entries in the changelog, because the package has never been
available for FC6. Is this ok?

Both spec and srpm files are available here:
ftp://open-gnss.org/pub/fedora/gedit-plugins/

"Session Saver" plugin gives:

Traceback (most recent call last):
  File "/usr/lib/gedit-2/plugins/sessionsaver/__init__.py", line 141, in deactivate
    window.get_data(self.WINDOW_DATA_KEY).deactivate()
  File "/usr/lib/gedit-2/plugins/sessionsaver/__init__.py", line 122, in deactivate
    manager = self._window.get_ui_manager()
AttributeError: 'SessionSaverWindowHelper' object has no attribute '_window'

File > Saved sessions > Manage saved sessions
  Close button doesn't work

File > Saved sessions > Save current session
  Save button does something, but doesn't close the dialog

Restoring saved sessions creates corrupted windows.

Conclusively, the Session Saver plugin seems unstable.

Traceback (most recent call last):
  File "/usr/lib/gedit-2/plugins/sessionsaver/dialogs.py", line 39, in
on_session_changed
    self.row_changed(self.on_get_path(piter), piter)
TypeError: iter should be a GtkTreeIter
Traceback (most recent call last):
  File "/usr/lib/gedit-2/plugins/sessionsaver/dialogs.py", line 39, in
on_session_changed
    self.row_changed(self.on_get_path(piter), piter)
TypeError: iter should be a GtkTreeIter


Traceback (most recent call last):
  File "/usr/lib/gedit-2/plugins/sessionsaver/dialogs.py", line 150, in on_response
    self.plugin.sessions.save()
  File "/usr/lib/gedit-2/plugins/sessionsaver/store.py", line 124, in save
    os.path.makedirs(dirname)
AttributeError: 'module' object has no attribute 'makedirs'

"Character Map" plugin
  Does it work?
  Unlike the colour picker, it does not show up anywhere.

(In reply to comment #15)
> "Character Map" plugin
>   Does it work?
>   Unlike the colour picker, it does not show up anywhere.

It is available on the sidebar.

Created attachment 151384 [details]
gedit-plugins spec file.

(In reply to comment #14)
> Traceback (most recent call last):
>   File "/usr/lib/gedit-2/plugins/sessionsaver/dialogs.py", line 39, in
> on_session_changed
>     self.row_changed(self.on_get_path(piter), piter)
> TypeError: iter should be a GtkTreeIter
> Traceback (most recent call last):
>   File "/usr/lib/gedit-2/plugins/sessionsaver/dialogs.py", line 39, in
> on_session_changed
>     self.row_changed(self.on_get_path(piter), piter)
> TypeError: iter should be a GtkTreeIter
> 
> 
> Traceback (most recent call last):
>   File "/usr/lib/gedit-2/plugins/sessionsaver/dialogs.py", line 150, in
on_response
>     self.plugin.sessions.save()
>   File "/usr/lib/gedit-2/plugins/sessionsaver/store.py", line 124, in save
>     os.path.makedirs(dirname)
> AttributeError: 'module' object has no attribute 'makedirs'
> 

This has been fixed in the attached spec file. The bug will reported upstream
once *.gnome.org gets back on line.

I think most of the issues you mentioned have been addressed. Do you have any
other comment?

Traceback (most recent call last):
  File "/usr/lib/gedit-2/plugins/sessionsaver/dialogs.py", line 35, in
on_session_added
    self.row_inserted(self.on_get_path(piter), piter)
TypeError: iter should be a GtkTreeIter

** (gedit:12835): CRITICAL **: gedit_commands_load_uris: assertion `(uris !=
NULL) && (uris->data != NULL)' failed

"Manage saved sessions..." still opens very damaged windows here, and
hence I still think the plugin ought to be excluded.

I have uploaded a new version that disables the sessionsaver plugin, and also
removes two .la files. Spec file and source rpm can be found here:
ftp://open-gnss.org/pub/fedora/gedit-plugins

Fine.

$ rpmlint gedit-plugins-2.18.0-2.fc7.src.rpm 
W: gedit-plugins mixed-use-of-spaces-and-tabs (spaces: line 1, tab: line 13)

$ rpmlint gedit-plugins-2.18.0-2.fc7.i386.rpm 
W: gedit-plugins incoherent-version-in-changelog 2.18.0.2 2.18.0-2.fc7

Upstream provides a shorter tar.bz2, too.

APPROVED

New Package CVS Request
=======================
Package Name: gedit-plugins
Short Description: Collection of plugins for gedit
Owners: trond.danielsen
Branches: FC-6
InitialCC: 

cvs done