[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#374185: marked as done (mkvmlinuz: syntax errors in script)

Your message dated Tue, 16 Jan 2007 13:47:01 +0000
with message-id <E1H6oeT-0000Bn-Ud@ries.debian.org>
and subject line Bug#374185: fixed in mkvmlinuz 30
has caused the attached Bug report to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what I am
talking about this indicates a serious mail system misconfiguration
somewhere.  Please contact me immediately.)

Debian bug tracking system administrator
(administrator, Debian Bugs database)
--- Begin Message --- 
Package: mkvmlinuz
Version: 22
Severity: grave
Justification: make package in question unusable

Hi,

mkvmlinuz currently fails to work at all. There are a number of syntax
errors in the script, which would need to be fixed before it can ever
hope to run successfully. Did whoever uploaded this actually test it
before uploading?

Anyway. Patch follows:

--- mkvmlinuz.orig	2006-06-17 20:03:05.000000000 +0200
+++ /usr/sbin/mkvmlinuz	2006-06-17 20:13:40.000000000 +0200
@@ -60,7 +60,7 @@
 esac
 
 # use non-option arguments as release version and kernel image file if needed
-shift $((OPTIND-1))
+shift $(( $OPTIND - 1 ))
 if test -z "$release" -a -n "$1"; then
     release=$1
 fi
@@ -115,7 +115,8 @@
 if test -z "$noinitrd" -a -z "$initrd"; then
 
     # guess the location of an initrd, but don't try too hard
-    if initrd=${kernel/vmlinux/initrd.img}; test -r $initrd; then
+    initrd=$(echo $kernel | sed -e 's/vmlinux/initrd.img/')
+    if test -r $initrd; then
 	:
     else
 	initrd=""

-- System Information:
Debian Release: testing/unstable
  APT prefers unstable
  APT policy: (500, 'unstable'), (500, 'testing')
Architecture: powerpc (ppc)
Shell:  /bin/sh linked to /bin/dash
Kernel: Linux 2.6.16-2-powerpc
Locale: LANG=nl_BE.UTF-8@euro, LC_CTYPE=nl_BE.UTF-8@euro (charmap=UTF-8)

Versions of packages mkvmlinuz depends on:
ii  binutils             2.16.1cvs20060413-1 The GNU assembler, linker and bina
ii  debconf [debconf-2.0 1.5.2               Debian configuration management sy

mkvmlinuz recommends no packages.

-- debconf information excluded

--- End Message ---
--- Begin Message --- 
Source: mkvmlinuz
Source-Version: 30

We believe that the bug you reported is fixed in the latest version of
mkvmlinuz, which is due to be installed in the Debian FTP archive:

mkvmlinuz_30.dsc
  to pool/main/m/mkvmlinuz/mkvmlinuz_30.dsc
mkvmlinuz_30.tar.gz
  to pool/main/m/mkvmlinuz/mkvmlinuz_30.tar.gz
mkvmlinuz_30_powerpc.deb
  to pool/main/m/mkvmlinuz/mkvmlinuz_30_powerpc.deb



A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to 374185@bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Aurélien G�R�ME <ag@roxor.cx> (supplier of updated mkvmlinuz package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing ftpmaster@debian.org)


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Format: 1.7
Date: Sun, 14 Jan 2007 14:41:03 +0100
Source: mkvmlinuz
Binary: mkvmlinuz
Architecture: source powerpc
Version: 30
Distribution: unstable
Urgency: low
Maintainer: Debian Kernel Team <debian-kernel@lists.debian.org>
Changed-By: Aurélien G�R�ME <ag@roxor.cx>
Description: 
 mkvmlinuz  - create a kernel to boot a PowerPC machine from Open Firmware
Closes: 374185 381787 401384
Changes: 
 mkvmlinuz (30) unstable; urgency=low
 .
   * Fix arithmetic syntax error. (Closes: #374185)
   * Add myself to the Uploaders field.
   * Bump Standards-Version to 3.7.2.
   * Fix some expressions in long description.
   * Fix lintian errors and warnings.
     + Add po-debconf to the Build-Depends field.
     + Depend explicitly on bash (>= 3) to avoid a simple dependency
       on an essential package.
     + Add dh_strip in debian/rules, because a binary was left
       unstripped.
     + Remove debian/conffiles as debhelper (>= 4) adds conffiles
       under /etc automatically.
     + Add the #DEBHELPER# token to debian/postinst.
     + Move debian/kernel-image.postinst to debian/kernel-image/postinst
       and debian/kernel-image.postrm to debian/kernel-image/postrm,
       because it prevents lintian from believing they are maintainer
       scripts for the packaging system.
     + Add debian/lintian.overrides for debconf-is-not-a-registry,
       because we need a debconf setting from mkvmlinuz in kernel
       scripts.
   * Fix silent failure by adding some checks. (Closes: #381787)
   * Add the XS-Vcs-Svn field.
   * Add Portuguese translation thanks to Rui Branco. (Closes: #401384)
   * Add debugging symbols during build process.
Files: 
 e1847f834236f50bfb24fe3a8cb680f6 688 devel optional mkvmlinuz_30.dsc
 766c9f91f029a970969993dd23849320 56988 devel optional mkvmlinuz_30.tar.gz
 7d9f48a2e1fac8883af7e4c9293af22f 66194 devel optional mkvmlinuz_30_powerpc.deb

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.5 (GNU/Linux)

iD8DBQFFrNYF2WTeT3CRQaQRAtDmAJ43xpfvbtLIIdpyKAqOOjnWm5jZ1QCfekMl
j07rG+TCrGm5BDTXvyMLJSA=
=hmCs
-----END PGP SIGNATURE-----

--- End Message ---
Reply to: