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

lingo build fails on HelloWorld C because invalid JSON string #36

Closed
erlingrj opened this issue Aug 28, 2023 · 4 comments
Closed

lingo build fails on HelloWorld C because invalid JSON string #36

erlingrj opened this issue Aug 28, 2023 · 4 comments

Comments

@erlingrj
Copy link
Collaborator

Building ...
Running "/home/erling/tools/lingua-franca/build/install/lf-cli/bin/lfc" "--json={\"out\":\"/home/erling/dev/lf/lf-ex/my-synth/target\",\"properties\":{\"no-compile\":false},\"src\":\"/home/erling/dev/lf/lf-ex/my-synth/./src/Main.lf\"}"
Unmatched argument at index 4: 'false'

I think it might be because --no-compile=false is not a valid command line argument. If you want to invoke the target compiler, just dont pass any --no-compile arg

@tanneberger
Copy link
Member

This is a huge problem with lfc.

The properties field is getting handled wrong. All the fields in properties are getting treated as command line parameters and not for example not target properties.

@lhstrh
Copy link
Member

lhstrh commented Oct 2, 2023

This is a huge problem with lfc.

The properties field is getting handled wrong. All the fields in properties are getting treated as command line parameters and not for example not target properties.

I will go ahead and address this in lf-lang/lingua-franca#2008.

@lhstrh lhstrh added this to the Initial Release milestone Oct 10, 2023
@lhstrh
Copy link
Member

lhstrh commented Oct 10, 2023

This hinges on lf-lang/lingua-franca#2035 and associated PRs that I'm still working on.

@lhstrh
Copy link
Member

lhstrh commented Nov 6, 2023

Since lf-lang/lingua-franca#2008 has been merged, these problems should be fixed. There was a bug that ignored the output directory, but that is fixed in lf-lang/lingua-franca#2088.

@lhstrh lhstrh closed this as completed Nov 6, 2023
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