Skip to content

build graphviz 13.x #363

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

Merged
merged 3 commits into from
Jul 15, 2025
Merged

build graphviz 13.x #363

merged 3 commits into from
Jul 15, 2025

Conversation

songmeo
Copy link
Contributor

@songmeo songmeo commented Jul 14, 2025

Build graphviz 13.x from source

Closes #321

@songmeo songmeo requested a review from pavel-kirienko July 14, 2025 22:24
@coveralls
Copy link

coveralls commented Jul 14, 2025

Coverage Status

coverage: 93.906%. remained the same
when pulling 17cce81 on 321-pin-graphviz-version
into 651013f on master.

@songmeo songmeo marked this pull request as draft July 14, 2025 22:39
@songmeo songmeo force-pushed the 321-pin-graphviz-version branch 4 times, most recently from 050cfed to fbd92ca Compare July 15, 2025 11:26
@songmeo songmeo force-pushed the 321-pin-graphviz-version branch from fbd92ca to bad26bb Compare July 15, 2025 11:47
@songmeo songmeo marked this pull request as ready for review July 15, 2025 12:16
@songmeo songmeo force-pushed the 321-pin-graphviz-version branch from e32295d to 03e0fbc Compare July 15, 2025 13:11
Copy link
Member

@pavel-kirienko pavel-kirienko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please bump the patch version, otherwise CD will fail with a tag conflict.

@@ -50,6 +50,14 @@ jobs:
python --version
ip link show

# Install Graphviz 13.x
wget https://gitlab.com/api/v4/projects/4207231/packages/generic/graphviz-releases/13.1.0/ubuntu_24.04_graphviz-13.1.0-cmake.deb
sudo apt install ./ubuntu_24.04_graphviz-13.1.0-cmake.deb
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IIRC apt doesn't have a stable command-line interface, so in automation scripts you should be using apt-get instead.

@songmeo songmeo force-pushed the 321-pin-graphviz-version branch 2 times, most recently from 7772b39 to 2e657db Compare July 15, 2025 13:48
@songmeo songmeo force-pushed the 321-pin-graphviz-version branch from 2e657db to 17cce81 Compare July 15, 2025 18:38
@songmeo songmeo merged commit 0b926ea into master Jul 15, 2025
14 checks passed
@songmeo songmeo deleted the 321-pin-graphviz-version branch July 15, 2025 19:17
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.

[docs] Class diagrams are scaled incorrectly
3 participants