This repository has been archived by the owner on Feb 23, 2021. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 75
building out documentation and content resources #47
Open
indie
wants to merge
667
commits into
intel:master
Choose a base branch
from
indie:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Test that filter backup module files (files starting with ~) was accidentally reversed in e6039e6e3b98d6ab91252a5012d76279b1fac6e8, this patch restore initial behavior. (From OE-Core master rev: b2eb846ee12989add7a7ca8bbf45f293a3a7e56d) (From OE-Core rev: 00ff958fec53e55cc475c1b31fb9813d97872ceb) Signed-off-by: Michaël Burtin <[email protected]> Signed-off-by: Richard Purdie <[email protected]> Signed-off-by: Robert Yang <[email protected]> Signed-off-by: Richard Purdie <[email protected]>
midori depends on webkit-gtk which could not build for mips64. [YOCTO #5141] (From OE-Core master rev: abadeb934d4f41288c4fde6a4e5df2b124326326) (From OE-Core rev: 672cd50a39a697b53c337a79c34fab05b48e0920) Signed-off-by: Jackie Huang <[email protected]> Signed-off-by: Richard Purdie <[email protected]> Signed-off-by: Robert Yang <[email protected]> Signed-off-by: Richard Purdie <[email protected]>
The PACKAGECONFIG flags were iterated over using dict.items(), but this returns the items in an undefined order. As this order determines the EXTRA_OECONF append order, we can get EXTRA_OECONF which are functionally equivalent, but whose contents differ, resulting in not using shared state archives we should be using. (From OE-Core master rev: 843a5dd8f8f0461e286d9fdb3ba55205b4275f88) (From OE-Core rev: 73f77c195e1af3df594eecce2cab47ee963d5c2e) Signed-off-by: Christopher Larson <[email protected]> Signed-off-by: Richard Purdie <[email protected]> Signed-off-by: Robert Yang <[email protected]> Signed-off-by: Richard Purdie <[email protected]>
mDNS name resolution is a key part of mDNS, so if the DISTRO_FEATURE is enabled then install libnss-mdns. (From OE-Core master rev: ef2ee68778be8e5336cd33ab6551bce1d56047b6) (From OE-Core rev: fb72bebc999aef7bb006a7ba273eaa96376a4016) Signed-off-by: Ross Burton <[email protected]> Signed-off-by: Saul Wold <[email protected]> Signed-off-by: Richard Purdie <[email protected]> Signed-off-by: Robert Yang <[email protected]> Signed-off-by: Richard Purdie <[email protected]>
If the same username exists on both target and the build host, but the uids differ, and we start target via NFS, then the uid for the user will be incorrect on target. For example, if postfix's uid on host is 119 and on target is 1024, then if we start target via NFS, the uid for postfix will be 119. The root cause is that when we use runqemu-extract-sdk to generate the NFS rootfs for later use, the tar command will respect the username instead of uid. So if PSEUDO_PASSWD environment is not set correctly, the host /etc/passwd will be used, resulting in wrong uids. The situation for gid is completely analogous to that of uid. It's almost impossible for the runqemu-extract-sdk to guess the correct location of the needed password file merely based on the target tarball name. This patch solves this problem by adding the '--numeric-owner' option to the tar command so that the uid/gid will be used when extracting the tarball using runqemu-extract-sdk. In this situation, we'll always get the correct uid/gid after extracting the tarball. [YOCTO #5364] (From OE-Core master rev: acce6ff1a77cfd29e3868faa89b120becb58bbbf) (From OE-Core rev: c2baac739a521d1edd408a24f6b1fece8f755218) Signed-off-by: Chen Qi <[email protected]> Signed-off-by: Saul Wold <[email protected]> Signed-off-by: Richard Purdie <[email protected]> Signed-off-by: Robert Yang <[email protected]> Signed-off-by: Richard Purdie <[email protected]>
Python 2.4 does not support the 'b' string literal or the keyword 'as' in exception handling. Python 3 does not accept the old method of exception handling and defaults to unicode. The b() function converts strings to bytes on Python 3 and using sys.exc_info() avoids the exception handling syntax. (From OE-Core master rev: 1e2ec5f576f167673d7980737826987fefdc74a9) (From OE-Core rev: 343127f2f81be337596d3eacbbc92278e82ce574) Signed-off-by: Konrad Scherer <[email protected]> Signed-off-by: Saul Wold <[email protected]> Signed-off-by: Richard Purdie <[email protected]> Signed-off-by: Robert Yang <[email protected]> Signed-off-by: Richard Purdie <[email protected]>
As the same reason to powerpc64, mips64 also need the flag. (From OE-Core master rev: d6f3cb0d71c3b6739365f085b6d5a5e20f329fa5) (From OE-Core rev: 9c4b604ea0d81bc1de224b35ae160f87be6bcf7b) Signed-off-by: Wenzong Fan <[email protected]> Signed-off-by: Richard Purdie <[email protected]> Signed-off-by: Robert Yang <[email protected]> Signed-off-by: Richard Purdie <[email protected]>
An incorrect process name in the nfsserver initscript prevented rpc.statd from being shut down. root@qemux86-64:~# /etc/init.d/nfsserver start creating NFS state directory: done starting 8 nfsd kernel threads: done starting mountd: done starting statd: done root@qemux86-64:~# ps | grep rpc.statd 650 root 10532 S /usr/sbin/rpc.statd 654 root 4720 S grep rpc.statd root@qemux86-64:~# /etc/init.d/nfsserver stop stopping statd: done stopping mountd: done stopping nfsd: done root@qemux86-64:~# ps | grep rpc.statd 650 root 10532 S /usr/sbin/rpc.statd 662 root 4720 S grep rpc.statd As this daemon drops a pid file,simply use that instead. Also add some initialization checks so the daemons are not left partially started in the absence of kernel nfsd support. (From OE-Core master rev: 37e70a28e9cfc773bd70f09d7129295ce891ae18) (From OE-Core rev: 5f22bad97a3bacb87cefb54ffd785d359c58aec0) Signed-off-by: Andy Ross <[email protected]> Signed-off-by: Qiang Chen <[email protected]> Signed-off-by: Richard Purdie <[email protected]> Signed-off-by: Robert Yang <[email protected]> Signed-off-by: Richard Purdie <[email protected]>
…eads nfsserver restart without killing kernel threads worked when portmap was the rpc publishing process and portmap was restarted. When rpcbind replaces portmap, nfsserver restart in this way does not work after an rpcbind restart. Steps to reproduce: 1). Make ext3 filesystem image on local host. cd /root dd if=/dev/zero of=test bs=1024K count=50 mkfs.ext3 -F test 2). runqemu qemux86-64 mkdir /mnt/wrtest mount -t ext3 -o loop test /mnt/wrtest echo "/mnt/wrtest *(sync,rw,no_subtree_check,no_root_squash)" > /etc/exports /etc/init.d/rpcbind restart /etc/init.d/nfsserver restart showmount -e localhost mkdir wrtest mount -t nfs localhost:/mnt/wrtest wrtest mount: mounting localhost:/mnt/wrtest on wrtest failed: Connection refused Modifying the nfsserver script to kill and restart kernel threads on restart makes the problem go away and is consistent with current RHEL/SUSE and Ubuntu/Debian mechanisms of handling the nfs server. (From OE-Core master rev: 1a96b8d7dfc490fc61bbd470a8b09065750cd563) (From OE-Core rev: d1b5e944656807c9db9cbe5d08d7b4bd8daeb826) Signed-off-by: Rich Dubielzig <[email protected]> Signed-off-by: Qiang Chen <[email protected]> Signed-off-by: Richard Purdie <[email protected]> Signed-off-by: Robert Yang <[email protected]> Signed-off-by: Richard Purdie <[email protected]>
(From OE-Core master rev: 89d7d46947d9bb8c7bf568c65e52d5bbe159027f) (From OE-Core rev: 7c6504c6c059ba6b37f88143801ac8137267cf83) Signed-off-by: Konrad Scherer <[email protected]> Signed-off-by: Richard Purdie <[email protected]> Signed-off-by: Robert Yang <[email protected]> Signed-off-by: Richard Purdie <[email protected]>
When installing the SDK to another location than the default one, qmake will look for libraries, headers, etc. in the default location. That's because the paths are hard-coded in the binary itself. Luckily, QT allows to override this using a qt.conf file installed in the same directory with the application executable. However, we already have a patch that allows for the installation of qt.conf in another place and read the location from QT_CONF_PATH environment variable. Hence, install qt.conf in ${sysconfdir}. This will allow other apps, that use QLibraryInfo class, to find it. [YOCTO #5339] (From OE-Core master rev: 23f88695683a8e428375a8ccb6be935347a8768c) (From OE-Core rev: 0e71811a1c3285a71cbca682cb62c1563d3e74ee) Signed-off-by: Laurentiu Palcu <[email protected]> Signed-off-by: Richard Purdie <[email protected]> Signed-off-by: Robert Yang <[email protected]> Signed-off-by: Richard Purdie <[email protected]>
This will allow apps using QLibraryInfo class to find qt.conf. [YOCTO #5339] (From OE-Core master rev: fffa4c37c49b169f663d28612b9251819cef9577) (From OE-Core rev: 8dc1d62c5dc161ba606cebe27f6fe900699646f7) Signed-off-by: Laurentiu Palcu <[email protected]> Signed-off-by: Richard Purdie <[email protected]> Signed-off-by: Robert Yang <[email protected]> Signed-off-by: Richard Purdie <[email protected]>
There is no need for += when using append hence removed and added a leading space appropriately (From OE-Core master rev: fb9cde0fc1a54b073edf5979f4cb7dc297b790fd) (From OE-Core rev: 586db07af01da9d7772b7088a20886b506e09422) Signed-off-by: Khem Raj <[email protected]> Signed-off-by: Richard Purdie <[email protected]> Signed-off-by: Robert Yang <[email protected]> Signed-off-by: Richard Purdie <[email protected]>
libpseudo.so is always installed into ${prefix}/lib/, not ${libdir}, so fix these paths; and skip libdir WARN_QA checking to ignore the warning in 64bit and multilib enabled system (From OE-Core master rev: 47c7850c025994685aa1811057f4f9a5f0f2a3ae) (From OE-Core rev: 1929d4ef17652a3eb825942041908d679773244f) Signed-off-by: Roy Li <[email protected]> Signed-off-by: Richard Purdie <[email protected]> Signed-off-by: Robert Yang <[email protected]> Signed-off-by: Richard Purdie <[email protected]>
- Wayland support depends on wayland-egl, which is provided by mesa. (From OE-Core master rev: a1a379b3c9728a06b086b4c1f06f663f54d7d37d) (From OE-Core rev: 8c75d888a5e4cf7fc2c92df730d80224f5ffa99a) Signed-off-by: Andreas Oberritter <[email protected]> Signed-off-by: Saul Wold <[email protected]> Signed-off-by: Richard Purdie <[email protected]> Signed-off-by: Robert Yang <[email protected]> Signed-off-by: Richard Purdie <[email protected]>
While ELF_C_RDWR_MMAP was used, elf_begin invoked mmap() to map file into memory. While the file's bss Offset has a large number, elf_update caculated file size by __elf64_updatenull_wrlock and the size was enlarged. In this situation, elf_update invoked ftruncate to enlarge the file, and memory size (elf->maximum_size) also was incorrectly updated. There was segment fault in elf_end which invoked munmap with the length is the enlarged file size, not the mmap's length. Before the above operations, invoke elf_begin/elf_update/elf_end with ELF_C_RDWR and ELF_F_LAYOUT set to enlarge the above file, it could make sure the file is safe for the following elf operations. [YOCTO #5356] https://bugzilla.redhat.com/show_bug.cgi?id=1019707 https://bugzilla.redhat.com/show_bug.cgi?id=1020842 (From OE-Core master rev: 35c8b1ac7c3b1e4209b1e30d1dbd1a457286b97b) (From OE-Core rev: a82322a982dc97ebc95f3fc45f9ad98bed947ad9) Signed-off-by: Hongxu Jia <[email protected]> Signed-off-by: Saul Wold <[email protected]> Signed-off-by: Richard Purdie <[email protected]> Signed-off-by: Robert Yang <[email protected]> Signed-off-by: Richard Purdie <[email protected]>
When using the tar executable in the buildtools, tar will execute gzip. If this happens before zlib-native is built, then the gzip on the host will be used and can fail if the libz in the buildtools is not compatible. Adding pigz to the build tools avoids this host contamination. (From OE-Core master rev: af6424e8c2bf3a938fddabc669c0956d68964ed0) (From OE-Core rev: dd9945dd510d6e7764721bec5573591a0ad69ba4) Signed-off-by: Konrad Scherer <[email protected]> Signed-off-by: Saul Wold <[email protected]> Signed-off-by: Richard Purdie <[email protected]> Signed-off-by: Robert Yang <[email protected]> Signed-off-by: Richard Purdie <[email protected]>
This seems to be an obsolete check - we don't have any problems with image creation under selinux, so remove it. (From OE-Core master rev: 12e81eceab9e0a483765566ad3791b14718195b5) (From OE-Core rev: 28830d3988047023d3b47bcaf320f5efa4428da6) Signed-off-by: Tom Zanussi <[email protected]> Signed-off-by: Saul Wold <[email protected]> Signed-off-by: Richard Purdie <[email protected]> Signed-off-by: Robert Yang <[email protected]> Signed-off-by: Richard Purdie <[email protected]>
(From OE-Core master rev: 3af8f2e0697a9523d3b505ba4c48eca35f6de3a9) (From OE-Core rev: 438032411ea5d71a33b7b030752193867a90b9f7) Signed-off-by: Jonathan Liu <[email protected]> Signed-off-by: Saul Wold <[email protected]> Signed-off-by: Richard Purdie <[email protected]> Signed-off-by: Robert Yang <[email protected]> Signed-off-by: Richard Purdie <[email protected]>
ldd sysroots/x86_64-linux/usr/bin/gdk-pixbuf-query-loaders.real <snip> libz.so.1 => /sysroots/x86_64-linux/usr/bin/../../usr/lib/libz.so.1 (0x00007fab55393000) If zlib-native has not been unpacked, host libz is used which can fail. (From OE-Core master rev: 8422c759ae674856aaaee176eab5a395a620443c) (From OE-Core rev: b9ae15b45768d25c44a9484b4a156a15da548bd9) Signed-off-by: Konrad Scherer <[email protected]> Signed-off-by: Saul Wold <[email protected]> Signed-off-by: Richard Purdie <[email protected]> Signed-off-by: Robert Yang <[email protected]> Signed-off-by: Richard Purdie <[email protected]>
(From OE-Core master rev: 9032c10cc882a96acdfd0739f090d121ab625a18) (From OE-Core rev: c191cb79019482a5c6a404e02184bae40ff9f84a) Signed-off-by: Roy Li <[email protected]> Signed-off-by: Saul Wold <[email protected]> Signed-off-by: Richard Purdie <[email protected]> Signed-off-by: Robert Yang <[email protected]> Signed-off-by: Richard Purdie <[email protected]>
/etc/init.d/dbus-1 use "set -e" to let the script exit when any command failes. This will cause "/etc/init.d/dbus-1 status" command can't display messages when dbus is stopped. (From OE-Core master rev: 9844b5e2a544b2c2f76aac497c3a2cdfcc46577c) (From OE-Core rev: 486d5d7e891df3fb2ce8d975d13625b11334814b) Signed-off-by: Lu Chong <[email protected]> Signed-off-by: Saul Wold <[email protected]> Signed-off-by: Richard Purdie <[email protected]> Signed-off-by: Robert Yang <[email protected]> Signed-off-by: Richard Purdie <[email protected]>
* Add the openssl-conf package to the list of packages to be created. This package contains the openssl.cnf file which is used by both the openssl executable in the openssl package and the libcrypto library. * This is to avoid messages like: WARNING: can't open config file: /usr/lib/ssl/openssl.cnf * When running "openssl req" to request and generate a certificate the command will fail without the openssl.cnf file being installed on the target system. * Made this package an RRECOMMENDS for libcrypto since: * libcrypto is a RDEPENDS for the openssl package * Users can specify a configuration file at another location so it is not stricly required and many commands will work without it (with warnings) (From OE-Core master rev: 5c3ec044838e23539f9fe4cc74da4db2e5b59166) (From OE-Core rev: bf6ef555caf92b2a013f15d258bf40997247a150) Signed-off-by: Chase Maupin <[email protected]> Signed-off-by: Qiang Chen <[email protected]> Signed-off-by: Saul Wold <[email protected]> Signed-off-by: Richard Purdie <[email protected]> Signed-off-by: Robert Yang <[email protected]> Signed-off-by: Richard Purdie <[email protected]>
Allow the user to provide additional packages to this image. This lets core-image-basic behave like all other core-image* recipes (which do support CORE_IMAGE_EXTRA_INSTALL), as well as match the documentation which suggests this as the mode to extend any core-image* image. v2 - drop redundant setting of CORE_IMAGE_EXTRA_INSTALL (From OE-Core master rev: 5faabf398819d40b55c46bc83ae03942d115024b) (From OE-Core rev: cdda59d2850b163ced4dfc847c4e114f8592ae52) Signed-off-by: Gary Thomas <[email protected]> Signed-off-by: Richard Purdie <[email protected]> Signed-off-by: Robert Yang <[email protected]> Signed-off-by: Richard Purdie <[email protected]>
BitBake currently allows using the same quotes outside and inside the value, but it isn't really right, looks odd and might stop working in future. (From OE-Core master rev: 0af9cf31851896276a219170001047406f45de50) (From OE-Core rev: 4051c98ebfe79614d1284b38442d5a3290bb4ad1) Signed-off-by: Paul Eggleton <[email protected]> Signed-off-by: Richard Purdie <[email protected]> Signed-off-by: Robert Yang <[email protected]> Signed-off-by: Richard Purdie <[email protected]>
in commit fe039170236080291c0220476a5809774f82ee5c Author: Muhammad Shakeel <[email protected]> Date: Wed Oct 2 10:55:32 2013 +0000 systemd-compat-units: Use correct run-postinsts script link OE-Core commit 75a1492 has moved run-postinsts script execution from S98 to S99 in rcS.d. run-postinsts.service should check for this script and run it on first boot rather than S98run-postinsts, which is for opkg/dpkg. the link was corrected but the mentioned commit is not available. Instead of reverting, we use the same variable as opkg for init script ordering and drop a note in case somebody wants to change default. (From OE-Core master rev: 7aabc9408fb382f0ae39f9932b6d9ac391528b76) (From OE-Core rev: 91a51e10392c0f802d6c329ad2b11ae82c55f171) Signed-off-by: Andreas Müller <[email protected]> Signed-off-by: Saul Wold <[email protected]> Signed-off-by: Richard Purdie <[email protected]> Signed-off-by: Robert Yang <[email protected]> Signed-off-by: Richard Purdie <[email protected]>
PowerPC toolchains can use the OS "linux" or "linux-gnuspe". This patch links them together so the one cross-canadian toolchain can support both. GCC_FOR_TARGET is set for the GCC recipe as otherwise configure can pick up an incorrect value. [YOCTO #5354] (From OE-Core master rev: a1d6331238982b0c5d39b0a18794f6654b00d46a) (From OE-Core rev: b114b045687776ebc5c805e1f19758e7e37eebf2) Signed-off-by: Richard Purdie <[email protected]> Signed-off-by: Saul Wold <[email protected]> Signed-off-by: Richard Purdie <[email protected]> Signed-off-by: Robert Yang <[email protected]> Signed-off-by: Richard Purdie <[email protected]>
This class points the inheritor, if it is a target, to directories in the target sysroot, so we want to be sure the .vapi files are there. (From OE-Core master rev: 2da8bbd47686f54efeec521d521f176f6aeb8d39) (From OE-Core rev: e68307c3fb0a02efe5e0789a58686f343c842707) Signed-off-by: Joe Slater <[email protected]> Signed-off-by: Saul Wold <[email protected]> Signed-off-by: Richard Purdie <[email protected]> Signed-off-by: Robert Yang <[email protected]> Signed-off-by: Richard Purdie <[email protected]>
Without this patch filenames containing spaces do not get into the final ext2/3/4 filsystem. [YOCTO #5401] (From OE-Core master rev: 1350b461ed0c9d4afa1ab909a5b1ff60fb160c97) (From OE-Core rev: 62d01b10508f86ca825ebc24773dfa2b485b4292) Signed-off-by: Saul Wold <[email protected]> Signed-off-by: Richard Purdie <[email protected]> Signed-off-by: Robert Yang <[email protected]> Signed-off-by: Richard Purdie <[email protected]>
The vtX terminfo files aren't being copied on systems where bash isn't the default shell (debian, etc.). I removed the bash specific syntax so the files are properly copied on these systems. (From OE-Core master rev: edd7d53c6149b27d5636a458db91650c8c400612) (From OE-Core rev: ccb25cd9df9ed4b84b74170336cfde5c1dc3d013) Signed-off-by: Seth Bollinger <[email protected]> Signed-off-by: Saul Wold <[email protected]> Signed-off-by: Richard Purdie <[email protected]> Signed-off-by: Robert Yang <[email protected]> Signed-off-by: Richard Purdie <[email protected]>
Provide a luv splash screen for grub's use, which informs the user that we're currently running boot tests. Signed-off-by: Matt Fleming <[email protected]>
For luvOS we need to install more than one boot loader, plus a variety of files to do boot time tests. Roll our own class which gives us much more control over the installation process and creation of the grub config. Signed-off-by: Matt Fleming <[email protected]>
There are tools such as sbsigntool that require a native version gnu-efi. Add support for building gnu-efi-native. Cc: Stefan Stanacar <[email protected]> Signed-off-by: Matt Fleming <[email protected]>
sbsigntool is a useful utility for cryptographically signing binaries for use in UEFI systems. Signed-off-by: Matt Fleming <[email protected]>
The tool cannot be built without this patch. We don't really need the man pages in this project. Signed-off-by: Ricardo Neri <[email protected]>
LuvOS is intended to be an automated testing tool. Thus, it does really require much interaction from the user other than inserting the bootable media, and removing it when tests are complete. Thus, we inform better to the user what to do when the tests are complete. Signed-off-by: Ricardo Neri <[email protected]>
Rather than hardcoding the package version, utilize the PV variable to define the configure path. This will prevent build issues when moving to a newer version. Signed-off-by: Ricardo Neri <[email protected]>
1090 is the lastest release of the BIOS Implementation Test Suite (BITS). These are the changes as per BITS relase notes: *Update to ACPICA version 20140325 *Backport GRUB2 commit 4e42521d8c9232b6ee9eac7d8b4945a7479de781 to preserve 16-byte alignment of the stack on EFI calls * python: Support the csv module Signed-off-by: Ricardo Neri <[email protected]>
This patch was merged in V14.07.00 of fwts but we're currently on V14.03.01. We're seeing microcode test failures on a number of Haswell machines, which shouldn't really be failures. This patch from upstream fixes that. Signed-off-by: Matt Fleming <[email protected]>
A recent fix to change a securebootcert failure into an info message was merged into upstream fwts. We're hitting this false positive on a number of machines. Signed-off-by: Matt Fleming <[email protected]>
We've no need to turn a data structure on the stack into a pointer and use get_user() on a kernel address, and in fact, doing so is likely to return -EFAULT, because the argument isn't a valid user address. This bug was introduced in commit c2268a9 ("fwts: Copied the structure from userland locally in kernel space"), and causes a uefirttime failure. Cc: Pradeep Gaddam <[email protected]> Signed-off-by: Matt Fleming <[email protected]>
Things don't work across the board right now, anyhow. People have even experienced hangs, and there's no way we can diagnose those hangs without displaying the test log on the screen. It's not ideal, and going forward we definitely need to satisfy both requirements (easy debugging and pretty graphics), but if need to pick one or the other, easy debugging wins every time. Signed-off-by: Matt Fleming <[email protected]>
Providing verbose kernel output on the serial console during boot is a useful way to debug issues. It also provides a much more informative message of what is currently happening. Enable both the standard ttyS0 device and ttyPCH0 which is the device used on the Intel Minnowboard. Tested-by: Gayatri Kammela <[email protected]> Tested-by: Ricardo Neri <[email protected]> Signed-off-by: Matt Fleming <[email protected]>
There's some latency somewhere in the test result pipeline, and it's impossible to trace noticable hangs when writing test results to the console to the offending unit test. Just simplify the pipeline, and pipe the result output directly to the console and results files instead of passing it through another instance of gawk before it hits the console, since hunting down buffer-related delays in gawk is extremely tedious. There's no user-visible change with this patch - it's preparatory work for a later patch that aggressively flushes the gawk output buffer. Tested-by: Gayatri Kammela <[email protected]> Tested-by: Ricardo Neri <[email protected]> Signed-off-by: Matt Fleming <[email protected]>
The internal buffering that gawk does makes the test output pretty useless, because it's not always possible to tell which test is currently running. For example, a test may have completed but the output will not appear on the screen until the output buffer fills and is subsequently flushed. Effectively all the unit test results from a single test suite are output as one block. The wakealarm test from fwts provides a good illustration of the user-visible problem. This test takes a number of seconds to complete, but because all the fwts results are output in one go, it's not possible to attribute delays to any one individual unit test. Explicitly flush all open file descriptors and pipes anytime that we print something from gawk. This gives much better user interaction when looking at the serial console because it's now possible to figure out which tests have the longest latencies. Whenever a unit test begins execution a message will be printed on the serial console immediately, e.g. [+] wakealarm... and when it finishes (in this case after multiple seconds) the result will be printed too, [+] wakealarm... passed Tested-by: Gayatri Kammela <[email protected]> Tested-by: Ricardo Neri <[email protected]> Signed-off-by: Matt Fleming <[email protected]>
As more LUV versions are released and the user base grows, it is important to know which particular version of LUV a given user is running. Knowing the version makes it easier to provide support and comments regarding bugs and supported features. Containing a summary of all the tests, luv.results is a good place to print the LUV version. The version is pulled from the /etc/issue file, which is updated with every release. While here, update also the welcome message in the console to depict the LUV version. Signed-off-by: Ricardo Neri <[email protected]> Signed-off-by: Matt Fleming <[email protected]>
Signed-off-by: Ricardo Neri <[email protected]> Signed-off-by: Matt Fleming <[email protected]>
Windows will not mount and display more than one partition on a removable disk. Make the results partiion the first partition on the image so that users are able to easily find the test results. Having this one partition restriction isn't a problem otherwise, since there's little value in auto-mounting the other partition (the EFI System Partition) anyway. Of course, we're assuming firmware is smart enough to find the EFI System Partition based on the partition table. Fixes issue intel#23. Reported-by: William Leara <[email protected]> Debugged-by: Ricardo Neri <[email protected]> Tested-by: Gayatri Kammela <[email protected]> Signed-off-by: Matt Fleming <[email protected]>
If no file system type is specified when creating partitions using parted (as is currently the case in the luv-live-image recipe) a default value is used, which happens to be 83 (Linux). Brian reports that this is causing the luv-results partition to not be mounted and displayed correctly on his Windows 7 machine, and that some of the Windows disk management tools are getting kinda confused. Specify "fat32" when creating partitions, so that the partition label matches reality. Reported-by: Brian Richardson <[email protected]> Signed-off-by: Matt Fleming <[email protected]>
Signed-off-by: Ricardo Neri <[email protected]> Signed-off-by: Matt Fleming <[email protected]>
meghadey
pushed a commit
that referenced
this pull request
Mar 1, 2018
Changes in DBI 1.638: Fix UTF-8 support for warn/croak calls within DBI internals, thanks to pali #53 Fix dependency on Storable for perl older than 5.8.9, thanks to H.Merijn Brand. Add DBD::Mem driver, a pure-perl in-memory driver using DBI::DBD::SqlEngine, thanks to Jens Rehsack #42 Corrected missing semicolon in example in documentation, thanks to pali #55 Changes in DBI 1.637 - 16th August 2017: Fix use of externally controlled format string (CWE-134) thanks to pali #44 This could cause a crash if, for example, a db error contained a %. https://cwe.mitre.org/data/definitions/134.html Fix extension detection for DBD::File related drivers Fix tests for perl without dot in @inc RT#120443 Fix loss of error message on parent handle, thanks to charsbar #34 Fix disappearing $_ inside callbacks, thanks to robschaber #47 Fix dependency on Storable for perl older than 5.8.9 Allow objects to be used as passwords without throwing an error, thanks to demerphq #40 Allow $sth NAME_* attributes to be set from Perl code, re #45 Added support for DBD::XMLSimple thanks to nigelhorne #38 Documentation updates: Improve examples using eval to be more correct, thanks to pali #39 Add cautionary note to prepare_cached docs re refs in %attr #46 Small POD changes (Getting Help -> Online) thanks to openstrike #33 Adds links to more module names and fix typo, thanks to oalders #43 Typo fix thanks to bor #37 Signed-off-by: Tim Orling <[email protected]> Signed-off-by: Armin Kuster <[email protected]>
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.