Skip to content

Update makefile/dockerfile#321

Merged
izar merged 1 commit intoOWASP:masterfrom
NoodlesNZ:fix-makefile
Apr 13, 2026
Merged

Update makefile/dockerfile#321
izar merged 1 commit intoOWASP:masterfrom
NoodlesNZ:fix-makefile

Conversation

@NoodlesNZ
Copy link
Copy Markdown
Contributor

Changes:

  • Fix broken URLs in Dockerfile
  • Bump base image for Dockerfile
  • Update to use poetry to manage dependencies
  • Use poetry to run commands in Makefile so that the virtual environment is used
  • Add entrypoint.sh to remove complexity/overloading in Makefile

The docker image can now be used like this:

docker run --rm -v $(pwd):/work pytm:latest dfd

or

docker run --rm -v $(pwd):/work pytm:latest dfd other_tm.py

This means the docker image can be used without the Makefile which makes it more portable

make can still be used as per normal, but it removes the complexity of trying to use docker as a shell (which didn't seem to work well)

Update to use poetry to manage dependencies
Add entrypoint.sh to remove complexity/overloading in makefile
@NoodlesNZ NoodlesNZ requested a review from izar as a code owner April 13, 2026 05:46
Copy link
Copy Markdown
Collaborator

@izar izar left a comment

Choose a reason for hiding this comment

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

Thanks!

@izar izar merged commit 0424682 into OWASP:master Apr 13, 2026
3 checks passed
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