maxlath / wikibase-cli

read and edit a Wikibase instance from the command line
MIT License
226 stars 24 forks source link

make clear in readme if the the tool can work on Windows #82

Closed vojtechhuser closed 2 years ago

maxlath commented 4 years ago

I don't have a Windows machine at hand, could you try to npm install wikibase-cli@latest and tell me how it goes? There was a major version released yesterday, and this issue #72 should have been fixed

VladimirAlexiev commented 4 years ago

@vojtechhuser I use Win10 and just upgraded to the latest version and didn't have the issue #72. (I've reported some seg faults previously). @maxlath Will keep you posted if I hit any errors.

VojtechDostal commented 2 years ago

I've just been trying to install this on Windows but I've ran into similar issues as those reported in #72

Install with npm seemed to go smoothly - as far as I can tell, I'm not an expert - but running the "wd summary Q5" command from my command line returns an error - "Sh is not a recognizable command...", which seems to be similar to what others have reported.

maxlath commented 2 years ago

@VojtechDostal do you get the same error with wb summary Q5 --instance https://www.wikidata.org?

VojtechDostal commented 2 years ago

@VojtechDostal do you get the same error with wb summary Q5 --instance https://www.wikidata.org?

That does work! What have missed or forgotten to configure? :)

maxlath commented 2 years ago

wd is just a convenience executable that calls wb, but in its current implementation, it explicitly requires to do that in a sh environment. Maybe removing that first line would let the choice of the shell type to the executing context: could you try to remove that line in your local installation and see if wd summary Q5 works?

VojtechDostal commented 2 years ago

After removing the 1st line from "wd" file, I restarted the command line window and tried "wd summary Q5" again but the problem remains the same

image

Command seem to work for my if I always include the instance:

wb add-qualifier Q1018410$FCBA6A36-C936-4A8E-9C5B-40C31BEFD87C' P3831 Q1263068 --instance https://www.wikidata.org

-->

https://www.wikidata.org/w/index.php?title=Q1018410&diff=prev&oldid=1569186941

maxlath commented 2 years ago

I rewrote wd as a node script in 2a4eaa0, published in v15.16.1: can you upgrade and retry wd summary Q5?

VojtechDostal commented 2 years ago

I rewrote wd as a node script in 2a4eaa0, published in v15.16.1: can you upgrade and retry wd summary Q5?

I upgraded and tried again, attaching results:

image

So there seems to be a different error this time.

maxlath commented 2 years ago

new attempt :D 38a9ea8 published in v15.16.3

VojtechDostal commented 2 years ago

Sorry for taking a bit longer :)

This seemed to have worked. After update, I tried the following command:

wd update-claim Q4466817$72dfdfa0-4c71-7806-6d8c-dc7b316148a6 "Thuja occidentalis ʽSmaragd’" --> https://www.wikidata.org/w/index.php?title=Q4466817&diff=prev&oldid=1573856361

Thanks! 👍

maxlath commented 2 years ago

Regarding the original issue, I added a mention that Windows should be part of the supported system: a1b780a