Henry:
This spec file has no %description value.
I'd remove this comments:
# The following shorthand works fine...
[...]
# Here's the "longhand" version:
# %files
# %defattr (-, root, bin)
# %dir %attr (0755, root, bin) %{_bindir}
# %{_bindir}/*
# %dir %attr (0755, root, bin) %{_libdir}/irssi
# %dir %attr (0755, root, bin) %{_libdir}/irssi/modules
# %{_libdir}/irssi/modules/*
# %{perl_archlib}/Irssi.pm
# %dir %attr (0755, root, bin) %{perl_archlib}/Irssi
# %{perl_archlib}/Irssi/*
# %dir %attr (0755, root, bin) %{perl_archlib}/auto/Irssi
# %{perl_archlib}/auto/Irssi/Irssi.bs
# %{perl_archlib}/auto/Irssi/Irssi.so
# %dir %attr (0755, root, bin) %{perl_archlib}/auto/Irssi/Irc
# %{perl_archlib}/auto/Irssi/Irc/*
# %dir %attr (0755, root, bin) %{perl_archlib}/auto/Irssi/TextUI
# %{perl_archlib}/auto/Irssi/TextUI/*
# %dir %attr (0755, root, bin) %{perl_archlib}/auto/Irssi/UI
# %{perl_archlib}/auto/Irssi/UI/*
# %dir %attr (0755, root, other) %{_datadir}/irssi
# %dir %attr(0755, root, bin) %{_datadir}/irssi/*
# %{_datadir}/irssi/*/*
# %dir %attr(0755, root, bin) %{_mandir}
# %dir %attr(0755, root, bin) %{_mandir}/*
# %{_mandir}/*/*
# %{_sysconfdir}/irssi.conf
# In order to include /etc/irssi.conf, is a root
# package required (which in this case would contain one file)?
> Please help to review the spec file of Irssi, I want to move it from SFE
> to spec-fils-other/core, any comment is welcome.
>
> Thanks,
> Henry
