Skip to content

Conversation

@jacomago
Copy link
Contributor

Update the pbrawclient to the latest

Has dependency on google protobuf 4.33 now

Checklist

  • Testing:

    • The feature has automated tests
    • [ x] Tests were run (Tested by me and @georgweiss on our archivers.)
    • If not, explain how you tested your changes
  • Documentation:

    • The feature is documented
    • The documentation is up to date
    • Release notes:
      • Added an entry if the change is breaking or significant
      • Added an entry when adding a new feature

@jacomago jacomago self-assigned this Nov 11, 2025
Copy link
Collaborator

@georgweiss georgweiss left a comment

Choose a reason for hiding this comment

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

By the way...
You need to update protobuf version in dependencies/phoebus-target/pom.xml
Oh, and please update dependencies/phoebus-target/.classpath for all those Eclipse IDE users out there.

@jacomago jacomago force-pushed the pbrawclient-update-2.2 branch from 2d1be29 to b54f524 Compare November 11, 2025 13:17
@shroffk
Copy link
Member

shroffk commented Nov 12, 2025

@jacomago since you are working on publishing this jar to maven central... in the future we should be able to complete get rid of the install-jar nonsense I had to cook up right?

@kasemir
Copy link
Collaborator

kasemir commented Nov 12, 2025

We need the Oracle JDBC jar to access archived data. For the longest time, Oracle's licensing prevented them from being available in Maven central, requiring the local install-jar nonsense that you had to cook up.
But it looks like that changed, https://medium.com/oracledevs/your-own-way-oracle-jdbc-drivers-19-7-0-0-on-maven-central-9a7dbb648995 Will try that later today

@jacomago
Copy link
Contributor Author

@jacomago since you are working on publishing this jar to maven central... in the future we should be able to complete get rid of the install-jar nonsense I had to cook up right?

Yeah hopefully. But it looked a lot of work to setup. So I thought I woudl do this upgrade first.

@shroffk shroffk merged commit 4f3790e into ControlSystemStudio:master Nov 12, 2025
2 checks passed
@kasemir kasemir mentioned this pull request Nov 12, 2025
2 tasks
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.

4 participants