[Pkg-sysvinit-devel] Bug#699087: sysvinit: avoid potentional bugs, more efficient code and cleanups

Sun Jan 27 10:44:08 UTC 2013

Package: sysvinit
Version: 2.88dsf-39
Severity: wishlist
Tags: patch

Please find attached a series of patches that mostly cleanup some scripts,
but also avoid potentional bugs and make the code more efficient.

Mostly checking if there's interest for further work along those lines.
Among the patches, these:

	0015-List-fstab-files-without-forking-ls-grep-nor-sed-and.patch
	0016-Use-shell-builtin-redirection-instead-of-using-a-tou.patch
	0017-Use-guaranteed-builtin-instead-of-sometimes-true-for.patch
	0018-Use-proper-test.patch
	0019-Don-t-fork-and-simplify-value-extractions-from-proc-.patch
	0020-Use-safer-inverted-logic.patch

may be more interesting.

-- System Information:
Debian Release: 7.0
  APT prefers unstable
  APT policy: (500, 'unstable'), (500, 'testing'), (1, 'experimental')
Architecture: i386 (i686)

Kernel: Linux 3.2.0-4-686-pae (SMP w/2 CPU cores)
Locale: LANG=C, LC_CTYPE=C (charmap=ANSI_X3.4-1968)
Shell: /bin/sh linked to /bin/bash

Versions of packages sysvinit depends on:
ii  debianutils     4.3.4
ii  initscripts     2.88dsf-39
ii  libc6           2.13-38
ii  libselinux1     2.1.9-5
ii  libsepol1       2.1.4-3
ii  sysv-rc         2.88dsf-39
ii  sysvinit-utils  2.88dsf-39

sysvinit recommends no packages.

sysvinit suggests no packages.

-- no debconf information

Cheers,

-- 
Cristian
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Uniformize-style-no-code-changes.patch
Type: text/x-diff
Size: 4391 bytes
Desc: 
URL: <http://lists.alioth.debian.org/pipermail/pkg-sysvinit-devel/attachments/20130127/d4c9458e/attachment-0021.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-Whitespace-cleanup.-No-code-shanges.patch
Type: text/x-diff
Size: 12237 bytes
Desc: 
URL: <http://lists.alioth.debian.org/pipermail/pkg-sysvinit-devel/attachments/20130127/d4c9458e/attachment-0022.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0003-More-whitespace-cleanup.patch
Type: text/x-diff
Size: 2553 bytes
Desc: 
URL: <http://lists.alioth.debian.org/pipermail/pkg-sysvinit-devel/attachments/20130127/d4c9458e/attachment-0023.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0004-Even-more-whitespace-uniformization.-No-code-changes.patch
Type: text/x-diff
Size: 924 bytes
Desc: 
URL: <http://lists.alioth.debian.org/pipermail/pkg-sysvinit-devel/attachments/20130127/d4c9458e/attachment-0024.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0005-Remove-useless-return-0.patch
Type: text/x-diff
Size: 1183 bytes
Desc: 
URL: <http://lists.alioth.debian.org/pipermail/pkg-sysvinit-devel/attachments/20130127/d4c9458e/attachment-0025.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0006-Add-explicit-return-status.patch
Type: text/x-diff
Size: 1392 bytes
Desc: 
URL: <http://lists.alioth.debian.org/pipermail/pkg-sysvinit-devel/attachments/20130127/d4c9458e/attachment-0026.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0007-Remove-some-useless-quotes.patch
Type: text/x-diff
Size: 1704 bytes
Desc: 
URL: <http://lists.alioth.debian.org/pipermail/pkg-sysvinit-devel/attachments/20130127/d4c9458e/attachment-0027.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0008-Cleanup-useless-null-string-test.patch
Type: text/x-diff
Size: 1106 bytes
Desc: 
URL: <http://lists.alioth.debian.org/pipermail/pkg-sysvinit-devel/attachments/20130127/d4c9458e/attachment-0028.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0009-Remove-useless-control-operators.patch
Type: text/x-diff
Size: 1430 bytes
Desc: 
URL: <http://lists.alioth.debian.org/pipermail/pkg-sysvinit-devel/attachments/20130127/d4c9458e/attachment-0029.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0010-Remove-totally-useless-quotes.patch
Type: text/x-diff
Size: 12803 bytes
Desc: 
URL: <http://lists.alioth.debian.org/pipermail/pkg-sysvinit-devel/attachments/20130127/d4c9458e/attachment-0030.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0011-Uniformize-tests.patch
Type: text/x-diff
Size: 3840 bytes
Desc: 
URL: <http://lists.alioth.debian.org/pipermail/pkg-sysvinit-devel/attachments/20130127/d4c9458e/attachment-0031.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0012-Remove-useless-n-test-operator.patch
Type: text/x-diff
Size: 1941 bytes
Desc: 
URL: <http://lists.alioth.debian.org/pipermail/pkg-sysvinit-devel/attachments/20130127/d4c9458e/attachment-0032.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0013-Uniformize-logical-test-operator.patch
Type: text/x-diff
Size: 1126 bytes
Desc: 
URL: <http://lists.alioth.debian.org/pipermail/pkg-sysvinit-devel/attachments/20130127/d4c9458e/attachment-0033.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0014-Remove-more-useless-quotes.patch
Type: text/x-diff
Size: 1125 bytes
Desc: 
URL: <http://lists.alioth.debian.org/pipermail/pkg-sysvinit-devel/attachments/20130127/d4c9458e/attachment-0034.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0016-Use-shell-builtin-redirection-instead-of-using-a-tou.patch
Type: text/x-diff
Size: 1888 bytes
Desc: 
URL: <http://lists.alioth.debian.org/pipermail/pkg-sysvinit-devel/attachments/20130127/d4c9458e/attachment-0035.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0016-Use-shell-builtin-redirection-instead-of-using-a-tou.patch
Type: text/x-diff
Size: 1888 bytes
Desc: 
URL: <http://lists.alioth.debian.org/pipermail/pkg-sysvinit-devel/attachments/20130127/d4c9458e/attachment-0036.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0017-Use-guaranteed-builtin-instead-of-sometimes-true-for.patch
Type: text/x-diff
Size: 1463 bytes
Desc: 
URL: <http://lists.alioth.debian.org/pipermail/pkg-sysvinit-devel/attachments/20130127/d4c9458e/attachment-0037.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0018-Use-proper-test.patch
Type: text/x-diff
Size: 2020 bytes
Desc: 
URL: <http://lists.alioth.debian.org/pipermail/pkg-sysvinit-devel/attachments/20130127/d4c9458e/attachment-0038.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0019-Don-t-fork-and-simplify-value-extractions-from-proc-.patch
Type: text/x-diff
Size: 1425 bytes
Desc: 
URL: <http://lists.alioth.debian.org/pipermail/pkg-sysvinit-devel/attachments/20130127/d4c9458e/attachment-0039.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0020-Use-safer-inverted-logic.patch
Type: text/x-diff
Size: 3815 bytes
Desc: 
URL: <http://lists.alioth.debian.org/pipermail/pkg-sysvinit-devel/attachments/20130127/d4c9458e/attachment-0040.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0021-Small-whitespace-cleanup.patch
Type: text/x-diff
Size: 1063 bytes
Desc: 
URL: <http://lists.alioth.debian.org/pipermail/pkg-sysvinit-devel/attachments/20130127/d4c9458e/attachment-0041.patch>