Skip to content

Conversation

@hnanchahal
Copy link
Contributor

@hnanchahal hnanchahal commented Nov 18, 2020

Copy link
Member

@arm4b arm4b left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for PR!

Left one comment to address.

Architecture: any
Pre-Depends: dpkg (>= 1.16.16), ${pre:Depends}, ${misc:Pre-Depends}, adduser
Depends: ${shlibs:Depends}, ${misc:Depends}, sudo, adduser, ${Depends}, libssl-dev, libffi-dev, git, libpam0g, openssh-server, openssh-client, bash, netbase
Depends: ${shlibs:Depends}, ${misc:Depends}, sudo, adduser, ${Depends}, libssl-dev, libffi-dev, git, libpam0g, openssh-server, openssh-client, bash, netbase, libldap2-dev, libsasl2-dev
Copy link
Member

@arm4b arm4b Nov 18, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Per StackStorm/st2#5081 (comment)
we'll need to add these dependencies via Build-Depends: as they're required only for build.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@arm4b
Copy link
Member

arm4b commented Nov 19, 2020

Dockerfiles updates (StackStorm/st2packaging-dockerfiles#93) worked, merging as tests now ✔️

@arm4b arm4b merged commit 4a8a5a5 into StackStorm:master Nov 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants