[Cruisecontrolrb-users] Git support - kinda works, have design questions
Jason Sallis
jsallis at gmail.com
Tue May 6 20:28:25 EDT 2008
Alex, have you considered using a shortened version of the revision
hash for display? With git, you can do 'git rev-parse --short HEAD' to
get a more readable but unique enough revision hash to id the build
with. I'm not sure if mercurial or other distributed vc systems have
such an option, but it would work for git at least. Just a thought.
- Jason
On 6-May-08, at 3:53 PM, Alexey Verkhovsky wrote:
> As of now, I can create a project from Git repository, run a builder
> on it, and even run the dashboard. Build pages look ugly because
> build.label in git is very long, a bunch of unit tests is failing, and
> I had to disable some functionality, namely displaying source control
> errors in the dashboard.
>
> SourceControl::AbstractAdapter#execute_with_error_log (formerly known
> as Subversion::execute_with_error_log) had a bold assumption that any
> command writing anything in stderr has failed. This is not true for
> Git. Actually (and this really sucks), I even found a case where Git
> command returns a non-zero exit code upon successful completion.
> Namely, "git remote update" - I had to replace that with lower level
> "git fetch origin" that seems to do the right thing. None of that is
> documented behavior, as far as I can tell.
>
> Anyhow, it works for some value of "works" now. ThoughtWorks IS needs
> to punch a hole in the firewall so that ccrb.tw.com can talk to
> RubyForge Git server. Once that is done, I am deploying the current
> tip.
>
> Meantime, there is one pretty important design decision to be made.
> Commit id in distributed source control systems tends to be a very
> long random hex string. So, our convention of using revision number as
> build labels doesn't work - neither for displaying, nor for ordering
> builds.
>
> If we still use those hashes for build labels, we'll need to order
> builds by mtime of a build directory and display abbreviated build
> labels instead of full ones. Or we need to generate our own build
> numbers. Either way sucks, in its own special way. Does anyone have a
> strong preference or a better idea?
>
> --
> Alexey Verkhovsky
> CruiseControl.rb [http://cruisecontrolrb.thoughtworks.com]
> RubyWorks [http://rubyworks.thoughtworks.com]
> _______________________________________________
> Cruisecontrolrb-users mailing list
> Cruisecontrolrb-users at rubyforge.org
> http://rubyforge.org/mailman/listinfo/cruisecontrolrb-users
More information about the Cruisecontrolrb-users
mailing list