summaryrefslogtreecommitdiff
path: root/buildstream
diff options
context:
space:
mode:
authorChandan Singh <csingh43@bloomberg.net>2019-04-19 22:02:00 +0100
committerChandan Singh <chandan@chandansingh.net>2019-04-22 12:28:15 +0000
commit1cbc5e63dc10c7676237fa923273a3937d6977ca (patch)
tree3b5f337ab360dc9f41ec706de074c57f086a251e /buildstream
parent5ed907bed61dac1168e89b6e0534ae74bf9af6aa (diff)
downloadbuildstream-1cbc5e63dc10c7676237fa923273a3937d6977ca.tar.gz
stack.py: Stop creating empty bst directorychandan/fix-assemble-stack
It seems like this hack was added as a workaround for OSTree's limitations to commit an empty directory, originally in https://gitlab.com/BuildStream/buildstream/commit/91940a6cba193ac0e20c01008335617847be27a5. Since we do not have this limitation anymore, stop creating this empty directory. Fixes #998.
Diffstat (limited to 'buildstream')
-rw-r--r--buildstream/plugins/elements/stack.py10
1 files changed, 1 insertions, 9 deletions
diff --git a/buildstream/plugins/elements/stack.py b/buildstream/plugins/elements/stack.py
index b26281fe4..97517ca48 100644
--- a/buildstream/plugins/elements/stack.py
+++ b/buildstream/plugins/elements/stack.py
@@ -55,15 +55,7 @@ class StackElement(Element):
# Just create a dummy empty artifact, its existence is a statement
# that all this stack's dependencies are built.
vrootdir = sandbox.get_virtual_directory()
-
- # XXX FIXME: This is currently needed because the artifact
- # cache wont let us commit an empty artifact.
- #
- # We need to fix the artifact cache so that it stores
- # the actual artifact data in a subdirectory, then we
- # will be able to store some additional state in the
- # artifact cache, and we can also remove this hack.
- vrootdir.descend('output', 'bst', create=True)
+ vrootdir.descend('output', create=True)
# And we're done
return '/output'