Open rezeau opened 4 months ago
You may want to leave some more information. I can't reproduce/confirm this issue.
I'd just like to add a word of advice here. Be aware that the H5P CLI tool, when using the setup command, will use the sources of the master branches of the github repositories. That's usually not the release version, but a development version. If you use the H5P CLI tool blindly to build H5P files in order to install these on some other H5P integration, you risk installing untested buggy code on that platform and potentially even spread it if people download the respective content and upload it on their system. See also https://github.com/h5p/h5p-cli/issues/55
So, if that applies to your case, please rather pack the libraries that you have been working on, and then install only those on the other platform, and then create demo content or whatever on that platform.
Thanks for the advice, @otacke ! I will be careful with contents created with the H5P CLI tool...
Here is more information as requested by @otacke Scenario to test issue with a simple H5P content
Still cannot reproduce this. Might be an issue with running adm-zip
inside the H5P CLI tool on Windows (?) only.
I cannot reproduce this either. On Windows it is recommended that you run the h5p-cli commands inside git bash.
@devland and @otacke I am still experiencing this issue. When you say "cannot reproduce" do you mean you have actually tried my scenario on a Windows platform? @devland Running the h5p-cli commands in git bash does not enter in consideration when Re-using the contents. @otacke probably has the correct explanation, so, how about making adm-zip work on Windows?
@rezeau Nope. Did not try this on Windows. Otherwise I could have looked into the cause instead of speculating. You didn't mention Windows at all before, so I merely guessed because I know you that are bold enough to use it.
@otacke Yes, I am bold/foolish enough to stick to Windows.😵💫
In the h5p-cli tools the pack library
command works OK. That command calls the adm-zip
script, but only the first part of it.
The export content
command (used by the Re-use link in the server interface) starts with exporting all the libraries needed by the particular content that is being exported, then it continues with the module.exports
function.
As far as I can see the Windows bug lies somewhere in that function. It would be great if a fix could be found.
@rezeau I don't know how much impact creating issues on github has. From what I experience, currently what does not exist on H5P Group's JIRA instance, does not have a good chance to be tackled any time soon except for some lucky cases.
@otacke Done at https://h5ptechnology.atlassian.net/browse/HFP-4159 Probably not the best of bug reports but can't do better at the moment. And of course I don't expect any fix to happen any time soon. "Blessed is he who expects nothing, for he shall never be disappointed" Alexander Pope.
@rezeau Sure, but expecting nothing sounds a lot like living half-asleep. I am not the maintainer anyway ;-)
the CLI interface offers to Reuse Content as Download etc but it does not work: when importing the saved content into another platform! I get error message "A valid content folder is missing". Same problem if I use the CLI Export content command.