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 qi.c...@windriver.com Previously, if CONFIG_FEATURE_INDIVIDUAL was enabled for busybox, yocto-based systems could start correctly. This is because if busybox is built as individual

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 qi.c...@windriver.com Previously, if CONFIG_FEATURE_INDIVIDUAL was enabled for busybox, yocto-based

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, qi.c...@windriver.com mailto:qi.c...@windriver.com wrote: From: Chen Qi qi.c...@windriver.com mailto:qi.c...@windriver.com Remove the pkg_postinst_${PN} from this recipe, as it's redundant. It

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

2013-06-13 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 qi.c...@windriver.com This patch enables us to split the busybox into two binaries, one containing suid applications, and the other containing nosuid

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

2013-06-16 Thread ChenQi
, ChenQi qi.c...@windriver.com 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

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, qi.c...@windriver.com wrote: From: Chen Qi 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 update-alternatives does. But it

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 rongqing...@windriver.com enable read-only-rootfs to make /tmp be able to be written when using sysvinit Signed-off-by: Roy.Li rongqing...@windriver.com --- meta/classes/image-live.bbclass |2 ++ 1 file changed, 2

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

2013-06-28 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 qi.c...@windriver.com 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

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:

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

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 s...@linux.intel.com 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

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 s...@linux.intel.com 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

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

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 qi.c...@windriver.com 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

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 qi.c...@windriver.com 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

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 qi.c...@windriver.com We now can have a 'one-binary' version of busybox, or 'two-binary' version of busybox, controlled by the 'BUSYBOX_SPLIT_SUID' variable. This

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

2013-07-10 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 qi.c...@windriver.com 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

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 qi.c...@windriver.com 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

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

2013-07-16 Thread ChenQi
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_settings to ChenQi/extrausers. The following changes since commit a63229917a5708de2d161aba0d67168ce0da6365

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 qi.c...@windriver.com 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

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

2013-07-16 Thread ChenQi
: 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: add a new bbclass

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 qi.c...@windriver.com Changes from V4: 1. Take into consideration the possibility of whitespaces in /etc/passwd and /etc/group 2. Change bbfatal

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, qi.c...@windriver.com 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

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 qi.c...@windriver.com 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

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

2013-07-24 Thread ChenQi
://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 still can't get

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 muhammad_shak...@mentor.com 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:

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 qi.c...@windriver.com To avoid generating ssh keys every time a system with read-only rootfs

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, qi.c...@windriver.com wrote: From: Chen Qi qi.c...@windriver.com 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

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, qi.c...@windriver.com 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

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 qi.c...@windriver.com This diretory needs to be writable, the following error will appear at system start-up. /etc/rc5.d/S20irattach: /etc/sysconfig/irda: Read-only

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 qi.c...@windriver.com 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

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 qi.c...@windriver.com 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

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

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

2013-08-01 Thread ChenQi
-extraction script 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

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 qi.c...@windriver.com 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

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

2013-08-02 Thread ChenQi
repository at: git://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

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

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 please

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, qi.c...@windriver.com mailto:qi.c...@windriver.com wrote: From: Chen Qi qi.c...@windriver.com mailto:qi.c...@windriver.com The init script for irda writes configuration items to /etc/sysconfig/irda

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 qi.c...@windriver.com 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, qi.c...@windriver.com mailto:qi.c...@windriver.com wrote: From: Chen Qi

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 qi.c...@windriver.com 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 qi.c...@windriver.com

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

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:

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:

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

2013-08-18 Thread ChenQi
://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 ChenQi/read-only

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 qi.c...@windriver.com 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

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

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 rongqing...@windriver.com Signed-off-by: Roy.Li rongqing...@windriver.com --- meta/recipes-core/systemd/systemd/run-ptest |2 ++ meta/recipes-core/systemd/systemd_204.bb| 18 +- 2 files changed,

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

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

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 qi.c...@windriver.com 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

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
file descriptors, 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

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

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 qi.c...@windriver.com The do_install_ptest_base function uses 'type -t' command to check

Re: [OE-core] [PATCH 4/6] populate-volatile.sh: improve this script

2013-02-17 Thread ChenQi
On 02/06/2013 10:40 PM, Richard Purdie wrote: On Mon, 2013-01-28 at 17:05 +0800, qi.c...@windriver.com wrote: From: Chen Qi qi.c...@windriver.com Here's a list of improvements: 1) Make it run correctly at rootfs time. 2) Handle link config items more reasonably. 3) Support read-only rootfs. 4)

Re: [OE-core] [PATCH 1/4] image.bbclass: add a hook funtion to support readonly rootfs

2013-02-19 Thread ChenQi
On 02/19/2013 07:39 PM, Bernhard Reutner-Fischer wrote: On 19 February 2013 08:38, qi.c...@windriver.com wrote: From: Chen Qi qi.c...@windriver.com read_only_rootfs_hook: this funtion is invoked only if 'read-only-rootfs' is in IMAGE_FEATURES. It tweaks some basic files and makes populate-

Re: [OE-core] [PATCH 3/4] populate-volatile.sh: add ROOT_DIR variable to support running at rootfs time

2013-02-19 Thread ChenQi
On 02/19/2013 07:56 PM, Bernhard Reutner-Fischer wrote: On 19 February 2013 08:38, qi.c...@windriver.com wrote: From: Chen Qi qi.c...@windriver.com For populate-volatile.sh script to run correctly both at rootfs time and at system boot time, it needs to be aware of which situation it is now

Re: [OE-core] [PATCH 1/1] hwclock.sh: check for the existence of /etc/default/rcS

2013-02-19 Thread ChenQi
On 02/19/2013 07:10 PM, Bernhard Reutner-Fischer wrote: On 19 February 2013 08:59, qi.c...@windriver.com wrote: From: Chen Qi qi.c...@windriver.com /etc/default/rcS might be missing if the init manager is not sysvinit. So we have to check for the existence of this file before sourcing it.

Re: [OE-core] [PATCH V2 4/5] populate-volatile.sh: improve the handling of link config items

2013-02-20 Thread ChenQi
On 02/20/2013 05:18 PM, Bernhard Reutner-Fischer wrote: On 20 February 2013 08:16, ChenQi qi.c...@windriver.com wrote: On 02/20/2013 03:10 PM, Bernhard Reutner-Fischer wrote: On 20 February 2013 06:55:33 qi.c...@windriver.com wrote: From: Chen Qi qi.c...@windriver.com Previously, if there's

Re: [OE-core] [PATCH V2 5/5] populate-volatile.sh: remove trailing whitespaces

2013-02-20 Thread ChenQi
On 02/20/2013 10:19 PM, Bernhard Reutner-Fischer wrote: On 20 February 2013 06:55, qi.c...@windriver.com wrote: From: Chen Qi qi.c...@windriver.com Remove trailing whitespaces. Signed-off-by: Chen Qi qi.c...@windriver.com --- .../initscripts-1.0/populate-volatile.sh |6 +++---

Re: [OE-core] [PATCH 1/1] sysvinit: start .sh scripts correctly

2013-02-25 Thread ChenQi
On 02/26/2013 10:00 AM, Saul Wold wrote: On 02/25/2013 05:39 PM, qi.c...@windriver.com wrote: From: Chen Qi qi.c...@windriver.com Previously, scripts which end with '.sh' were sourced, so the arguments like 'start' and 'stop' were just ignored. This resulted in some init scripts not being

Re: [OE-core] [PATCH 1/1] sysvinit: start .sh scripts correctly

2013-02-26 Thread ChenQi
On 02/26/2013 07:04 PM, Burton, Ross wrote: On 26 February 2013 01:39, qi.c...@windriver.com wrote: - - case $1 in - *.sh) - # Source shell script for speed. - ( - trap - INT QUIT TSTP - scriptname=$1 -

Re: [OE-core] [PATCH 0/1] meta-skeleton: specify the correct path to core-image-sato.bb

2013-03-20 Thread ChenQi
://git.pokylinux.org/poky-contrib ChenQi/meta-skeleton http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=ChenQi/meta-skeleton Chen Qi (1): meta-skeleton: specify the correct path to core-image-sato.bb .../images/core-image-multilib-example.bb |2 +- 1 file changed, 1 insertion

Re: [OE-core] [PATCH 1/1] xserver-nodm: change start runlevel from 2 5 to 2 3 4 5

2013-03-27 Thread ChenQi
On 03/27/2013 07:11 AM, Richard Purdie wrote: On Tue, 2013-03-26 at 15:18 +0800, qi.c...@windriver.com wrote: From: Chen Qi qi.c...@windriver.com This script is intended to start at runlevel 2 3 4 5, as specified in its HEAD INFO area. Perhaps the header is wrong? The fact that it was not

Re: [OE-core] [PATCH] initscripts: RDEPENDS sed to remove sed: command not found.

2013-03-27 Thread ChenQi
On 03/28/2013 01:15 AM, Anders Roxell wrote: On Tue, Mar 26, 2013 at 10:17 AM, ChenQi qi.c...@windriver.com mailto:qi.c...@windriver.com wrote: On 03/26/2013 04:20 PM, anders.rox...@gmail.com mailto:anders.rox...@gmail.com wrote: From: Anders Roxell anders.rox...@enea.com

Re: [OE-core] [oe-core v3] rpm: split out run-postinsts

2013-04-01 Thread ChenQi
On 03/28/2013 12:18 PM, Zhenhua Luo wrote: 1. Split out run-postinsts script into separated package, sometimes only the postinsts script is required to run all postinsts scripts in /etc/rpm-postinsts/ instead of the whole rpm package. 2. Set ROOTFS_PKGMANAGE_BOOTSTRAP to rpm-postinsts

Re: [OE-core] [PATCH 1/1] core-image-sato.bb: increase free space size

2013-04-01 Thread ChenQi
On 04/02/2013 10:59 AM, Khem Raj wrote: On Apr 1, 2013, at 7:30 PM, qi.c...@windriver.com wrote: + +# Override IMAGE_OVERHEAD_FACTOR to increase the free space size on sato images. +IMAGE_OVERHEAD_FACTOR = 2.0 what is the size of image before and after ? roughly 100M - 300M. smart

Re: [OE-core] [PATCH 1/1] core-image-sato.bb: increase free space size

2013-04-03 Thread ChenQi
On 04/02/2013 06:15 PM, Gary Thomas wrote: On 2013-04-01 20:30, qi.c...@windriver.com wrote: From: Chen Qi qi.c...@windriver.com Sato images, by default, have about 100M free space. This is not enough and it blocks automatic testing. This patch fixes this problem by overriding the

Re: [OE-core] [PATCH] dpkg, opkg, rpm-postinst: fix overwriting the run-postinstall script

2013-04-11 Thread ChenQi
On 04/11/2013 02:42 PM, Laurentiu Palcu wrote: If multiple package managers are installed in the image, they will overwrite each other's run-postinsts script, resulting in postinstalls not beeing run at all at first boot. What this patch does: * checks whether opkg/dpks/rpm is actually used

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

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 qi.c...@windriver.com 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

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 qi.c...@windriver.com Enable postinst logging by checking the configuration in /etc/ default/postinst. In this way, the postinst logging is enabled if 'debug-tweaks' is in

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 qi.c...@windriver.com Enable postinst logging by checking the configuration in /etc/ default/postinst. In this way, the postinst logging is enabled if 'debug-tweaks' is in

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

2013-05-28 Thread ChenQi
-manual: Added xmlto and dblatex 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

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

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 qi.c...@windriver.com 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

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 qi.c...@windriver.com If a package inherits update-rc.d.bbclass

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 qi.c...@windriver.com 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

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 fix. and I stand

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

2013-09-26 Thread ChenQi
...@windriver.com 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

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: qi.chen-cwa4wttnnzf54taoqty...@public.gmane.org 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

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 qi.c...@windriver.com 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

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 \

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

2013-10-16 Thread ChenQi
-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 No Output for buildhistory-diff *) With the patch Below is the output. chenqi@pek-qchen1-u12u4:~/poky/build$ buildhistory

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 qi.c...@windriver.com 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

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 qi.c...@windriver.com 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

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, qi.c...@windriver.com wrote: From: Chen Qi qi.c...@windriver.com 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

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, qi.c...@windriver.com wrote: From: Chen Qi qi.c...@windriver.com If we build a minimal image with iproute2 installed, the following error

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 around

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:

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

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, qi.c...@windriver.com 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

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 1/1] sanity.bbclass: check for validity of TMPDIR

2013-11-14 Thread ChenQi
On 11/14/2013 09:12 PM, Richard Purdie wrote: On Thu, 2013-11-14 at 21:07 +0800, qi.c...@windriver.com wrote: From: Chen Qi qi.c...@windriver.com TMPDIR must be an absolute path, otherwise, the build will fail. Special characters in TMPDIR will also cause build failures. This patch enables

Re: [OE-core] [PATCH 0/1] sanity.bbclass: check for validity of TMPDIR

2013-11-14 Thread ChenQi
that I include it into the valid char set? ** TESTING RECORDS Test Case 1: use `bitbake minimal' on paths containing special chars Here's the output. === TMPDIR = /home/chenqi/wrlinux-build/directory-names/dir-with-special

Re: [OE-core] [PATCH 1/1] sanity.bbclass: check for validity of TMPDIR

2013-11-14 Thread ChenQi
On 11/15/2013 02:01 AM, Saul Wold wrote: On 11/14/2013 07:49 AM, Mark Hatle wrote: On 11/14/13, 7:42 AM, Phil Blundell wrote: On Thu, 2013-11-14 at 06:27 -0700, Gary Thomas wrote: Also, is - actually valid? I seem to recall having problems when my build tree had the hyphen (-) in the path.

  1   2   3   4   5   6   >