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

Bug#507300: marked as done (debian-cd: [hppa] command line parameter passed with incorrect quoting)

Your message dated Mon, 12 Jan 2009 16:02:03 +0000
with message-id <E1LMPEp-0007cU-Lo@ries.debian.org>
and subject line Bug#507300: fixed in debian-cd 3.1.0
has caused the Debian Bug report #507300,
regarding debian-cd: [hppa] command line parameter passed with incorrect quoting
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 this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org

507300: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=507300
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message ---
Package: debian-cd
Version: 3.0.2
Severity: minor

The boot-hppa script currently has:
add_mkisofs_opt $CDROOT/../$1.mkisofs_opts "-hppa-cmdline '0/vmlinux,root=/dev/ram,initrd=0/ramdisk,ramdisk_size=$RSIZE"'

During interaction with the HPPA bootloader this results in the following
being displayed:
'0/vmlinux root=/dev/ram initrd=0/ramdisk ramdisk_size=13890' console=ttyS0 TERM=vt102
 0: '0/vmlinux
 1: root=/dev/ram
 2: initrd=0/ramdisk
 3: ramdisk_size=13890'
 4: console=ttyS0
 5: TERM=vt102

Note the quotes in lines 0 and 3.

After the installer has been booted we get this:
# cat /proc/cmdline
root=/dev/ram ramdisk_size=13890' console=ttyS0 TERM=vt102 palo_kernel=0'0/vmlinux

I currently don't see any bad effects from this, but would like to add
support for KERNEL_PARAMS, which could result in for example:
# cat /proc/cmdline
root=/dev/ram ramdisk_size=13890 desktop=xfce' ...

Which in turn results in the following in the cdebconf database:
Name: tasksel/desktop
Template: debian-installer/dummy
Value: xfce'"'

Which _is_ likely to break things.

Not sure how this could be solved. Possibly this is a mkisofs bug rather
than a debian-cd one.

Attachment: signature.asc
Description: This is a digitally signed message part.

--- End Message ---
--- Begin Message ---
Source: debian-cd
Source-Version: 3.1.0

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

  to pool/main/d/debian-cd/debian-cd_3.1.0.dsc
  to pool/main/d/debian-cd/debian-cd_3.1.0.tar.gz
  to pool/main/d/debian-cd/debian-cd_3.1.0_all.deb

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

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

Debian distribution maintenance software
Frans Pop <fjp@debian.org> (supplier of updated debian-cd 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)

Hash: SHA1

Format: 1.8
Date: Mon, 12 Jan 2009 16:07:35 +0100
Source: debian-cd
Binary: debian-cd
Architecture: source all
Version: 3.1.0
Distribution: unstable
Urgency: low
Maintainer: Debian CD Group <debian-cd@lists.debian.org>
Changed-By: Frans Pop <fjp@debian.org>
 debian-cd  - Tools for building (Official) Debian CD set
Closes: 503911 506616 506668 507300
 debian-cd (3.1.0) unstable; urgency=low
   [ Frans Pop ]
   * Don't include brltty on netinst images as it pulls in libicu which causes
     a 5MB size increase of the image, which is out of proportion given its use.
     This should not affect users as netinst installs should normally use a
     mirror and thus the package will be installed from there if needed.
     brltty will still get included on full CDs and DVDs through forcd1.
   * Remove references to no longer used files rawlist.exclude and list.exclude.
   * Clean up no longer needed compatibility code in lenny x86 boot script.
   [ Steve McIntyre ]
   * Rename list2cds to sort_deps.
   * Fix how directory trees are created in update-cd. Closes: #503911.
   * Update update_popcon to cope better with errors; now uses curl instead
     of wget.
   [ Raphael Hertzog ]
   * Don't try to copy sbm on etch CD images as it got removed.
   * Also extract isolinux.bin from the syslinux package in the case of etch
     CD images (since it's no more provided by debian-cd itself).
   [ Frans Pop ]
   * Rework the way tasks are handled:
     - all tasks files are moved to ./tasks/<codename> subdirectories so they
       can be more easily kept up-to-date with specific distro releases
     - always generate the debian-installer and tasksel tasks dynamically:
       . all task files for the desired suite are copied to the working
         directory at the beginning of each build, and are used from there
       . the debian-installer and tasksel tasks are no longer included in
         releases but are always created automatically at build time; this
         means users no longer need to run the generate_di_* scripts or the
         update_tasksel script
       . the popcon task file will remain included as a static file, but plan
         is to add an option to update it automatically for each build; reason
         is that updating it requires network access
     - bump version to 3.1
   * Update the contrib/testingcds script based on its current version on
   * Clean up and update interesting-fromcd23 and exclude task files for lenny.
   * Fix error in expansion of %ARCH% variables in D-I image location envvars.
   * Fix i386/amd64 multi-arch support in the lenny x86 boot script so that the
     isolinux config gets modified correctly independent of the order in which
     the arches are specified.
   * Workaround #505243 by modifying the syslinux configuration for amd64/i386
     multi-arch images and using ifcpu64.c32 to autodetect 32/64-bit systems
     when a user hits enter from the isolinux help screens. Based on research
     done by Franklin Piat.
   [ Steve McIntyre ]
   * Stop copying the /tools/ directory on from the mirror; it's going away.
   * Add support for multiple desktop tasks on a single disk
   * Add support for tracking which packages we've extracted binaries from,
     so that we can archive them (both binaries and sources). If we need
     anything, we will list it in CD$N.pkgs_extracted so that external
     scripts can pick it up and do whatever's needed.
   * Minor changes to the interface of tools/which_deb to accommodate that:
     now just lists the files *within* the mirror; it's up to callers to
     prepend ${MIRROR} as needed.
   [ Frans Pop ]
   * make_image: ensure correct quoting of variables when calling genisoimage.
     Closes: #507300.
   * boot-hppa: support KERNEL_PARAMS (so desktop= can be passed).
   * boot-hppa: stop passing redundant boot parameters root and ramdisk_size.
   * Split out languages supported by D-I and tasksel from task.list files.
     Instead of duplicating what language tasks should be included in each
     task.list.* file, let update_tasks add them dynamically at runtime from a
     single list of languages supported by tasksel and D-I. This also makes
     maintaining the lists a lot less painful. Closes: #506616.
   * Indicate which tasks are "secondary" by adding a "-" suffix in task.list.*
     files instead of hardcoding it in update_tasks. Closes: #506668.
   * Add new file tasks/README.tasksel to document the way tasksel tasks are
     used when determining the order in which packages are added to images.
   * Update list of languages supported by tasksel and D-I for Lenny.
   * Add support for a separate 'LXDE desktop environment' CD.
   * Add support for a 'light desktop environment' CD (LXDE + Xfce).
     Includes a framework to manipulate the isolinux configuration for x86 so
     that a user can select which desktop he wishes to install.
   * Add support to create an 'all desktops' DVD.
     This new task will include all four desktop environments supported by
     tasksel (GNOME, KDE, LXDE and Xfce). On x86 it will also add an option in
     the isolinux menu (under Advanced options) to select which DE to install,
     but GNOME will remain default.
   * Add support for a separate 'GNOME desktop environment' CD.
     This new task is mostly added for completeness. Main difference with the
     default task file is that no server tasks are included.
   * Add a suffix "generic" to the default task files to make the overall naming
     scheme more consistent. It also allows to make update_tasks more generic.
   * Update exclude list for Lenny:
     - remove aspell* packages as they do not included on CD1 anyway and
       excluding them also means they get excluded from DVD1;
     - remove xen/vserver linux-image packages as they do not get included on
       CD1 anyway;
     - xmail and cdebconf were added as they got pulled in through alternative
       dependencies, which is no longer an issue.
   * Give priority to Key packages over regular packages for secondary tasks
     (similar to what we already did for primary tasks).
   * README.easy-build: document how to use custom packages and D-I images.
   * Add myself as uploader.
 19bd7e8f74fe9ca847ee23a6d1b1fef607d62cd2 1013 debian-cd_3.1.0.dsc
 26bc627f84293d9930a7b597f1bec200f6c67cc1 593597 debian-cd_3.1.0.tar.gz
 6de674145488e57ab3a1edbf03c47083887999b7 601382 debian-cd_3.1.0_all.deb
 72c2f26106775ce82d9ac02ce667bf383209c64b57aaace507bfc25629af764d 1013 debian-cd_3.1.0.dsc
 9bcc40b3bf7edf51e46d8af1dec87ca1bb9f3fc43ad0a6172e19f7c1917c981e 593597 debian-cd_3.1.0.tar.gz
 db1aa7e230816948cca85f19f0acae7ed0e1a214891e1cdcc29b6887980725bf 601382 debian-cd_3.1.0_all.deb
 9d17ee606340bf252a3186360dbd2834 1013 admin extra debian-cd_3.1.0.dsc
 0c5332894f6723fc2645ee63b9bc523f 593597 admin extra debian-cd_3.1.0.tar.gz
 183e1d4e5e5d9a382b39fee44733c93d 601382 admin extra debian-cd_3.1.0_all.deb

Version: GnuPG v1.4.9 (GNU/Linux)


--- End Message ---

Reply to: