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

Allows EO resource files to be compiled and echoed to the command line. #122

Closed
wants to merge 8 commits into from

Conversation

nqafield
Copy link
Contributor

@nqafield nqafield commented Feb 18, 2017

You can use the following EO compiler commands from the command line. (See below for a caveat.)

  • --help (as before)
  • --demo (lists the available EO files)
  • --demo <filename> (outputs the EO file followed by the compiled Java)

Caveat: I've only tested this as IntelliJ "configurations" with program arguments.

Note: The tests are a bit sporadic. Possibly too thorough in a couple of places. And non-existent in others. (I can add more if you like.)

Hopefully improves on this (#116) closed PR.

Trying out smaller classes, a bit of TDD, test naming, different
formatting etc.

Probably just need to crack on soon and get it in a form that Qulice
will be happy with.
Qulice is happy, but there's still plenty to do.
Allows EO resource files to be compiled and echoed to the command line.
@nqafield
Copy link
Contributor Author

Argh! It works on my machine...

@nqafield nqafield mentioned this pull request Feb 18, 2017
@g4s8
Copy link
Contributor

g4s8 commented Feb 19, 2017

@pa9ey looks like you are using windows-style \ path separator for resources in DemoCommand and ProrgamTest. Try to replace it with unix-style separator: /, like this one in DemoCommand:

-        final String path = "org\\eolang\\compiler\\eo\\";
+        final String path = "org/eolang/compiler/eo/";

@nqafield
Copy link
Contributor Author

Thanks again @g4s8! 👍 👍 👍

@g4s8
Copy link
Contributor

g4s8 commented Mar 11, 2017

@rultor merge?

@rultor
Copy link
Contributor

rultor commented Mar 11, 2017

@rultor merge?

@g4s8 Thanks for your request. @yegor256 Please confirm this.

@nqafield nqafield closed this Aug 6, 2017
@0crat
Copy link

0crat commented Aug 6, 2017

I can't do it for technical reasons, I'm very sorry. If you don't know what to do, submit this error as a ticket here:

1.0-SNAPSHOT ${buildNumber} 2017-08-04T07:36:32
java.lang.IllegalStateException: java.lang.reflect.InvocationTargetException
	at com.zerocracy.farm.reactive.StkGroovy.process(StkGroovy.java:115)
	at com.zerocracy.farm.StkSafe.process(StkSafe.java:89)
	at com.zerocracy.farm.reactive.Brigade.process(Brigade.java:103)
	at com.zerocracy.farm.reactive.Brigade.process(Brigade.java:82)
	at com.zerocracy.farm.reactive.FlushAction.process(FlushAction.java:85)
	at com.zerocracy.farm.reactive.FlushAction.exec(FlushAction.java:67)
	at com.zerocracy.farm.reactive.FlushAction.exec(FlushAction.java:35)
	at org.cactoos.func.FuncOf.lambda$new$3(FuncOf.java:88)
	at org.cactoos.func.FuncOf.apply(FuncOf.java:104)
	at org.cactoos.func.IoCheckedFunc.apply(IoCheckedFunc.java:66)
	at org.cactoos.func.UncheckedFunc.apply(UncheckedFunc.java:59)
	at org.cactoos.func.RunnableOf.run(RunnableOf.java:89)
	at com.jcabi.log.VerboseRunnable.run(VerboseRunnable.java:197)
	at com.zerocracy.farm.reactive.RunnableWithTrigger.run(RunnableWithTrigger.java:52)
	at java....

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

Successfully merging this pull request may close these issues.

4 participants