8000 add requiremenets by oczkers · Pull Request #1 · hunterjm/futgui · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

add requiremenets #1

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from
Closed

add requiremenets #1

wants to merge 1 commit into from

Conversation

oczkers
Copy link
Contributor
@oczkers oczkers commented Oct 30, 2015

Now it should download & install external packages before "compiling" script. Considering that You're making gui app, i'll recommend building "stable" versions on Your own for every platform (mac/win at least) and attach binaries (all python files attached) so not only geeks could launch it ;-).

https://github.com/oczkers/futgui/tree/installable_as_library
Here I've made same changes to allow installation as a library (setup.py install) but it probably won't work because only .py files are installed.

@hunterjm
Copy link
Owner

The gui (currently) requires my fork of fut. Is there any way to specify that in requirements.txt?

I have releases here on github that has the pre-compiled binaries for both mac and windows. I'm currently using py2app for mac and pyinstaller for Windows, because I was having trouble getting cx-freeze working. Your addition to setup.py will fix the requirements build for mac, but not for windows where you use the pyinstaller spec.

I would really like to move to using cx-freeze so I can have the same build process for both platforms, but I gave up.

@oczkers
Copy link
Contributor Author
oczkers commented Oct 30, 2015

I've build only one or two aps to exe (py2exe and nuitka (converts to c++ first to gain performance)) so can't help You with this but managing requirements could be easily solved by pip oczkers@fc9be9c

Unfortunately setuptools and distutils can't download dependecies from specific repo so it mist be omitted oczkers@ed9dd2e

edit: decalring specific repo in requirements is super easy oczkers@20cedb8

@hunterjm
Copy link
Owner

Thanks for taking a look. I removed the dependency on my fork of your repo, so most of this is no longer needed. I added you to the contributors list, however, for the awesome work on your library!

@hunterjm hunterjm closed this Nov 10, 2015
@oczkers
Copy link
Contributor Author
oczkers commented Nov 10, 2015

I mentioned futgui in readme, hope it's ok with You?
futapi/fut@50f37ff

@hunterjm
Copy link
Owner

Absolutely, thanks for the mention!

hunterjm pushed a commit that referenced this pull request Nov 18, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0