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

Feeding LFC the JSON via stdin #31

Open
2 tasks
tanneberger opened this issue Jul 25, 2023 · 6 comments
Open
2 tasks

Feeding LFC the JSON via stdin #31

tanneberger opened this issue Jul 25, 2023 · 6 comments
Assignees
Labels
bug Something isn't working enhancement New feature or request good first issue Good for newcomers

Comments

@tanneberger
Copy link
Member

tanneberger commented Jul 25, 2023

JSON via Stdin

We currently pass JSON using the --json={} option, but it would be cleaner to use stdin.

Steps

  • modify lingo to give lfc properties via stdin @revol-xut
  • modify lfc to receive via stdin @lhstrh
@tanneberger tanneberger added bug Something isn't working enhancement New feature or request labels Jul 25, 2023
@tanneberger tanneberger added the good first issue Good for newcomers label Aug 22, 2023
@tanneberger tanneberger mentioned this issue Aug 22, 2023
4 tasks
@tanneberger
Copy link
Member Author

This is not relevant anymore we agreed that we ignore the problem and say that lingo only will work with the non gradle lfc version

@lhstrh
Copy link
Member

lhstrh commented Oct 1, 2023

I still think that it's worth doing this because it is cleaner than passing JSON as CLI args...

@lhstrh lhstrh reopened this Oct 1, 2023
@tanneberger
Copy link
Member Author

@lhstrh Do we want this in lingo v0.2?

@lhstrh
Copy link
Member

lhstrh commented Oct 1, 2023

Sure!

@tanneberger
Copy link
Member Author

How is the lfc site of things doing ?

@lhstrh
Copy link
Member

lhstrh commented Oct 1, 2023

I'm working on it. Still chasing down mysterious bugs: https://github.com/lf-lang/lingua-franca/actions/runs/6369165649

I will add more robust handling of the JSON as well...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants