Skip to content

Commit

Permalink
Fix cyclic dependency issue seen specifically with CI code (#165)
Browse files Browse the repository at this point in the history
* Fixe cyclic dependency issue seen specifically with CI code

* Install libX11-devel in the P10 dockerfile
  • Loading branch information
Nishidha authored Feb 15, 2023
1 parent 34891cd commit cb551f1
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion open_ce/build_tree.py
Original file line number Diff line number Diff line change
Expand Up @@ -509,7 +509,7 @@ def build_command_dependencies(self, node):
for dep_package in dep_packages:
terms += [next(x.build_command.name() for x in deps if dep_package == x.packages)]

return ", ".join(f"'{x}'" for x in terms)
return ", ".join(f"'{x}'" for x in terms if node.build_command.name() != x)

def remove_external_deps_from_dag(self):
'''
Expand Down
2 changes: 1 addition & 1 deletion open_ce/images/builder/Dockerfile-p10
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ ENV BUILD_USER=builder
ARG BUILD_ID=1084

RUN export ARCH="$(uname -m)" && \
yum repolist && yum install -y rsync openssh-clients diffutils procps git-lfs gcc-toolset-11 glibc-devel file libtirpc-devel psmisc && \
yum repolist && yum install -y rsync openssh-clients diffutils procps git-lfs gcc-toolset-11 glibc-devel file libtirpc-devel psmisc libX11-devel && \
# Create CICD Group
groupadd --non-unique --gid ${GROUP_ID} ${CICD_GROUP} && \
# Adduser Builder
Expand Down

0 comments on commit cb551f1

Please sign in to comment.