And while here, also:
- allocate UID and GID for litellm user
- add a startup script
- add sample configurations
Details
Details
- Reviewers
tagattie
Diff Detail
Diff Detail
- Repository
- R11 FreeBSD ports repository
- Lint
No Lint Coverage - Unit
No Test Coverage - Build Status
Buildable 68484 Build 65367: arc lint + arc unit
Event Timeline
Comment Actions
Hi @otis, thanks for an update to the port. I think it's a great improvements.
A major comment is that I plan to add PROXY option at a later date. (I'm waiting for some Azure-related ports to be updated.)
So I would prefer all optional dependencies needed for LiteLLM's proxy functionality put in PROXY_RUN_DEPENDS.
What do you think about it?
| GIDs | ||
|---|---|---|
| 328 | Seems a typo. | |
| misc/py-litellm/Makefile | ||
| 3 | Remove PORTREVISION. | |
| 17 | Please don't resort RUN_DEPENDS alphabetically. The original ordering is based on the project's pyproject.toml so that I can easily check updates on new dependencies and version restrictions. | |
| 24 | Put litellm-proxy-extras in the optional PROXY_RUN_DEPENDS. (See a note below.) | |
| 55 | I plan to add PROXY option later as I may have told. PROXY option will require a number of additional dependencies including gunicorn, redis and litellm-proxy-extras. So I would prefer to put those dependencies under PROXY_RUN_DEPENDS for earier update at a later date and consistency. | |