search

msys2 / MINGW-packages

Package scripts for MinGW-w64 targets to build under MSYS2.
https://packages.msys2.org
BSD 3-Clause "New" or "Revised" License
2.27k stars 1.22k forks source link

broken nsis #21999

Closed ciros88 closed 1 week ago

ciros88 commented 2 weeks ago

Description / Steps to reproduce the issue

  1. install mingw-w64-i686-nsis or mingw-w64-x86_64-nsis
  2. execute makensis /version or makensis /license

Expected behavior

display nsis version or license

Actual behavior

$ makensis /version\ Processing config: C:\msys64\mingw32\etc\nsisconf.nsh\ Can't open script "C:/msys64/version"

$ makensis /license\ Processing config: C:\msys64\mingw32\etc\nsisconf.nsh\ Can't open script "C:/msys64/license"

Verification

Windows Version

MINGW32_NT-10.0-22621

MINGW environments affected

Are you willing to submit a PR?

No response

raedrizqie commented 2 weeks ago

oopss.. sorry

ciros88 commented 2 weeks ago

you can use:

$ nss-config --version 3.104.0

@raedrizqie ure not getting the point: makensis is broken, it is not about printing the version itself

raedrizqie commented 2 weeks ago

i think you need to run it under cmd or powershell ?

ciros88 commented 2 weeks ago

i run it in msys2/mingw32/mingw64 shell (as it should be)

raedrizqie commented 2 weeks ago

yes.. you can run cmd/powershell inside mintty

ciros88 commented 2 weeks ago

this is not the point, the issue is that makensis doesn't work in msys2 shell (where is supposed to, and also where i personally need it)

ArchGuyWu commented 1 week ago

this is not the point, the issue is that makensis doesn't work in msys2 shell (where is supposed to, and also where i personally need it)

You can see this: https://www.msys2.org/docs/filesystem-paths/

ArchGuyWu commented 1 week ago

Automatic Unix ⟶ Windows Path Conversion is enabled by default.

ciros88 commented 1 week ago

@ArchGuyWu can you give me a proper context? exactly what is your suggested solution? have you tried it? thanks

ArchGuyWu commented 1 week ago

@ArchGuyWu can you give me a proper context? exactly what is your suggested solution? have you tried it? thanks

Don't use "/" for arguments in unix shell.

ArchGuyWu commented 1 week ago

"/" is used for path in unix system.

ciros88 commented 1 week ago

ok now makes sense, and: makensis -version v3.10

makensis -license Copyright (C) 1999-2023 Nullsoft and Contributors ...

gives the right output, thanks