Skip to content

Commit b993417

Browse files
authored
Merge pull request #103 from bastelfreak/ezbakefoo
CI: Add support for checking PR branches
2 parents c8b08ea + 2f25871 commit b993417

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

tasks/build.rake

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -116,7 +116,10 @@ namespace :vox do
116116

117117
libs_to_build_manually.each do |lib, config|
118118
puts "Checking out #{lib}"
119-
run_command("git clone -b #{config[:branch]} #{config[:repo]} #{deps_tmp}/#{lib}", silent: false, print_command: true)
119+
# to be able to checkout github refs, e.g. 66/merge, we need to do an explicit fetch
120+
# this allows us to test on branches from pull requests
121+
# we can probably switch to git clone --revision $ref $url in the future, but that requires a newer git. EL9 is too old
122+
run_command("git clone --no-checkout #{config[:repo]} #{deps_tmp}/#{lib}; cd #{deps_tmp}/#{lib}; git fetch origin #{config[:branch]}; git checkout FETCH_HEAD", silent: false, print_command: true)
120123
end
121124

122125
puts "Starting container"

0 commit comments

Comments
 (0)