contentauth / c2patool

Command line tool for displaying and adding C2PA manifests
Apache License 2.0
94 stars 30 forks source link

Fix windows release #132

Closed jackfarzan closed 11 months ago

jackfarzan commented 11 months ago

Changes in this pull request

Switch from tar/compress to 7z for windows release

Checklist

codecov-commenter commented 11 months ago

Codecov Report

Merging #132 (30c87bc) into main (74220bf) will not change coverage. The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@           Coverage Diff           @@
##             main     #132   +/-   ##
=======================================
  Coverage   83.91%   83.91%           
=======================================
  Files           3        3           
  Lines         373      373           
=======================================
  Hits          313      313           
  Misses         60       60           
jackfarzan commented 11 months ago

This looks like a corruption of the change. I approve the Compress-Archive change, but not the 7z thing

Here's an example of it running successfully with 7z https://github.com/jackfarzan/c2patool/actions/runs/5658765421/job/15330803932

Compress-Archive does not exist on the GH windows runner.