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

Command beamdasm.disassemble not found #299

Open
doorgan opened this issue Oct 7, 2022 · 2 comments
Open

Command beamdasm.disassemble not found #299

doorgan opened this issue Oct 7, 2022 · 2 comments

Comments

@doorgan
Copy link

doorgan commented Oct 7, 2022

OTP versions tried: 24.3.3, 25.0
Elixir versions tried: 1.13, 1.14

Operating systems tried: Windows 10, Ubuntu 20.04, MacOS

Expected behavior:

  1. I click "Disassemble BEAM" on a .beam file
  2. The disassembled beam files are shown

Actual behavior:

  1. I click "Disassemble BEAM" on a .beam file
  2. Nothing happens
  3. Running the extension via the command pallette produces this error message:
    image

Steps to reproduce:
For erlang files:

  1. In an empty folder, create a file with a module test1.erl:
-module(test1).
-export([foo/0]).

foo() -> 42.
  1. Run erlc test1.erl
  2. Right click on the generated test1.beam file and click "Disassemble BEAM"
  3. Nothing happens.

For elixir files:

  1. In an empty folder, create a file with a module test1.ex:
defmodule Test1 do
  def foo, do: 42
end
  1. Run elixirc test1.ex
  2. Right click on the generated Elixir.Test1.beam file and click "Disassemble BEAM"
  3. Nothing happens.
@doorgan
Copy link
Author

doorgan commented Oct 7, 2022

I just rebuilt the vsix file and published it, with no changes other than bumping the engine version which wasn't even required, and it now works

I published the fork here https://marketplace.visualstudio.com/items?itemName=Dorgandash.beamdasm-fork as a temporary workaround
I'll gladly remove the fork from the marketplace once the original is fixed

@scout119
Copy link
Owner

scout119 commented Oct 7, 2022

Thank you for reporting. I will update this weekend. Please remove the fork

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

No branches or pull requests

2 participants