[PATCH] scripts: add new script

[PATCH] scripts: add new script

[Ed Bartosh]

Added 'native' convenience shell script to run native tools.
Example of usage:
  > bitbake bmap-tools-native
  > native bmaptool --version

Signed-off-by: Ed Bartosh <ed.bartosh@linux.intel.com>
---
 scripts/native | 48 ++++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 48 insertions(+)
 create mode 100755 scripts/native

diff --git a/scripts/native b/scripts/native
new file mode 100755
index 0000000..5e904b0
--- /dev/null
+++ b/scripts/native
@@ -0,0 +1,48 @@
+#!/bin/sh
+#
+# Copyright (c) 2016,  Intel Corporation.
+# All Rights Reserved
+#
+# This program is free software;  you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation; either version 2 of the License, or
+# (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY;  without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See
+# the GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program;  if not, see <http://www.gnu.org/licenses/>
+#
+
+#
+# This script is for running tools from native oe sysroot
+#
+
+if [ $# -lt 1 -o "$1" == '--help' -o "$1" == '-h' ]; then
+    echo "Usage: $0 <native tool> [parameters]"
+    exit 1
+fi
+
+SYSROOT_SETUP_SCRIPT=`which oe-find-native-sysroot 2> /dev/null`
+if [ -z "$SYSROOT_SETUP_SCRIPT" ]; then
+        echo "Error: Unable to find oe-find-native-sysroot script"
+        exit 1
+fi
+. $SYSROOT_SETUP_SCRIPT
+
+OLDPATH=$PATH
+
+# look for a tool only in native sysroot
+PATH=$OECORE_NATIVE_SYSROOT/usr/bin:$OECORE_NATIVE_SYSROOT/bin:$OECORE_NATIVE_SYSROOT/usr/sbin:$OECORE_NATIVE_SYSROOT/sbin
+tool=`/usr/bin/which $1 2>/dev/null`
+
+if [ -n "$tool" ] ; then
+    # add old path to allow usage of host tools
+    PATH=$PATH:$OLD_PATH $@
+else
+    echo "Error: Unable to find '$1' in native sysroot"
+    exit 1
+fi
-- 
2.1.4

Re: [PATCH] scripts: add new script

[Christopher Larson]

[-- Attachment #1: Type: text/plain, Size: 594 bytes --]

On Wed, Sep 28, 2016 at 2:55 AM, Ed Bartosh <ed.bartosh@linux.intel.com>
wrote:

> Added 'native' convenience shell script to run native tools.
> Example of usage:
>   > bitbake bmap-tools-native
>   > native bmaptool --version
>
> Signed-off-by: Ed Bartosh <ed.bartosh@linux.intel.com>
>

‘add new script’ is not a useful commit message summary. You don’t even
mention what script you add until the message body.
-- 
Christopher Larson
clarson at kergoth dot com
Founder - BitBake, OpenEmbedded, OpenZaurus
Maintainer - Tslib
Senior Software Engineer, Mentor Graphics

[-- Attachment #2: Type: text/html, Size: 1102 bytes --]

[PATCH v2] scripts: add new script 'native'

[Ed Bartosh]

Added 'native' convenience shell script to run native tools.
Example of usage:
  > bitbake bmap-tools-native
  > native bmaptool --version

Signed-off-by: Ed Bartosh <ed.bartosh@linux.intel.com>
---
 scripts/native | 48 ++++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 48 insertions(+)
 create mode 100755 scripts/native

diff --git a/scripts/native b/scripts/native
new file mode 100755
index 0000000..496e34f
--- /dev/null
+++ b/scripts/native
@@ -0,0 +1,48 @@
+#!/bin/sh
+#
+# Copyright (c) 2016,  Intel Corporation.
+# All Rights Reserved
+#
+# This program is free software;  you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation; either version 2 of the License, or
+# (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY;  without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See
+# the GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program;  if not, see <http://www.gnu.org/licenses/>
+#
+
+#
+# This script is for running tools from native oe sysroot
+#
+
+if [ $# -lt 1 -o "$1" = '--help' -o "$1" = '-h' ] ; then
+    echo "Usage: $0 <native tool> [parameters]"
+    exit 1
+fi
+
+SYSROOT_SETUP_SCRIPT=`which oe-find-native-sysroot 2> /dev/null`
+if [ -z "$SYSROOT_SETUP_SCRIPT" ]; then
+        echo "Error: Unable to find oe-find-native-sysroot script"
+        exit 1
+fi
+. $SYSROOT_SETUP_SCRIPT
+
+OLDPATH=$PATH
+
+# look for a tool only in native sysroot
+PATH=$OECORE_NATIVE_SYSROOT/usr/bin:$OECORE_NATIVE_SYSROOT/bin:$OECORE_NATIVE_SYSROOT/usr/sbin:$OECORE_NATIVE_SYSROOT/sbin
+tool=`/usr/bin/which $1 2>/dev/null`
+
+if [ -n "$tool" ] ; then
+    # add old path to allow usage of host tools
+    PATH=$PATH:$OLD_PATH $@
+else
+    echo "Error: Unable to find '$1' in native sysroot"
+    exit 1
+fi
-- 
2.1.4

Re: [PATCH v2] scripts: add new script 'native'

[Jérémy Rosen]

"native" might be a bit too generic...

could I suggest oe_native-run or bitbake-native ?

something that says it's yocto-galaxy specific and can be autocompleted 
with a reasonable prefix...


Jérémy Rosen


On 30/09/2016 17:02, Ed Bartosh wrote:
> Added 'native' convenience shell script to run native tools.
> Example of usage:
>    > bitbake bmap-tools-native
>    > native bmaptool --version
>
> Signed-off-by: Ed Bartosh <ed.bartosh@linux.intel.com>
> ---
>   scripts/native | 48 ++++++++++++++++++++++++++++++++++++++++++++++++
>   1 file changed, 48 insertions(+)
>   create mode 100755 scripts/native
>
> diff --git a/scripts/native b/scripts/native
> new file mode 100755
> index 0000000..496e34f
> --- /dev/null
> +++ b/scripts/native
> @@ -0,0 +1,48 @@
> +#!/bin/sh
> +#
> +# Copyright (c) 2016,  Intel Corporation.
> +# All Rights Reserved
> +#
> +# This program is free software;  you can redistribute it and/or modify
> +# it under the terms of the GNU General Public License as published by
> +# the Free Software Foundation; either version 2 of the License, or
> +# (at your option) any later version.
> +#
> +# This program is distributed in the hope that it will be useful,
> +# but WITHOUT ANY WARRANTY;  without even the implied warranty of
> +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See
> +# the GNU General Public License for more details.
> +#
> +# You should have received a copy of the GNU General Public License
> +# along with this program;  if not, see <http://www.gnu.org/licenses/>
> +#
> +
> +#
> +# This script is for running tools from native oe sysroot
> +#
> +
> +if [ $# -lt 1 -o "$1" = '--help' -o "$1" = '-h' ] ; then
> +    echo "Usage: $0 <native tool> [parameters]"
> +    exit 1
> +fi
> +
> +SYSROOT_SETUP_SCRIPT=`which oe-find-native-sysroot 2> /dev/null`
> +if [ -z "$SYSROOT_SETUP_SCRIPT" ]; then
> +        echo "Error: Unable to find oe-find-native-sysroot script"
> +        exit 1
> +fi
> +. $SYSROOT_SETUP_SCRIPT
> +
> +OLDPATH=$PATH
> +
> +# look for a tool only in native sysroot
> +PATH=$OECORE_NATIVE_SYSROOT/usr/bin:$OECORE_NATIVE_SYSROOT/bin:$OECORE_NATIVE_SYSROOT/usr/sbin:$OECORE_NATIVE_SYSROOT/sbin
> +tool=`/usr/bin/which $1 2>/dev/null`
> +
> +if [ -n "$tool" ] ; then
> +    # add old path to allow usage of host tools
> +    PATH=$PATH:$OLD_PATH $@
> +else
> +    echo "Error: Unable to find '$1' in native sysroot"
> +    exit 1
> +fi

Re: [PATCH v2] scripts: add new script 'native'

[Ulf Magnusson]

On Sun, Oct 2, 2016 at 4:15 PM, Jérémy Rosen <jeremy.rosen@smile.fr> wrote:
> "native" might be a bit too generic...
>
> could I suggest oe_native-run or bitbake-native ?
>
> something that says it's yocto-galaxy specific and can be autocompleted with
> a reasonable prefix...
>
>
> Jérémy Rosen

I think overly namespacing things that aren't likely to clash in
practice gets annoying. I would've gone for "run-native" (more
descriptive and English-like, less generic), but how about something
simple and easy to type like "oe-native" or "oe-run-native" if a
prefix is added? That's consistent with the naming scheme used for
many other scripts in scripts/ too.

Cheers,
Ulf

>
>
>
> On 30/09/2016 17:02, Ed Bartosh wrote:
>>
>> Added 'native' convenience shell script to run native tools.
>> Example of usage:
>>    > bitbake bmap-tools-native
>>    > native bmaptool --version
>>
>> Signed-off-by: Ed Bartosh <ed.bartosh@linux.intel.com>
>> ---
>>   scripts/native | 48 ++++++++++++++++++++++++++++++++++++++++++++++++
>>   1 file changed, 48 insertions(+)
>>   create mode 100755 scripts/native
>>
>> diff --git a/scripts/native b/scripts/native
>> new file mode 100755
>> index 0000000..496e34f
>> --- /dev/null
>> +++ b/scripts/native
>> @@ -0,0 +1,48 @@
>> +#!/bin/sh
>> +#
>> +# Copyright (c) 2016,  Intel Corporation.
>> +# All Rights Reserved
>> +#
>> +# This program is free software;  you can redistribute it and/or modify
>> +# it under the terms of the GNU General Public License as published by
>> +# the Free Software Foundation; either version 2 of the License, or
>> +# (at your option) any later version.
>> +#
>> +# This program is distributed in the hope that it will be useful,
>> +# but WITHOUT ANY WARRANTY;  without even the implied warranty of
>> +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See
>> +# the GNU General Public License for more details.
>> +#
>> +# You should have received a copy of the GNU General Public License
>> +# along with this program;  if not, see <http://www.gnu.org/licenses/>
>> +#
>> +
>> +#
>> +# This script is for running tools from native oe sysroot
>> +#
>> +
>> +if [ $# -lt 1 -o "$1" = '--help' -o "$1" = '-h' ] ; then
>> +    echo "Usage: $0 <native tool> [parameters]"
>> +    exit 1
>> +fi
>> +
>> +SYSROOT_SETUP_SCRIPT=`which oe-find-native-sysroot 2> /dev/null`
>> +if [ -z "$SYSROOT_SETUP_SCRIPT" ]; then
>> +        echo "Error: Unable to find oe-find-native-sysroot script"
>> +        exit 1
>> +fi
>> +. $SYSROOT_SETUP_SCRIPT
>> +
>> +OLDPATH=$PATH
>> +
>> +# look for a tool only in native sysroot
>>
>> +PATH=$OECORE_NATIVE_SYSROOT/usr/bin:$OECORE_NATIVE_SYSROOT/bin:$OECORE_NATIVE_SYSROOT/usr/sbin:$OECORE_NATIVE_SYSROOT/sbin
>> +tool=`/usr/bin/which $1 2>/dev/null`
>> +
>> +if [ -n "$tool" ] ; then
>> +    # add old path to allow usage of host tools
>> +    PATH=$PATH:$OLD_PATH $@
>> +else
>> +    echo "Error: Unable to find '$1' in native sysroot"
>> +    exit 1
>> +fi
>
>
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [PATCH v2] scripts: add new script 'native'

[Ulf Magnusson]

On Sun, Oct 2, 2016 at 4:30 PM, Ulf Magnusson <ulfalizer@gmail.com> wrote:
> On Sun, Oct 2, 2016 at 4:15 PM, Jérémy Rosen <jeremy.rosen@smile.fr> wrote:
>> "native" might be a bit too generic...
>>
>> could I suggest oe_native-run or bitbake-native ?
>>
>> something that says it's yocto-galaxy specific and can be autocompleted with
>> a reasonable prefix...
>>
>>
>> Jérémy Rosen
>
> I think overly namespacing things that aren't likely to clash in
> practice gets annoying. I would've gone for "run-native" (more
> descriptive and English-like, less generic), but how about something
> simple and easy to type like "oe-native" or "oe-run-native" if a
> prefix is added? That's consistent with the naming scheme used for
> many other scripts in scripts/ too.
>
> Cheers,
> Ulf

I just noticed that the script's already in though, so not much point
arguing. ;)

Cheers,
Ulf

>
>>
>>
>>
>> On 30/09/2016 17:02, Ed Bartosh wrote:
>>>
>>> Added 'native' convenience shell script to run native tools.
>>> Example of usage:
>>>    > bitbake bmap-tools-native
>>>    > native bmaptool --version
>>>
>>> Signed-off-by: Ed Bartosh <ed.bartosh@linux.intel.com>
>>> ---
>>>   scripts/native | 48 ++++++++++++++++++++++++++++++++++++++++++++++++
>>>   1 file changed, 48 insertions(+)
>>>   create mode 100755 scripts/native
>>>
>>> diff --git a/scripts/native b/scripts/native
>>> new file mode 100755
>>> index 0000000..496e34f
>>> --- /dev/null
>>> +++ b/scripts/native
>>> @@ -0,0 +1,48 @@
>>> +#!/bin/sh
>>> +#
>>> +# Copyright (c) 2016,  Intel Corporation.
>>> +# All Rights Reserved
>>> +#
>>> +# This program is free software;  you can redistribute it and/or modify
>>> +# it under the terms of the GNU General Public License as published by
>>> +# the Free Software Foundation; either version 2 of the License, or
>>> +# (at your option) any later version.
>>> +#
>>> +# This program is distributed in the hope that it will be useful,
>>> +# but WITHOUT ANY WARRANTY;  without even the implied warranty of
>>> +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See
>>> +# the GNU General Public License for more details.
>>> +#
>>> +# You should have received a copy of the GNU General Public License
>>> +# along with this program;  if not, see <http://www.gnu.org/licenses/>
>>> +#
>>> +
>>> +#
>>> +# This script is for running tools from native oe sysroot
>>> +#
>>> +
>>> +if [ $# -lt 1 -o "$1" = '--help' -o "$1" = '-h' ] ; then
>>> +    echo "Usage: $0 <native tool> [parameters]"
>>> +    exit 1
>>> +fi
>>> +
>>> +SYSROOT_SETUP_SCRIPT=`which oe-find-native-sysroot 2> /dev/null`
>>> +if [ -z "$SYSROOT_SETUP_SCRIPT" ]; then
>>> +        echo "Error: Unable to find oe-find-native-sysroot script"
>>> +        exit 1
>>> +fi
>>> +. $SYSROOT_SETUP_SCRIPT
>>> +
>>> +OLDPATH=$PATH
>>> +
>>> +# look for a tool only in native sysroot
>>>
>>> +PATH=$OECORE_NATIVE_SYSROOT/usr/bin:$OECORE_NATIVE_SYSROOT/bin:$OECORE_NATIVE_SYSROOT/usr/sbin:$OECORE_NATIVE_SYSROOT/sbin
>>> +tool=`/usr/bin/which $1 2>/dev/null`
>>> +
>>> +if [ -n "$tool" ] ; then
>>> +    # add old path to allow usage of host tools
>>> +    PATH=$PATH:$OLD_PATH $@
>>> +else
>>> +    echo "Error: Unable to find '$1' in native sysroot"
>>> +    exit 1
>>> +fi
>>
>>
>> --
>> _______________________________________________
>> Openembedded-core mailing list
>> Openembedded-core@lists.openembedded.org
>> http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [PATCH v2] scripts: add new script 'native'

[Paul Eggleton]

On Sun, 02 Oct 2016 16:44:01 Ulf Magnusson wrote:
> On Sun, Oct 2, 2016 at 4:30 PM, Ulf Magnusson <ulfalizer@gmail.com> wrote:
> > On Sun, Oct 2, 2016 at 4:15 PM, Jérémy Rosen <jeremy.rosen@smile.fr> 
wrote:
> >> "native" might be a bit too generic...
> >> 
> >> could I suggest oe_native-run or bitbake-native ?
> >> 
> >> something that says it's yocto-galaxy specific and can be autocompleted
> >> with a reasonable prefix...
> > 
> > I think overly namespacing things that aren't likely to clash in
> > practice gets annoying. I would've gone for "run-native" (more
> > descriptive and English-like, less generic), but how about something
> > simple and easy to type like "oe-native" or "oe-run-native" if a
> > prefix is added? That's consistent with the naming scheme used for
> > many other scripts in scripts/ too.
> 
> I just noticed that the script's already in though, so not much point
> arguing. ;)

There is - just because it got merged doesn't mean it can't be renamed, in 
fact if we are going to rename it then now is the time to do that, rather than 
waiting until people are used to the current name.

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre

Re: [PATCH v2] scripts: add new script 'native'

[Ed Bartosh]

On Sun, Oct 02, 2016 at 04:30:58PM +0200, Ulf Magnusson wrote:
> On Sun, Oct 2, 2016 at 4:15 PM, Jérémy Rosen <jeremy.rosen@smile.fr> wrote:
> > "native" might be a bit too generic...
> >
> > could I suggest oe_native-run or bitbake-native ?
> >
> > something that says it's yocto-galaxy specific and can be autocompleted with
> > a reasonable prefix...
> >
> >
> > Jérémy Rosen
> 
> I think overly namespacing things that aren't likely to clash in
> practice gets annoying. I would've gone for "run-native" (more
> descriptive and English-like, less generic), but how about something
> simple and easy to type like "oe-native" or "oe-run-native" if a
> prefix is added? That's consistent with the naming scheme used for
> many other scripts in scripts/ too.
> 

I'm ok to rename it. I agree that 'native' may look too generic, but
I don't know about any existing tool with this name.

Anyway, let me know which name I should rename it to. Feel free to send a
patch too. I personally like 'oe-native' more than other proposals.

--
Regards,
Ed

Re: [PATCH v2] scripts: add new script 'native'

[Randy MacLeod]

On 2016-10-02 03:37 PM, Paul Eggleton wrote:
>> "native" might be a bit too generic...
>> > >>
>> > >> could I suggest oe_native-run or bitbake-native ?
>> > >>
>> > >> something that says it's yocto-galaxy specific and can be autocompleted
>> > >> with a reasonable prefix...
>> >

I also prefer an oe prefix.
oe-native, oe-run-native, or oe-exec-native would work for me.


-- 
# Randy MacLeod. SMTS, Linux, Wind River
Direct: 613.963.1350 | 350 Terry Fox Drive, Suite 200, Ottawa, ON, 
Canada, K2K 2W5

Re: [PATCH v2] scripts: add new script 'native'

[Paul Eggleton]

On Tue, 04 Oct 2016 12:22:56 Randy MacLeod wrote:
> On 2016-10-02 03:37 PM, Paul Eggleton wrote:
> >> "native" might be a bit too generic...
> >> 
> >> > >> could I suggest oe_native-run or bitbake-native ?
> >> > >> 
> >> > >> something that says it's yocto-galaxy specific and can be
> >> > >> autocompleted
> >> > >> with a reasonable prefix...
> 
> I also prefer an oe prefix.
> oe-native, oe-run-native, or oe-exec-native would work for me.

Well, someone needs to send a patch, any volunteers?

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre

Re: [PATCH v2] scripts: add new script 'native'

[Ulf Magnusson]

On Thu, Oct 6, 2016 at 5:02 AM, Paul Eggleton
<paul.eggleton@linux.intel.com> wrote:
> On Tue, 04 Oct 2016 12:22:56 Randy MacLeod wrote:
>> On 2016-10-02 03:37 PM, Paul Eggleton wrote:
>> >> "native" might be a bit too generic...
>> >>
>> >> > >> could I suggest oe_native-run or bitbake-native ?
>> >> > >>
>> >> > >> something that says it's yocto-galaxy specific and can be
>> >> > >> autocompleted
>> >> > >> with a reasonable prefix...
>>
>> I also prefer an oe prefix.
>> oe-native, oe-run-native, or oe-exec-native would work for me.
>
> Well, someone needs to send a patch, any volunteers?
>
> Cheers,
> Paul

Sent one (and caved in to peer pressure re. the prefix).

Cheers,
Ulf

Re: [PATCH v2] scripts: add new script 'native'

[Paul Eggleton]

On Thu, 06 Oct 2016 05:20:20 Ulf Magnusson wrote:
> On Thu, Oct 6, 2016 at 5:02 AM, Paul Eggleton
> 
> <paul.eggleton@linux.intel.com> wrote:
> > On Tue, 04 Oct 2016 12:22:56 Randy MacLeod wrote:
> >> On 2016-10-02 03:37 PM, Paul Eggleton wrote:
> >> >> "native" might be a bit too generic...
> >> >> 
> >> >> > >> could I suggest oe_native-run or bitbake-native ?
> >> >> > >> 
> >> >> > >> something that says it's yocto-galaxy specific and can be
> >> >> > >> autocompleted
> >> >> > >> with a reasonable prefix...
> >> 
> >> I also prefer an oe prefix.
> >> oe-native, oe-run-native, or oe-exec-native would work for me.
> > 
> > Well, someone needs to send a patch, any volunteers?
> > 
> > Cheers,
> > Paul
> 
> Sent one (and caved in to peer pressure re. the prefix).

Thanks!

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre