Skip to content

CI upgrades - #68

Merged
Smattr merged 5 commits into
mainfrom
smattr/eb0225f8-2da3-4ee9-a664-fb0a26359918
Aug 12, 2026
Merged

CI upgrades#68
Smattr merged 5 commits into
mainfrom
smattr/eb0225f8-2da3-4ee9-a664-fb0a26359918

Conversation

@Smattr

@Smattr Smattr commented Aug 12, 2026

Copy link
Copy Markdown
Owner

No description provided.

Smattr added 5 commits August 12, 2026 20:01
This seems to have been how I should have configured this to begin
with¹ instead of hard coding versions that go missing when packages are
upgraded.

This is adapted from commit dc1edfbae4bf065574786eb9b2984e875a5a8181
from Rumur.²

¹ https://www.freshports.org/devel/py-pytest/
² Smattr/rumur@dc1edfb
Clang Static Analyzer says:

  [MEDIUM] libvimcat/src/read.c:89:13: File position of the stream might
    be 'indeterminate' after a failed operation. Can cause undefined
    behavior [unix.Stream]
      int c = getc(f);
              ^
  [MEDIUM] libvimcat/src/read.c:89:13: Read function called when stream
    is in EOF state. Function has no effect [unix.Stream]
      int c = getc(f);
              ^
  [MEDIUM] libvimcat/src/read.c:105:15: File position of the stream
    might be 'indeterminate' after a failed operation. Can cause
    undefined behavior [unix.Stream]
        int n = getc(f);
                ^
  [MEDIUM] libvimcat/src/read.c:105:15: Read function called when stream
    is in EOF state. Function has no effect [unix.Stream]
        int n = getc(f);
                ^
  [MEDIUM] libvimcat/src/read.c:115:13: File position of the stream
    might be 'indeterminate' after a failed operation. Can cause
    undefined behavior [unix.Stream]
          if (ERROR(ungetc(n, f) == EOF)) {
              ^

Reported-by: Clang Static Analyzer 20.1.8
@Smattr
Smattr force-pushed the smattr/eb0225f8-2da3-4ee9-a664-fb0a26359918 branch from a379dd7 to cd0548e Compare August 12, 2026 10:20
@Smattr
Smattr merged commit 642dbe7 into main Aug 12, 2026
8 checks passed
@Smattr
Smattr deleted the smattr/eb0225f8-2da3-4ee9-a664-fb0a26359918 branch August 12, 2026 10:22
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