12223 2017-10-13 19:00:06 +0000 opkg marks package as installed despite a dependency failing to install 2021-07-08 15:05:24 +0000 1 1 6 Yocto Project Subprojects opkg opkg 0.3.5 All Multiple ACCEPTED Undecided minor next-patch 1 michael.j.hansen alex.stewart --- --- No (bug/feature does not impact docs) oldest_to_newest 77573 0 michael.j.hansen 2017-10-13 19:00:06 +0000 What steps will reproduce the problem? ###################################### 1. Create package 'a', which includes conflict_file. 2. Create package 'b', which DEPENDS on 'c'. 3. Create package 'c', which also includes conflict_file. 4. Install 'a' 5. Install 'b' What is the expected output? What do you see instead? ###################################### Expected: Packages 'b' and 'c' should not be installed. Observed: Opkg correctly reports an error and non-zero exit status after attempting to install 'b'. However, 'b' is marked as installed despite its dependency 'c' not being installed. 77574 1 4080 michael.j.hansen 2017-10-13 19:01:36 +0000 Created attachment 4080 Test case for 12223 77649 2 alejandro.delcastillo 2017-10-17 22:44:07 +0000 So I tried the same case on dpkg, and this is what I got root@29d4823d742c:/tmp# apt-get install b Reading package lists... Done Building dependency tree Reading state information... Done The following additional packages will be installed: c The following NEW packages will be installed: b c 0 upgraded, 2 newly installed, 0 to remove and 0 not upgraded. Need to get 0 B/1274 B of archives. After this operation, 0 B of additional disk space will be used. Do you want to continue? [Y/n] Y WARNING: The following packages cannot be authenticated! c b Install these packages without verification? [y/N] y Get:1 file:/tmp feed/ c 1.0 [630 B] Get:2 file:/tmp feed/ b 1.0 [644 B] debconf: delaying package configuration, since apt-utils is not installed Selecting previously unselected package c. (Reading database ... 16254 files and directories currently installed.) Preparing to unpack /tmp/feed/c_1.0.deb ... Unpacking c (1.0) ... dpkg: error processing archive /tmp/feed/c_1.0.deb (--unpack): trying to overwrite '/usr/myfile', which is also in package a 1.0 Selecting previously unselected package b. Preparing to unpack /tmp/feed/b_1.0.deb ... Unpacking b (1.0) ... Errors were encountered while processing: /tmp/feed/c_1.0.deb E: Sub-process /usr/bin/dpkg returned an error code (1) Running dpkg --list: ii a 1.0 amd64 this is a iU b 1.0 amd64 this is b Which is saying that package a is fully installed, package b is in unpacked state and package c is not installed at all. In opkg, we get package a and b installed, and C half-installed, which is a state that makes sense (but not ideal). Opkg state diagram is simpler than dpkg, so this is more or less by design. An improvement over the current state would be to be able to rollback C, but that is probably going to be fairly hard. I am not opposed to opkg becoming more resilient, and more dpkg-wise, but its probably low on the priority list as its main purpose is to be a simplified version of dpkg, which means some corners are cut. but would certainly take that change in, if someone else take the task on. 4080 2017-10-13 19:01:36 +0000 2017-10-13 19:01:36 +0000 Test case for 12223 issue12223.py text/x-python 1318 michael.j.hansen IyEvdXNyL2Jpbi9lbnYgcHl0aG9uMwoKIyBXaGF0IHN0ZXBzIHdpbGwgcmVwcm9kdWNlIHRoZSBw cm9ibGVtPwojIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjCiMKIyAxLiBD cmVhdGUgcGFja2FnZSAnYScsIHdoaWNoIGluY2x1ZGVzIGNvbmZsaWN0X2ZpbGUuCiMgMi4gQ3Jl YXRlIHBhY2thZ2UgJ2InLCB3aGljaCBERVBFTkRTIG9uICdjJy4KIyAzLiBDcmVhdGUgcGFja2Fn ZSAnYycsIHdoaWNoIGluY2x1ZGVzIGNvbmZsaWN0X2ZpbGUuCiMgNC4gSW5zdGFsbCAnYScKIyA1 LiBJbnN0YWxsICdiJwojCiMgV2hhdCBpcyB0aGUgZXhwZWN0ZWQgb3V0cHV0PyBXaGF0IGRvIHlv dSBzZWUgaW5zdGVhZD8KIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIwoj IEV4cGVjdGVkOiAgUGFja2FnZXMgJ2InIGFuZCAnYycgc2hvdWxkIG5vdCBiZSBpbnN0YWxsZWQu CiMKIyBPYnNlcnZlZDogIE9wa2cgY29ycmVjdGx5IHJlcG9ydHMgYW4gZXJyb3IgYW5kIG5vbi16 ZXJvIGV4aXQgc3RhdHVzIGFmdGVyCiMgYXR0ZW1wdGluZyB0byBpbnN0YWxsICdiJy4gIEhvd2V2 ZXIsICdiJyBpcyBtYXJrZWQgYXMgaW5zdGFsbGVkIGRlc3BpdGUgaXRzCiMgZGVwZW5kZW5jeSAn Yycgbm90IGJlaW5nIGluc3RhbGxlZC4KIwoKaW1wb3J0IG9zCmltcG9ydCBvcGssIGNmZywgb3Br Z2NsCgpvcGsucmVncmVzc19pbml0KCkKCndpdGggb3BlbignY29uZmxpY3RfZmlsZScsICd3Jykg YXMgZjoKICAgIGYud3JpdGUoJ1Rlc3QnKQoKYSA9IG9way5PcGsoUGFja2FnZT0iYSIpCmEud3Jp dGUoZGF0YV9maWxlcz1bJ2NvbmZsaWN0X2ZpbGUnXSkKYiA9IG9way5PcGsoUGFja2FnZT0iYiIs IERlcGVuZHM9ImMiKQpiLndyaXRlKCkKYyA9IG9way5PcGsoUGFja2FnZT0iYyIpCmMud3JpdGUo ZGF0YV9maWxlcz1bJ2NvbmZsaWN0X2ZpbGUnXSkKCm8gPSBvcGsuT3BrR3JvdXAoKQpvLm9wa19s aXN0LmV4dGVuZChbYSwgYiwgY10pCm8ud3JpdGVfbGlzdCgpCgpvcGtnY2wudXBkYXRlKCkKCm9w a2djbC5pbnN0YWxsKCJhIikKaWYgbm90IG9wa2djbC5pc19pbnN0YWxsZWQoImEiKToKICAgIG9w ay5mYWlsKCJQYWNrYWdlICdhJyBmYWlsZWQgdG8gaW5zdGFsbCIpCgpvcGtnY2wuaW5zdGFsbCgi YiIpCmlmIG9wa2djbC5pc19pbnN0YWxsZWQoImMiKToKICAgIG9way5mYWlsKCJQYWNrYWdlICdj JyBpbnN0YWxsZWQgZGVzcGl0ZSBjb25mbGljdCIpCmlmIG9wa2djbC5pc19pbnN0YWxsZWQoImIi KToKICAgIG9way5mYWlsKCJQYWNrYWdlICdiJyBpbnN0YWxsZWQgd2l0aG91dCBkZXBlbmRlbmN5 ICdjJyIpCg==