Roundup Tracker - Issues

Issue 2550720

classification
Title: Install on Win 7, fails to detect missing python dependency
Type: compile error Severity: major
Components: Installation Versions: 1.4
process
Status: new Resolution:
Dependencies: Superseder:
Assigned To: Nosy List: ber, cscortes, rouilj, techtonik
Priority: Keywords:

Created on 2011-08-15 14:36 by cscortes, last changed 2016-07-10 18:47 by rouilj.

Messages
msg4376 Author: [hidden] (cscortes) Date: 2011-08-15 14:36
Missing pyWin32, Mark Harmond's windows library for python. I suppose
the installer could check for this and report the missing dependency.  I
have done this myself with DotNet dlls, and some installers give you a
chance to write a routine to check complex missing dependencies. 
Although the easiest way I can think of is wrap an import statement with
a try catch and report either success or failure.  

workaround, before you install roundup, make sure you have install
pywin32 from Mark Hammond's web site.
msg4377 Author: [hidden] (ber) Date: 2011-08-15 15:08
Luis,
thanks for creating this issue!

I wonder if we could just include the pyWin32 library in the installer?
msg4379 Author: [hidden] (cscortes) Date: 2011-08-16 15:31
I don't know.  I guess I would make a distinction between needing pywin
for the installer and needing it for roundup.

If the installer needs it then yes it could be included, but I think
that roundup needs it.  The problem is that there is a different pywin32
library for every version of python (2.5,2.6,2.7,...) so it is easier
just to throw an error and ask the user to figure out which version he
needs to install for his python setup.
msg4382 Author: [hidden] (ber) Date: 2011-08-16 18:53
Ah, you are right of course, that we cannot know the python binary unless we 
also include python itself. ;)

I wonder what roundup would use pywin, though.
msg5817 Author: [hidden] (rouilj) Date: 2016-07-10 18:47
The pyWin32 dependency was removed in 1.5.0 for most uses except
running roundup as a windows service.

I think it was used for portalocker. From CHANGES.txt:

- pywin32 is no longer required to run on Windows (anatoly techtonik)
- Rewritten portalocker.py logic in ctypes for Windows (anatoly techtonik)

I think this issue is a partial replacement for: issue2550718
History
Date User Action Args
2016-07-10 18:47:45rouiljsetnosy: + techtonik, rouilj
messages: + msg5817
2011-08-16 18:53:48bersetmessages: + msg4382
2011-08-16 15:31:31cscortessetmessages: + msg4379
2011-08-15 15:08:23bersetnosy: + ber
messages: + msg4377
2011-08-15 14:36:38cscortescreate