Merge pull request #242 from lf-lang/fix-compiler

Updated compiler and compiler-flags property, removed cmake property

Author: cmnrd

docs/reference/target-declaration.mdx

@@ -28,15 +28,14 @@ A target specification may have optional parameters, the names and values of whi
 - [**build-type**](#build-type): One of Debug (the default), Release, RelWithDebInfo and MinSizeRel.
 - [**cargo-dependencies**](#cargo-dependencies): (Rust only) list of dependencies to include in the generated Cargo.toml file.
 - [**cargo-features**](#cargo-features): (Rust only) List of string names of features to include.
-- [**cmake**](#cmake): Whether to use cmake for building.
 - [**cmake-include**](#cmake): List of paths to cmake files to guide compilation.
 - [**compiler**](#compiler): A string giving the name of the target language compiler to use.
+- [**compiler-flags**](#compiler-flags): An arrays of strings giving options to be passed to the target compiler.
 - [**docker**](#docker): A boolean to generate a Dockerfile.
 - [**external-runtime-path**](#external-runtime-path): Specify a pre-compiled external runtime library located to link to instead of the default.
 - [**export-dependency-graph**](#export-dependency-graph): To export the reaction dependency graph as a dot graph (for debugging).
 - [**fast**](#fast): A boolean specifying to execute as fast as possible without waiting for physical time to match logical time.
 - [**files**](#files): An array of paths to files or directories to be copied to the directory that contains the generated sources.
-- [**flags**](#flags): An arrays of strings giving options to be passed to the target compiler.
 - [**logging**](#logging): An indicator of how much information to print when executing the program.
 - [**no-compile**](#no-compile): If true, then do not invoke a target language compiler. Just generate code.
 - [**no-runtime-validation**](#no-runtime-validation): If true, disable runtime validation.
@@ -57,13 +56,12 @@ c={
     auth: <true or false>
     build: <string>,
     build-type: <Debug, Release, RelWithDebInfo, or MinSizeRel>,
-    cmake: <true or false>,
     cmake-include: <string or list of strings>,
     compiler: <string>,
+    compiler-flags: <string or list of strings>,
     docker: <true or false>,
     fast: <true or false>,
     files: <string or list of strings>,
-    flags: <string or list of strings>,
     logging: <error, warning, info, log, debug>,
     no-compile: <true or false>,
     protobufs: <string or list of strings>,
@@ -76,6 +74,7 @@ cpp={
 `target Cpp {
     build-type: <Debug, Release, RelWithDebInfo, or MinSizeRel>,
     cmake-include: <string or list of strings>,
+    compiler: <string>,
     external-runtime-path: <string>,
     export-dependency-graph <true or false>,
     fast: <true or false>,
@@ -276,35 +275,6 @@ This is a list of features of the generated crate. Supported are:
 </ShowIf>
 </ShowIfs>
 
-## cmake
-
-<ShowIfs>
-<ShowIf py rs ts>
-This target does not support the `cmake` target option.
-</ShowIf>
-<ShowIf cpp>
-The C++ target does not support the `cmake` target option because it always uses `cmake`.
-</ShowIf>
-<ShowIf c>
-```lf-c
-target C {
-    cmake: <true or false>
-};
-```
-
-This will enable or disable the CMake-based build system (the default is `true`). Enabling the CMake build system will result in a `CMakeLists.txt` being generated in the `src-gen` directory. This `CMakeLists.txt` is then used when `cmake` is invoked by the LF runtime (either the `lfc` or the IDE). Alternatively, the generated program can be built manually. To do so, in the `src-gen/ProgramName` directory, run:
-
-```sh
-mkdir build && cd build
-cmake ../
-make
-```
-
-If `cmake` is disabled, `gcc` is directly invoked after code generation by default. In this case, additional target properties, such as [compiler](#compiler) and [flags](#flags) can be used to gain finer control over the compilation process.
-</ShowIf>
-</ShowIfs>
-
-
 <ShowOnly c cpp>
 
 ## cmake-include
@@ -361,25 +331,33 @@ See [`AsyncCallback.lf`](https://github.com/lf-lang/lingua-franca/blob/master/te
 <ShowIf ts py rs>
 This target does not support the `compiler` target option.
 </ShowIf>
+<ShowIf c cpp>
+This parameter is a string giving the name of the compiler to use. Normally
+CMake selects the best compiler for your system, but you can use this parameter
+to point to your preferred compiler. Possible values are, for instance, `gcc`,
+`g++`, `clang`, or `clang++`.
+</ShowIf>
+</ShowIfs>
+
+## compiler-flags
+
+<ShowIfs>
+<ShowIf cpp py ts rs>
+This target does not support the `compiler-flags` parameter.
+</ShowIf>
 <ShowIf c>
-This parameter is a string giving the name of the C compiler to use.
-It is used only when [cmake](#cmake) is set to `false`. For example:
+This parameter is a list of strings giving additional arguments to pass to the target language compiler. For example:
 
 ```lf-c
 target C {
-    cmake: false,
-    compiler: "cc",
+    compiler-flags: ["-g", "-I/usr/local/include", "-L/usr/local/lib", "-lpaho-mqtt3c"],
 };
 ```
 
-The `compiler` option here specifies to use `cc` rather than `gcc`.
-</ShowIf>
-<ShowIf cpp>
-This parameter is a string giving the name of the C++ compiler to use. Normally
-CMake selects the best compiler for your system, but you can use this parameter
-to point it to your preferred C++ compiler.
-</ShowIf>
+The `compiler-flags` option specifies to include debug information in the compiled code (`-g`); a directory to search for include files (`-I/usr/local/include`); a directory to search for library files (`-L/usr/local/lib`); a library to link with (`-lpaho-mqtt3c`, which will link with file `libpaho-mqtt3c.so`).
 
+**Note**: Using the `compiler-flags` parameter is strongly discouraged, although supported. Flags are compiler-specific, and thus interfere with CMake's ability to find the most suitable compiler for each platform. In a similar fashion, we recommend against the use of the `compiler` standard parameter for the same reason. A better solution is to provide a `cmake-include` file.
+</ShowIf>
 </ShowIfs>
 
 ## docker
@@ -494,29 +472,6 @@ Moreover, the `files` target specification works in conjunction with the `import
 </ShowIf>
 </ShowIfs>
 
-## flags
-
-<ShowIfs>
-<ShowIf cpp py ts rs>
-This target does not support the `flags` parameter.
-</ShowIf>
-<ShowIf c>
-This parameter is a list of strings giving additional arguments to pass to the target language compiler.
-It is used only when [cmake](#cmake) is set to `false`. For example:
-
-```lf-c
-target C {
-    cmake: false,
-    flags: ["-g", "-I/usr/local/include", "-L/usr/local/lib", "-lpaho-mqtt3c"],
-};
-```
-
-The `flags` option specifies to include debug information in the compiled code (`-g`); a directory to search for include files (`-I/usr/local/include`); a directory to search for library files (`-L/usr/local/lib`); a library to link with (`-lpaho-mqtt3c`, which will link with file `libpaho-mqtt3c.so`).
-
-**Note**: Using the `flags` standard parameter when `cmake` is enabled is strongly discouraged, although supported. Flags are compiler-specific, and thus interfere with CMake's ability to find the most suitable compiler for each platform. In a similar fashion, we recommend against the use of the `compiler` standard parameter for the same reason. A better solution is to provide a `cmake-include` file, as described next.
-</ShowIf>
-</ShowIfs>
-
 ## logging
 
 <ShowIfs>

versioned_docs/version-0.6.0/reference/target-declaration.mdx

@@ -28,15 +28,14 @@ A target specification may have optional parameters, the names and values of whi
 - [**build-type**](#build-type): One of Debug (the default), Release, RelWithDebInfo and MinSizeRel.
 - [**cargo-dependencies**](#cargo-dependencies): (Rust only) list of dependencies to include in the generated Cargo.toml file.
 - [**cargo-features**](#cargo-features): (Rust only) List of string names of features to include.
-- [**cmake**](#cmake): Whether to use cmake for building.
 - [**cmake-include**](#cmake): List of paths to cmake files to guide compilation.
 - [**compiler**](#compiler): A string giving the name of the target language compiler to use.
+- [**compiler-flags**](#compiler-flags): An arrays of strings giving options to be passed to the target compiler.
 - [**docker**](#docker): A boolean to generate a Dockerfile.
 - [**external-runtime-path**](#external-runtime-path): Specify a pre-compiled external runtime library located to link to instead of the default.
 - [**export-dependency-graph**](#export-dependency-graph): To export the reaction dependency graph as a dot graph (for debugging).
 - [**fast**](#fast): A boolean specifying to execute as fast as possible without waiting for physical time to match logical time.
 - [**files**](#files): An array of paths to files or directories to be copied to the directory that contains the generated sources.
-- [**flags**](#flags): An arrays of strings giving options to be passed to the target compiler.
 - [**logging**](#logging): An indicator of how much information to print when executing the program.
 - [**no-compile**](#no-compile): If true, then do not invoke a target language compiler. Just generate code.
 - [**no-runtime-validation**](#no-runtime-validation): If true, disable runtime validation.
@@ -57,13 +56,12 @@ c={
     auth: <true or false>
     build: <string>,
     build-type: <Debug, Release, RelWithDebInfo, or MinSizeRel>,
-    cmake: <true or false>,
     cmake-include: <string or list of strings>,
     compiler: <string>,
+    compiler-flags: <string or list of strings>,
     docker: <true or false>,
     fast: <true or false>,
     files: <string or list of strings>,
-    flags: <string or list of strings>,
     logging: <error, warning, info, log, debug>,
     no-compile: <true or false>,
     protobufs: <string or list of strings>,
@@ -76,6 +74,7 @@ cpp={
 `target Cpp {
     build-type: <Debug, Release, RelWithDebInfo, or MinSizeRel>,
     cmake-include: <string or list of strings>,
+    compiler: <string>,
     external-runtime-path: <string>,
     export-dependency-graph <true or false>,
     fast: <true or false>,
@@ -276,35 +275,6 @@ This is a list of features of the generated crate. Supported are:
 </ShowIf>
 </ShowIfs>
 
-## cmake
-
-<ShowIfs>
-<ShowIf py rs ts>
-This target does not support the `cmake` target option.
-</ShowIf>
-<ShowIf cpp>
-The C++ target does not support the `cmake` target option because it always uses `cmake`.
-</ShowIf>
-<ShowIf c>
-```lf-c
-target C {
-    cmake: <true or false>
-};
-```
-
-This will enable or disable the CMake-based build system (the default is `true`). Enabling the CMake build system will result in a `CMakeLists.txt` being generated in the `src-gen` directory. This `CMakeLists.txt` is then used when `cmake` is invoked by the LF runtime (either the `lfc` or the IDE). Alternatively, the generated program can be built manually. To do so, in the `src-gen/ProgramName` directory, run:
-
-```sh
-mkdir build && cd build
-cmake ../
-make
-```
-
-If `cmake` is disabled, `gcc` is directly invoked after code generation by default. In this case, additional target properties, such as [compiler](#compiler) and [flags](#flags) can be used to gain finer control over the compilation process.
-</ShowIf>
-</ShowIfs>
-
-
 <ShowOnly c cpp>
 
 ## cmake-include
@@ -361,25 +331,33 @@ See [`AsyncCallback.lf`](https://github.com/lf-lang/lingua-franca/blob/master/te
 <ShowIf ts py rs>
 This target does not support the `compiler` target option.
 </ShowIf>
+<ShowIf c cpp>
+This parameter is a string giving the name of the compiler to use. Normally
+CMake selects the best compiler for your system, but you can use this parameter
+to point to your preferred compiler. Possible values are, for instance, `gcc`,
+`g++`, `clang`, or `clang++`.
+</ShowIf>
+</ShowIfs>
+
+## compiler-flags
+
+<ShowIfs>
+<ShowIf cpp py ts rs>
+This target does not support the `compiler-flags` parameter.
+</ShowIf>
 <ShowIf c>
-This parameter is a string giving the name of the C compiler to use.
-It is used only when [cmake](#cmake) is set to `false`. For example:
+This parameter is a list of strings giving additional arguments to pass to the target language compiler. For example:
 
 ```lf-c
 target C {
-    cmake: false,
-    compiler: "cc",
+    compiler-flags: ["-g", "-I/usr/local/include", "-L/usr/local/lib", "-lpaho-mqtt3c"],
 };
 ```
 
-The `compiler` option here specifies to use `cc` rather than `gcc`.
-</ShowIf>
-<ShowIf cpp>
-This parameter is a string giving the name of the C++ compiler to use. Normally
-CMake selects the best compiler for your system, but you can use this parameter
-to point it to your preferred C++ compiler.
-</ShowIf>
+The `compiler-flags` option specifies to include debug information in the compiled code (`-g`); a directory to search for include files (`-I/usr/local/include`); a directory to search for library files (`-L/usr/local/lib`); a library to link with (`-lpaho-mqtt3c`, which will link with file `libpaho-mqtt3c.so`).
 
+**Note**: Using the `compiler-flags` parameter is strongly discouraged, although supported. Flags are compiler-specific, and thus interfere with CMake's ability to find the most suitable compiler for each platform. In a similar fashion, we recommend against the use of the `compiler` standard parameter for the same reason. A better solution is to provide a `cmake-include` file.
+</ShowIf>
 </ShowIfs>
 
 ## docker
@@ -498,29 +476,6 @@ Moreover, the `files` target specification works in conjunction with the `import
 </ShowIf>
 </ShowIfs>
 
-## flags
-
-<ShowIfs>
-<ShowIf cpp py ts rs>
-This target does not support the `flags` parameter.
-</ShowIf>
-<ShowIf c>
-This parameter is a list of strings giving additional arguments to pass to the target language compiler.
-It is used only when [cmake](#cmake) is set to `false`. For example:
-
-```lf-c
-target C {
-    cmake: false,
-    flags: ["-g", "-I/usr/local/include", "-L/usr/local/lib", "-lpaho-mqtt3c"],
-};
-```
-
-The `flags` option specifies to include debug information in the compiled code (`-g`); a directory to search for include files (`-I/usr/local/include`); a directory to search for library files (`-L/usr/local/lib`); a library to link with (`-lpaho-mqtt3c`, which will link with file `libpaho-mqtt3c.so`).
-
-**Note**: Using the `flags` standard parameter when `cmake` is enabled is strongly discouraged, although supported. Flags are compiler-specific, and thus interfere with CMake's ability to find the most suitable compiler for each platform. In a similar fashion, we recommend against the use of the `compiler` standard parameter for the same reason. A better solution is to provide a `cmake-include` file, as described next.
-</ShowIf>
-</ShowIfs>
-
 ## logging
 
 <ShowIfs>