[SCM] ci-tooling packaging branch, master, updated. 1b62b7a2264ed1069d4ab853d9e9e2d7e3855b43
Harald Sitter
apachelogger-guest at moszumanska.debian.org
Thu Jan 7 09:32:03 UTC 2016
Gitweb-URL: http://git.debian.org/?p=pkg-kde/ci-tooling.git;a=commitdiff;h=1b62b7a
The following commit has been merged in the master branch:
commit 1b62b7a2264ed1069d4ab853d9e9e2d7e3855b43
Author: Harald Sitter <sitter at kde.org>
Date: Thu Jan 7 10:32:00 2016 +0100
restore compatibility with previous build_source glob behavior
if the source dir does not exist the previous code would ignore it where
the new code doesn't and actively raises
to prevent test fails restore the old behavior support, but ideally we
should deprecate this behavior asap on account of it being rubbish
---
lib/ci/build_source.rb | 12 +++++++++---
1 file changed, 9 insertions(+), 3 deletions(-)
diff --git a/lib/ci/build_source.rb b/lib/ci/build_source.rb
index c14ed79..9f41c2a 100644
--- a/lib/ci/build_source.rb
+++ b/lib/ci/build_source.rb
@@ -40,7 +40,6 @@ module CI
def copy_source
# copy sources around
- p BUILD_DIR
FileUtils.rm_rf(BUILD_DIR, verbose: true)
# Allow support for format 1.0 and quilt
@@ -51,8 +50,15 @@ module CI
end
FileUtils.mkpath("#{BUILD_DIR}/source")
- # NOTE: /. is fileutils notation for recursive content
- FileUtils.cp_r("#{source_dir}/.", "#{BUILD_DIR}/source/", verbose: true)
+ # Legacy behavior was to ignore missing source directories. Using the
+ # fileutils /. notation does not allow for this though.
+ # FIXME: deprecate away from this and let missing sources fail!
+ # if a native doesn't have packaging or a non-native no source that is
+ # rather a problem
+ if Dir.exist?(source_dir)
+ # NOTE: /. is fileutils notation for recursive content
+ FileUtils.cp_r("#{source_dir}/.", "#{BUILD_DIR}/source/", verbose: true)
+ end
%w(.bzr .git .hg .svn debian).each do |dir|
FileUtils.rm_rf(Dir.glob("#{BUILD_DIR}/source/**/#{dir}"))
--
ci-tooling packaging
More information about the pkg-kde-commits
mailing list