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

Bug#666766: marked as done (debian-installer: 00CDMountPoint apt configuration file has an incorrect syntax)

Your message dated Tue, 25 Feb 2014 16:31:04 +0300
with message-id <20140225133104.GA31169@mraw.org>
and subject line Re: Bug#666766: debian-installer: 00CDMountPoint apt configuration file has an incorrect syntax
has caused the Debian Bug report #666766,
regarding debian-installer: 00CDMountPoint apt configuration file has an incorrect syntax
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
immediately.)


-- 
666766: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=666766
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message --- 
Package: debian-installer
Severity: normal

Hello maintainers of the debian-installer,

First of all, thank you for your hard work.


The apt configuration file /etc/apt/apt.conf.d/00CDMountPoint looks like
generated by d-i [1].

Its syntax is not entirely correct, which causes annoying warnings for those
people who use cupt package manager.

Now it's:

| Acquire::cdrom {
|   mount "/media/cdrom";
| }
| Dir::Media::MountPath "/media/cdrom";

apt.conf(5) require semicolons after closing parentheses as well, so the right
syntax would be:

| Acquire::cdrom {
|   mount "/media/cdrom";
| };
| Dir::Media::MountPath "/media/cdrom";

(note the semicolon in the third line).

I wanted to provide a patch but couldn't find the file in the sources of d-i
[2]. Please let me know anything else I can do to have this fixed in the wheezy's
debian-installer.

[1] http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=611569#30
[2] git://git.debian.org/d-i/debian-installer.git

-- System Information:
Debian Release: wheezy/sid
  APT prefers testing
  APT policy: (990, 'testing'), (500, 'stable-updates'), (500, 'unstable'), (500, 'stable'), (1, 'experimental')
Architecture: i386 (i686)

Kernel: Linux 3.2.0-1-686-pae (SMP w/2 CPU cores)
Locale: LANG=fi_FI.UTF-8, LC_CTYPE=fi_FI.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

--- End Message ---
--- Begin Message --- 
Version: 1.124

Eugene V. Lyubimkin <jackyf@debian.org> (2012-04-01):
> Its syntax is not entirely correct, which causes annoying warnings for
> those people who use cupt package manager.
> 
> Now it's:
> 
> | Acquire::cdrom {
> |   mount "/media/cdrom";
> | }
> | Dir::Media::MountPath "/media/cdrom";
> 
> apt.conf(5) require semicolons after closing parentheses as well, so
> the right syntax would be: […]

This manpage doesn't say that AFAICT. It even points to:
  /usr/share/doc/apt/examples/apt.conf

which ends with:
  DPkg
  {
    // Probably don't want to use force-downgrade..
    Options {"--force-overwrite";}
  }

so I'm not sure cupt is right in complaining about that.

> I wanted to provide a patch but couldn't find the file in the sources
> of d-i [2]. Please let me know anything else I can do to have this
> fixed in the wheezy's debian-installer.
> 
> [1] http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=611569#30
> [2] git://git.debian.org/d-i/debian-installer.git

Anyway, base-installer has:
| commit 722410e864eccf3cc83bb8bb655079a99eb347d3
| Author: Joey Hess <joey@kitenet.net>
| Date:   Sat Apr 7 16:02:27 2012 -0400
| 
|     Add missing semicolon to /etc/apt/apt.conf.d/00CDMountPoint.

so closing with the appropriate version.

Mraw,
KiBi.

Attachment: signature.asc
Description: Digital signature


--- End Message ---
Reply to: