Skip to content
This repository

forced to npm install in various plugin directories #343

Closed
hallahan opened this Issue · 8 comments

3 participants

Nicholas Hallahan Ward Cunningham cScarlson
Nicholas Hallahan

It is required to do npm install in the following plugin directories for Node to be able to run:
logwatch
linkmap
parse

This took some detective work, and would not be obvious to someone just trying to install the project for the first time. We need to make this automatic.

We have server code in the client directory tree. This is confusing.

Also, linkmap crashed for me, and I did a fix that I will do a pull request for shortly.

Ward Cunningham

Hmm. I think we're moving to a model where the plugins are managed independently from the core. This counts as a shaky first step. Do you have a suggestion as to how it should be handled?

Nicholas Hallahan

Well, as it stands, the core crashes if the server plugins are not installed. I would either put server code in the server/plugins dir or take the plugins out of the client dir for clarity's sake.

Perhaps we could have an install script that insures that npm install is run for client, server, and the plug-ins?

Ward Cunningham

We have code that runs on both the client and the server.

Nicholas Hallahan

Indeed, so maybe just take the plugins up one dir level?

Ward Cunningham

Yes, it makes sense for plugins to be at the same level as client & server directories. There are probably other options that we will want to consider. For example, a deployment configuration could be different than development. Also, we should optimize npm based deployment however that works.

Ward Cunningham

I've added a try-catch around the plugin server start code. This gets the server going and reports reasonable diagnostics. Client side of plugins are tolerant of server-side failures.

4dd298b (continue serving when plugins fail to start)

cScarlson

Added an update to issue #345, since Ward's try-catch implementation -- still some errors however.

Ward Cunningham

We might want to eventually come up with a quick way to "install" every plugin to make it easier to get a full site up and running. For the moment we're happy to have site operators know what they are serving. So I'll close this issue for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.