summaryrefslogtreecommitdiffstats
path: root/meta/lib
diff options
context:
space:
mode:
authorPaul Barker <paul@paulbarker.me.uk>2014-10-26 19:36:22 +0000
committerRichard Purdie <richard.purdie@linuxfoundation.org>2014-11-04 10:27:10 +0000
commit365947e9d1ed5b32ab81ee8e7a23872a0bb4b169 (patch)
treeb1fef37b04e8560fb40b8f0c289846c2b46a8615 /meta/lib
parent9e07fbbd6f6bc6517c52e54e7118bc9c73f3ffc1 (diff)
downloadpoky-365947e9d1ed5b32ab81ee8e7a23872a0bb4b169.tar.gz
package_manager: Fix BAD_RECOMMENDATIONS for opkg
In package_manager.py, when using opkg as the packager, the command 'opkg <args> info <pkg>' is called to get information about each pkg in BAD_RECOMMENDATIONS in a format that can be written to the status file. The 'Status: ...' line is modified and all other lines are passed through. Changing the verbosity level argument for this command will change what it written into the status file. Crucially, with the default verbosity level, no blank lines are being printed by the opkg command and so no blank lines are being written to the status file to separate each package entry. The package parsing code in opkg expects package entries in the status file to be separated by at least one blank line. If no blank line is seen, the next package entry is interpreted as a continuation of the last package entry, but the new values overwrite the old values. So with the default verbosity level, a blank line follows some package entries and these are parsed. The others are dropped due to the lack of blank lines. As the verbosity increases, more debugging messages add blank lines and more packages are parsed. The solution to ensure that this works correctly regardless of the verbosity level is simply add a blank line after the output of 'opkg info' is written to the status file, ensuring that the next package is separated from the current package. [YOCTO #6816] (From OE-Core rev: 3fa24eee41c26fecd5e4f680082288ec772d2de9) Signed-off-by: Paul Barker <paul@paulbarker.me.uk> Cc: Chris Carr <chris.carr@ge.com> Signed-off-by: Ross Burton <ross.burton@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta/lib')
-rw-r--r--meta/lib/oe/package_manager.py4
1 files changed, 4 insertions, 0 deletions
diff --git a/meta/lib/oe/package_manager.py b/meta/lib/oe/package_manager.py
index e1ec75e52b..1af5c86f58 100644
--- a/meta/lib/oe/package_manager.py
+++ b/meta/lib/oe/package_manager.py
@@ -1411,6 +1411,10 @@ class OpkgPM(PackageManager):
1411 else: 1411 else:
1412 status.write(line + "\n") 1412 status.write(line + "\n")
1413 1413
1414 # Append a blank line after each package entry to ensure that it
1415 # is separated from the following entry
1416 status.write("\n")
1417
1414 ''' 1418 '''
1415 The following function dummy installs pkgs and returns the log of output. 1419 The following function dummy installs pkgs and returns the log of output.
1416 ''' 1420 '''