-
Notifications
You must be signed in to change notification settings - Fork 3
add rpm dependencies to build on centOS7 #71
Conversation
afaik it existed as "XlsxWriter" during my first test in epel? Now it doesn't exist anymore, and it needs to be custom-build anyhow, so better stick to naming conventions
Regarding invoke, see also #72 |
@gsiv can you please take a look on the changes. |
It should be a separate discussion whether it is needed, so we shouldn't include it right now.
The python-pyramid-mailer version looks good.
|
I am very confused now, as my setup should be more or less the same as yours. pytz: introduced into ringo with commit a58f82b (May 2017). It is a pre-existing package in most distributions (including CentOS)
|
the xlsxwriter problem was due to a typo during package building (in which my package used the default values instead of the ones I gave it). |
About the future: (Strangely, my install of CentOS 7, which should be quite similar to yours, only sees python-future, and no future) edit: terrible pun, also it is |
After discussion: |
Updating setup.cfg so that the rpm has proper requirements.
It is tested in a docker environment by successfully running "ringo-admin --help".
pyramid-mailer
in that specific version was introduced in setup.py a few months ago.XlsxWriter
I am not sure; that package built with this name on my setup.