Bug 206871

Version:	0
Release:	0.1.%{_snap}

This is a very early package. Is it really suitable at this stage for inclusion
into FE?

Yes, it's already usable and used by many people, even though it is a CVS snapshot.

It does not build here:

błąd: Znaleziono zainstalowane (ale niespakietowane) pliki:
   /usr/lib/ekg2/plugins/rot13.so


Błędy budowania RPM-a:
    Znaleziono zainstalowane (ale niespakietowane) pliki:
   /usr/lib/ekg2/plugins/rot13.so
[rpm-build@X ekg2-0]$

which means that /usr/lib/ekg2/plugins/rot13.so file is not listed in %files
section.

I'm going to have to duck out of this one - work pressures mean I can keep up
with what I have but not much else. Reassigning this BZ to the default nobody to
pick up

I get following error:
błąd: Znaleziono zainstalowane (ale niespakietowane) pliki:
   /usr/lib64/ekg2/plugins/readline.so
   /usr/lib64/ekg2/plugins/rot13.so
   /usr/share/ekg2/plugins/readline/vars-pl.tx

http://rpm.greysector.net/extras/ekg2.spec
http://rpm.greysector.net/extras/ekg2-0-0.2.20061119.src.rpm

This one actually builds in mock.

The main package looks good, but you have to fix some issues in subpackages.
-xosd, -logsqlite, -jabber and -gadu-gadu subpackage contain documentation files
in %{_datadir}/ekg2/[sub_package_name]. Firstly, you should either simply mark
them as %doc in ekg2 datadir or don't even install them there, just put it in
standard documentation folder. Secondly, Polish doc files in subpackages have
wrong charset - iconv them to utf8.
And the thing I've already mentioned at IRC channel is to change "pluginami"
to "wtyczkami" in description of main package.

1. I found another issue now. When trying to install it I get following error:
Preparing...                ########################################### [100%]
        file /usr/libexec/ioctld from install of ekg2-0-0.2.20061119 conflicts 
with file from package ekg-1.7-0.3.rc2.fc6

ekg and ekg2 come to conflict with each other.

2. How about metapackages? Typying only `yum install ekg2` a normal user won't
get a usable ekg2 package. Trying to connect to any protocol just ends with an
ugly "Nieprawidłowy identyfikator użytkownika" (Incorrect user id) - he may get
discouraged by it. Maybe move a ekg2 content to ekg2-core and make main package
as a metapackage which installs ekg2-core and ekg2-gadu-gadu (or even
ekg2-jabber) would be a good solution? What's your opinion?

I agree. The same should probably be done with gg2. I'll fix the above issues
shortly.

http://rpm.greysector.net/extras/ekg2.spec
http://rpm.greysector.net/extras/ekg2-0-0.3.20070203.src.rpm

Addressed the above issues.

http://rpm.greysector.net/extras/ekg2.spec
http://rpm.greysector.net/extras/ekg2-0-0.4.20070203.src.rpm

Even more issues fixed:
- renamed perl and python bindings packages                                    
                                        
- added gpgme-devel BR                                                         
                                        
- added BuildConflicts: readline-devel                                         
                                        
- removed redundant BRs                                                        
                                        
- built HTML docs                                                              
                                        
- built ncurses plugin with -fno-strict-aliasing to avoid infinite loop        
                                        
  in unicode build

REVIEW:
 * source matches upstram (fec378012b1387b67cf226d40a152ebf)
!* %{?dist} tag not present
 * package is licensed under a GPL license; license text is included
 * rpmlint output:
E: ekg2 no-binary
W: ekg2 no-documentation
W: perl-ekg2 no-documentation
W: ekg2-gadu-gadu no-documentation
W: ekg2-gpg no-documentation
W: ekg2-jabber no-documentation
W: ekg2-logsqlite no-documentation
W: ekg2-ioctld no-documentation
W: ekg2-xosd no-documentation
W: ekg2-gtk2 no-documentation
W: python-ekg2 no-documentation
we can safely ignore it since ekg2 is a metapackage
 * BRs looks fine (mock fc6/x86_64 builds good)
 * provides and requires look sane
 * not relocatable
 * all directories owned well
 * %clean section present
 * BuildRoot's good
 * subpackages: python-ekg2 should be renamed to ekg2-python, because it's not 
python module but ekg2 extension to support python

THINGS to do:
 - add %{?dist} tag
 - rename python-ekg2 to ekg2-python

(In reply to comment #12)
>  - rename python-ekg2 to ekg2-python

The naming guidelines say pretty much the opposite:
http://fedoraproject.org/wiki/Packaging/NamingGuidelines#head-8756a3bce652c376d7ba3908461b638784b6952d
https://bugzilla.redhat.com/223618

This was discussed in a recent packaging committee meeting and no changes were
made to the naming guidelines.

(In reply to comment #13)
> https://bugzilla.redhat.com/223618

Obexftp seems to be different case. Obexftp puts its python files to standard
 %{python_sitelib} and %{python_sitearch} directories so that's in fact a 
python module.
Ekg2 doesn't create new python module (`import ekg2` give you nothing) so this 
is not python module but only an extension to ekg2 to support python. Naming 
Guidelines talks about "(python modules)" but IMO ekg2-python is not a python 
module.

That POV was represented in the packaging meeting too, and it does indeed have
some merit.  Maybe we need more discussion and clarifications.

By the way, just from skimming the specfile, the 
%{perl_vendorarch}/auto/Ekg2/Irc/ dir appears to be unowned.

(In reply to comment #15)
> That POV was represented in the packaging meeting too, and it does indeed have
> some merit.  Maybe we need more discussion and clarifications.
> 
We probably need.

> By the way, just from skimming the specfile, the 
> %{perl_vendorarch}/auto/Ekg2/Irc/ dir appears to be unowned.

Fact, missed that...

http://rpm.greysector.net/extras/ekg2-0-0.6.20070427.src.rpm
http://rpm.greysector.net/extras/ekg2.spec

ekg2 now requires libgsm, under review in bug 228186.

 * rpmlint:
W: ekg2-python no-documentation
W: ekg2-voip no-documentation
E: ekg2 no-binary
W: ekg2 no-documentation
W: ekg2-gtk2 no-documentation
W: ekg2-logsqlite no-documentation
W: ekg2-gpg no-documentation
W: ekg2-ioctld no-documentation
W: ekg2-jabber no-documentation
W: perl-ekg2 no-documentation
E: perl-ekg2 script-without-shebang /usr/share/ekg2/scripts/sample.pl
E: perl-ekg2 script-without-shebang /usr/share/ekg2/scripts/dupa.pl
E: perl-ekg2 script-without-shebang /usr/share/ekg2/scripts/irc.pl
W: ekg2-gadu-gadu no-documentation
W: ekg2-xosd no-documentation

 * use versioned dependencies for the main package (current solution breaks 
updates)

 * fix perl-ekg2 issues reported by rpmlint

http://rpm.greysector.net/extras/ekg2-0-0.7.20070427.src.rpm
http://rpm.greysector.net/extras/ekg2.spec

How about now?

I like it now.
Approved.

New Package CVS Request
=======================
Package Name: ekg2
Short Description: Multi-protocol instant messaging and chat client
Owners: rpm
Branches: F-7 FC-6
InitialCC: 

cvs done.

built for devel