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@xxxxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |limb@xxxxxxxxxxxx AssignedTo|nobody@xxxxxxxxxxxxxxxxx |limb@xxxxxxxxxxxx --- Comment #1 from Jon Ciesla <limb@xxxxxxxxxxxx> 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: 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. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review