All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCHv2 0/3][eclipse-poky] Add option to allow building from local repo
@ 2012-12-10  8:13 mail
  2012-12-10  8:13 ` [PATCHv2 1/3] scripts/build.sh: Added function to use the local repository for building mail
                   ` (3 more replies)
  0 siblings, 4 replies; 6+ messages in thread
From: mail @ 2012-12-10  8:13 UTC (permalink / raw)
  To: yocto; +Cc: Timo Mueller

From: Timo Mueller <timo.mueller@bmw-carit.de>

Hi,

I've rebased the patch series. It should now apply to the current master.

Best regards,
	 Timo

From the original cover letter:
<snip>
if you build eclipse-poky with the provided build script it will always use
the upstream version of the IDE.
During development I wanted to use my local repository to make sure that my
changes don't break the build system. Therefor I added an option to the build
script to allow building from the local eclipse-poky git repository.
</snip>

Timo Mueller (3):
  scripts/build.sh: Added function to use the local repository for
    building
  scripts/build.sh: Added help option to the cmdline
  script/build.sh: Added option to enable building from local
    repository.

 scripts/build.sh |   30 ++++++++++++++++++++++++++----
 1 files changed, 26 insertions(+), 4 deletions(-)

-- 
1.7.7.6



^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCHv2 1/3] scripts/build.sh: Added function to use the local repository for building
  2012-12-10  8:13 [PATCHv2 0/3][eclipse-poky] Add option to allow building from local repo mail
@ 2012-12-10  8:13 ` mail
  2012-12-10  8:13 ` [PATCHv2 2/3] scripts/build.sh: Added help option to the cmdline mail
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 6+ messages in thread
From: mail @ 2012-12-10  8:13 UTC (permalink / raw)
  To: yocto; +Cc: Timo Mueller

From: Timo Mueller <timo.mueller@bmw-carit.de>

If USE_LOCAL_GIT_REPO is set to 1 the build scripts uses the local
repository of the eclipse poky project for building instead of the
upstream project. The local repository is derived from the location
for the build script.
---
 scripts/build.sh |    8 ++++++++
 1 files changed, 8 insertions(+), 0 deletions(-)

diff --git a/scripts/build.sh b/scripts/build.sh
index de81ce3..cbb7375 100755
--- a/scripts/build.sh
+++ b/scripts/build.sh
@@ -65,6 +65,8 @@ check_env ()
   fi 
 }
 
+USE_LOCAL_GIT_REPO=0
+
 if [ $# -ne 2 ] && [ $# -ne 3 ]; then 
    help
 fi
@@ -97,6 +99,12 @@ mkdir ${BUILD_DIR} || fail $? "Create temporary build directory ${BUILD_DIR}"
 
 #git clone
 GIT_URL=git://git.pokylinux.org/eclipse-poky.git
+if [ $USE_LOCAL_GIT_REPO -eq 1 ]; then
+	SCRIPT_DIR=`dirname $0`
+	GIT_DIR=`readlink -f ${SCRIPTDIR}\..`
+	GIT_URL="file://${GIT_DIR}"
+fi
+
 GIT_DIR=${BUILD_SRC}
 #mkdir ${GIT_DIR}
 #cp -r features/ ${GIT_DIR}
-- 
1.7.7.6



^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [PATCHv2 2/3] scripts/build.sh: Added help option to the cmdline
  2012-12-10  8:13 [PATCHv2 0/3][eclipse-poky] Add option to allow building from local repo mail
  2012-12-10  8:13 ` [PATCHv2 1/3] scripts/build.sh: Added function to use the local repository for building mail
@ 2012-12-10  8:13 ` mail
  2012-12-10  8:13 ` [PATCHv2 3/3] script/build.sh: Added option to enable building from local repository mail
  2012-12-10 17:36 ` [PATCHv2 0/3][eclipse-poky] Add option to allow building from local repo Paul Eggleton
  3 siblings, 0 replies; 6+ messages in thread
From: mail @ 2012-12-10  8:13 UTC (permalink / raw)
  To: yocto; +Cc: Timo Mueller

From: Timo Mueller <timo.mueller@bmw-carit.de>

Calling the build script with the -h option will now show the usage.
The cmdline parsing can be extended to add new options in the future.
---
 scripts/build.sh |   18 ++++++++++++++----
 1 files changed, 14 insertions(+), 4 deletions(-)

diff --git a/scripts/build.sh b/scripts/build.sh
index cbb7375..693dcf7 100755
--- a/scripts/build.sh
+++ b/scripts/build.sh
@@ -3,12 +3,13 @@
 help ()
 {
   echo "Build the Yocto Eclipse plugins"
-  echo "Usage: $0 <branch name> <release name>";
+  echo "Usage: $0 [OPTIONS] BRANCH_NAME RELEASE_NAME [TAG_NAME]";
   echo ""
   echo "Options:"
-  echo "<branch name> - git branch name to build upon"
-  echo "<release name> - release name in the final output name"
-  echo "[tag name] - git tag name to build upon. defaults to master if not set"
+  echo "-h - display this help and exit"
+  echo "BRANCH_NAME - git branch name to build upon"
+  echo "RELEAES_NAME - release name in the final output name"
+  echo "TAG_NAME - git tag name to build upon. defaults to HEAD if not set"
   echo ""
   echo "Example: $0 master r0 M1.1_rc1";
   exit 1;
@@ -66,6 +67,15 @@ check_env ()
 }
 
 USE_LOCAL_GIT_REPO=0
+while getopts ":h" opt; do
+	case $opt in
+		h)
+			help
+			;;
+	esac
+done
+shift $(($OPTIND - 1))
+
 
 if [ $# -ne 2 ] && [ $# -ne 3 ]; then 
    help
-- 
1.7.7.6



^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [PATCHv2 3/3] script/build.sh: Added option to enable building from local repository.
  2012-12-10  8:13 [PATCHv2 0/3][eclipse-poky] Add option to allow building from local repo mail
  2012-12-10  8:13 ` [PATCHv2 1/3] scripts/build.sh: Added function to use the local repository for building mail
  2012-12-10  8:13 ` [PATCHv2 2/3] scripts/build.sh: Added help option to the cmdline mail
@ 2012-12-10  8:13 ` mail
  2012-12-10 17:36 ` [PATCHv2 0/3][eclipse-poky] Add option to allow building from local repo Paul Eggleton
  3 siblings, 0 replies; 6+ messages in thread
From: mail @ 2012-12-10  8:13 UTC (permalink / raw)
  To: yocto; +Cc: Timo Mueller

From: Timo Mueller <timo.mueller@bmw-carit.de>

If the script is called with the -l option the local git repository is
used instead of the upstream repository.
---
 scripts/build.sh |    6 +++++-
 1 files changed, 5 insertions(+), 1 deletions(-)

diff --git a/scripts/build.sh b/scripts/build.sh
index 693dcf7..54081d5 100755
--- a/scripts/build.sh
+++ b/scripts/build.sh
@@ -7,6 +7,7 @@ help ()
   echo ""
   echo "Options:"
   echo "-h - display this help and exit"
+  echo "-l - use local git repository"
   echo "BRANCH_NAME - git branch name to build upon"
   echo "RELEAES_NAME - release name in the final output name"
   echo "TAG_NAME - git tag name to build upon. defaults to HEAD if not set"
@@ -67,11 +68,14 @@ check_env ()
 }
 
 USE_LOCAL_GIT_REPO=0
-while getopts ":h" opt; do
+while getopts ":lh" opt; do
 	case $opt in
 		h)
 			help
 			;;
+		l)
+			USE_LOCAL_GIT_REPO=1
+			;;
 	esac
 done
 shift $(($OPTIND - 1))
-- 
1.7.7.6



^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCHv2 0/3][eclipse-poky] Add option to allow building from local repo
  2012-12-10  8:13 [PATCHv2 0/3][eclipse-poky] Add option to allow building from local repo mail
                   ` (2 preceding siblings ...)
  2012-12-10  8:13 ` [PATCHv2 3/3] script/build.sh: Added option to enable building from local repository mail
@ 2012-12-10 17:36 ` Paul Eggleton
  2012-12-11  8:11   ` Timo Müller
  3 siblings, 1 reply; 6+ messages in thread
From: Paul Eggleton @ 2012-12-10 17:36 UTC (permalink / raw)
  To: mail; +Cc: yocto

Hi Timo,

On Monday 10 December 2012 09:13:41 mail@timomueller.eu wrote:
> From: Timo Mueller <timo.mueller@bmw-carit.de>
> 
> Hi,
> 
> I've rebased the patch series. It should now apply to the current master.
> 
> Best regards,
> 	 Timo
> 
> From the original cover letter:
> <snip>
> if you build eclipse-poky with the provided build script it will always use
> the upstream version of the IDE.
> During development I wanted to use my local repository to make sure that my
> changes don't break the build system. Therefor I added an option to the
> build script to allow building from the local eclipse-poky git repository.
> </snip>
> 
> Timo Mueller (3):
>   scripts/build.sh: Added function to use the local repository for
>     building
>   scripts/build.sh: Added help option to the cmdline
>   script/build.sh: Added option to enable building from local
>     repository.
> 
>  scripts/build.sh |   30 ++++++++++++++++++++++++++----
>  1 files changed, 26 insertions(+), 4 deletions(-)

Can you please include Signed-off-by in your commit messages?

Thanks,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCHv2 0/3][eclipse-poky] Add option to allow building from local repo
  2012-12-10 17:36 ` [PATCHv2 0/3][eclipse-poky] Add option to allow building from local repo Paul Eggleton
@ 2012-12-11  8:11   ` Timo Müller
  0 siblings, 0 replies; 6+ messages in thread
From: Timo Müller @ 2012-12-11  8:11 UTC (permalink / raw)
  To: Paul Eggleton; +Cc: yocto

Hi Paul,

Paul Eggleton wrote, On 10.12.2012 18:36:
> Hi Timo,
>
> On Monday 10 December 2012 09:13:41 mail@timomueller.eu wrote:
>> From: Timo Mueller <timo.mueller@bmw-carit.de>
>>
>> Hi,
>>
>> I've rebased the patch series. It should now apply to the current master.
>>
>> Best regards,
>> 	 Timo
>>
>>  From the original cover letter:
>> <snip>
>> if you build eclipse-poky with the provided build script it will always use
>> the upstream version of the IDE.
>> During development I wanted to use my local repository to make sure that my
>> changes don't break the build system. Therefor I added an option to the
>> build script to allow building from the local eclipse-poky git repository.
>> </snip>
>>
>> Timo Mueller (3):
>>    scripts/build.sh: Added function to use the local repository for
>>      building
>>    scripts/build.sh: Added help option to the cmdline
>>    script/build.sh: Added option to enable building from local
>>      repository.
>>
>>   scripts/build.sh |   30 ++++++++++++++++++++++++++----
>>   1 files changed, 26 insertions(+), 4 deletions(-)
>
> Can you please include Signed-off-by in your commit messages?

Sure! I'll resend the series.

>
> Thanks,
> Paul
>

Best regards,
Timo


^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2012-12-11  8:11 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-12-10  8:13 [PATCHv2 0/3][eclipse-poky] Add option to allow building from local repo mail
2012-12-10  8:13 ` [PATCHv2 1/3] scripts/build.sh: Added function to use the local repository for building mail
2012-12-10  8:13 ` [PATCHv2 2/3] scripts/build.sh: Added help option to the cmdline mail
2012-12-10  8:13 ` [PATCHv2 3/3] script/build.sh: Added option to enable building from local repository mail
2012-12-10 17:36 ` [PATCHv2 0/3][eclipse-poky] Add option to allow building from local repo Paul Eggleton
2012-12-11  8:11   ` Timo Müller

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.