koordinates / kart-qgis-plugin

A QGIS integration plugin for Kart repositories. Distributed version-control for geospatial and tabular data.
https://kartproject.org
GNU General Public License v2.0
37 stars 16 forks source link

Show log returns empty window with error: No such option: --graph #92

Closed andreatitolo closed 1 year ago

andreatitolo commented 1 year ago

Describe the bug Attempting to show the log for any repositories I have first returns a "Kart failed" error, then displays an empty log, no graph.

Kart failed with the following message: 
Usage: kart_cli log [OPTIONS] [REVISIONS] [--] [FILTERS]
Try 'kart_cli log --help' for help.
Error: No such option: --graph 

However, in the terminal, kart log shows the correct log.

To Reproduce Steps to reproduce the behavior:

  1. Open QGIS with the Kart repositories pane
  2. Click on any repo and "Show Log.."
  3. Return the above error
  4. Kart plugin displays an empty log

Expected behavior I expected to see the log graph.

Version info (this is important!):

Additional context I haven't used kart for a two-three weeks, but I remember having no troubles seeing the log before. I updated to 0.12.0 and this issue started to show up (I guess?).

rcoup commented 1 year ago

@andreatitolo yes, I think we've changed kart log to not (immediately) pass everything to git log with v0.12.0, which is where the bug is coming from.

@olsen232 the Kart QGIS plugin uses kart log --graph. Is it easiest to add --graph to the options that kart log understands, or pass all unknown options through to git log automatically? Or is there a change we can make in the plugin instead?

olsen232 commented 1 year ago

@craigds selected Kart 0.12 as the time when unknown options would no longer be supported by Kart log, and did some prep for it but we missed this particular invocation. He's away today but I won't work against his plan - instead, to move forward, the following is required:

  1. Add --graph to the options kart log supports - see https://github.com/koordinates/kart/pull/784, already merged
  2. Use --output-format=text:FORMAT or -otext:FORMAT instead of --format=FORMAT when calling kart log from the plugin - see https://github.com/koordinates/kart-qgis-plugin/pull/93
  3. Release new Kart and new plugin
olsen232 commented 1 year ago

Fixed as of v1.0.6, which works with Kart 0.12.2