Bug#881406: apt-file: drop Contents header removal?

2018-08-26 Thread Paul Wise
On Sat, 11 Nov 2017 19:00:39 +0800 Paul Wise wrote:

> I noticed that reprepro (#881405) still has a header.

This has now been fixed.

> Unfortunately the reprepro header is not compatible with the current
> workaround in apt-file. We could fix that by replacing the first
> match with line 1. Alternatively we could just drop the workaround,
> removing it should speed up apt-file by not invoking sed. I guess
> most apt-file users aren't using reprepro generated repositories that
> also have Contents files available. Also, the header from reprepro
> doesn't cause quite as bad issues as the dak and launchpad header, it
> just means that there is a package called FILE with a single file in
> it called LOCATION.

Given all this (especially the speed-up) I think the workaround should
now be removed from apt-file.

-- 
bye,
pabs

https://wiki.debian.org/PaulWise



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


Bug#881406: apt-file: drop Contents header removal?

2017-11-11 Thread Paul Wise
Package: apt-file
Version: 3.1.5
Severity: wishlist

In #676642 we added a workaround for the Contents file having a header
breaking some lookups. The Debian (#841997) and Ubuntu (LP#1638219)
archives have now removed this header but unfortunately I noticed that
reprepro (#881405) still has a header. Unfortunately the reprepro header is not 
compatible with the current workaround in apt-file. We could fix that by 
replacing the first match with line 1. Alternatively we could just drop the 
workaround, removing it should speed up apt-file by not invoking sed. I guess 
most apt-file users aren't using reprepro 
generated repositories that also have Contents files available. Also,
the header from reprepro doesn't cause quite as bad issues as the dak
and launchpad header, it just means that there is a package called FILE
with a single file in it called LOCATION.

https://bugs.debian.org/676642
https://bugs.debian.org/841997
https://bugs.launchpad.net/bugs/1638219

-- System Information:
Debian Release: buster/sid
  APT prefers testing-debug
  APT policy: (900, 'testing-debug'), (900, 'testing'), (800, 
'unstable-debug'), (800, 'unstable'), (790, 'buildd-unstable'), (700, 
'experimental-debug'), (700, 'experimental'), (690, 'buildd-experimental')
Architecture: amd64 (x86_64)

Kernel: Linux 4.13.0-1-amd64 (SMP w/4 CPU cores)
Locale: LANG=en_AU.utf8, LC_CTYPE=en_AU.utf8 (charmap=UTF-8), 
LANGUAGE=en_AU.utf8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)

Versions of packages apt-file depends on:
ii  apt  1.5
ii  libapt-pkg-perl  0.1.33
ii  liblist-moreutils-perl   0.416-1+b3
ii  libregexp-assemble-perl  0.36-1
ii  perl 5.26.0-8

apt-file recommends no packages.

apt-file suggests no packages.

-- no debconf information

-- 
bye,
pabs

https://wiki.debian.org/PaulWise


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