Skip to content
This repository has been archived by the owner on Nov 9, 2022. It is now read-only.

roxy help missing a deployment command #176

Closed
JackYoungblood opened this issue Jan 20, 2014 · 4 comments
Closed

roxy help missing a deployment command #176

JackYoungblood opened this issue Jan 20, 2014 · 4 comments
Labels

Comments

@JackYoungblood
Copy link

When I'm at the root roxy directory and type ml -h I do not see "new" listed as a deployment command. This leads to some confusion when trying to create a new project.

@paxtonhare
Copy link
Contributor

There's a bug in the help code. It's a pain to fix. I suggest you do this:

ml new -h

That should return the help for new.

--Paxton

On Mon, Jan 20, 2014 at 1:51 PM, nizdobs [email protected] wrote:

When I'm at the root roxy directory and type ml -h I do not see "new"
listed as a deployment command. This leads to some confusion when trying to
create a new project.


Reply to this email directly or view it on GitHubhttps://github.com//issues/176
.

@grtjn
Copy link
Contributor

grtjn commented Jan 21, 2014

Listing new in def self.usage in Help.rb should be no problem, nor is adding a def self.new to the same class. Ruby only complains if you call ml.rb with only 'new' as parameter. But it is possible to add '-h' as second param if there are no other params. That would at least integrate the usage documentation.

I can imagine though that executing a 'new' command inside Ruby might be an issue (to actually perform the work for 'new' instead of just shoing help). A work-around for that could be to rename 'new' command param to something like 'doNew' in the first lines of ml.rb. Provided you would want to do the 'new' work inside Ruby; the shell script code seems to work just fine..

Interested in a pull request?

@dmcassel dmcassel added the bug label Apr 18, 2014
@grtjn
Copy link
Contributor

grtjn commented Apr 22, 2014

On second thought, there is a good reason to not want the 'new' usage inside the Roxy Ruby code. If you put the ml-script in some bin directory (available in your path), and run it from a random directory, there will likely be no deploy/lib/ml.rb available. In that case, only ./ml new is appropriate. The ml-script is enough self-contained to download the Roxy files from git.

@paxtonhare
Copy link
Contributor

fixed in dev

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

4 participants