Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CI: add Alpine multi-platform CI #2894

Closed
wants to merge 1 commit into from
Closed

CI: add Alpine multi-platform CI #2894

wants to merge 1 commit into from

Conversation

neheb
Copy link
Collaborator

@neheb neheb commented Jan 13, 2024

No description provided.

@ghost
Copy link

ghost commented Jan 13, 2024

👇 Click on the image for a new way to code review

Review these changes using an interactive CodeSee Map

Legend

CodeSee Map legend

@neheb neheb force-pushed the ms branch 3 times, most recently from 5a67742 to 6f22ad2 Compare January 13, 2024 01:29
Copy link

codecov bot commented Jan 13, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (28fa956) 64.57% compared to head (cd963b0) 63.93%.
Report is 2 commits behind head on main.

❗ Current head cd963b0 differs from pull request most recent head 94c4ce7. Consider uploading reports for the commit 94c4ce7 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2894      +/-   ##
==========================================
- Coverage   64.57%   63.93%   -0.64%     
==========================================
  Files         104      104              
  Lines       22196    22400     +204     
  Branches    10882    10877       -5     
==========================================
- Hits        14332    14322      -10     
- Misses       5622     5854     +232     
+ Partials     2242     2224      -18     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@neheb neheb force-pushed the ms branch 4 times, most recently from ea4c59e to 54576f6 Compare January 13, 2024 02:27
@neheb
Copy link
Collaborator Author

neheb commented Jan 13, 2024

hrm no new failures. Maybe some will pop up if I apply this to the CMake tests.

@neheb neheb force-pushed the ms branch 5 times, most recently from 5257449 to f6b5cb4 Compare January 13, 2024 04:59
@neheb
Copy link
Collaborator Author

neheb commented Jan 13, 2024

I hate conan.

@neheb neheb force-pushed the ms branch 8 times, most recently from fece63e to 1b5b083 Compare January 19, 2024 02:50
@neheb
Copy link
Collaborator Author

neheb commented Jan 19, 2024

2024-01-19T03:18:41.1850664Z FAIL: test_run (github.test_CVE_2018_12265.AdditionOverflowInLoaderExifJpeg.test_run)
2024-01-19T03:18:41.1851309Z ----------------------------------------------------------------------
2024-01-19T03:18:41.1851701Z Traceback (most recent call last):
2024-01-19T03:18:41.1852232Z   File "/home/runner/work/exiv2/exiv2/tests/system_tests.py", line 652, in test_run
2024-01-19T03:18:41.1852852Z     self.compare_stderr(i, command, processed_stderr, stderr)
2024-01-19T03:18:41.1853490Z   File "/home/runner/work/exiv2/exiv2/tests/system_tests.py", line 773, in compare_stderr
2024-01-19T03:18:41.1854039Z     self._compare_output(
2024-01-19T03:18:41.1854539Z   File "/home/runner/work/exiv2/exiv2/tests/system_tests.py", line 745, in _compare_output
2024-01-19T03:18:41.1855098Z     self.assertMultiLineEqual(
2024-01-19T03:18:41.1855909Z AssertionError: 'Erro[319 chars]ata area; ignored.\n' != 'Erro[319 chars]ata area; ignored.\nUncaught exception: Overflow in addition\n'
2024-01-19T03:18:41.1857144Z   Error: Upper boundary of data for directory Image, entry 0x00fe is out of bounds: Offset = 0x0000002a, size = 64, exceeds buffer size by 22 Bytes; truncating the entry
2024-01-19T03:18:41.1858173Z   Warning: Directory Image, entry 0x0201: Strip 0 is outside of the data area; ignored.
2024-01-19T03:18:41.1858879Z   Warning: Directory Image, entry 0x0201: Strip 7 is outside of the data area; ignored.
2024-01-19T03:18:41.1859431Z + Uncaught exception: Overflow in addition
2024-01-19T03:18:41.1859784Z  : Standard error does not match

ping @kevinbackhouse

@neheb
Copy link
Collaborator Author

neheb commented Jan 19, 2024

nvm, that's probably a bug.

@neheb neheb force-pushed the ms branch 5 times, most recently from 9e7f99e to cd963b0 Compare February 7, 2024 22:16
The Alpine ones handle the newest.

Signed-off-by: Rosen Penev <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant