[Buildbot-devel] Problem with build.steps.source.Git on Windows with revision from sourcestamp

[Russell Gallop]

I have updated the master to 0.8.3p1 and it does not fix it.

The commands seem different between Linux and Windows build slaves. e.g.

checkoutstep=Git(repourl=gitrepo,
                 mode="clobber",
                 workdir="build/repo",
                 haltOnFailure=True)


Then the windows build does:

C:\Program Files\Git\cmd\git.CMD init

 Initialized empty Git repository in

C:/buildbot/slave/windows_xp_32bit/build/repo/.git/

C:\Program Files\Git\cmd\git.CMD reset --hard 4190595005d7e193ffa156b4ddfcb6006a751db8

fatal: Could not parse object '4190595005d7e193ffa156b4ddfcb6006a751db8'.

C:\Program Files\Git\cmd\git.CMD rev-parse HEAD

HEAD

fatal: ambiguous argument 'HEAD': unknown revision or path not in the working tree.
Use '--' to separate paths from revisions

Which does not make much sense to me for checking out code.

Linux does:

rm -rf /local/swnb/buildbot/slave/centos_4.8_32bit/build/repo

/usr/local/bin/git init

 Initialized empty Git repository in

/local/swnb/buildbot/slave/centos_4.8_32bit/build/repo/.git/

/usr/local/bin/git reset --hard 4190595005d7e193ffa156b4ddfcb6006a751db8

 fatal: Could not parse object '4190595005d7e193ffa156b4ddfcb6006a751db8'.

/usr/local/bin/git fetch -t git://git/repo/.git +master

 ...

/usr/local/bin/git reset --hard 4190595005d7e193ffa156b4ddfcb6006a751db8

 HEAD is now at 4190595 Updated: infr_scripts_pl verif_tests_sw

/usr/local/bin/git branch -M master

/usr/local/bin/git rev-parse HEAD

 4190595005d7e193ffa156b4ddfcb6006a751db8


I am using:
-Windows XP
-msys git 1.7.0.2
-buildmaster 0.8.3p1
-buildslave 0.8.3

Still puzzled...
Russ


Dustin J. Mitchell wrote:
> On Fri, Jan 14, 2011 at 11:58 AM, Amber Yust <ayust at yelp.com> wrote:
>   
>> Have you grabbed 0.8.3p1 for the buildslave? I believe 0.8.3 had some Git
>> issues.
>>     
>
> This particular issue was a problem in 0.8.2, actually, and is fixed
> in 0.8.3 (and p1).  p1 only added fixes for the GitPoller, so it's
> irrelevant here.
>
> Dustin
>