Skip to content
This repository has been archived by the owner on Apr 13, 2023. It is now read-only.

CeylonRunTool should not try to interpret command arguments #4821

Open
CeylonMigrationBot opened this issue Oct 27, 2012 · 6 comments
Open

Comments

@CeylonMigrationBot
Copy link

[@quintesse] The current run tool doesn't permit the following:

ceylon run --verbose org.codejive.ceylon.httpd/1.0.2 --port 8080

That's pretty bad and I think we should fix this for M4.
(Sorry Stef ;)

[Migrated from ceylon/ceylon-runtime#26]

@CeylonMigrationBot
Copy link
Author

[@quintesse] I see that you can fix this by inserting -- before the module name, but I don't think that for "run" this should be necessary. But at least this means we can probably downgrade this from "high prio" and "M4".

@CeylonMigrationBot
Copy link
Author

[@FroMage] Aw maaaan.....

@CeylonMigrationBot
Copy link
Author

[@FroMage] M6.

@CeylonMigrationBot
Copy link
Author

[@quintesse] This should now be fixed.... mostly.... the example uses --verbose which is exactly the only option that is hard-coded in the Launcher, at a point where we don't actually know anything about the tool we're running.

@CeylonMigrationBot
Copy link
Author

[@quintesse] I'm taking a break from this, I'm going to have to discuss with @tombentley how to proceed on this because the amount of work involved for the gain it gets us is way out of proportion.

The --verbose option is now handled at least (and generically by all our tools), what remains is --help which is what is causing the headaches.

NB: there is another hard-coded option that has to be handled so early in the startup that the tool chain can't be involved yet (a chicken and egg problem): --sysrep. Right now I see no way that a user program could use that option without resorting to the use of the -- fence argument. Hopefully there won't be much user programs that want to use the --sysrep option themselves.

@CeylonMigrationBot
Copy link
Author

[@FroMage] Moving to 1.2, unfortunately.

@CeylonMigrationBot CeylonMigrationBot added this to the 1.3 milestone Nov 14, 2015
quintesse added a commit that referenced this issue Nov 14, 2015
quintesse added a commit that referenced this issue Nov 14, 2015
…andard ceylon tool base class, so it inherits the default "verbose" option implementation (#4821)
quintesse added a commit that referenced this issue Nov 14, 2015
…andard ceylon tool base class, so it inherits the default "verbose" option implementation (#4821)
quintesse added a commit that referenced this issue Nov 14, 2015
…tools now uses the standard ceylon tool base class, so it inherits the default "verbose" option implementation (#4821)
quintesse added a commit that referenced this issue Nov 14, 2015
…ct on the arguments being set for the target tool (#4821)
quintesse added a commit that referenced this issue Nov 14, 2015
…g executed to set up the logggers appropiately (#4821)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants