Skip to content

refactor: Replace yq with bbb-yq-go (based on debian yq-go) - #837

Merged
antobinary merged 3 commits into
v4.0.x-releasefrom
v4.0.x-release-yq-go
Jul 15, 2026
Merged

refactor: Replace yq with bbb-yq-go (based on debian yq-go)#837
antobinary merged 3 commits into
v4.0.x-releasefrom
v4.0.x-release-yq-go

Conversation

@antobinary

@antobinary antobinary commented Jul 9, 2026

Copy link
Copy Markdown
Member
  • In this PR we're switching back to mikefarah syntax but invoking yq-go, now added via bbb-yq-go package from BigBlueButton's packages repository.

bbb-yq-go (packaging mikefarah/yq binary from https://github.com/mikefarah/yq/releases) matches the mikefarah/yq syntax BigBlueButton relied on in previous BBB iterations under the name yq. When upgrading to Ubuntu 24.04 in #826 we tried adopting the tool called yq already present in Ubuntu 24.04. However, the syntax was different (overcome in #826 and bigbluebutton/bigbluebutton#24805) and the behavior turned out to not be identical either.

  • I am including here a tweak to need_ppa as it was causing apt-key complaints and errors - discovered during the work on the PR
Reading package lists...
Repository: 'Types: deb
URIs: https://ppa.launchpadcontent.net/martin-uni-mainz/yq-go/ubuntu/
Suites: noble
Components: main
'
Description:
yq go builds and it build dependencies
More info: https://launchpad.net/~martin-uni-mainz/+archive/ubuntu/yq-go
Adding repository.
Found existing deb entry in /etc/apt/sources.list.d/martin-uni-mainz-ubuntu-yq-go-noble.sources
+ apt-key list 4B77C2225D3BBDB3
+ grep -q -E '1024|4096'
Warning: apt-key is deprecated. Manage keyring files in trusted.gpg.d instead (see apt-key(8)).
+ err 'Unable to setup PPA for ppa:martin-uni-mainz/yq-go'
+ say 'Unable to setup PPA for ppa:martin-uni-mainz/yq-go'
+ echo 'bbb-install: Unable to setup PPA for ppa:martin-uni-mainz/yq-go'
bbb-install: Unable to setup PPA for ppa:martin-uni-mainz/yq-go
+ exit 1
Error: Process completed with exit code 1.

Note, this matches bigbluebutton/bigbluebutton PR bigbluebutton/bigbluebutton#25398

Many thanks to @christmart who ported yq-go from Debian into a launchpad ppa. Initially I adopted that ppa but decided it's best for the project to be in control of which yq version we're using. See bigbluebutton/bigbluebutton#25398 for our packaging.

@antobinary
antobinary marked this pull request as ready for review July 10, 2026 01:34
@antobinary

Copy link
Copy Markdown
Member Author

I'm considering invalidating commands with yq only by adding:

# Ubuntu's archive 'yq' is the kislyuk/Python tool with different write semantics;
# BBB uses yq-go. Remove it so a stale 'yq …' fails loudly instead of misbehaving.
if dpkg -s yq >/dev/null 2>&1; then apt-get purge -y yq; fi

This should prevent people from going astray when following old commands. Any opinions?

@antobinary antobinary changed the title refactor: Replace yq with yq-go from ppa:martin-uni-mainz/yq-go refactor: Replace yq with bbb-yq-go (based on debian yq-go) Jul 15, 2026
@antobinary
antobinary merged commit fabffef into v4.0.x-release Jul 15, 2026
2 checks passed
@antobinary
antobinary deleted the v4.0.x-release-yq-go branch July 22, 2026 19:25
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