Skip to content
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

Can't seem to include scripts in nuget package #189

Open
jeremiahredekop opened this issue Feb 24, 2015 · 7 comments
Open

Can't seem to include scripts in nuget package #189

jeremiahredekop opened this issue Feb 24, 2015 · 7 comments

Comments

@jeremiahredekop
Copy link

I am attempting to include .csx scripts in nuget packages as per the docs. The package installs successfully, but the .csx files are not registered by mmbot when it starts up.

I tried including them in 3 separate paths:

  • lib
  • lib\net45
  • (root)
  • content

Am I doing something wrong? The documentation wasn't clear about how to include scripts in nuget packages.

@PeteGoo
Copy link
Member

PeteGoo commented Feb 24, 2015

Hmmm. It's been a while since I tried this method. The nuget folder
resolution can be a bit mad. In the meantime you can of course just drop
then in a scripts folder.

I'll try to take a look and see of I can figure it out
On 25 Feb 2015 04:24, "Jeremiah Redekop" notifications@github.com wrote:

I am attempting to include .csx scripts in nuget packages as per the docs.
The package installs succesfully, but the .csx files are not registered by
mmbot when it starts up.

I tried including them in 3 seperate paths:

  • lib
  • lib\net45
  • (root)

Am I doing something wrong? The documentation wasn't clear about how to
include scripts in nuget packages.


Reply to this email directly or view it on GitHub
#189.

@jamesbascle
Copy link
Contributor

As I recall, the script loader doesn't look in /packages to find
.CSX's. You could make the .csx scripts install to something like
../../scripts
On Feb 24, 2015 1:22 PM, "Peter Goodman" notifications@github.com wrote:

Hmmm. It's been a while since I tried this method. The nuget folder
resolution can be a bit mad. In the meantime you can of course just drop
then in a scripts folder.

I'll try to take a look and see of I can figure it out
On 25 Feb 2015 04:24, "Jeremiah Redekop" notifications@github.com wrote:

I am attempting to include .csx scripts in nuget packages as per the
docs.
The package installs succesfully, but the .csx files are not registered
by
mmbot when it starts up.

I tried including them in 3 seperate paths:

  • lib
  • lib\net45
  • (root)

Am I doing something wrong? The documentation wasn't clear about how to
include scripts in nuget packages.


Reply to this email directly or view it on GitHub
#189.


Reply to this email directly or view it on GitHub
#189 (comment).

@jeremiahredekop
Copy link
Author

I suppose another tactic would be to use powershell to force copy .csx files into the script folder...

@jeremiahredekop
Copy link
Author

Thanks guys, my comment was on an stale browser, so hadn't read your replies yet 😄

My mmbot robot is on an azure worker role and I am trying to automate all the dependencies, including scripts.

@jamesbascle
Copy link
Contributor

There is actually some code in the pipeline to automatically download
packages that contain IMMBotScripts to the packages folder and load them up
on startup. Unfortunately the code that will properly clean up after an
UPDATE to a new version of a package hasn't made it to the distributed
version yet.

On Tue, Feb 24, 2015 at 1:33 PM, Jeremiah Redekop notifications@github.com
wrote:

Thanks guys, my comment was on an stale browser, so hadn't read your
replies yet [image: 😄]

My mmbot robot is on an azure worker role and I am trying to automate all
the dependencies, including scripts.


Reply to this email directly or view it on GitHub
#189 (comment).

@jamesbascle
Copy link
Contributor

For instance, we can do a fresh install (with the same ini ofc) by:

cinst mmbot -force
mmbot
(inside mmbot)
mmbot update package mmbot.hipchat restart
mmbot update package . restart

Unfortunately there is some manual intervention required until a new
version goes out to clean up the old versions of dependent packages.

That might be another route to go down if you don't mind having .CS files
w/ IMMBotScript in them instead of .csx files.

On Tue, Feb 24, 2015 at 1:35 PM, James Bascle rbascle9@gmail.com wrote:

There is actually some code in the pipeline to automatically download
packages that contain IMMBotScripts to the packages folder and load them up
on startup. Unfortunately the code that will properly clean up after an
UPDATE to a new version of a package hasn't made it to the distributed
version yet.

On Tue, Feb 24, 2015 at 1:33 PM, Jeremiah Redekop <
notifications@github.com> wrote:

Thanks guys, my comment was on an stale browser, so hadn't read your
replies yet [image: 😄]

My mmbot robot is on an azure worker role and I am trying to automate all
the dependencies, including scripts.


Reply to this email directly or view it on GitHub
#189 (comment).

@jeremiahredekop
Copy link
Author

Thanks James - will review when I get more cycles to spend on this.

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

No branches or pull requests

3 participants