upshot icon indicating copy to clipboard operation
upshot copied to clipboard

Plugins

Open mikeknoop opened this issue 11 years ago • 7 comments

I forked upshot here: https://github.com/mikeknoop/upshot

For the most part, this was done to rip out the DropBox dependency and replace it with a custom uploading mechanism so I could handle the image saving server-side.

I also made a few other tweaks, added sound support, etc. But the biggest thing I thought upshot could use is a plugin architecture. DropBox was intertwined in many places (including hard to modify .nib files).

Have you considered what a plugin architecture might look like? You could use my branch as a reference for a "cleaner slate" that already has most DB stuff ripped out.

mikeknoop avatar Jun 20 '13 05:06 mikeknoop

Oh this looks like you've done some very nice work there, thank you! Looks ilke a lot of these changes would be very useful for this upstream project, so thank you for sharing :)

I haven't thought in detail about how a plugin infrastructure could look like (that's different from a modular structure, which would at least allow to add additional services more easily. I agree Dropbox is currently at the core of this app).

I am not opposed to it, but maybe enabling a good, structurally sound modular architecture is a better investment than enabling arbitrary plugin code that can hook into the compiled .app.?

On the other hand, I can see how that wouldn't scale if people wanted to see 37 different upload services supported.

fwenzel avatar Jun 20 '13 05:06 fwenzel

@fwenzel I had a co-worker try to install my fork (3 of them, in fact) and they all couldn't launch the app. One tried to recompile and launch and was hitting this:

7/12/13 9:32:00.128 AM UpShot[60864]: A Python runtime not could be located.
You may need to install a framework build of Python, or edit the PyRuntimeLocations
array in this application's Info.plist file.

Is that something you're familiar with? Would love guidance so I can fix my build! Thanks.

mikeknoop avatar Jul 12 '13 14:07 mikeknoop

I actually run Python 2.7 from homebrew, perhaps that makes a difference? Otherwise the way I set it up is like the README says: Make a virtualenv, activate it, install dependencies and make a build.

Are you able to create the app and launch it on your own computer but it doesn't work on your coworkers' machines? Or does it not work for you either? What output do you get from the build command?

fwenzel avatar Jul 12 '13 15:07 fwenzel

I also installed Python 2.7 via homebrew and my fork works for me, I was trying to tease out what the differences between our machines might be.

I assumed that once you created the .app that it would bake in all the dependencies but I guess that might not be the case.

My co-worker says he did install Python 2.7 via brew but thinks he needed to also do brew install python --framework.

mikeknoop avatar Jul 12 '13 16:07 mikeknoop

I assumed that once you created the .app that it would bake in all the dependencies but I guess that might not be the case.

That's how it's supposed to be yes.

My co-worker says he did install Python 2.7 via brew but thinks he needed to also do brew install python --framework.

If so, then let me know and I have to update the readme!

fwenzel avatar Jul 12 '13 17:07 fwenzel

Just to clarify, everyone except me has problems running the .app container that Ive distributed with the repo.

If you're willing, Id like to hear if it works for you or if the binary dies on launch. On Jul 12, 2013 10:03 AM, "Fred Wenzel" [email protected] wrote:

I assumed that once you created the .app that it would bake in all the dependencies but I guess that might not be the case.

That's how it's supposed to be yes.

My co-worker says he did install Python 2.7 via brew but thinks he needed to also do brew install python --framework.

If so, then let me know and I have to update the readme!

— Reply to this email directly or view it on GitHubhttps://github.com/fwenzel/upshot/issues/52#issuecomment-20889701 .

mikeknoop avatar Jul 12 '13 17:07 mikeknoop

Sure, zip it up and shoot me an email (or link)

fwenzel avatar Jul 12 '13 20:07 fwenzel