From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga14.intel.com (mga14.intel.com [143.182.124.37]) by mail.openembedded.org (Postfix) with ESMTP id 9D93E6B19D for ; Thu, 18 Jul 2013 09:43:20 +0000 (UTC) Received: from azsmga001.ch.intel.com ([10.2.17.19]) by azsmga102.ch.intel.com with ESMTP; 18 Jul 2013 02:43:20 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.89,692,1367996400"; d="scan'208";a="333244391" Received: from unknown (HELO helios.localnet) ([10.252.122.61]) by azsmga001.ch.intel.com with ESMTP; 18 Jul 2013 02:43:19 -0700 From: Paul Eggleton To: openembedded-core@lists.openembedded.org Date: Thu, 18 Jul 2013 10:43:18 +0100 Message-ID: <3464105.VeimZdSK2P@helios> Organization: Intel Corporation User-Agent: KMail/4.10.4 (Linux/3.8.0-26-generic; KDE/4.10.4; i686; ; ) In-Reply-To: References: MIME-Version: 1.0 Subject: Re: [CONSOLIDATED PULL 00/40] Review and ACK X-BeenThere: openembedded-core@lists.openembedded.org X-Mailman-Version: 2.1.12 Precedence: list List-Id: Patches and discussions about the oe-core layer List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 18 Jul 2013 09:43:20 -0000 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" On Wednesday 17 July 2013 23:34:31 Saul Wold wrote: > Ross, Paul: > > Please review this set and ACK as appropriate, I will rebase based > on your input. This this is on the AB down, we have known issues with > systemd. This addresses many of the M2 related issues and fixes the > testimage infrasturcture on teh AutoBuilder > > Please look at Qi's "EXTRA_USER" changes > > There are a couple of patches that are on Poky, that need to be merged > one is to tiny and the other is to include the securiy flags for the LSB > images. That way they get tested with 1/2 the images built > > Sau! > > The following changes since commit cf1790d992f067be8d5f9894458f55f6f1bdc61f: > > lib/oeqa: fix dependecy check (2013-07-15 10:29:27 -0700) > > are available in the git repository at: > > git://git.openembedded.org/openembedded-core-contrib sgw/stage > > http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=sgw/ > stage > > Bruce Ashfield (3): > linux-yocto/3.8: update META srcrev > linux-yocto/3.4: mips: fix gcc 4.8 compilation > linux-yocto/3.8: restore qemumips64 SRCREV > > Chen Qi (5): > tar: remove an erroneous patch > useradd_base.bbclass: add a new bbclass > useradd.bbclass: code refactor > extrausers.bbclass: add a new bbclass I agree with Otavio, the first two patches should be squashed together. Otherwise the changes look OK. > local.conf.sample.extended: add example for EXTRA_USERS_PARAMS > > Cristian Iorga (2): > bluez4: conflicts with bluez5 > bluez5: new package for v5.7 Didn't you find these caused world build failures? > Felipe F. Tonello (3): > pulseaudio: Added cli-protocol-unix to pulseaudio-misc > valgrind: added perl dependency > pulseaudio: Removed avahi forced dependency > > Hongxu Jia (2): > udev-extraconf: allow users in disk group to write vfat filesystem > xuser-account: add xuser to disk group > > Joe Slater (1): > curl: add upstream status to patch > > Jonathan Liu (1): > gdb: add PACKAGECONFIG for Python support > > Kai Kang (1): > tcl: fix typo of upstream status in patch file > > Khem Raj (4): > gcc-4.8: Fix on-device SDK C++ runtime issue > uclibc: Build ldd/ldcondif/iconv for main uclibc recipe > eglibc-2.18: Remove unused patches > eglibc-2.18: Update to latest snapshot > > Mark Hatle (4): > sanity.bbclass: Update gcc sanity check > terminal.bbclass: Fix BB_RUNFMT processing > busybox: fix ip reference in simple.script > dbus-ptest: Disable python module check > > Paul Eggleton (4): > qt-mobility: tighten up config generation and use PACKAGECONFIG > classes/insane: fix regression in libdir QA regex > classes/insane: allow libdir QA check to be skipped using INSANE_SKIP > scripts/contrib/*: fix arithmetic bashism > > Roy.Li (1): > dpkg: backport a patch to support mipsn32el and mips64el Not sure if case is a big issue for Upstream-Status values but this one has "Upstream-Status: BackPort". > > Saul Wold (2): > Upstream-Status: Correct capitalization > buildtools: testimage needs the mmap python If we're changing other things I'd like to see the commit message corrected for the above (buildtools -> buildtools-tarball and python -> python module; also the commit message should go into a little bit more detail e.g. "Add python-mmap which is needed by the new testimage class on machines that need buildtools-tarball installed". > Simon Busch (1): > wpa-supplicant: backport patch to fix dbus interface for autoscan > functionality > > Stefan Stanacar (1): > populate_sdk_base.bbclass: use new perm option for find > > Tudor Florea (1): > python: Install Python test suite and run it as ptest. The run-ptest script here has hardcoded /usr/lib/ for ${libdir}, we should ideally avoid this (e.g. through use of sed at do_install). Also I'd like to know if the ptest package could simply depend on ${PN}-modules rather than having a hardcoded list. I'll follow up with the submitter. > Yi Zhao (3): > iproute2: Fix alternative link for ip command > run-postinsts: Fix typo > rpm-postinsts.bb: Fix typo > > Zhenhua Luo (1): > oprofile: add Upstream-Status for several patches Cheers, Paul -- Paul Eggleton Intel Open Source Technology Centre