ScreenToGif icon indicating copy to clipboard operation
ScreenToGif copied to clipboard

Improve chocolatey package

Open majkinetor opened this issue 5 years ago • 12 comments

ffmpeg and gifski could be added as choco dependencies. Gifski doesn't exist as choco package yet but I am willing to make it.

That way

  • doing cinst screentogif would install all of them and configure correctly.
  • doing choco update ... would update only dependencies that actually have a new version
  • it is also possible to install only main package without dependencies using --ignore-dependencies cmd line argument.

What do you think ?

If you accept, I can do relevant work. I also think package should be embedded (i.e. contain binary inside, not just link)

majkinetor avatar Jan 14 '19 15:01 majkinetor

@majkinetor , It would be nice, if you add lossygif to Chocolatey too. See my table — it a best GIF compression tool.

Thanks.

Kristinita avatar Jan 30 '19 15:01 Kristinita

@Kristinita

They don't provide binaries so its a bit complicated as I need to build them myself which I generally do not do.

majkinetor avatar Jan 30 '19 16:01 majkinetor

@majkinetor Hi, sorry for taking so long.

I would like to push this forward, at least with FFmpeg.

With Gifski and SharpDX, it's a little bit different. I think that it's best to bundle together.

NickeManarin avatar Feb 28 '20 19:02 NickeManarin

Sure, just add 1 line to nuspec file, that it depends on ffmpeg. I am personally involved in keeping this package up to date all the time as you can see in the version history.

If you need any help let me know.

majkinetor avatar Feb 28 '20 20:02 majkinetor

Strange, it's not working on my tests in here:

I added these lines to the nuspec:

<metadata>
  <package>
    [...]
    <dependencies>
      <dependency id="ffmpeg" version="4.2.2"/>
    </dependencies>
  </metadata>
</package>

And I'm trying to test with this command: choco install -fdv .\screentogif.nuspec

I end up with this error:

screentogif not installed. An error occurred during installation:
 Unable to resolve dependency 'ffmpeg (≥ 4.2.2)'.

NickeManarin avatar Mar 03 '20 22:03 NickeManarin

Forget it, it's working now with this command, running the same folder as the package:

choco install -s "'.;https://chocolatey.org/api/v2/'" -fdv screentogif

NickeManarin avatar Mar 03 '20 22:03 NickeManarin

Now for the Gifski and SharpDX, I think that I could simply ship my own MSI file. But I'm not sure how existent "choco" installations of ScreenToGif would work.

Because the current package is installed via the portable version.

NickeManarin avatar Mar 03 '20 22:03 NickeManarin

Now for the Gifski and SharpDX, I think that I could simply ship my own MSI file.

Why not just add them to portable installation directly. Why MSI for 1 simple exe and a library ?

Like suggested I can create gifsky package. If you add it to deps, the exe will then be available on the PATH. If you don't want to depend on choco, you should embedd it.

majkinetor avatar Mar 05 '20 07:03 majkinetor

I'll try to embedd both Gifski.dll and SharpDX.dll.

NickeManarin avatar Mar 05 '20 16:03 NickeManarin

I just got a message saying Netframework 4.8 is required as well... Could be added as another dep: https://chocolatey.org/packages/netfx-4.8

lucastheisen avatar Mar 23 '20 19:03 lucastheisen

@lucastheisen Yes, it can be. Thanks.

NickeManarin avatar Mar 31 '20 14:03 NickeManarin

Is it possible to get an installer in addition to the current portable version, or at least an install argument? Cause the current package is potable only

brian6932 avatar Jul 16 '21 06:07 brian6932