Hello Chris,

in Case of the CMake files I already did some things here.

I would push the CMake files to compile the scources on the branch, so you only 
have to bind the Pom.xml.

Best regards

Markus

Freundliche Grüße

Markus Sommer
Geschäftsführer

isb innovative software businesses GmbH
Otto-Lilienthal-Strasse 2
D - 88046 Friedrichshafen

Tel.:    +49 (0) 7541 3834-14
Mob:  +49 (0) 171 537 8437
Fax:     +49 (0) 7541 3834-20
E-Mail: som...@isb-fn.de
Web: www.isb-fn.de 

Geschäftsführer: Markus Sommer, Thomas Zeler
Sitz: Friedrichshafen

Registergericht: Amtsgericht Ulm HRB-Nr. 631624
Important Note: This e-mail and any attachments are confidential, may contain 
trade secrets and may well also be legally privileged or otherwise protected 
from disclosure. If you have received it in error, you are on notice of its 
status. 
Please notify us immediately by reply e-mail and then delete his e-mail and any 
attachment from your system. If you are not the intended recipient please 
understand that you must not copy this e-mail or any attachments or disclose 
the contents to any other person. Thank you.


-----Ursprüngliche Nachricht-----
Von: Christofer Dutz <christofer.d...@c-ware.de> 
Gesendet: Dienstag, 28. Mai 2019 10:22
An: dev@plc4x.apache.org
Betreff: Re: [BUILDS] What build system to use?

Hi all,

I'll try to summarize responses to all (

So in general you are quite happy as it currently is, however the C++ module 
could use simplification. I would suggest that I get rid of the individual 
pom.xml files and all the packaging and unpacking and change the CMake files to 
use relative paths in the project structure?
In general you are ok with using maven to trigger a native build tool (cmake, 
python setup.py or dotnet) and to have this build the entire language part of 
that particular module. I could make the maven plugin work in a way that it can 
generate code for multiple specs into different directories (think it already 
allows this anyway) and to run this at the language-root. So a build for any 
language would be:

1. Generate the code using the plc4x-maven-plugin 2. Execute the native build

Beyond that we would need:

- Clean by providing extra rules to the maven-clean-plugin to clean up the 
language dependent directories and resources.
- If we want to ship artifacts ... find a way to do so in the package, deploy 
and install phases (For now this is mostly disabled anyway ... but we should 
come up with something)

Things that are important here is a way to provide the version from maven to 
the native build tool as otherwise we would have problems releasing. I think I 
have implemented this at least for dotnet and python ... have to double-check 
with CMake.

I am not focused on using Thrift at all ... it was something you folks came up 
with and I simply made it work reliably in our build. I do agree that I would 
prefer not having to build the thrift binary in advance. If you come up with an 
alternative, I'm happy to replace this.
The reason for us building it, was that the Thrift project is not distributing 
thrift compiler binaries for anything except windows. I could contact them and 
donate our thrift config to the project and hope they start also distributing 
other binaries, but for now there was just no way to tun the thrift compiler 
without manually pre-installing it on every system. We could remove that thrift 
compilation and extend the pre-flight-check and README documentation with 
instructions on how to install thrift, but then again the list of things to 
install is continuously growing. 

I do have some strong Anti-Docker feelings. First of all ... it does require 
jumping some extra Hoops on Mac and Windows (Here we need a VM to run docker in 
... this has been dealt with, but I think it could complicate things). The far 
worse problem is that the current way of distributing Docker images is a huge 
mine-field Apache distribution-policy wise. I wouldn't want to audit any Docker 
images to not include category X stuff and I would like to avoid even visiting 
this war-field ;-) It's currently sort of in a "we'll do it and decide what to 
do policy wise sometime in the future" ... 

Regarding the splitting into individual poms ... we have already done this ... 
if you change into the plc4j directory and run a "mvn install" it will only 
build the java part and you won't need any special profiles to be enabled ... 
so not quite sure what you need beyond that ... I usually only build the 
modules I'm working on and do a full build quite often only before committing.

Regarding documentation ... this is a big to-do I have on my list ... 
describing which steps in which order are executed for which reason. Just 
didn't have the time to yet. But I'll happily prioritize this up, if it helps.

Ufff ... now I'm also starting to produce monster emails ;-)

Chris

Am 27.05.19, 17:22 schrieb "Dr. Julian Feinauer" 
<j.feina...@pragmaticminds.de>:

    Hi all,
    
    first, thanks Chris for bringing this up.
    I also wanted to start this discussion on the list but thank you very much 
for doing it and the nice writeup.
    
    We now have 4 languages (Java, C++, C# and Python) and hopefully more to 
come (C would be really nice, I personally think and as some of you may know, 
I'm a C master now).
    So I think it is a good time now to "loosen" the coupling between the 
languages (at least until code-gen comes, then we have to reconsider).
    I also have the hope that we will grow our community and have "subgroups" 
that take care of these languages.
    So, I definitely favor the "native build system" approach which.
    
    On the other hand, I totally agree with Chris, that we have to keep an eye 
on keeping things release-able.
    There are two general options for that... separate repos and separate 
release cycles or, as Chris suggests, some kind of "meta-automation".
    I cannot judge (have not enough experience) to see which one of both 
approaches is worse, but I think I prefer the latter.
    
    I just checked how Apache Arrow handles this (they also have separate 
language bindings, see https://github.com/apache/arrow) and it looks like they 
have the latter approach and bind it together via CMake.
    
    To come to a conclusion I think our best bet is to keep it kind of as-is 
but do this a bit more explicit. That means,
    * every lang has a subfolder and uses a native build tool
    * we provide an opportunity to bind all that together to allow for "joint" 
builds and releases
    
    But, two other things which are wort notice.
    I agree with Björn that we should rethink the current way we handle thrift, 
as it feels a bit unnatural for me ( well, that’s well known by now).
    One option could be to provide a docker container for that (arrow does it 
that way, I think) to perform the code generation.
    
    Second, I suggest to consider splitting maven into to "separate" poms.
    This means we use maven as "multi-build-tool" on one hand to orchestrate 
the build for us but have a separate java maven build for the maven subfolder.
    I cannot say whether maven is the optimal solution for the 
"multi-build-tool", but I guess that’s always hard and our maven integration 
currently works pretty nice and flawless so it would be good to keep that.
    
    Perhaps Chris as maven expert can say something about if it is feasible to 
do what I suggest and makes any sense?
    
    Julian
    
    Am 27.05.19, 17:03 schrieb "Bjoern Hoeper" <hoe...@ltsoft.de>:
    
        Hey everyone,
        
        I would also opt for Maven being at least the triggering build system 
because it also integrates well with Jenkins and the mechanisms behind it are 
quite understandable.
        I think the main point we need to solve is to get some kind of 
documentation which build step in which profile triggers what, with which set 
of parameters to make it easier for people not working on the native parts to 
debug stuff. I think Chris Preflight check already adds a lot to this point.
        I think for .NET we found quite a nice solution because it integrates 
well. For C++ it is always difficult to get everything working with different 
architectures and stuff. But I think the current Maven / CMake Integration is 
as good as it gets.
        
        One question that remains open for me is if it is really necessary to 
build Thrift and all of Boost during the C++ compilation because both take a 
lot of time and are quite error prone.
        
        So on the bottom line +1 for the integrated Maven solution. But I think 
it is not a real yes/no question but more of a "yes...but" kind of thing.
        
        Best Regards
        Björn
        
        
        
        -----Ursprüngliche Nachricht-----
        Von: Christofer Dutz <christofer.d...@c-ware.de> 
        Gesendet: Montag, 27. Mai 2019 14:42
        An: dev@plc4x.apache.org
        Betreff: [BUILDS] What build system to use?
        
        Hi all,
        
        as the discussion had come up multiple times now … mainly on Slack, I 
would like to do a dedicated discussion here on the list about the topic.
        
        Up to now we had built the build to generally build every module with 
maven, utilizing plugins to enable triggering builds with things like CMake.
        The maven build then took care of packing the results and handling the 
dependencies (At least this was how I built things for the C++ module) With the 
plc4net however we’re using an execution of “dotnet” to build a solution.
        In this case the build is entirely handled by dotnet and all maven does 
is trigger the build and fail if the execution fails with any non 0 return code.
        We are currently doing something similar with python: where we’re just 
calling python to execute the setup.py script.
        
        What is your opinion on how we should do our builds?
        If we go down the path like I initially setup C++, we have the benefit 
of utilizing the maven ecosystem and we will not have any problems during 
releases.
        It however comes with some comfort disadvantages for the “native” guys. 
But things like code-generation will work nicely.
        
        If we go down a non-maven path we will have to deal with all of this 
ourselves … we will have to come up with a way to handle everything ourselves 
Or learn how to do it in python and CMake and dotnet, and … The other thing is 
that we can’t integrate the driver generation as easy as with maven.
        Either we manually execute the code generation multiple times with 
maven to generate multiple output directories before executing the individual 
build system Or we have to build custom generators for every build-system we 
are using.
        
        Even if I like the way I setup the C++ build, I doubt all people will 
like it as it requires them to do things differently than they are used to.
        But I would strongly object implementing multiple code-generators.
        
        One option would be to split up the plc4x git repo into multiple repos 
– each one for one language … so we’d have a plc4j, plc4cpp, plc4net and 
plc4python.
        Each of these would be dedicated to one particular language and have 
build tools that fit them. However I strongly object this option, as it would 
require the Release manager to understand and setup each of these environments.
        
        Another option which I think would be a valid compromise, would be to 
have all languages in one repo the way we currently have it and Maven as 
triggering build system.
        The code is generated by maven but the build itself is handled by the 
particular build system.
        This will require people working exclusively in VisualStudio or some 
Python IDE, to manually run a “mvn generate-sources” first, but I think that’s 
a reasonable restriction.
        
        The benefit is that releasing this should be possible with our current 
release process which has a limited setup cost for the Release Manager.
        
        I would opt for the hybrid option with Maven as initiating build system 
for releases and code generation and CI stuff but to have the native build 
system for every language to build the individual parts.
        Namely this would be:
        
          *   Plc4cpp: CMake
          *   Plc4net: dotnet
          *   Plc4Python: python setup.py
        
        Not quite sure how we can get the test-results to be reported correctly.
        
        What are your thoughts on this?
        
        Chris
        
        
        
    
    

Reply via email to