ACK+cmnt: [kteam-tools][PATCH] git-build-kernel: Don't hard-code the location of the build script

[Kamal Mostafa]

On Fri, Apr 20, 2018 at 09:12:50AM +0200, Juerg Haefliger wrote:
> Ping.
> 
> On 03/26/2018 04:40 PM, Juerg Haefliger wrote:
> > Instead, deduce its location from the location of the post-receive hook.
> > With this, one can use a local build script, for example for testing or
> > special builds.
> > 
> > Signed-off-by: Juerg Haefliger <juergh at canonical.com>
> > ---
> >  git-build-kernel/post-receive | 5 +++--
> >  1 file changed, 3 insertions(+), 2 deletions(-)
> > 
> > diff --git a/git-build-kernel/post-receive b/git-build-kernel/post-receive
> > index dd8451f35dd5..43671f97a82a 100755
> > --- a/git-build-kernel/post-receive
> > +++ b/git-build-kernel/post-receive
> > @@ -2,8 +2,9 @@
> >  # .git/hooks/post-receive --> git-build-kernel automatic push builder
> >  # Kamal Mostafa <kamal at canonical.com>
> >  
> > -### directory containing the git-build-kernel script
> > -GBK_DIR=/usr3/ubuntu/kteam-tools/git-build-kernel
> > +### directory containing the git-build-kernel script (the same directory where
> > +### the post-receive hook is located)
> > +GBK_DIR="$(dirname "$(readlink -e "${BASH_SOURCE[0]}")")"
> >  
> >  option_only_build_refs_build=0
> >  option_only_build_predefined_refs=1
> > 
> 
> 

Acked-by: Kamal Mostafa <kamal at canonical.com>

And here's a patch to the README that could go along with the code change.

diff --git a/git-build-kernel/README b/git-build-kernel/README
index 03b8c00..875ffb9 100644
--- a/git-build-kernel/README
+++ b/git-build-kernel/README
@@ -75,8 +75,9 @@ Notes:
 
 To set up your git push auto-builder
 
-1. On the build server (tangerine or gomeisa), cd into your own ubuntu-*
-   repo .git/hooks/ directory, and symlink to my post-receive script:
+1. On the build server, cd into your own ubuntu-* repo .git/hooks/ directory,
+   and create a symlink to the builder's post-receive script.  This
+   post-receive MUST be created as a symlink (not a copy):
 
 	cd .../ubuntu-something/.git/hooks/
 	ln -s /usr3/ubuntu/kteam-tools/git-build-kernel/post-receive .