Nicklas Larsson

> > file does not recognize sub-command CHMOD > > [CHMOD new in 3.19](https://cmake.org/cmake/help/latest/command/file.html#chmod) We have to work-around that one, perhaps [execute_process](https://cmake.org/cmake/help/latest/command/execute_process.html)?

> > > > file does not recognize sub-command CHMOD > > > > > > > > > [CHMOD new in 3.19](https://cmake.org/cmake/help/latest/command/file.html#chmod) > > > > > > We...

> Ah! `configure_file` is configure-time, so `execute_process` should work. `execute_process(COMMAND /bin/chmod 755 ${GISBASE}/${script_file_name})` seems to work for me.

I have the same issue with [pdal](https://ports.macports.org/port/PDAL), after a [renaming it to lowercase](https://github.com/macports/macports-ports/commit/ad254d9b202df2aef0567c2fcbfaae7f785813eb). I noticed also that the web livecheck doesn't detect updates, while `port livecheck pdal` does. So this...

You could perhaps do a version check, like: https://github.com/OSGeo/grass/blob/76bcffb3571a2df5843c38a05a9e9e37434c5ebb/scripts/db.dropcolumn/db.dropcolumn.py#L97-L106 and use the syntax supported for older versions only there (even if it is slower).

I wonder if this shouldn't be approached in a different way. There _is_ C API (driver specific) for this (`db_begin_transaction()`, `db_commit_transaction()`). Anyways, it should have been `START TRANSACTION` and `COMMIT`,...

> > Anyways, it should have been `START TRANSACTION` and `COMMIT`, not `BEGIN TRANSACTION` and `END TRANSACTION`. > > or BEGIN as alias for START TRANSACTION. Yes, for the drivers...

Injecting sql code in `db.execute` breaks current behaviour, so that is not a good option. I was originally thinking of directly use C API in Python, like in: https://github.com/OSGeo/grass/blob/7413740dd81c11c2909ad10e7c3161fc097088f9/gui/wxpython/vdigit/wxdigit.py#L575 or...

I like where this is going. Something like this is what I had in mind, although it is way beyond the original intent of this PR. But I don't mind....

https://github.com/OSGeo/grass/pull/3636/commits/093c461380326dccd897db142fc603d50aea2490: great initiative!