Skip to content

Feature/bump erlang opt version 27#73

Merged
dhruvjain99 merged 36 commits into
mainfrom
feature/bump-erlang-opt-version-27
Jul 18, 2025
Merged

Feature/bump erlang opt version 27#73
dhruvjain99 merged 36 commits into
mainfrom
feature/bump-erlang-opt-version-27

Conversation

@VivekPipaliya23

Copy link
Copy Markdown

Proposed Changes

This upgrade will ensure the compilation on Erlang OTP 27.

Types of Changes

What types of changes does your code introduce to this project?
Put an x in the boxes that apply

  • Bugfix (non-breaking change which fixes issue #XXXX)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation (correction or otherwise)
  • Cosmetics (whitespace, styles...)
  • DevOps (Build scripts, pipelines...)

Checklist

Put an x in the boxes that apply. You can also fill these out after creating
the PR. If you're unsure about any of them, don't hesitate to ask on the
mailing list. We're here to help! This is simply a reminder of what we are
going to look for before merging your code.

  • I have read the CODE_OF_CONDUCT.md document
  • All tests pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if needed)
  • Any dependent changes have been merged and published in related repositories
  • I have updated changelog (At the bottom of the release version)
  • I have squashed all my commits into one before merging

Further Comments

If this is a relatively large or complex change, kick off the discussion by
explaining why you chose the solution you did and what alternatives you
considered, etc.

@VivekPipaliya23 VivekPipaliya23 force-pushed the feature/bump-erlang-opt-version-27 branch 2 times, most recently from 94935d8 to b304c5a Compare May 26, 2025 06:26
@VivekPipaliya23 VivekPipaliya23 force-pushed the feature/bump-erlang-opt-version-27 branch from 897fa16 to 3499ddf Compare June 30, 2025 05:40
@VivekPipaliya23 VivekPipaliya23 force-pushed the feature/bump-erlang-opt-version-27 branch from 3499ddf to 4a89663 Compare July 3, 2025 08:51
@nerijus

nerijus commented Jul 9, 2025

Copy link
Copy Markdown

Thanks a lot, it almost built without problems with erlang 26.2.5.13. The error left is:

===> Running cuttlefish schema generator
escript: exception error: no function clause matching 
                 rlx_resolve:format_error({bad_app_file,
                                           "/usr/lib64/erlang/lib/debugger-5.2.1.1/ebin/debugger.app",
                                           {error,enoent}}) (/home/runner/work/rebar3/rebar3/vendor/relx/src/rlx_resolve.erl, line 10)
  in function  rebar3:handle_error/2 (/home/runner/work/rebar3/rebar3/apps/rebar/src/rebar3.erl, line 376)
  in call from escript:run/2 (escript.erl, line 750)
  in call from escript:start/1 (escript.erl, line 277)
  in call from init:start_em/1 
  in call from init:do_boot/3 
make: *** [Makefile:27: rel] Error 127

Edit: The problem was that my erlang install does not have debugger.app. The following diff helped:

--- rebar.config.orig   2025-07-09 21:34:33.000000000 +0000
+++ rebar.config        2025-07-09 21:57:20.116099407 +0000
@@ -88,7 +88,6 @@
         xmerl,
         vmq_server,
         vernemq_dev,
-        debugger,
         {cuttlefish, load},
         {vmq_plumtree, load},
         {plumtree, load},

So it is not related to this PR.

Comment thread .github/workflows/pr.yml
Comment thread .github/workflows/pr.yml Outdated
Comment thread .github/workflows/pr.yml Outdated
Comment thread .github/workflows/pr.yml
Comment thread run-tests-with-retry.sh
Comment thread apps/vmq_server/src/vmq_reg_redis_trie.erl
Comment thread apps/vmq_server/test/vmq_cluster_SUITE.erl
Comment thread apps/vmq_webhooks/test/vmq_webhooks_SUITE.erl
Comment thread apps/vmq_server/test/vmq_cluster_SUITE.erl
Comment thread apps/vmq_server/test/vmq_cluster_SUITE.erl
Comment thread apps/vmq_webhooks/test/webhooks_handler.erl
@dhruvjain99 dhruvjain99 merged commit 13d3c41 into main Jul 18, 2025
7 checks passed
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.

3 participants