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

How to add ndisasm.exe to PATH? #29

Closed
StackOverflowExcept1on opened this issue Oct 1, 2022 · 1 comment
Closed

How to add ndisasm.exe to PATH? #29

StackOverflowExcept1on opened this issue Oct 1, 2022 · 1 comment

Comments

@StackOverflowExcept1on
Copy link

I'm using ndisasm in my batch file, why it wasn't added to the PATH variable?

Last stable version of NASM includes that
image

name: Build
on:
  push:
  pull_request:
  workflow_dispatch:

jobs:
  build:
    if: "!contains(github.event.head_commit.message, '[ci skip]')"
    runs-on: windows-latest

    steps:
      - uses: actions/checkout@v2

      - uses: ilammy/setup-nasm@v1

      - name: Build project
        run: build.bat
        shell: cmd
'ndisasm' is not recognized as an internal or external command, operable program or batch file.
@ilammy
Copy link
Owner

ilammy commented Oct 22, 2022

Thanks for pointing it out 👍

I've published an updated v1.4.0 which installs ndisasm as well.

@ilammy ilammy closed this as completed Oct 22, 2022
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