Hello Kari,

I tried your patch sent to #170825.

I think it deals with 217862 and 270545.

I'm not sure it fixes 20471 (I'm not sure I tried the right sequence).
And it does not deal with the use case described at the beginning of
#170825.

170825 is about a virtual package:
A_1 Provides: A-1
A_2 Provides: A-2
C Depends: A-1

The sequence:
        dpkg -i A_1
        dpkg -i C
        dpkg -i A_2
should fail when installing A_2

(In this case, A_2 has no pkg->installed.depended, so your code is not
even executed)

I'm attaching equivs control files in case you want to build packages for
testing (build them with equivs-build <control_file>).

In your patch, you should also change the internerr message (in case a user
fall there, it may be interresting to have a message different from the
previous one).

Also, I don't understand the "deferred installs do not apply here".
We are going into this code path when --force-depends is used.

Thanks for taking care of this bug anyway.
I will run with this patch, and will inform you if I find any side effect.

Kind Regards,
-- 
Nekral
Section: misc
Priority: optional
Standards-Version: 3.6.2

Package: test-dpkg-170825-a-virt
Version: 1
Provides: test-dpkg-170825-a-1
Description: short description
 long description
Section: misc
Priority: optional
Standards-Version: 3.6.2

Package: test-dpkg-170825-a-virt
Version: 2
Provides: test-dpkg-170825-a-2
Description: short description
 long description
Section: misc
Priority: optional
Standards-Version: 3.6.2

Package: test-dpkg-170825-c-virt
Version: 1
Depends: test-dpkg-170825-a-1
Description: short description
 long description

Reply via email to