Skip to content

Skip and report libraries that fail instead of aborting the index - #94

Merged
adrpo merged 1 commit into
OpenModelica:masterfrom
adrpo:report-skipped-libraries
Nov 3, 2026
Merged

adrpo merged 1 commit into
OpenModelica:masterfrom
adrpo:report-skipped-libraries

Conversation

@adrpo

@adrpo adrpo commented Oct 29, 2026

Copy link
Copy Markdown
Member

Fixes #61

A library that cannot be handled no longer stops "Update Package Index". It is left out, listed, and the job goes unstable and sends the list by mail.

Changes

Tests

tests/test_problems.py:

  • write_problems: writing the list, and removing a stale file
  • genindex.main on a rawdata.json with a libless ref: it is skipped from index.json and reported
  • load_library records the omc error of a library that does not load

Locally all 41 tests pass except test_run_check_missing, which needs GITHUB_AUTH (set in CI). The Jenkinsfile change can only be checked by a run on Jenkins.


generated by Claude Code

🤖 Generated with Claude Code

…enModelica#61)

genindex raised on a rawdata.json ref without libraries (LEPSE v1.2.1)
and stopped the whole "Update Package Index" job. It now leaves that ref
out of the index and goes on with the others.

updateinfo and genindex write what they skipped to
updateinfo-problems.txt and genindex-problems.txt (libraries that did
not load, refs kept from or dropped from the previous run, repositories
that could not be fetched, duplicate library files). The Jenkinsfile
marks the job unstable when either file exists and mails the list to
PROBLEMS_EMAIL (default adrian.pop@liu.se, empty sends nothing).

Fixes OpenModelica#61

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@adrpo adrpo self-assigned this Oct 29, 2026
@adrpo
adrpo requested a review from AnHeuermann October 29, 2026 18:38
@adrpo
adrpo merged commit f94b9b0 into OpenModelica:master Nov 3, 2026
4 checks passed
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.

Handle libraries that fail to parse

2 participants