miniweb icon indicating copy to clipboard operation
miniweb copied to clipboard

dynamic plugin system

Open gaebor opened this issue 6 years ago • 3 comments

gaebor avatar Mar 10 '18 08:03 gaebor

Thanks for the patches.

To be honest, I actually prefer to remove unused code rather than adding more, i.e. make it even simpler.

As you see from the commit log, beyond my initial effort to clean it up, add the directory listing stuff, and making it behave nicer across platforms (and I do use it on more than one platform), it's been mostly stagnant except for rare minor fixes.

I don't plan to develop it further in a meaningful way, so not sure what to do with this PR - it does add meaningful features which someone would need to maintain.

If you do plan to develop it further though, I can add a link at my readme to your fork at some stage. What say you?

avih avatar Mar 13 '18 20:03 avih

Don't accept if you don't want to.

I'm not sure that I'll develop myself, but your patches and fixes helped a lot. The original stanleyhuang implementation didn't even work for (windows) x64.

thx anyway

gaebor avatar Mar 13 '18 20:03 gaebor

OK. I don't plan to merge these patches, but I'll leave leave the PR open at least for a while, in case someone wants to discuss it.

avih avatar Mar 13 '18 21:03 avih