Updated...
Thanks,
Henry
Brian Cameron ??:
>
> 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
>
