Bug#828633: mount in testing/unstable should conflict with old bash-completion

2016-06-26 Thread Andreas Henriksson
Control: tags -1 = moreinfo

Hello!

Apparently I'm stressing this too much as well being short on time
currently...

On Sun, Jun 26, 2016 at 08:12:26PM +0200, Sven Joachim wrote:
> On 2016-06-26 23:22 +1000, Russell Coker wrote:
> 
> > Below is the results of "apt-get dist-upgrade" to upgrade from Jessie to
> > testing.  If I manually upgrade bash-completion first then mount can be
> > upgraded without problems.
> >
> > Preparing to unpack .../mount_2.28-5_amd64.deb ...
> > Unpacking mount (2.28-5) over (2.27.1-3.1) ...
> > dpkg: error processing archive 
> > /var/cache/apt/archives/mount_2.28-5_amd64.deb (--unpack):
> >  trying to overwrite '/usr/share/bash-completion/completions/mount', which 
> > is also in package bash-completion 1:2.1-4.3
> 
> How could that happen, considering that bash-completion 1:2.1-4.3 is
> precisely the version which dropped the file in question, and there is
> not even a newer version in the archive?  Did you have a local
> bash-completion package with the same version as the official one and
> different contents installed?

I also find this peculiar when looking closer at it. That version
of bash-completion should not have the mount completion file.

> 
> On 2016-06-26 19:01 +0200, Sven Joachim wrote:
> 
> > On 2016-06-26 18:42 +0200, Andreas Henriksson wrote:
> >
> >> Thanks for your bug report. I apparently forgot to bump the version
> >> of the already existing Breaks/Replaces statements in previous upload.
> >> Fixed in git, will be part of next upload.
> >
> > The fix in git is wrong (at least not sufficient), though.
> 
> Or at least unrelated, since changes to the util-linux package will not
> fix file conflicts in the mount package.

Thanks for catching my mistake. I pushed a revert

> 
> > You bumped the Breaks/Replaces combination in util-linux, but really
> > it needs to be changed in mount, removing the spurious tilde:
> >
> > --8<---cut here---start->8---
> > diff --git a/debian/control b/debian/control
> > index cef4980..61f66ba 100644
> > --- a/debian/control
> > +++ b/debian/control
> > @@ -78,8 +78,8 @@ Section: admin
> >  Pre-Depends: ${misc:Pre-Depends}, ${shlibs:Depends}
> >  Depends: ${misc:Depends}
> >  Suggests: nfs-common (>=1:1.1.0-13)
> > -Breaks: bash-completion (<< 1:2.1-4.3~)
> > -Replaces: bash-completion (<< 1:2.1-4.3~)
> > +Breaks: bash-completion (<< 1:2.1-4.3)
> > +Replaces: bash-completion (<< 1:2.1-4.3)

This change makes no sense to me. When declaring relationships against
specific debian revisions including a trailing tilde is recommended
to enable backports for example.

> >  Multi-Arch: foreign
> >  Description: tools for mounting and manipulating filesystems
> >   This package provides the mount(8), umount(8), swapon(8),
> > --8<---cut here---end--->8---
> 
> Scratch that, this does not make any sense.  I should not comment on bug
> reports during football half-time breaks.

I guess we agree. not sure what change is needed here really.

Russel could you please enlighten us how this could happen?

(I'll try to find time during debconf to set up a chroot and test upgrade
to verify..)

Regards,
Andreas Henriksson



Bug#828633: mount in testing/unstable should conflict with old bash-completion

2016-06-26 Thread Sven Joachim
On 2016-06-26 23:22 +1000, Russell Coker wrote:

> Below is the results of "apt-get dist-upgrade" to upgrade from Jessie to
> testing.  If I manually upgrade bash-completion first then mount can be
> upgraded without problems.
>
> Preparing to unpack .../mount_2.28-5_amd64.deb ...
> Unpacking mount (2.28-5) over (2.27.1-3.1) ...
> dpkg: error processing archive /var/cache/apt/archives/mount_2.28-5_amd64.deb 
> (--unpack):
>  trying to overwrite '/usr/share/bash-completion/completions/mount', which is 
> also in package bash-completion 1:2.1-4.3

How could that happen, considering that bash-completion 1:2.1-4.3 is
precisely the version which dropped the file in question, and there is
not even a newer version in the archive?  Did you have a local
bash-completion package with the same version as the official one and
different contents installed?

On 2016-06-26 19:01 +0200, Sven Joachim wrote:

> On 2016-06-26 18:42 +0200, Andreas Henriksson wrote:
>
>> Thanks for your bug report. I apparently forgot to bump the version
>> of the already existing Breaks/Replaces statements in previous upload.
>> Fixed in git, will be part of next upload.
>
> The fix in git is wrong (at least not sufficient), though.

Or at least unrelated, since changes to the util-linux package will not
fix file conflicts in the mount package.

> You bumped the Breaks/Replaces combination in util-linux, but really
> it needs to be changed in mount, removing the spurious tilde:
>
> --8<---cut here---start->8---
> diff --git a/debian/control b/debian/control
> index cef4980..61f66ba 100644
> --- a/debian/control
> +++ b/debian/control
> @@ -78,8 +78,8 @@ Section: admin
>  Pre-Depends: ${misc:Pre-Depends}, ${shlibs:Depends}
>  Depends: ${misc:Depends}
>  Suggests: nfs-common (>=1:1.1.0-13)
> -Breaks: bash-completion (<< 1:2.1-4.3~)
> -Replaces: bash-completion (<< 1:2.1-4.3~)
> +Breaks: bash-completion (<< 1:2.1-4.3)
> +Replaces: bash-completion (<< 1:2.1-4.3)
>  Multi-Arch: foreign
>  Description: tools for mounting and manipulating filesystems
>   This package provides the mount(8), umount(8), swapon(8),
> --8<---cut here---end--->8---

Scratch that, this does not make any sense.  I should not comment on bug
reports during football half-time breaks.

Cheers,
   Sven



Bug#828633: mount in testing/unstable should conflict with old bash-completion

2016-06-26 Thread Sven Joachim
On 2016-06-26 18:42 +0200, Andreas Henriksson wrote:

> Control: tags -1 + pending
> Control: severity -1 serious
>
> Hello!
>
> Thanks for your bug report. I apparently forgot to bump the version
> of the already existing Breaks/Replaces statements in previous upload.
> Fixed in git, will be part of next upload.

The fix in git is wrong (at least not sufficient), though.  You bumped
the Breaks/Replaces combination in util-linux, but really it needs to be
changed in mount, removing the spurious tilde:

--8<---cut here---start->8---
diff --git a/debian/control b/debian/control
index cef4980..61f66ba 100644
--- a/debian/control
+++ b/debian/control
@@ -78,8 +78,8 @@ Section: admin
 Pre-Depends: ${misc:Pre-Depends}, ${shlibs:Depends}
 Depends: ${misc:Depends}
 Suggests: nfs-common (>=1:1.1.0-13)
-Breaks: bash-completion (<< 1:2.1-4.3~)
-Replaces: bash-completion (<< 1:2.1-4.3~)
+Breaks: bash-completion (<< 1:2.1-4.3)
+Replaces: bash-completion (<< 1:2.1-4.3)
 Multi-Arch: foreign
 Description: tools for mounting and manipulating filesystems
  This package provides the mount(8), umount(8), swapon(8),
--8<---cut here---end--->8---

Cheers,
   Sven



Bug#828633: mount in testing/unstable should conflict with old bash-completion

2016-06-26 Thread Andreas Henriksson
Control: tags -1 + pending
Control: severity -1 serious

Hello!

Thanks for your bug report. I apparently forgot to bump the version
of the already existing Breaks/Replaces statements in previous upload.
Fixed in git, will be part of next upload.

Regards,
Andreas Henriksson



Bug#828633: mount in testing/unstable should conflict with old bash-completion

2016-06-26 Thread Russell Coker
Package: mount
Version: 2.28-5
Severity: normal

Below is the results of "apt-get dist-upgrade" to upgrade from Jessie to
testing.  If I manually upgrade bash-completion first then mount can be
upgraded without problems.

Preparing to unpack .../mount_2.28-5_amd64.deb ...
Unpacking mount (2.28-5) over (2.27.1-3.1) ...
dpkg: error processing archive /var/cache/apt/archives/mount_2.28-5_amd64.deb 
(--unpack):
 trying to overwrite '/usr/share/bash-completion/completions/mount', which is 
also in package bash-completion 1:2.1-4.3
Errors were encountered while processing:
 /var/cache/apt/archives/mount_2.28-5_amd64.deb
E: Sub-process /usr/bin/dpkg returned an error code (1)

-- System Information:
Debian Release: stretch/sid
  APT prefers testing
  APT policy: (500, 'testing'), (500, 'stable')
Architecture: amd64 (x86_64)

Kernel: Linux 4.4.0-1-amd64 (SMP w/4 CPU cores)
Locale: LANG=C, LC_CTYPE=C (charmap=ANSI_X3.4-1968)
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)

Versions of packages mount depends on:
ii  libblkid1  2.27.1-3.1
ii  libc6  2.22-11
ii  libmount1  2.28-5
ii  libselinux12.3-2
ii  libsmartcols1  2.28-5
ii  libudev1   230-2

mount recommends no packages.

Versions of packages mount suggests:
ii  nfs-common  1:1.2.8-9

-- no debconf information