Re: [OE-core] [PATCH 1/1] sysvinit: modify bootlog location

2013-05-13 Thread ChenQi
On 05/14/2013 06:36 AM, Slater, Joseph wrote: -Original Message- From: Saul Wold [mailto:s...@linux.intel.com] Sent: Friday, May 10, 2013 3:00 PM To: Burton, Ross; Slater, Joseph Cc: openembedded-core@lists.openembedded.org Subject: Re: [OE-core] [PATCH 1/1] sysvinit: modify bootlog loc

Re: [OE-core] [PATCH 1/4] image.bbclass: add postinst_enable_logging

2013-05-16 Thread ChenQi
On 05/17/2013 02:32 AM, Saul Wold wrote: On 05/16/2013 10:56 AM, qi.c...@windriver.com wrote: From: Chen Qi Add a function postinst_enable_logging, so that when 'debug-tweaks' is in IMAGE_FEATURES, we create .enable_postinst_logging under /etc, which is used by run-postinst scripts to determin

Re: [OE-core] [PATCH V2 2/4] dpkg: modify the run-postinst script to enable postinst logging

2013-05-20 Thread ChenQi
On 05/21/2013 12:09 AM, Paul Eggleton wrote: On Monday 20 May 2013 10:50:29 qi.c...@windriver.com wrote: From: Chen Qi Enable postinst logging by checking the configuration in /etc/ default/postinst. In this way, the postinst logging is enabled if 'debug-tweaks' is in IMAGE_FEATURES, and at t

Re: [OE-core] [PATCH V2 4/4] rpm-postinsts.bb: enable postinst logging

2013-05-20 Thread ChenQi
On 05/20/2013 11:38 PM, Saul Wold wrote: On 05/19/2013 07:50 PM, qi.c...@windriver.com wrote: From: Chen Qi Enable postinst logging by checking the configuration in /etc/ default/postinst. In this way, the postinst logging is enabled if 'debug-tweaks' is in IMAGE_FEATURES, and at the same tim

Re: [OE-core] [PATCH 0/2] switch to using busybox instead of tinylogin

2013-05-27 Thread ChenQi
ex packages for docs (2013-05-22 17:23:41 +0100) are available in the git repository at: git://git.pokylinux.org/poky-contrib <http://git.pokylinux.org/poky-contrib> ChenQi/tinylogin-busybox http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=ChenQi/tinylogin-bu

Re: [OE-core] [PATCH 1/1] busybox: fix to support FEATURE_INDIVIDUAL

2013-06-02 Thread ChenQi
On 05/30/2013 08:26 PM, Saul Wold wrote: On 05/30/2013 05:51 PM, qi.c...@windriver.com wrote: From: Chen Qi Previously, if CONFIG_FEATURE_INDIVIDUAL was enabled for busybox, yocto-based systems could start correctly. This is because if busybox is built as individual apps, '/bin/busybox' may n

Re: [OE-core] [PATCH 1/1] busybox: fix to support FEATURE_INDIVIDUAL

2013-06-02 Thread ChenQi
On 06/03/2013 12:10 PM, Saul Wold wrote: On 06/02/2013 07:19 PM, ChenQi wrote: On 05/30/2013 08:26 PM, Saul Wold wrote: On 05/30/2013 05:51 PM, qi.c...@windriver.com wrote: From: Chen Qi Previously, if CONFIG_FEATURE_INDIVIDUAL was enabled for busybox, yocto-based systems could start

Re: [OE-core] [PATCH 1/8] busybox: remove the postinst part of the recipe

2013-06-07 Thread ChenQi
On 06/07/2013 08:32 PM, Otavio Salvador wrote: On Fri, Jun 7, 2013 at 3:13 AM, > wrote: From: Chen Qi mailto:qi.c...@windriver.com>> Remove the pkg_postinst_${PN} from this recipe, as it's redundant. It basically wants to do the same thing as the up

Re: [OE-core] [PATCH 4/8] busybox: add the ability to split the busybox binary

2013-06-12 Thread ChenQi
On 06/12/2013 04:26 AM, Bernhard Reutner-Fischer wrote: On Fri, Jun 07, 2013 at 02:13:58PM +0800, qi.c...@windriver.com wrote: From: Chen Qi This patch enables us to split the busybox into two binaries, one containing suid applications, and the other containing nosuid apps. Add a variable, BU

Re: [OE-core] [PATCH 4/8] busybox: add the ability to split the busybox binary

2013-06-16 Thread ChenQi
, ChenQi wrote: On 06/12/2013 04:26 AM, Bernhard Reutner-Fischer wrote: On Fri, Jun 07, 2013 at 02:13:58PM +0800, qi.c...@windriver.com wrote: Hi Bernhard, Thank you very much for your review and suggestions! I went through your methods below very carefully and I tried your patch out. Here are

Re: [OE-core] [PATCH V2 1/9] busybox: remove the postinst part of the recipe

2013-06-17 Thread ChenQi
On 06/18/2013 01:52 AM, Otavio Salvador wrote: On Mon, Jun 17, 2013 at 2:49 AM, wrote: From: Chen Qi Remove the pkg_postinst_${PN} from this recipe, as it's redundant. It basically wants to do the same thing as the update-alternatives does. But it doesn't do it well. Signed-off-by: Chen Qi

Re: [OE-core] [PATCH 1/2] image-live.bbclass: enable read-only-rootfs IMAGE_FEATURE

2013-06-23 Thread ChenQi
On 06/19/2013 02:37 PM, rongqing...@windriver.com wrote: From: "Roy.Li" enable read-only-rootfs to make /tmp be able to be written when using sysvinit Signed-off-by: Roy.Li --- meta/classes/image-live.bbclass |2 ++ 1 file changed, 2 insertions(+) diff --git a/meta/classes/image-live.

Re: [OE-core] [PATCH 1/1] udev-cache: take a read-only rootfs into consideration

2013-06-27 Thread ChenQi
On 06/28/2013 01:59 PM, Saul Wold wrote: On 06/27/2013 10:00 PM, qi.c...@windriver.com wrote: From: Chen Qi In case of a read-only rootfs, no matter the /etc is read-only or it's bind mounted with tmpfs, the udev cache handling should be skipped, as the data cannot be persisted between reboots

Re: [OE-core] [oe-commits] Chen Qi : busybox: add the ability to split the busybox binary

2013-06-30 Thread ChenQi
On 06/28/2013 07:56 PM, Martin Jansa wrote: On Tue, Jun 25, 2013 at 04:37:58PM +, g...@git.openembedded.org wrote: Module: openembedded-core.git Branch: master Commit: e5a1442819dfb74e86a6f69da008ba6908c8bbc7 URL: http://git.openembedded.org/?p=openembedded-core.git&a=commit;h=e5a1442819

Re: [OE-core] [oe-commits] Chen Qi : busybox: add the ability to split the busybox binary

2013-07-01 Thread ChenQi
On 07/01/2013 03:33 PM, Martin Jansa wrote: Hi Chen, 1) install image created before this change on target device (which doesn't include coreutils, wget and is depending on usable busybox) 2) build feeds which include this change 3) run "opkg update && opkg upgrade" on target device (or equiv

Re: [OE-core] [PATCH 4/4 v2] udev: Add RRECOMMENDS for udev-cache to enable the cache

2013-07-02 Thread ChenQi
On 07/02/2013 04:41 PM, Koen Kooi wrote: Op 1 jul. 2013, om 22:48 heeft Saul Wold het volgende geschreven: This will enable the udev cache for sysvinit systems, which does improve post-first boots. Can I please get some numbers on that? I haven't used it for years since it actually slowed

Re: [OE-core] [PATCH] udev: Add RRECOMMENDS for udev-cache to enable the cache

2013-07-02 Thread ChenQi
On 07/02/2013 04:42 PM, Koen Kooi wrote: Op 1 jul. 2013, om 20:58 heeft Saul Wold het volgende geschreven: This will enable the udev cache for sysvinit systems, which does improve post-first boots. But will it stay disabled for non-sysv init systems? I think so. After all, it's not recomm

Re: [OE-core] [PATCH 02/11] busybox: Upgrade to v1.21.0

2013-07-04 Thread ChenQi
Hi Radu, I sent a patch a few days ago, which updates busybox to 1.21.1. 1.21.1 is now available and claimed to be 'stable'. Please have a look at the patch. Any comment or suggestion is welcome. Thanks, Chen Qi On 07/04/2013 10:57 PM, Radu Moisan wrote: Removed obsolete patches Added a few f

Re: [OE-core] [PATCH 1/1] busybox: upgrade to stable 1.21.1

2013-07-04 Thread ChenQi
On 07/05/2013 06:59 AM, Saul Wold wrote: On 07/02/2013 07:39 PM, qi.c...@windriver.com wrote: From: Chen Qi Merged or backported patches are dropped. The wget_dl_dir_fix.patch was submitted more than 1 year ago, it's about the -P option behavior, and it's not accepted, so I dropped this patch

Re: [OE-core] [PATCH 1/1] image.bbclass: add a method to add/delete/modify user/group settings

2013-07-05 Thread ChenQi
On 07/05/2013 04:39 PM, Martin Jansa wrote: On Fri, Jul 05, 2013 at 02:07:28PM +0800, qi.c...@windriver.com wrote: From: Chen Qi We may want to add a user or group which does not logically belong to any specific package. For example, we may want to add a user with the name 'tester' to our imag

Re: [OE-core] [PATCH 1/1] busybox: fix the on-target upgrade problem

2013-07-08 Thread ChenQi
On 07/06/2013 06:27 PM, Martin Jansa wrote: On Wed, Jul 03, 2013 at 12:48:12PM +0800, qi.c...@windriver.com wrote: From: Chen Qi We now can have a 'one-binary' version of busybox, or 'two-binary' version of busybox, controlled by the 'BUSYBOX_SPLIT_SUID' variable. This makes on-target upgrade

Re: [OE-core] [PATCH 1/1] image.bbclass: add a method to add/delete/modify user/group settings

2013-07-09 Thread ChenQi
On 07/09/2013 07:20 AM, Saul Wold wrote: On 07/04/2013 11:07 PM, qi.c...@windriver.com wrote: Since this gets run every time for image creation, we can be sure an image won't be using someform of login, so finding login.defs might not be the best thing to do. take the poky.tiny example and

Re: [OE-core] [PATCH 1/1] init-live.sh: allow users to choose the boot action

2013-07-10 Thread ChenQi
On 07/10/2013 04:32 PM, Tomas Frydrych wrote: Hi, On 10/07/13 08:12, qi.c...@windriver.com wrote: From: Chen Qi Previously, the boot action for live images was totally determined by the LABEL value in the boot parameters (/proc/cmdline). This behavior is not very convenient for users. The us

Re: [OE-core] [PATCH V2 1/1] image.bbclass: add a method to add/delete/modify user/group settings

2013-07-10 Thread ChenQi
On 07/11/2013 04:02 AM, Saul Wold wrote: On 07/09/2013 11:28 PM, qi.c...@windriver.com wrote: From: Chen Qi We may want to add a user or group which does not logically belong to any specific package. For example, we may want to add a user with the name 'tester' to our image. Besides, we may wa

Re: [OE-core] [PATCH V4 0/3] Add a method for image level user/group configuration

2013-07-15 Thread ChenQi
ass rename usersettings.bbclass to extrausers.bbclass rename USER_GROUP_SETTINGS to EXTRA_USERS_PARAMS 2. Add an example in the extrauser.bbclass to show how to use this bbclass 3. Report the incorrect setting in EXTRA_USERS_PARAMS. Note the branch has also changed from ChenQi/user_group_s

Re: [OE-core] [PATCH V5 3/3] extrausers.bbclass: add a new bbclass

2013-07-16 Thread ChenQi
On 07/17/2013 07:47 AM, Saul Wold wrote: On 07/15/2013 11:27 PM, qi.c...@windriver.com wrote: From: Chen Qi This class is dedicated to image level user/group configuration. It inherits useradd_base.bbclass. Users need to inherit this class in their layers or local.conf to make the setting of

Re: [OE-core] [PATCH V5 0/3] Add a method for image level user/group configuration

2013-07-16 Thread ChenQi
a6365: meta-yocto-bsp: update reference board SRCREVs (2013-07-10 09:45:51 +0100) are available in the git repository at: git://git.pokylinux.org/poky-contrib ChenQi/extrausers http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=ChenQi/extrausers Chen Qi (3): useradd_base.bbclass: a

Re: [OE-core] [PATCH V5 0/3] Add a method for image level user/group configuration

2013-07-16 Thread ChenQi
On 07/17/2013 01:27 PM, ChenQi wrote: On 07/17/2013 05:40 AM, Saul Wold wrote: On 07/15/2013 11:27 PM, qi.c...@windriver.com wrote: From: Chen Qi Changes from V4: 1. Take into consideration the possibility of whitespaces in /etc/passwd and /etc/group 2. Change bbfatal in perform_usermod and

Re: [OE-core] [PATCH 1/1] busybox-syslog: respect /etc/syslog-startup.conf in systemd

2013-07-18 Thread ChenQi
On 07/19/2013 01:07 AM, Burton, Ross wrote: On 18 July 2013 08:05, wrote: 1. The configuration file, /etc/syslog-startup.conf, is ignored. This is really confusing, as we are used to the behaviour of changing the configuration in /etc/syslog-startup.conf for the syslog service. Thi

Re: [OE-core] [PATCH 1/3] rpm: remove the obsolete rpm-postinsts.bb

2013-07-22 Thread ChenQi
On 07/23/2013 07:49 AM, Saul Wold wrote: On 07/19/2013 02:39 AM, qi.c...@windriver.com wrote: From: Chen Qi Now that the postinsts of rpm, deb and ipk are all handled in the one central place, run-postinsts.bb, the rpm-postinsts.bb recipe is actually obsolete now. Remove this recipe to avoid c

Re: [OE-core] RFC: meta-ro-rootfs approach and volatiles vs tmpfiles.d

2013-07-24 Thread ChenQi
http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=ChenQi/read-only-rootfs-in-live-images I'm really interested on this topic. From my understanding, this RFC mainly focuses on read-only rootfs support for systemd based images, right? I've read though this thread carefully, but st

Re: [OE-core] [PATCH] systemd: Remove init.d dir if systemd unit file is present and sysvinit not a distro feature

2013-07-25 Thread ChenQi
On 07/23/2013 06:03 PM, Shakeel, Muhammad wrote: From: Muhammad Shakeel If systemd is supported DISTRO_FEATURE and sysvinit is not and also if systemd_unitdir contains a service file(s) then no need to keep init.d scripts for sysvinit compatibility. Signed-off-by: Muhammad Shakeel --- meta/

Re: [OE-core] [PATCH 9/9] Generate ssh keys at rootfs creation time in case of a read-only rootfs

2013-07-28 Thread ChenQi
Thanks for all the reviews and comments. I'll drop this patch and send out a version 2 of this patchset. Best Regards, Chen Qi On 07/26/2013 03:39 PM, qi.c...@windriver.com wrote: From: Chen Qi To avoid generating ssh keys every time a system with read-only rootfs starts, we generate ssh keys

Re: [OE-core] [PATCH V2 10/10] openssh: make /etc/ssh directory writable in read-only rootfs

2013-07-29 Thread ChenQi
On 07/29/2013 11:59 PM, Burton, Ross wrote: On 29 July 2013 03:33, wrote: From: Chen Qi If the rootfs is read-only and the ssh keys are not available at system start-up, the init script will generate ssh keys into /etc/ssh, thus causing a 'read-only file system' error. Make this directory w

Re: [OE-core] [PATCH V2 09/10] lighttpd: make /www diretory writable in read-only rootfs

2013-07-29 Thread ChenQi
On 07/29/2013 11:56 PM, Burton, Ross wrote: On 29 July 2013 03:33, wrote: +echo "/www /var/volatile/www" > ${D}${sysconfdir}/default/readonly/lighttpd /www is the default lighttpd document root, where the web sites are stored, so putting them in a tmpfs would be very wrong indeed. Why do

Re: [OE-core] [PATCH V2 08/10] irda-utils: make /etc/sysconfig writable in read-only rootfs

2013-07-29 Thread ChenQi
On 07/29/2013 11:26 PM, Phil Blundell wrote: On Mon, 2013-07-29 at 10:33 +0800, qi.c...@windriver.com wrote: From: Chen Qi This diretory needs to be writable, the following error will appear at system start-up. /etc/rc5.d/S20irattach: /etc/sysconfig/irda: Read-only file system The whole

Re: [OE-core] RFC: meta-ro-rootfs approach and volatiles vs tmpfiles.d

2013-07-31 Thread ChenQi
On 07/31/2013 02:17 AM, Chris Larson wrote: On Wed, Jul 24, 2013 at 7:51 PM, ChenQi <mailto:qi.c...@windriver.com>> wrote: You can get more information from the bug link below. The related bugs are listed in the blocks list of this bug. https://bugzilla.yoctoproject.org/sho

Re: [OE-core] RFC: meta-ro-rootfs approach and volatiles vs tmpfiles.d

2013-07-31 Thread ChenQi
On 07/31/2013 02:53 AM, Chris Larson wrote: On Wed, Jul 24, 2013 at 7:51 PM, ChenQi <mailto:qi.c...@windriver.com>> wrote: You can get more information from the bug link below. The related bugs are listed in the blocks list of this bug. https://bugzilla.yoctoproject.org/sho

Re: [OE-core] [PATCH] base-files: remove symbolic link from /etc/mtab to /proc/mounts

2013-07-31 Thread ChenQi
On 07/31/2013 04:19 PM, Chunrong Guo wrote: *If /etc/mtab is a symlink to /proc/self/mounts, mounting fails because addmntent returns an error. As a consequence, the Private directory is unusable because /proc/self/mounts is read-only feature. Could you please detail your

Re: [OE-core] [PATCH 1/1] buildtools-tarball: unset OECORE_NATIVE_SYSROOT

2013-08-01 Thread ChenQi
t needs to get the location of native sysroot. And it does so by grep 'OECORE_NATIVE_SYSROOT=' expression in $env_setup_script. Please see more details below (code snippet from populate_sdk_base.bbclass). chenqi@pek-qchen1-d1:~/poky$ grep -i oecore_native_sysroot meta/classes/popul

Re: [OE-core] [PATCH 1/1] buildtools-tarball: unset OECORE_NATIVE_SYSROOT

2013-08-01 Thread ChenQi
On 08/02/2013 02:12 AM, Saul Wold wrote: On 07/29/2013 06:50 PM, qi.c...@windriver.com wrote: From: Chen Qi When building a qemu image inside the environment created by the buildtools-tarball, the qemu image cannot be started, as the runqemu script uses the tunctl binary which cannot be found

Re: [OE-core] [PATCH 0/1] scripts/contrib/mkefidisk.sh: fix bashism

2013-08-02 Thread ChenQi
://git.pokylinux.org/poky-contrib ChenQi/fix-bashism http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=ChenQi/fix-bashism Chen Qi (1): scripts/contrib/mkefidisk.sh: fix bashism scripts/contrib/mkefidisk.sh |8 1 file changed, 4 insertions(+), 4 deletions(-) Please

Re: [OE-core] Initial review of ChenQi/read-only-rootfs-in-live-images

2013-08-04 Thread ChenQi
Hi Chris, Thanks for your review and please see comments inline. (You might be reviewing the old patchset, please see the new one if you have time.) On 08/02/2013 11:50 PM, Chris Larson wrote: Greetings, Nice work on this branch. The bind based approach seems clean. I do have some comments:

Re: [OE-core] Initial review of ChenQi/read-only-rootfs-in-live-images

2013-08-04 Thread ChenQi
For convenience, here's a link below in case people may wondering which patchset we are talking about. http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=ChenQi/read-only-rootfs-in-live-images //Chen Qi On 08/05/2013 10:18 AM, ChenQi wrote: Hi Chris, Thanks for your review and p

Re: [OE-core] [PATCH V4 09/10] irda-utils: fix for read-only rootfs

2013-08-07 Thread ChenQi
On 08/07/2013 10:18 PM, Chris Larson wrote: On Wed, Aug 7, 2013 at 1:08 AM, > wrote: From: Chen Qi mailto:qi.c...@windriver.com>> The init script for irda writes configuration items to /etc/sysconfig/irda if that file is not available in system. B

Re: [OE-core] [PATCH V4 02/10] use a uniform way to determine whether rootfs is read-only

2013-08-07 Thread ChenQi
On 08/08/2013 04:29 AM, Otavio Salvador wrote: On Wed, Aug 7, 2013 at 5:08 AM, wrote: ... --- a/meta/recipes-core/initscripts/initscripts-1.0/functions +++ b/meta/recipes-core/initscripts/initscripts-1.0/functions @@ -58,3 +58,17 @@ status() { fi return 3 } + +# Determine whether

Re: [OE-core] [PATCH V4 09/10] irda-utils: fix for read-only rootfs

2013-08-07 Thread ChenQi
On 08/08/2013 10:23 AM, Khem Raj wrote: On Wednesday, August 7, 2013, ChenQi <mailto:qi.c...@windriver.com>> wrote: > On 08/07/2013 10:18 PM, Chris Larson wrote: > > On Wed, Aug 7, 2013 at 1:08 AM, <mailto:qi.c...@windriver.com>> wrote: >> >> Fro

Re: [OE-core] [PATCH V4 08/10] populate-volatile.sh: use 'cp -a' to avoid potential problem

2013-08-07 Thread ChenQi
On 08/07/2013 04:08 PM, qi.c...@windriver.com wrote: From: Chen Qi Previously, dead links in target directory will not be copied. This is incorrect as dead links are not uncommon in our rootfs. So we use '-a' option instead. Signed-off-by: Chen Qi --- .../initscripts-1.0/populate-volatile.s

Re: [OE-core] [oe-commits] Hongxu Jia : util-linux: use u-a for su

2013-08-09 Thread ChenQi
Hi Martin, I think this is the same problem we discussed before. The thread is "busybox: fix the on-target upgrade problem". The problem roots in opkg's update-alternatives. At that time, I made a patch to fix this problem. I didn't send it out because I then found some other problems with upda

Re: [OE-core] [oe-commits] Hongxu Jia : util-linux: use u-a for su

2013-08-09 Thread ChenQi
On 08/09/2013 03:22 PM, Martin Jansa wrote: On Fri, Aug 09, 2013 at 02:47:22PM +0800, Hongxu Jia wrote: On 08/09/2013 01:25 AM, Martin Jansa wrote: On Wed, Jul 31, 2013 at 05:51:35AM +, g...@git.openembedded.org wrote: Module: openembedded-core.git Branch: master Commit: 6190c69a9f78aeef95

Re: [OE-core] [oe-commits] Chen Qi : useradd.bbclass: add a new base class and code refactor

2013-08-15 Thread ChenQi
On 08/16/2013 01:47 AM, Martin Jansa wrote: On Wed, Jul 24, 2013 at 10:26:48AM +, g...@git.openembedded.org wrote: Module: openembedded-core.git Branch: master Commit: 2a57bb7e9a7e154578aa7cb9aeebdf398a54ec00 URL: http://git.openembedded.org/?p=openembedded-core.git&a=commit;h=2a57bb7e9a

Re: [OE-core] [PATCH V5 00/10] Make read-only rootfs work with live images

2013-08-18 Thread ChenQi
tion. 3. Fix a new bug#4937 (https://bugzilla.yoctoproject.org/show_bug.cgi?id=4937) The following changes since commit f63e7f4323368c0d6fe7a1d44393a7e15652d4f2: subversion: Add patch to use neon 0.30 (2013-08-07 07:43:47 +0100) are available in the git repository at: git://git.pokylinux.org/poky-contrib Che

Re: [OE-core] [PATCH 4/6] openssh: add init.d/sshd status command for LSB compliance

2013-08-22 Thread ChenQi
On 08/22/2013 03:02 PM, jhuang0 wrote: On 8/22/2013 2:42 PM, Khem Raj wrote: Hi Jackie One thing comes to mind is that these scripts are hardcoding the binaries and may be it would be nicer to replace the bindir, libdir and so on at install time, that would make them flexible where /usr is

Re: [OE-core] [PATCH 3/4] runqemu-internal: change the lock acquire/release logic

2013-08-27 Thread ChenQi
On 08/27/2013 06:09 PM, Richard Purdie wrote: On Tue, 2013-08-27 at 15:08 +0800, qi.c...@windriver.com wrote: From: Chen Qi Checking whether the lock file exists is sufficient for runqemu. The contents of the lock file is not important. What happens if the runqemu script crashes and leaves th

Re: [OE-core] [PATCH 3/4] runqemu-internal: change the lock acquire/release logic

2013-08-27 Thread ChenQi
On 08/27/2013 06:14 PM, Phil Blundell wrote: On Tue, 2013-08-27 at 15:08 +0800, qi.c...@windriver.com wrote: This patch simplifies the lock acquire/release logic by removing the flock mechanism. I'm not sure that "simplifies" is an accurate description of the results of removing flock. How abo

Re: [OE-core] [PATCH] systemd: add ptest

2013-08-28 Thread ChenQi
On 08/28/2013 04:39 PM, rongqing...@windriver.com wrote: From: "Roy.Li" Signed-off-by: Roy.Li --- meta/recipes-core/systemd/systemd/run-ptest |2 ++ meta/recipes-core/systemd/systemd_204.bb| 18 +- 2 files changed, 19 insertions(+), 1 deletion(-) create mode 100

Re: [OE-core] Busybox adduser cannot create system user "no uids left"

2013-08-29 Thread ChenQi
On 08/29/2013 11:19 PM, Samuel Stirtzel wrote: Hi, with the current master/HEAD it seems impossible to create system users with busybox. root@overo:~# which adduser /usr/sbin/adduser root@overo:~# ls -la /usr/sbin/adduser lrwxrwxrwx1 root root19 Aug 29 2013 /usr/sbin/addus

Re: [OE-core] Failure

2013-09-01 Thread ChenQi
On 09/02/2013 10:08 AM, Otavio Salvador wrote: Hello, I got this build failure today; I am about to leave so I removed the tmpdir and will let it running during night. I post more info if/when I reproduce it again: Patch redhat-portability.diff does not apply (enforce with -f) ERROR: Function f

Re: [OE-core] [PATCH 2/2] sysklogd: lower the ALTERNATIVE_PRIORITY in case of systemd

2013-09-02 Thread ChenQi
On 09/02/2013 07:58 PM, Peter A. Bigot wrote: On 09/02/2013 06:40 AM, qi.c...@windriver.com wrote: From: Chen Qi The sysklogd package hasn't got systemd support yet. So in case of a systemd based system, the commands and corresponding configuration files should have a lower priority than that

Re: [OE-core] [PATCH 0/2] busybox & sysklogd: weigh busybox's syslogd/klogd utilities over those from sysklogd in case of systemd

2013-09-02 Thread ChenQi
ptors, not a float (2013-09-02 09:16:24 +0100) are available in the git repository at: git://git.pokylinux.org/poky-contrib ChenQi/systemd-syslog http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=ChenQi/systemd-syslog Chen Qi (2): busybox-syslog: add configuration file /etc/s

Re: [OE-core] [PATCH 1/1] image.bbclass: leave metadata in place if a PM is installed in the image

2013-09-05 Thread ChenQi
On 09/05/2013 07:11 PM, Laurentiu Palcu wrote: On Thu, Sep 05, 2013 at 11:43:54AM +0100, Phil Blundell wrote: On Thu, 2013-09-05 at 10:15 +0200, Martin Jansa wrote: On Thu, Sep 05, 2013 at 10:31:10AM +0300, Laurentiu Palcu wrote: rootfs_uninstall_unneeded () { - if ${@base_contains("IM

Re: [OE-core] [PATCH 1/2] ptest.bbclass: fix error on ubuntu host

2013-09-05 Thread ChenQi
Just noticed the do_configure_ptest_base and do_compile_ptest_base also have this problem. I'll send out a new patchset. //Chen Qi On 09/05/2013 08:49 PM, qi.c...@windriver.com wrote: From: Chen Qi The do_install_ptest_base function uses 'type -t' command to check whether do_install_ptest i

Re: [OE-core] [PATCH] udev-extraconf: Avoid mounting unknown filesystems

2013-09-10 Thread ChenQi
Hi Otavio, This patch caused a new bug. So I'm afraid maybe this patch is not totally correct. https://bugzilla.yoctoproject.org/show_bug.cgi?id=4487 The problem is that for a cdrom device, there's no ID_FS_TYPE. Any idea? Besides, could you please detail a little about what problem this patc

Re: [OE-core] [PATCH 1/1] update-rc.d.bbclass: change RRECOMMENDS to RDEPENDS

2013-09-12 Thread ChenQi
On 09/12/2013 06:25 PM, Martin Jansa wrote: On Thu, Sep 12, 2013 at 05:43:55PM +0800, qi.c...@windriver.com wrote: From: Chen Qi If a package inherits update-rc.d.bbclass, it actually needs it to update the symlinks under /etc/rc?.d/ directories. This is a runtime dependency. For example, if w

Re: [OE-core] [PATCH 1/1] update-rc.d.bbclass: change RRECOMMENDS to RDEPENDS

2013-09-12 Thread ChenQi
On 09/12/2013 08:18 PM, Martin Jansa wrote: On Thu, Sep 12, 2013 at 07:18:12PM +0800, ChenQi wrote: On 09/12/2013 06:25 PM, Martin Jansa wrote: On Thu, Sep 12, 2013 at 05:43:55PM +0800, qi.c...@windriver.com wrote: From: Chen Qi If a package inherits update-rc.d.bbclass, it actually needs

Re: [OE-core] [PATCH 1/1] image.bbclass: do not automatically remove update-rc.d

2013-09-21 Thread ChenQi
ping On 09/13/2013 10:09 AM, qi.c...@windriver.com wrote: From: Chen Qi In rootfs_uninstall_unneeded function, the update-rc.d package would be removed if no 'package-management' and no delayed postinsts. However, in update-rc.d.bbclass, the update-rc.d package is only recommended. Thus, if N

Re: [OE-core] [PATCH 1/1] image.bbclass: do not automatically remove update-rc.d

2013-09-22 Thread ChenQi
On 09/22/2013 04:44 PM, Richard Purdie wrote: On Sun, 2013-09-22 at 11:13 +0800, ChenQi wrote: ping I replied to this: """ This is not correct. The correct behaviour here is to uninstall these packages *if* they are installed. Just removing it from the list is not the real

Re: [OE-core] [PATCH 0/1] sysvinit: fix missing of boot log

2013-09-26 Thread ChenQi
en Qi The following changes since commit 4d8ab87d25ea6e59dc88375813ef88ee0ac813d0: build-appliance: Update SRCREV (2013-09-24 18:09:12 +0100) are available in the git repository at: git://git.pokylinux.org/poky-contrib ChenQi/bootlogd http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=C

Re: [OE-core] [PATCH V2 1/1] sysvinit: fix missing of boot log

2013-09-26 Thread ChenQi
On 09/26/2013 06:50 PM, Enrico Scholz wrote: writes: Besides, we need to make sure that tmpfs has been mounted. Otherwise, the boot log will still be missing. ... case "$ACTION" in start) + mount -a -t tmpfs 2>/dev/null This will mount filesystems out-of-order. E.g. it

Re: [OE-core] [PATCH V2 1/1] sysvinit: fix missing of boot log

2013-10-07 Thread ChenQi
ping On 09/26/2013 06:02 PM, qi.c...@windriver.com wrote: From: Chen Qi Previously, our system had no boot log even if the bootlogd daemon was started correctly. The root cause is that the log file doesn't exist when starting the bootlogd. Add '-c' option to bootlogd so that it will create th

Re: [OE-core] Setting root password

2013-10-11 Thread ChenQi
On 10/11/2013 10:35 PM, Jack Mitchell wrote: What is the best way to set the root password for an image during build time? I have come across this snippet: ROOTFS_POSTPROCESS_COMMAND += " \ sed 's%^root:[^:]*:%root:${PASSWD}:%' \ < ${IMAGE_ROOTFS}/etc/shadow \ > ${IMA

Re: [OE-core] [PATCH 1/1] buildhistory.bbclass: always record PKG, PKGE, PKGV and PKGR

2013-10-16 Thread ChenQi
istory. 1. bitbake chown-example 2. mv ../meta-local/recipes-core/chown-example/chown-example_1.0.bb ../meta-local/recipes-core/chown-example/chown-example_2.0.bb 3. bitbake chown-example 4. buildhistory-diff Testing Result --- *) Without the patch *) With the patch chenqi@p

Re: [OE-core] [PATCH 1/1] image_types.bbclass: add --numeric-owner option to tar command

2013-10-20 Thread ChenQi
On 10/18/2013 10:57 PM, Richard Purdie wrote: On Fri, 2013-10-18 at 15:47 +0800, qi.c...@windriver.com wrote: From: Chen Qi 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. F

Re: [OE-core] [PATCH V2 1/1] sysvinit: fix missing of boot log

2013-10-29 Thread ChenQi
ping Any comment on this one? Best Regards, Chen Qi On 09/26/2013 06:02 PM, qi.c...@windriver.com wrote: From: Chen Qi Previously, our system had no boot log even if the bootlogd daemon was started correctly. The root cause is that the log file doesn't exist when starting the bootlogd. Add

Re: [OE-core] [PATCH 1/1] iproute2: explicitly add 'bash' to its RDEPENDS

2013-10-30 Thread ChenQi
On 10/31/2013 11:32 AM, Khem Raj wrote: On Tue, Oct 29, 2013 at 7:07 PM, wrote: From: Chen Qi If we build a minimal image with iproute2 installed, the following error will appear during rootfs. error: Can't install iproute2-3.10.0-r0.0@i586: no package provides /bin/bash The problem is tha

Re: [OE-core] [PATCH 1/1] iproute2: explicitly add 'bash' to its RDEPENDS

2013-10-30 Thread ChenQi
On 10/31/2013 01:39 PM, Saul Wold wrote: On 10/30/2013 09:40 PM, ChenQi wrote: On 10/31/2013 11:32 AM, Khem Raj wrote: On Tue, Oct 29, 2013 at 7:07 PM, wrote: From: Chen Qi If we build a minimal image with iproute2 installed, the following error will appear during rootfs. error: Can&#

Re: [OE-core] [PATCH V2 1/1] sysvinit: fix missing of boot log

2013-11-01 Thread ChenQi
On 11/01/2013 06:44 PM, Richard Purdie wrote: On Tue, 2013-10-29 at 20:33 +0800, ChenQi wrote: ping Any comment on this one? Yes, shouldn't the boot ordering be tweaked to ensure the tmpfs is available rather than force mounting it within this init script? This looks like a hack aroun

Re: [OE-core] [PATCH V2 0/5] fix WARNING: QA Issue on kmod/udev/nfs-utils (unsafe references [/ references to /usr/lib])

2013-11-08 Thread ChenQi
On 11/04/2013 07:39 PM, Richard Purdie wrote: On Mon, 2013-11-04 at 17:56 +0800, Hongxu Jia wrote: Change in V2: The fourth patch fixed the QA Issue on udev, it was missed on previous version. //Hongxu The following changes since commit f3541226b8b1187e79dec0f6f9f3c58cedf9ac9b: bitbake: h

Re: [OE-core] [PATCH 1/8] udev: fix dependency and location of udevadm

2013-11-10 Thread ChenQi
On 11/10/2013 06:54 AM, Phil Blundell wrote: On Sat, 2013-11-09 at 13:28 +0800, qi.c...@windriver.com wrote: + install -d ${D}${base_bindir} + mv ${D}${bindir}/udevadm ${D}${base_bindir}/udevadm + rmdir ${D}${bindir} This will fail if ${bindir} and ${base_bindir} are the same.

Re: [OE-core] [PATCH 2/8] initscripts: add setup-commands.sh

2013-11-10 Thread ChenQi
On 11/10/2013 07:00 AM, Phil Blundell wrote: On Sat, 2013-11-09 at 13:28 +0800, qi.c...@windriver.com wrote: +for dir in /usr/bin /usr/sbin; do +if [ ! -e $dir ]; then + if [ "$VERBOSE" != "no" ]; then + echo "WARN: $dir missing, setting up links to busybox" + fi +

Re: [OE-core] [PATCH 0/8] Fixes about unsafe-references QA warnings

2013-11-11 Thread ChenQi
On 11/11/2013 07:12 PM, Burton, Ross wrote: On 9 November 2013 05:28, wrote: This solution is based on the following two principles. 1. With /usr on a seperate partition, system should still boot without any error. 2. Without /usr, system should be able to boot into single user mode with erro

Re: [OE-core] [PATCH 1/8] udev: fix dependency and location of udevadm

2013-11-11 Thread ChenQi
On 11/11/2013 06:53 PM, Phil Blundell wrote: On Mon, 2013-11-11 at 10:18 +0800, ChenQi wrote: On 11/10/2013 06:54 AM, Phil Blundell wrote: On Sat, 2013-11-09 at 13:28 +0800, qi.c...@windriver.com wrote: + install -d ${D}${base_bindir} + mv ${D}${bindir}/udevadm ${D}${base_bindir

Re: [OE-core] [PATCH 2/8] initscripts: add setup-commands.sh

2013-11-11 Thread ChenQi
Hi Phil, First of all, thank you for your careful review and explanation. To conclude, you suggest modifying the init scripts and moving commands around if needed, right? And please see comments inline. On 11/11/2013 07:53 PM, Phil Blundell wrote: On Mon, 2013-11-11 at 10:52 +0800, ChenQi

Re: [OE-core] [PATCH 2/8] initscripts: add setup-commands.sh

2013-11-11 Thread ChenQi
On 11/11/2013 08:12 PM, Burton, Ross wrote: On 11 November 2013 02:52, ChenQi wrote: The problem here is that the init scripts under /etc/rcS.d/ need to execute commands like awk, dirname, and readlink which are from /usr. Yes. So why do you really need to support split /usr, and why isn&#

Re: [OE-core] [PATCH 1/1] initscripts: improve the way initscripts handle volatile storage

2012-11-13 Thread ChenQi
On 11/14/2012 07:14 AM, Saul Wold wrote: On 11/12/2012 03:33 AM, qi.c...@windriver.com wrote: From: Chen Qi Change the way how populate_volatile.sh handles link-type config items. Previously, if a link-type config item is encountered, the script does not handle it correctly. If the target exis

Re: [OE-core] [PATCH 1/1] initscripts: improve the way initscripts handle volatile storage

2012-11-13 Thread ChenQi
On 11/14/2012 07:14 AM, Saul Wold wrote: On 11/12/2012 03:33 AM, qi.c...@windriver.com wrote: From: Chen Qi Change the way how populate_volatile.sh handles link-type config items. Previously, if a link-type config item is encountered, the script does not handle it correctly. If the target exis

Re: [OE-core] [PATCH 1/1] mdadm: make udev rules dir configurable

2012-11-22 Thread ChenQi
On 11/20/2012 08:07 PM, Richard Purdie wrote: On Tue, 2012-11-20 at 17:34 +0800, qi.c...@windriver.com wrote: From: Chen Qi mdadm hardcodes the udev rules dir to be '/lib/udev/rules.d'. However, udev configs its base library directory to be ${base_libdir} instead of '/lib'. Thus if the target h

Re: [OE-core] [PATCH 1/1] autotools.bbclass: use relative path for acpaths whenever possible

2012-11-23 Thread ChenQi
On 11/22/2012 08:19 AM, Saul Wold wrote: On 11/08/2012 09:47 PM, qi.c...@windriver.com wrote: From: Chen Qi When the TMPDIR is very long, say, 410 characters, aclocal would fail because the argument list is too long. This patch is an effort to use relative path for acpaths whenever possible, a

Re: [OE-core] [PATCH] udev 182: Create a symlink of /lib/udev/udevd in /sbin

2012-11-27 Thread ChenQi
On 11/27/2012 10:15 PM, Otavio Salvador wrote: On Tue, Nov 27, 2012 at 10:57 AM, Shakeel, Muhammad wrote: From: Muhammad Shakeel From udev 174 changelog: "The udev daemon moved to /lib/udev/udevd. Non-systemd init systems and non-dracut initramfs image generators need to change the init scri

Re: [OE-core] [PATCH] udev 182: Create a symlink of /lib/udev/udevd in /sbin

2012-11-29 Thread ChenQi
.com] On Behalf Of Otavio Salvador Sent: Wednesday, November 28, 2012 3:52 PM To: ChenQi Cc: Shakeel, Muhammad; openembedded-core@lists.openembedded.org Subject: Re: [OE-core] [PATCH] udev 182: Create a symlink of /lib/udev/udevd in /sbin On Wed, Nov 28, 2012 at 12:26 AM, ChenQi wrote: On 11/2

Re: [OE-core] [PATCH] udev 182: Create a symlink of /lib/udev/udevd in /sbin

2012-11-29 Thread ChenQi
me some more information to help me? Thanks, Chen Qi -Original Message- From: otavio.salva...@gmail.com [mailto:otavio.salva...@gmail.com] On Behalf Of Otavio Salvador Sent: Wednesday, November 28, 2012 3:52 PM To: ChenQi Cc: Shakeel, Muhammad; openembedded-core@lists.openembedded.org Subject: Re

Re: [OE-core] [PATCH 2/3] qt4-native: make qt4-native work with long building path

2012-11-30 Thread ChenQi
On 11/30/2012 03:46 PM, Martin Jansa wrote: On Fri, Nov 30, 2012 at 11:08:43AM +0800, qi.c...@windriver.com wrote: From: Chen Qi If the TMPDIR has more than 256 chars, building qt4-native fails. This violates the 410 length limit of TMPDIR. This patch makes building qt4-native succeed with a l

Re: [OE-core] [PATCH 2/3] qt4-native: make qt4-native work with long building path

2012-11-30 Thread ChenQi
On 11/30/2012 04:11 PM, Martin Jansa wrote: On Fri, Nov 30, 2012 at 04:01:00PM +0800, ChenQi wrote: On 11/30/2012 03:46 PM, Martin Jansa wrote: On Fri, Nov 30, 2012 at 11:08:43AM +0800, qi.c...@windriver.com wrote: From: Chen Qi If the TMPDIR has more than 256 chars, building qt4-native

Re: [OE-core] [PATCH 0/3] [V2] Make poky work correctly with long TMPDIR

2012-11-30 Thread ChenQi
) are available in the git repository at: git://git.pokylinux.org/poky-contrib ChenQi/autotools http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=ChenQi/autotools Chen Qi (3): autotools.bbclass: use relative path for acpaths whenever possible qt4-native: make qt4-native work with

Re: [OE-core] [PATCH 2/3] qt4-native: make qt4-native work with long building path

2012-12-02 Thread ChenQi
On 11/30/2012 09:48 PM, Martin Jansa wrote: On Fri, Nov 30, 2012 at 06:39:23PM +0800, qi.c...@windriver.com wrote: From: Chen Qi If the TMPDIR has more than 256 chars, building qt4-native fails. This violates the 410 length limit of TMPDIR. This patch makes building qt4-native succeed with a l

Re: [OE-core] [PATCH 2/3] qt4-native: make qt4-native work with long building path

2012-12-03 Thread ChenQi
On 12/03/2012 04:18 PM, Martin Jansa wrote: On Mon, Dec 03, 2012 at 09:55:23AM +0800, qi.c...@windriver.com wrote: From: Chen Qi If the TMPDIR has more than 256 chars, building qt4-native fails. This violates the 410 length limit of TMPDIR. This patch makes building qt4-native succeed with a l

Re: [OE-core] [PATCH 2/4] ICU: add pkgconfig support

2012-12-03 Thread ChenQi
On 12/03/2012 07:50 PM, Burton, Ross wrote: On 3 December 2012 10:42, wrote: +ICU-3.6 has no pkgconfig, so I add it here. +But ICU has had pkgconfig support since 4.6. Is there a reason we can't just bump oe-core to 4.6? Ross I'm not sure. Maybe we bump ICU to the latest version later. (If

Re: [OE-core] [PATCH 1/3] autotools.bbclass: use relative path for acpaths whenever possible

2012-12-03 Thread ChenQi
Hi Richard, I saw that this series of patches have been merged except this one. Is there any problem with this patch? The thing is, if we don't fix autotool.bbclass like this, building coreutils will fail with a long TMPDIR (bug#2766). I tried to build coretuils with TMPDIR of 200 char length,

Re: [OE-core] [PATCH 0/4] [V3] Add pkgconfig support to four packages

2012-12-04 Thread ChenQi
changes since commit 247b6a3754c2c50318e2a73d79981ba0e9cb560b: gstreamer, gst-plugins*: fix localdata (2012-11-28 15:25:33 +) are available in the git repository at: git://git.pokylinux.org/poky-contrib ChenQi/pkgconfig2 http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=ChenQi/pkgconfig2 Che

Re: [OE-core] [PATCH 1/3] autotools.bbclass: use relative path for acpaths whenever possible

2012-12-04 Thread ChenQi
On 12/04/2012 04:39 PM, Richard Purdie wrote: On Tue, 2012-12-04 at 10:57 +0800, ChenQi wrote: Hi Richard, I saw that this series of patches have been merged except this one. Is there any problem with this patch? The thing is, if we don't fix autotool.bbclass like this, building core

  1   2   3   4   5   6   >