search

linuxserver / docker-nzbget

GNU General Public License v3.0
149 stars 83 forks source link

pip modules #69

Closed elmero1 closed 5 years ago

elmero1 commented 5 years ago

I use your docker container and a plugin for notification (nzb-notify). Unfortunately the notification plugin need additional pip modules: apprise >= 0.7.0 pynzbget chardet

Is it possible to include this in your container?

CHBMB commented 5 years ago

@thelamer @aptalca What do you think? We can either include the deps or do we defer this to a custom script?

thelamer commented 5 years ago

We do it for sab, but that was before custom modification logic existed. https://github.com/linuxserver/docker-sabnzbd/blob/master/Dockerfile#L40-L45 The notification stuff does seem like generally accepted as standard issue stuff for users.

CHBMB commented 5 years ago

I'll do a PR then. @thelamer