Skip to content
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

Support message containers introduced in RabbitMQ 3.13.0 #109

Merged
merged 2 commits into from
Jun 29, 2024

Conversation

gomoripeti
Copy link
Contributor

This commit makes the plugin depend on the mc module and hence 3.13.0+ at runtime.

Fixes #108

@@ -2,15 +2,15 @@ name: build
on: [push, pull_request]
jobs:
build:
runs-on: ubuntu-20.04
runs-on: ubuntu-22.04
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I was a bit too optimistic of just changing the CI and expecting it to just work.
I would need some help from your previous experience to make the build work.
I wonder why ppa:rabbitmq/rabbitmq-erlang did not work in the end?

@phil16727
Copy link

@gomoripeti your Build Failed, any chance you could fix that and publish version 0.63 of Plugin?

@gomoripeti
Copy link
Contributor Author

hi @phil16727 I am not the maintainer of this plugin so I definitely cannot publish a new version.

There is some issue with the build infrastructure on newer erlang versions. It is not easy to build elixir plugins for rabbitmq nowadays. I was hoping noxdafox can help based on his prior experience, but I will also try to look at it eventually (not sure when I will find the time)

@noxdafox
Copy link
Owner

noxdafox commented Jun 7, 2024

The issue is RMQ 3.13 requires Erlang 26.0 and it's a pain to get it installed on Ubuntu 22. Last attempts were unsuccessful.

I will try again once I have some spare time which is quite limited atm.

Thanks @gomoripeti for the contribution!

@dentarg
Copy link
Contributor

dentarg commented Jun 16, 2024

@gomoripeti
Copy link
Contributor Author

thanks a lot @dentarg I was going to try switching to setup-beam but I thought the issue is with mix_task_archive_deps which was not touched in 2 years

@gomoripeti
Copy link
Contributor Author

rebased to latest main
build still fails, this time because of mix_task_archive_deps which is definitely not compatible with latest elixir mix version (at least since 1.16.0)
Since rabbitmq/rabbitmq-server#8204 RabbitMQ itself stopped using mix_task_archive_deps.
Looking into what could be done.

This commit makes the plugin depend on the `mc` module and hence
3.13.0+ at runtime.

Fixes noxdafox#108
The later implemented in `mix_task_archive_deps` uses
`Mix.Dep.load_on_environment/1` which was removed in Elixir 1.16.0.
This plugin has no dependencies so the two tasks are mostly
equivalent.

The mix task `archive.build.elixir` is still used from
`mix_task_archive_deps`.
@islamsaka
Copy link

You really did an excellent job, @gomoripeti

@noxdafox
Copy link
Owner

Thanks for the contribution @gomoripeti!

@noxdafox noxdafox merged commit 031ebc8 into noxdafox:main Jun 29, 2024
1 check passed
@gomoripeti gomoripeti deleted the support_3_13 branch August 14, 2024 17:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
5 participants