This service will be undergoing maintenance at 00:00 UTC, 2016-08-01. It is expected to last about 1 hours
Bug 166343 - Review Request: nail - An enhanced implementation of the mailx command
Review Request: nail - An enhanced implementation of the mailx command
Status: CLOSED NEXTRELEASE
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Aurelien Bompard
David Lawrence
http://nail.sourceforge.net
:
Depends On:
Blocks: FE-ACCEPT
  Show dependency treegraph
 
Reported: 2005-08-19 10:27 EDT by Dmitry Butskoy
Modified: 2007-11-30 17:11 EST (History)
2 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2005-09-26 06:35:06 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
wtogami: fedora‑cvs+


Attachments (Terms of Use)

  None (edit)
Description Dmitry Butskoy 2005-08-19 10:27:41 EDT
Spec Name or Url: http://dmitry.butskoy.name/nail/nail.spec
SRPM Name or Url: http://dmitry.butskoy.name/nail/nail-11.25-1.src.rpm

Description: 
  Nail is an enhanced mail command, which provides the functionality
of the POSIX mailx command. It is derived from Berkeley Mail.
  Additionally to the POSIX features, nail can work with Maildir/ e-mail
storage format (as well as mailboxes), supports IMAP, POP3 and SMTP
procotols (including over SSL) to operate with remote hosts, handles mime
types and different charsets. There are a lot of other useful features,
see nail.html in the documentation.
  And as its ancient analogues, nail can be used as a mail script language,
both for sending and receiving mail.


Additional info:

  As now RedHat recommends to use new Maildir/ format instead of old mailboxes,
the "mail" command becomes useless for mail reading, because it supports the mailbox format only. Using nail can solve this issue.
  In addition, nail provides many useful features, for a long time pending
to be implemented in a command line interface.
Comment 1 Oliver Falk 2005-08-22 08:22:25 EDT
Good:
 - rpmlint is clean
 - License is OK
 - Works with my IMAP server. :-)

To be changed:
 - Remove the 'An' from the beginning of the Summary
 - BuildRoot is not Fedora standard:
%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
 - I would remove the leading spaces in the %description
 - Use %{_sbindir}/sendmail in the makeflags; Inconsitent use of macros
 - Do we really need the nail webpage???

Comment 2 Dmitry Butskoy 2005-08-22 09:01:33 EDT
" - Remove the 'An' from the beginning of the Summary"
" - BuildRoot is not Fedora standard:"
done.

" - I would remove the leading spaces in the %description"
  I use leading spaces for paragraphs. IMHO, the description will be more
readable if paragraphs will be used. Or it is not recommended?

" - Use %{_sbindir}/sendmail in the makeflags;"
done.

" - Do we really need the nail webpage???"
Why are not? :-) It seemed to me that there is no good README in this package,
therefore I have added a copy of its current homepage.

  See new release: http://dmitry.butskoy.name/nail/nail-11.25-2.src.rpm

Comment 3 Oliver Falk 2005-08-22 09:19:01 EDT
(In reply to comment #2)
> " - I would remove the leading spaces in the %description"
>   I use leading spaces for paragraphs. IMHO, the description will be more
> readable if paragraphs will be used. Or it is not recommended?

Good, if you *wanted* it like this.
 
> " - Do we really need the nail webpage???"
> Why are not? :-) It seemed to me that there is no good README in this package,
> therefore I have added a copy of its current homepage.

OK. I thought the man-page is enough. But you are right a good readme is fine to
have...
 
>   See new release: http://dmitry.butskoy.name/nail/nail-11.25-2.src.rpm

Add this bug #166343 to the changelog... And I approve it.
Comment 4 Michael Schwendt 2005-08-22 09:28:18 EDT
Blank lines in front of paragraphs are most common.
Leading spaces decrease readability and lead to a sort of stair-case effect.
Comment 5 Dmitry Butskoy 2005-08-22 09:31:16 EDT
  Add bug #166343 to changelog.
  (The same source, I did not increment release for this...)
Comment 6 Dmitry Butskoy 2005-08-22 09:41:01 EDT
  Also change leading spaces to blank lines.
Comment 7 Dmitry Butskoy 2005-09-07 07:03:40 EDT
  Oliver,

  Whether you had plans to review this package? As no I am "sponsored", you can
do it...
Comment 8 Dmitry Butskoy 2005-09-07 07:05:48 EDT
 Brrr...
 As NOW I am sponsored, you can do it (WTF typo... :-)))
Comment 9 Dmitry Butskoy 2005-09-14 06:04:12 EDT
  ping -b somebody  :)
Comment 10 Aurelien Bompard 2005-09-25 12:07:56 EDT
I'm not trying to steal this package from you Oliver, but it looks fine to me.
Since there's been no answer for about a month, here's my APPROVED vote.
Two minor remarks:
 - why don't you use "%{?_smp_mflags}" ? If the package doesn't build with it,
please add a comment above the make line. Else, please add it.
 - krb5-devel is already required by openssl-devel, so you can remove it from
the BuildRequires.
Comment 11 Dmitry Butskoy 2005-09-26 06:05:10 EDT
 %{?_smp_mflags} -- OK

  krb5-devel -- nail uses gssapi stuff immediately (and checks for it before
compiling). 
  There is a hypothetical possibility that openssl will use another kerberos
implementation (heimdal for example), therefore we cannot rely that
openssl-devel will always imply krb5-devel ...

New SRPM: http://dmitry.butskoy.name/nail/nail-11.25-3.src.rpm
New SPEC: http://dmitry.butskoy.name/nail/nail.spec

Comment 12 Aurelien Bompard 2005-09-26 06:16:03 EDT
Okay, go ahead and import.
Comment 13 Michael Schwendt 2005-10-02 17:57:58 EDT
> %install
> make DESTDIR=$RPM_BUILD_ROOT `cat makeflags` install

rm -rf $RPM_BUILD_ROOT

is missing as the first command in this %install section.
Comment 14 Dmitry Butskoy 2005-10-03 07:34:21 EDT
Thanks,
(Why rpmlint-0.71 has told nothing about this? :( )

changed in CVS.
Comment 15 Dmitry Butskoy 2007-07-25 08:35:47 EDT
Package Change Request
======================
Package Name: nail
New Branches: EL-4 EL-5

Note You need to log in before you can comment on or make changes to this bug.