Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update builder image #4601

Merged
merged 2 commits into from
Jul 9, 2019
Merged

Update builder image #4601

merged 2 commits into from
Jul 9, 2019

Conversation

emkll
Copy link
Contributor

@emkll emkll commented Jul 9, 2019

Status

Ready for review

Description of Changes

Towards #4550

Testing

  • CI (including deb tests) passes

Deployment

Dev env only

Checklist

If you made changes to the server application code:

  • Linting (make lint) and tests (make -C securedrop test) pass in the development container

If you made changes to securedrop-admin:

  • Linting and tests (make -C admin test) pass in the admin development container

If you made changes to the system configuration:

If you made non-trivial code changes:

  • I have written a test plan and validated it for this PR

If you made changes to documentation:

  • Doc linting (make docs-lint) passed locally

Builder image needed security updates
@emkll emkll added this to the 0.14.0 milestone Jul 9, 2019
@emkll emkll force-pushed the update-builder-20190709 branch from d2317fb to 0a48773 Compare July 9, 2019 17:28
- Fix typo in Makefile
- Add build-essential to resolve failures with deb-test ci target (see https://circleci.com/gh/freedomofpress/securedrop/30769)
- Add fix for click (see https://circleci.com/gh/freedomofpress/securedrop/30776)
@emkll emkll force-pushed the update-builder-20190709 branch from 495cfdb to e33e2b7 Compare July 9, 2019 17:56
@emkll emkll requested a review from redshiftzero July 9, 2019 18:32
@emkll emkll mentioned this pull request Jul 9, 2019
8 tasks
@redshiftzero redshiftzero merged commit b1bea49 into develop Jul 9, 2019
@emkll emkll deleted the update-builder-20190709 branch July 9, 2019 19:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants