setup.py do not work on Linux

jkakavas / creepy

A geolocation OSINT tool. Offers geolocation information gathering through social networking platforms.

http://www.geocreepy.com

GNU General Public License v3.0

412 stars 108 forks source link

setup.py do not work on Linux #30

Open petterreinholdtsen opened 8 years ago

petterreinholdtsen commented 8 years ago

Hi. The setup.py script do not work on Linux. There are three problems.

The cacerts.txt file is missing from the source tarball. Why is it listed in setup.py, as it isn't needed at least on Linux.
The imageformats list is only for Windows, and consist of paths that only work on the developers Windows machine.
There is no python module py2exe, and it isn't needed on Linux.

Would it be possibe to rewrite it to work on both Windows and Linux?

Also, perhaps the setup.py implementation can be changed to one that understand entry_point, to make it easier to install a wrapper to start the main program?

We disabled the parts that do not work on Debian, see https://sources.debian.net/patches/patch/creepy/1.4.1-3/06-linux-setup-py.patch/ for the patch we use there. It will break the Windows build, I guess, but I do not have a way to test it on Windows and wonder if you have any idea how to get a setup.py file that work on all the platforms you want it to work on.

jkakavas commented 8 years ago

I only use setup.py in order to build the executable for windows. It is not distributed and is not to be run on user systems. As such, I don't see any need in adjusting this to work for linux. Well, one could argue that this would give yet another way of "setting up" creepy in linux ( non debian based ) distributions ,but I don't really see the point. Can't you just ignore this ?

petterreinholdtsen commented 8 years ago

[Ioannis Kakavas]

I only use setup.py in order to build the executable for windows. It is not distributed and is not to be run on user systems.

As such, I don't see any need in adjusting this to work for linux. Well, one could argue that this would give yet another way of "setting up" creepy in linux ( non debian based ) distributions ,but I don't really see the point. Can't you just ignore this ?

The build system for creating a Debian package is using it, so ignoring it is not an option for me. We build and install the files into the package using the information in setup.py.

Happy hacking Petter Reinholdtsen

jkakavas commented 8 years ago

Ok, I see your point. I ll check your patch and will find a way to have it working for both.

AxRealGG commented 7 years ago

@petterreinholdtsen are you stupid? setup.py only for building on windows. For linux you should run CreepyMain.py from creepy folder. See installation guides on http://www.geocreepy.com/

jkakavas commented 7 years ago

Please, don't talk like that. Apart from being plain wrong, this is no way to converse on a public forum.

On Mar 19, 2017 11:22 AM, "AxRealGG" notifications@github.com wrote:

@petterreinholdtsen https://github.com/petterreinholdtsen are you stupid? setup.py only for building on windows. For linux you should run CreepyMain.py from creepy folder. See installation guides on http://www.geocreepy.com/

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/jkakavas/creepy/issues/30#issuecomment-287604117, or mute the thread https://github.com/notifications/unsubscribe-auth/AJzhKAKD7mhS09Oyjc1Mu8nNlegwflS7ks5rnPO-gaJpZM4HBpo4 .