Hi Otto,

well generally it's considered bad practice to commit generated code.
Especially in this case it mixes up with the real code. 

Yes: It also has the benefit of us being able to review code and I remember we 
used to use that for this in the early days of PLC4C.

Sebastian made the code generation add a comment to the files so at least the 
JetBrains IDEs will start yelling at you as soon as you start editing these 
files.

It was just something that popped my mind and I wanted to hear what you all 
think about it. 

If we keep doing it the way we currently do it, it's less work for me, so I'm 
happy with it.

The upside would be, that someone working on PLC4Go exclusively could 
theoretically update the generated code without requiring Maven ... but I would 
say it's something we should simply: That's the way it works ... deal with it 
;-)

Chris


-----Ursprüngliche Nachricht-----
Von: Otto Fowler <[email protected]> 
Gesendet: Freitag, 19. März 2021 13:30
An: [email protected]
Betreff: Re: [DISCUSS] Generate a standalone executable version of the 
maven-plugin and build tools?

My impression was that we checked in the code so that we could review the code 
/ code generation changes in github, not because of a lack of automation.  I 
don’t see what this would give us that we don’t have, other than being in a 
different shiny way.  Can you elaborate more on the problem this would solve?

> On Mar 19, 2021, at 05:55, Christofer Dutz <[email protected]> wrote:
> 
> Hi,
> 
> last week I managed to do some cool stuff with GraalVM compiling a Java 
> Application into a windows executable.
> Now I was thinking ... wouldn't this also work for our Maven plugin? Of 
> course, we probably need to add something to it or to create a new module 
> that uses the template engine and the mspec parser in a standalone 
> application.
> 
> So, If we had such an executable, we could probably automate the code 
> generation for C, Go and .Net too and wouldn't have to check-in generated 
> code.
> 
> What do you think?
> 
> 
> Chris

Reply via email to