eo icon indicating copy to clipboard operation
eo copied to clipboard

Failure of non-existent attributes

Open OlesiaSub opened this issue 1 year ago • 5 comments

This code worked fine.

image

Then I modified the code and replaced app with main. And I launched eoc dataize app (I know I need to have an app object to do it, but I was inattentive). And, as you can see below, instead of giving me the warning about the absence of the app object the compiler told me something about foo and bar, which are not even present in this program anymore. I think it may be quite misleading.

image

OlesiaSub avatar Aug 11 '22 11:08 OlesiaSub

@OlesiaSub indeed, a misleading behavior. This happened because you didn't run

$ eoc clean

Some files remained in .eoc directory, already compiled. I think we need to improve TranspileMojo. When generating Java files it has to maintain a Tojos catalog of them (maybe in target/eo/transpiled.csv). Before the transpilation of each .eo file, it has to read the list of files previously generated and delete them.

yegor256 avatar Aug 15 '22 10:08 yegor256

@mximp please, help (or delegate)

yegor256 avatar Aug 15 '22 10:08 yegor256

@Karina5005 is this something you can assist with?

mximp avatar Aug 17 '22 16:08 mximp

@mximp Do I understand correctly that when I do a transpile, I have to store the filenames in target/eo/transpiled.csv. The next time I want to transpile a file and it is in the target/eo/transpiled.csv., that means I must first remove the java files generated from that file.

Karina5005 avatar Sep 01 '22 08:09 Karina5005

@Karina5005 transpiled.csv is supposed to contain generated Java file names per EO program (.eo file). When transpiling a program you first check transpiled.csv and remove all files listed there for this program. And yes all newly generated files for a program you add to transpiled.csv

mximp avatar Sep 01 '22 08:09 mximp

@mximp @Karina5005 Looks like we can close this issue

Graur avatar Nov 14 '22 08:11 Graur

@OlesiaSub Can you please check & close if all fine.

mximp avatar Nov 14 '22 09:11 mximp

I've checked examples from the issue description and it seems that we don't have that problem anymore. Also we have implemented the required logic in #1170.

volodya-lombrozo avatar Mar 06 '23 13:03 volodya-lombrozo