Bug#511769: dpkg: don't put 'Essential' before 'Status' in /var/lib/dpkg/status

2009-01-14 Thread Eugene V. Lyubimkin
Package: dpkg
Version: 1.14.24
Severity: wishlist

The format of dpkg's /var/lib/dpkg/status is similar to format that
Packages uses, with one change: 'Status:' line just after 'Package:'
line. However, there is an exception from this rule: when package version
has 'Essential: yes' line, it's written before the 'Status' line.

My wish is remove this exception and put 'Essential: yes' line after
'Status:' line as dpkg does for all other package entry lines.

-- System Information:
Debian Release: 5.0
  APT prefers unstable
  APT policy: (500, 'unstable'), (1, 'experimental')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.28-rc7jackyf (SMP w/2 CPU cores)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/bash

Versions of packages dpkg depends on:
ii  coreutils 6.10-6 The GNU core utilities
ii  libc6 2.7-18 GNU C Library: Shared libraries
ii  lzma  4.43-14Compression method of 7z format in

dpkg recommends no packages.

Versions of packages dpkg suggests:
ii  apt   0.7.20 Advanced front-end for dpkg

-- no debconf information



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#511769: dpkg: don't put 'Essential' before 'Status' in /var/lib/dpkg/status

2009-01-14 Thread Raphael Hertzog
On Wed, 14 Jan 2009, Eugene V. Lyubimkin wrote:
 Package: dpkg
 Version: 1.14.24
 Severity: wishlist
 
 The format of dpkg's /var/lib/dpkg/status is similar to format that
 Packages uses, with one change: 'Status:' line just after 'Package:'
 line. However, there is an exception from this rule: when package version
 has 'Essential: yes' line, it's written before the 'Status' line.
 
 My wish is remove this exception and put 'Essential: yes' line after
 'Status:' line as dpkg does for all other package entry lines.

And what is the rationale for this change?

(Such a change needs a rationale, it won't be done just to please you)

Cheers,
-- 
Raphaël Hertzog

Le best-seller français mis à jour pour Debian Etch :
http://www.ouaza.com/livre/admin-debian/



--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#511769: dpkg: don't put 'Essential' before 'Status' in /var/lib/dpkg/status

2009-01-14 Thread Eugene V. Lyubimkin
Raphael Hertzog wrote:
 On Wed, 14 Jan 2009, Eugene V. Lyubimkin wrote:
 My wish is remove this exception and put 'Essential: yes' line after
 'Status:' line as dpkg does for all other package entry lines.
 
 And what is the rationale for this change?
 
 (Such a change needs a rationale, it won't be done just to please you)
It may (may, not essentially will) simplify some code that parse
/var/lib/dpkg/status. Now one can't rely that it will be always 'Status' line
after 'Package' entry. And, not related to previous, removing exception from
some rule usually leads to cleaner understandings and cleaner and smaller
code. If it's not the case, or such a change may break anything, then the
change is undesirable. This is only a wish, after all.

-- 
Eugene V. Lyubimkin aka JackYF, JID: jackyf.devel(maildog)gmail.com
Ukrainian C++ Developer, Debian Maintainer, APT contributor



signature.asc
Description: OpenPGP digital signature


Bug#511769: dpkg: don't put 'Essential' before 'Status' in /var/lib/dpkg/status

2009-01-14 Thread Guillem Jover
Hi!

On Wed, 2009-01-14 at 17:19:45 +0200, Eugene V. Lyubimkin wrote:
 Raphael Hertzog wrote:
  On Wed, 14 Jan 2009, Eugene V. Lyubimkin wrote:
   My wish is remove this exception and put 'Essential: yes' line after
   'Status:' line as dpkg does for all other package entry lines.

Hmm, I don't see any exception here, dpkg is always outputting the
Essential field (if it has a 'yes' value) just after the Package
field.

  And what is the rationale for this change?
  
  (Such a change needs a rationale, it won't be done just to please you)

 It may (may, not essentially will) simplify some code that parse
 /var/lib/dpkg/status.

Well arguably, no one except dpkg should be parsing the status file. I
know this might be tricky right now as dpkg does not provide a nicer
way to access the whole data, which for example apt might need. But I
hope to fix this during squeeze with the introduction of a libdpkg.

 Now one can't rely that it will be always 'Status' line after 'Package'
 entry.

And one shouldn't!

 And, not related to previous, removing exception from some rule
 usually leads to cleaner understandings and cleaner and smaller
 code. If it's not the case, or such a change may break anything,
 then the change is undesirable. This is only a wish, after all.

Again, I don't see the exception, for example dpkg will not care at
all about the ordering of the fields, and will happily parse all
fields from a reshuffled stanza w/o problems.

So, I'm not sure I see the point in this bug report.

regards,
guillem



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org