Skip to content

Replace Thrift package source with Github release link#111

Merged
fruffy merged 2 commits into
mainfrom
fruffy/dependency_simplification
Oct 26, 2025
Merged

Replace Thrift package source with Github release link#111
fruffy merged 2 commits into
mainfrom
fruffy/dependency_simplification

Conversation

@fruffy

@fruffy fruffy commented Jul 6, 2025

Copy link
Copy Markdown
Contributor

Also delete obsolete bridge-utils installation, it is provided via the package managers.

@fruffy fruffy force-pushed the fruffy/dependency_simplification branch 2 times, most recently from d272b9c to e23ba84 Compare July 6, 2025 17:49
@fruffy fruffy requested a review from jafingerhut September 5, 2025 18:10
@fruffy

fruffy commented Sep 5, 2025

Copy link
Copy Markdown
Contributor Author

@jafingerhut Fixes some CI failures that can occur. Github is more reliable than https://archive.apache.org

@fruffy

fruffy commented Oct 26, 2025

Copy link
Copy Markdown
Contributor Author

@jafingerhut Can you give this a review?

…te bridge-utils.

Signed-off-by: fruffy <fruffy@nyu.edu>
Signed-off-by: fruffy <fruffy@nyu.edu>
@fruffy fruffy force-pushed the fruffy/dependency_simplification branch from a72a42d to 7a2dc2f Compare October 26, 2025 00:02

@jafingerhut jafingerhut left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Look reasonable to me, as long as everything gets installed that still needs to be, and CI test results appear to show they do.

@fruffy fruffy merged commit 0ec2496 into main Oct 26, 2025
2 checks passed
@fruffy fruffy deleted the fruffy/dependency_simplification branch October 26, 2025 19:20
rcgoodfellow pushed a commit to oxidecomputer/tofino-sde that referenced this pull request Feb 21, 2026
jafingerhut pushed a commit to jafingerhut/open-p4studio that referenced this pull request Feb 26, 2026
Signed-off-by: fruffy <fruffy@nyu.edu>
Signed-off-by: Andy Fingerhut <andy_fingerhut@alum.wustl.edu>
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.

2 participants