Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Bug 1913129 - Remove Geckoview Streaming helper code #743

Merged
merged 1 commit into from
Aug 14, 2024

Conversation

badboy
Copy link
Member

@badboy badboy commented Aug 13, 2024

This is not in use anymore.
Last use removed in https://bugzilla.mozilla.org/show_bug.cgi?id=1877836

We don't remove support for parsing gecko_datapoint to stay compatible with already existing old definition files.

Pull Request checklist

  • Quality: This PR builds and tests run cleanly
    • make test runs without emitting any warnings
    • make lint runs without emitting any errors
  • Tests: This PR includes thorough tests or an explanation of why it does not
  • Changelog: This PR includes a changelog entry to CHANGELOG.md or an explanation of why it does not need one
    • Any breaking changes to language binding APIs are noted explicitly

@badboy badboy requested a review from a team as a code owner August 13, 2024 10:09
@badboy badboy requested review from rosahbruno and removed request for a team August 13, 2024 10:09
@badboy badboy force-pushed the remove-gvst-helper branch from b0fb095 to e882fee Compare August 13, 2024 10:15
@badboy
Copy link
Member Author

badboy commented Aug 13, 2024

Actually I think this is not even breaking.
~~The only user ever was us in FOG. ~~
We removed all of that use in Firefox 129.

It is breaking, because ESR 128 is also using glean_parser 14.
So what we probably need to do to unbreak 128/release is:

Revert the removal of the alias. Release that as a patch release.
Then re-revert that, do a major release (and just include this too).

@badboy badboy requested a review from travis79 August 14, 2024 12:07
This is not in use anymore.
Last use removed in https://bugzilla.mozilla.org/show_bug.cgi?id=1877836

We don't remove support for _parsing_ `gecko_datapoint` to stay
compatible with already existing old definition files.
@badboy badboy force-pushed the remove-gvst-helper branch from e882fee to ade3a88 Compare August 14, 2024 12:11
@badboy badboy changed the title Remove Geckoview Streaming helper code Bug 1913129 - Remove Geckoview Streaming helper code Aug 14, 2024
@badboy badboy enabled auto-merge (rebase) August 14, 2024 12:12
@badboy badboy merged commit 6e109c8 into main Aug 14, 2024
8 checks passed
@badboy badboy deleted the remove-gvst-helper branch August 14, 2024 16:50
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants