-
Notifications
You must be signed in to change notification settings - Fork 125
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
Making maraschino independent from external libs #12
Conversation
Installprocedure with this commit:
Much easier. |
I used packages from my /usr/local/python2.7 system. Dunno if this is gonna five troubles on machines that have no python 2.7. But it works on a minimul ubuntu install with python 2.7.2, and slackware too is confirmed on xbmc.org. I could change everything and use python-packages from 2.6 if it issues a lot on 2.6 installs. |
lol, a lot of requests, looks like a mess, but it's on stage with your repo. You could try mine everything still works. I moved files around (removed cluttering in rootfolder). |
i was unable to make a db file with the latest 'restructure' commit, but after i made it everything seems to work fine |
fixed db creation by setup.py. |
Running with python2.6 works on my Synology NAS |
Implemented on experimental branch. |
Is this on master? I do not remember merging anything into it. What happened to this? How did it close? Very confused right now, nothing related to this is in master's history and it does not seem to be included. @mrkipling can you check this out? I've been waiting on this pull so I can rebase experimental off master and do some more 'nightly' updates with the other pull requests. |
Would be a shame to see this get lost. Makes using maraschino on unraid server sooo much easier |
I'm not seeing any commits for this pull request any more. Perhaps they got lost when it was closed above? Mar2zz, would you be able to re-open the pull request and check that it is working on master? Doubt that I'll be able to get round to looking at it today, but I agree that it shouldn't get lost. |
I agree, makes it using it in any os extremely easy. Maybe @Mar2zz can re-do this? I'm thinking it got deleted when he re-forked the repo. Worst case scenario we have the modifications in the history of the experimental branch. |
@gugahoi, that's not such a big deal as experimental is going to be periodically reset to master (as it's just a testbed which will end up containing lots of stuff that didn't make the cut). Not to be confused with a "prerelease" or "beta" branch, should we go down that route in the future. |
I know I just meant that we can reproduce the modifications looking at the experimental branch if we absolutely have to. the history will contain the commits made by Mar2zz and with that we will be able to re-do them on master. |
Oh I see, I misunderstood what you meant. Yep, either way. Would be easiest if the pull request was re-opened so let's see if Mar2zz can do that. |
Can't reopen it, but I will import every module again in a few hours Op 02-01-12 13:10, Bradley Abrahams schreef:
|
Mar2zz, sounds good - although I'm slightly confused what you mean by "module", as I thought that you were adding external libraries to the repo? My advice would be as follows:
As for things like tools.py, I don't really mind too much, but it seems logical to keep it in the root perhaps? |
Regarding transmission, that isn't on master yet (haven't had time to look at it). |
It's here: #31 A slighty different approach this time, see my comment over there. Please let someone confirm it works before merging. (download my masterrepo and test. (sudo pip uninstall flask flask-sqlalchemy sqlalchemy jsonrpc cherrypy will uninstall all external python libs there were mandatory before). |
This pull requests enables cherrypy/jsonrpclib local, so users won't have to easy_install them. Also see my posts at xbmc.org about the why:
http:https://forum.xbmc.org/showthread.php?p=967659&posted=1#post967659
Now I am going to try to get flask and flask-sqlalchemy to be local.