[OE-core] [PATCH 4/4] oprofile: use dynamic root home directory

Martin Jansa martin.jansa at gmail.com
Wed Dec 19 11:06:04 UTC 2012


On Wed, Dec 19, 2012 at 05:39:39PM +0800, Kang Kai wrote:
> Use stub "ROOTHOME" to replace "/home/root" in oprofile-root.patch, then
> we can substitute it with configured root home directory.
> 
> Signed-off-by: Kang Kai <kai.kang at windriver.com>
> ---
>  meta/recipes-kernel/oprofile/oprofile.inc          |    1 +
>  .../oprofile/oprofile/oprofile-root.patch          |   23 ++++++++++++-------
>  2 files changed, 15 insertions(+), 9 deletions(-)
> 
> diff --git a/meta/recipes-kernel/oprofile/oprofile.inc b/meta/recipes-kernel/oprofile/oprofile.inc
> index b3ce852..0977065 100644
> --- a/meta/recipes-kernel/oprofile/oprofile.inc
> +++ b/meta/recipes-kernel/oprofile/oprofile.inc
> @@ -31,6 +31,7 @@ inherit autotools
>  
>  EXTRA_OECONF = "--with-kernel-support --without-x"
>  do_configure () {
> +        find . -type f | xargs sed -i 's#ROOTHOME#${ROOT_HOME}#'
>  	cp ${WORKDIR}/acinclude.m4 ${S}/
>  	autotools_do_configure

You should use tab here.

Cheers,

>  }
> diff --git a/meta/recipes-kernel/oprofile/oprofile/oprofile-root.patch b/meta/recipes-kernel/oprofile/oprofile/oprofile-root.patch
> index db63a8e..f3fe4c9 100644
> --- a/meta/recipes-kernel/oprofile/oprofile/oprofile-root.patch
> +++ b/meta/recipes-kernel/oprofile/oprofile/oprofile-root.patch
> @@ -4,6 +4,11 @@ Upstream-Status: inappropriate [OE specific]
>  
>  Signed-off-by: Mark Hatle <mark.hatle at windriver.com>
>  
> +Replace "/home/root" with stub "ROOTHOME", then substitute it with
> +real root home directory which is configured dymanically.
> +
> +Signed-off-by: Kang Kai <kai.kang at windriver.com>
> +
>  diff -ur oprofile-0.9.7.orig/doc/opcontrol.1.in oprofile-0.9.7/doc/opcontrol.1.in
>  --- oprofile-0.9.7.orig/doc/opcontrol.1.in	2011-07-04 21:25:04.000000000 -0500
>  +++ oprofile-0.9.7/doc/opcontrol.1.in	2012-07-02 14:02:16.358865420 -0500
> @@ -12,7 +17,7 @@ diff -ur oprofile-0.9.7.orig/doc/opcontrol.1.in oprofile-0.9.7/doc/opcontrol.1.i
>   .SH FILES
>   .TP
>  -.I /root/.oprofile/daemonrc
> -+.I /home/root/.oprofile/daemonrc
> ++.I ROOTHOME/.oprofile/daemonrc
>   Configuration file for opcontrol
>   .TP
>   .I /var/lib/oprofile/samples/
> @@ -24,7 +29,7 @@ diff -ur oprofile-0.9.7.orig/doc/oprofile.1 oprofile-0.9.7/doc/oprofile.1
>   Configuration files
>   .TP
>  -.I /root/.oprofile/daemonrc
> -+.I /home/root/.oprofile/daemonrc
> ++.I ROOTHOME/.oprofile/daemonrc
>   Configuration file for opcontrol
>   .TP
>   .I /opt/oprofile-0.9.7-rc3/share/oprofile/
> @@ -36,7 +41,7 @@ diff -ur oprofile-0.9.7.orig/doc/oprofile.1.in oprofile-0.9.7/doc/oprofile.1.in
>   Configuration files
>   .TP
>  -.I /root/.oprofile/daemonrc
> -+.I /home/root/.oprofile/daemonrc
> ++.I ROOTHOME/.oprofile/daemonrc
>   Configuration file for opcontrol
>   .TP
>   .I @prefix@/share/oprofile/
> @@ -48,7 +53,7 @@ diff -ur oprofile-0.9.7.orig/doc/oprofile.html oprofile-0.9.7/doc/oprofile.html
>                   <p>
>   		    Followed by list arguments for profiling set up. List of arguments
>  -		    saved in <code class="filename">/root/.oprofile/daemonrc</code>.
> -+		    saved in <code class="filename">/home/root/.oprofile/daemonrc</code>.
> ++		    saved in <code class="filename">ROOTHOME/.oprofile/daemonrc</code>.
>   		    Giving this option is not necessary; you can just directly pass one
>   		    of the setup options, e.g. <span class="command"><strong>opcontrol --no-vmlinux</strong></span>.
>   		  </p>
> @@ -57,7 +62,7 @@ diff -ur oprofile-0.9.7.orig/doc/oprofile.html oprofile-0.9.7/doc/oprofile.html
>                   <p>
>   		    Start data collection with either arguments provided by <code class="option">--setup</code>
>  -		or information saved in <code class="filename">/root/.oprofile/daemonrc</code>. Specifying
> -+		or information saved in <code class="filename">/home/root/.oprofile/daemonrc</code>. Specifying
> ++		or information saved in <code class="filename">ROOTHOME/.oprofile/daemonrc</code>. Specifying
>   		the addition <code class="option">--verbose</code> makes the daemon generate lots of debug data
>   		whilst it is running.
>   		</p>
> @@ -69,7 +74,7 @@ diff -ur oprofile-0.9.7.orig/doc/oprofile.xml oprofile-0.9.7/doc/oprofile.xml
>   		<listitem><para>
>   		    Followed by list arguments for profiling set up. List of arguments
>  -		    saved in <filename>/root/.oprofile/daemonrc</filename>.
> -+		    saved in <filename>/home/root/.oprofile/daemonrc</filename>.
> ++		    saved in <filename>ROOTHOME/.oprofile/daemonrc</filename>.
>   		    Giving this option is not necessary; you can just directly pass one
>   		    of the setup options, e.g. <command>opcontrol --no-vmlinux</command>.
>   		  </para></listitem>
> @@ -78,7 +83,7 @@ diff -ur oprofile-0.9.7.orig/doc/oprofile.xml oprofile-0.9.7/doc/oprofile.xml
>   		<listitem><para>
>   		    Start data collection with either arguments provided by <option>--setup</option>
>  -		or information saved in <filename>/root/.oprofile/daemonrc</filename>. Specifying
> -+		or information saved in <filename>/home/root/.oprofile/daemonrc</filename>. Specifying
> ++		or information saved in <filename>ROOTHOME/.oprofile/daemonrc</filename>. Specifying
>   		the addition <option>--verbose</option> makes the daemon generate lots of debug data
>   		whilst it is running.
>   		</para></listitem>
> @@ -90,7 +95,7 @@ diff -ur oprofile-0.9.7.orig/gui/oprof_start_util.cpp oprofile-0.9.7/gui/oprof_s
>   string const get_config_dir()
>   {
>  -	return "/root";
> -+	return "/home/root";
> ++	return "ROOTHOME";
>   }
>   
>   string daemon_pid;
> @@ -102,7 +107,7 @@ diff -ur oprofile-0.9.7.orig/utils/opcontrol oprofile-0.9.7/utils/opcontrol
>   
>   	# location for daemon setup information
>  -	SETUP_DIR="/root/.oprofile"
> -+	SETUP_DIR="/home/root/.oprofile"
> ++	SETUP_DIR="ROOTHOME/.oprofile"
>   	SETUP_FILE="$SETUP_DIR/daemonrc"
>   	SEC_SETUP_FILE="$SETUP_DIR/daemonrc_new"
>   
> -- 
> 1.7.5.4
> 
> 
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core at lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core

-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa at gmail.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.openembedded.org/pipermail/openembedded-core/attachments/20121219/fd1c2964/attachment-0002.sig>


More information about the Openembedded-core mailing list