nextcloud / news-updater

:newspaper: Fast, parallel feed updater for the News app; written in Python
GNU General Public License v3.0
105 stars 24 forks source link

Remove unsupported pip options. #39

Closed danielrheinbay closed 3 years ago

danielrheinbay commented 3 years ago

Remove pip options from README which are no longer supported, see https://github.com/pypa/pip/issues/7309

BernhardPosselt commented 3 years ago

Sure but how do you get the same behavior as before now? I think that needs to go into the README as well

danielrheinbay commented 3 years ago

The equivalent of --install-option="--install-scripts=/usr/bin" would be --prefix=/usr/bin. However, this was not needed on my machine; just executing sudo pip3 install nextcloud_news_updater resulted in nextcloud_news_updater to be installed to /usr/bin by default. How do we proceed? Shall we add --prefix=/usr/bin just to be on the safe side?

BernhardPosselt commented 3 years ago

If it works the same, then it's fine :) thanks for the PR