Changes between Version 11 and Version 12 of G8SourceLayout


Ignore:
Timestamp:
02/17/21 12:47:59 (4 years ago)
Author:
wenzeslaus
Comment:

--

Legend:

Unmodified
Added
Removed
Modified

  • G8SourceLayout

    v11 v12  
    4141* `include` contains things for make, so any includes would go here? E.g., Jinja templates if we would have some or more realistic example `*.cmake` files.
    4242* `include` does not contain grass, but the header files are there directly. Common source code layout nowadays is to have a directory named as the resulting directory in the distribution, i.e., not `include/*.h` but `include/grass/*.h`. This makes it easier for many code editing tools.
     43  * PR:1350
    4344* Python code in `lib/python` is next to C libraries which may look like it is a C library to communicate with Python.
    4445* `lib/python` suffers from the same as header files in `include`. In this case it is even little more serious because it breaks rules on how a Python package should look like. Again, dir named `grass` should be inserted before the dir with the subpackages.
    4546 * https://github.com/OSGeo/grass/issues/1100
     47 * PR:1356
    4648* `gui/wxpython` again similar to `include` and `lib/python` because there is no "package" directory, but a greater reorganization may be needed as the "wxgui package" is not even in the distribution and the subdirectories are all top-level packages.
    4749