Nicklas Larsson

Results 286 comments of Nicklas Larsson

Given this another thought. What about default to quarterly, with nix and docker monthly? Let's get this ticket moving, please.

> [@echoix](https://github.com/echoix) In the light of upcoming summit, do you want to reduce the frequency of some of the updates? I was also thinking this may become a very irritating...

> I think we should really consider testing the Dockerfiles on every PR to help us simplify situation like #4343. Docker images, just like Windows and macOS builds are one...

> > I think we should really consider testing the Dockerfiles on every PR to help us simplify situation like #4343. Docker images, just like Windows and macOS builds are...

Just a thought: would it be (reasonably) possible to trigger a docker run in a PR via something like `[docker ci]` (in the spirit of `[skip ci]`)?

> > There is a similar issue with file /vector/v.vol.rst/user3.c, please address that here too. > > Done. That one is a roughly similar issue (CID 1208515), but not the...

If I understand the static analyser's reasoning correctly: ```c while (edges && edges->delete) { // the check of 'edges' implies it potentially can be NULL e = edges; DELETE(edges, e);...

> > If my understanding of the above is right, it should probably be enough to add a `if (!edges) return;` before `e = edges->next;`. > > i guess using...

Thanks for the report @a-benini-2! (Small advice: to make the log content searchable, if there will be reason for another report in the future, please post the log as text)....

> The issue occurs on Windows 10 directly using either GRASS-GIS 8.4.0 or GRASS-GIS 8.5.0dev ([f0a997c](https://github.com/OSGeo/grass/commit/f0a997ca86ff93740e1549fbd6dc5bbf40b19441)) provided by OSGeo4W. The issue didn't occur using GRASS-GIS 8.3.2 on the same system....