[Bug 226139] Merge Review: mgetty

Tue Sep 16 15:50:44 UTC 2008

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

https://bugzilla.redhat.com/show_bug.cgi?id=226139

Jon Ciesla <limb at jcomserv.net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |limb at jcomserv.net
         AssignedTo|nobody at fedoraproject.org    |limb at jcomserv.net

--- Comment #1 from Jon Ciesla <limb at jcomserv.net>  2008-09-16 11:50:42 EDT ---
rpmlint on SRPM:

mgetty.src: W: mixed-use-of-spaces-and-tabs (spaces: line 183, tab: line 179)
The specfile mixes use of spaces and tabs for indentation, which is a cosmetic
annoyance.  Use either spaces or tabs for indentation, not both.

Fix, since there's the below to do as well. . .

rpmlint on RPMS:

The file can't be read by everybody. If this is expected (for security
reasons), contact your rpmlint distributor to get it added to the list of
exceptions for your distro (or add it to your local configuration if you
installed rpmlint from the source tarball).

mgetty.i386: E: non-readable /etc/mgetty+sendfax/dialin.config 0600
The file can't be read by everybody. If this is expected (for security
reasons), contact your rpmlint distributor to get it added to the list of
exceptions for your distro (or add it to your local configuration if you
installed rpmlint from the source tarball).

mgetty.i386: E: non-readable /etc/mgetty+sendfax/mgetty.config 0600
The file can't be read by everybody. If this is expected (for security
reasons), contact your rpmlint distributor to get it added to the list of
exceptions for your distro (or add it to your local configuration if you
installed rpmlint from the source tarball).

mgetty.i386: E: non-readable /sbin/mgetty 0700
The file can't be read by everybody. If this is expected (for security
reasons), contact your rpmlint distributor to get it added to the list of
exceptions for your distro (or add it to your local configuration if you
installed rpmlint from the source tarball).

mgetty.i386: E: non-standard-executable-perm /sbin/mgetty 0700
A standard executable should have permission set to 0755. If you get this
message, it means that you have a wrong executable permissions in some files
included in your package.

mgetty.i386: E: non-readable /usr/sbin/callback 0700
The file can't be read by everybody. If this is expected (for security
reasons), contact your rpmlint distributor to get it added to the list of
exceptions for your distro (or add it to your local configuration if you
installed rpmlint from the source tarball).

mgetty.i386: E: non-standard-executable-perm /usr/sbin/callback 0700
A standard executable should have permission set to 0755. If you get this
message, it means that you have a wrong executable permissions in some files
included in your package.

mgetty-sendfax.i386: W: non-standard-uid /var/spool/fax/incoming fax
A file in this package is owned by a non standard user. Standard users are:
root, bin, daemon, adm, lp, sync, shutdown, halt, mail, news, uucp, operator,
games, gopher, ftp, nobody.

mgetty-sendfax.i386: W: non-standard-uid /usr/lib/mgetty+sendfax/faxq-helper
fax
A file in this package is owned by a non standard user. Standard users are:
root, bin, daemon, adm, lp, sync, shutdown, halt, mail, news, uucp, operator,
games, gopher, ftp, nobody.

mgetty-sendfax.i386: E: setuid-binary /usr/lib/mgetty+sendfax/faxq-helper fax
04711
The file is setuid, this may be dangerous, especially if this  file is setuid
root.

mgetty-sendfax.i386: E: non-standard-executable-perm
/usr/lib/mgetty+sendfax/faxq-helper 04711
A standard executable should have permission set to 0755. If you get this
message, it means that you have a wrong executable permissions in some files
included in your package.

mgetty-sendfax.i386: W: non-standard-uid /var/spool/fax/outgoing fax
A file in this package is owned by a non standard user. Standard users are:
root, bin, daemon, adm, lp, sync, shutdown, halt, mail, news, uucp, operator,
games, gopher, ftp, nobody.

mgetty-voice.i386: E: non-readable /sbin/vgetty 0700
The file can't be read by everybody. If this is expected (for security
reasons), contact your rpmlint distributor to get it added to the list of
exceptions for your distro (or add it to your local configuration if you
installed rpmlint from the source tarball).

mgetty-voice.i386: E: non-standard-executable-perm /sbin/vgetty 0700
A standard executable should have permission set to 0755. If you get this
message, it means that you have a wrong executable permissions in some files
included in your package.

mgetty-voice.i386: E: non-readable /etc/mgetty+sendfax/voice.conf 0600
The file can't be read by everybody. If this is expected (for security
reasons), contact your rpmlint distributor to get it added to the list of
exceptions for your distro (or add it to your local configuration if you
installed rpmlint from the source tarball).

These are probably fine, but need explanation in the spec.

mgetty-sendfax.i386: W: summary-ended-with-dot Provides support for sending
faxes over a modem.
Summary ends with a dot.

mgetty-viewfax.i386: W: summary-ended-with-dot An X Window System fax viewer.
Summary ends with a dot.

mgetty-voice.i386: W: summary-ended-with-dot A program for using your modem and
mgetty as an answering machine.
Summary ends with a dot.

Trivial, but fix.

mgetty-viewfax.i386: E: only-non-binary-in-usr-lib
There are only non binary files in /usr/lib so they should be in /usr/share.

Not sure if this is fixable or not.  What are these?

mgetty-voice.i386: E: incoherent-logrotate-file /etc/logrotate.d/vgetty
Your logrotate file should be named /etc/logrotate.d/<package name>.

mgetty-voice.i386: E: incoherent-logrotate-file /etc/logrotate.d/vm
Your logrotate file should be named /etc/logrotate.d/<package name>.

mgetty-sendfax.i386: E: incoherent-logrotate-file /etc/logrotate.d/sendfax
Your logrotate file should be named /etc/logrotate.d/<package name>.

Fix, unless catastrophic breakage would ensue.

mgetty-voice.i386: W: file-not-utf8
/usr/share/doc/mgetty-voice-1.1.36/ChangeLog
The character encoding of this file is not UTF-8.  Consider converting it in
the specfile for example using iconv(1).

mgetty.i386: W: file-not-utf8 /usr/share/doc/mgetty-1.1.36/modems.db
The character encoding of this file is not UTF-8.  Consider converting it in
the specfile for example using iconv(1).

Fix.

mgetty-voice.i386: W: conffile-without-noreplace-flag /etc/logrotate.d/vgetty
A configuration file is stored in your package without the noreplace flag. A
way to resolve this is to put the following in your SPEC file:
%config(noreplace) /etc/your_config_file_here

mgetty-voice.i386: W: conffile-without-noreplace-flag /etc/logrotate.d/vm
A configuration file is stored in your package without the noreplace flag. A
way to resolve this is to put the following in your SPEC file:
%config(noreplace) /etc/your_config_file_here

mgetty-voice.i386: W: conffile-without-noreplace-flag
/etc/mgetty+sendfax/voice.conf
A configuration file is stored in your package without the noreplace flag. A
way to resolve this is to put the following in your SPEC file:
%config(noreplace) /etc/your_config_file_here

I can't think of a compelling reason not to fix this.  If there's one I'm
missing, document in the spec.

Mock rawhide build fails:

xecuting(%prep): /bin/sh -e /var/tmp/rpm-tmp.8qdaFe
+ umask 022
+ cd /builddir/build/BUILD
+ LANG=C
+ export LANG
+ unset DISPLAY
+ cd /builddir/build/BUILD
+ rm -rf mgetty-1.1.36
+ /usr/bin/gzip -dc /builddir/build/SOURCES/mgetty1.1.36-Jun15.tar.gz
+ /bin/tar -xf -
+ STATUS=0
+ '[' 0 -ne 0 ']'
+ cd mgetty-1.1.36
+ /bin/chmod -Rf a+rX,u+w,g-w,o-w .
+ mv policy.h-dist policy.h
Patch #0 (mgetty-1.1.29-config.patch):
+ echo 'Patch #0 (mgetty-1.1.29-config.patch):'
+ /bin/cat /builddir/build/SOURCES/mgetty-1.1.29-config.patch
+ /usr/bin/patch -s -p1 -b --suffix .config --fuzz=0
1 out of 2 hunks FAILED
 -- saving rejects to file Makefile.rej
RPM build errors:
error: Bad exit status from /var/tmp/rpm-tmp.8qdaFe (%prep)
    Bad exit status from /var/tmp/rpm-tmp.8qdaFe (%prep)
Child returncode was: 1
EXCEPTION: Command failed. See logs for output.
 # ['bash', '--login', '-c', 'rpmbuild -bb --target i386 --nodeps
builddir/build/SPECS/mgetty.spec']
Traceback (most recent call last):
  File "/usr/lib/python2.5/site-packages/mock/trace_decorator.py", line 70, in
trace
    result = func(*args, **kw)
  File "/usr/lib/python2.5/site-packages/mock/util.py", line 316, in do
    raise mock.exception.Error, ("Command failed. See logs for output.\n # %s"
% (command,), child.returncode)
Error: Command failed. See logs for output.
 # ['bash', '--login', '-c', 'rpmbuild -bb --target i386 --nodeps
builddir/build/SPECS/mgetty.spec']
LEAVE do --> EXCEPTION RAISED

Probably patch fuzz issue, so I can't verify if the BuildRequires are good.

Otherwise, full review is good.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.