electron-builder icon indicating copy to clipboard operation
electron-builder copied to clipboard

NSIS one click installer uses package name for install directory not productName

Open Krelborn opened this issue 5 years ago • 27 comments

  • Version: 22.4.1
  • Target: NSIS

When installing on Windows with the one click NSIS installer, the application gets installed to a directory which is named using the name key from the npm package.json rather than using the productName. If I set the one click option to false, the directory is named correctly. The actual user data for is always stored in a folder named using productName.

Krelborn avatar Mar 20 '20 13:03 Krelborn

Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward?

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

stale[bot] avatar May 19 '20 15:05 stale[bot]

Is anyone able to comment on this?

Krelborn avatar May 19 '20 18:05 Krelborn

I think it may be related https://github.com/electron-userland/electron-builder/issues/4933

mahnunchik avatar May 26 '20 10:05 mahnunchik

Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward?

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

stale[bot] avatar Jul 25 '20 11:07 stale[bot]

It is still relevant!

mahnunchik avatar Jul 26 '20 20:07 mahnunchik

It's not been fixed as of the latest version.

Krelborn avatar Jul 27 '20 07:07 Krelborn

Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward?

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

stale[bot] avatar Sep 25 '20 11:09 stale[bot]

пт, 25 сент. 2020 г., 14:04 stale[bot] [email protected]:

Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward?

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/electron-userland/electron-builder/issues/4803#issuecomment-698867882, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAF6VB55GBHQTG7IVBAV3DDSHR2NNANCNFSM4LQL5A7Q .

mahnunchik avatar Sep 25 '20 12:09 mahnunchik

Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward?

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

stale[bot] avatar Nov 24 '20 23:11 stale[bot]

relevent

mahnunchik avatar Nov 25 '20 08:11 mahnunchik

Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward?

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

stale[bot] avatar Jan 24 '21 10:01 stale[bot]

https://github.com/electron-userland/electron-builder/issues/4803#issuecomment-766321788

mahnunchik avatar Jan 25 '21 07:01 mahnunchik

We are running into this too. Disabled oneClick for now. 😢

tessro avatar Mar 25 '21 04:03 tessro

It's caused by the second parameter to getWindowsInstallationDirName, here: https://github.com/electron-userland/electron-builder/blob/da85087143d2c6a63faab4c44c28dc625326e9ee/packages/app-builder-lib/src/targets/nsis/NsisTarget.ts#L167

I tried to trace this back to understand the rationale, but it's a very old change and the file has been moved in git since it was implemented this way.

tessro avatar Mar 25 '21 04:03 tessro

Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward?

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

stale[bot] avatar Jun 02 '21 16:06 stale[bot]

https://github.com/electron-userland/electron-builder/issues/4803#issuecomment-853177497

mahnunchik avatar Jun 02 '21 16:06 mahnunchik

Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward?

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

stale[bot] avatar Sep 21 '21 22:09 stale[bot]

Well, I'm dealing with this issue too. The nsis installs the app into AppData/Local/Programs/{manifest.name} instead of the desired AppData/Local/Programs/{manifest.productName}, even though the productName is just a PascalCased name.

On the other hand, the user data folder the app creates is the desired AppData/Roaming/{manifest.productName}, so the app is now inconsistent in how it names its folders.

But I guess the stale bot has won :(

tomasklaen avatar Dec 15 '21 22:12 tomasklaen

This needs to be un-marked as stale. This is still a persistent and relevant issue. @mmaietta

tom-leamon avatar Dec 21 '23 09:12 tom-leamon

Yes I confirm this issue is still there and very annoying. It should use the productName

end3rbyte avatar Jul 17 '24 08:07 end3rbyte

Well, I'm dealing with this issue too. The nsis installs the app into AppData/Local/Programs/{manifest.name} instead of the desired AppData/Local/Programs/{manifest.productName}, even though the productName is just a PascalCased name.

On the other hand, the user data folder the app creates is the desired AppData/Roaming/{manifest.productName}, so the app is now inconsistent in how it names its folders.

But I guess the stale bot has won :(

I just noticed this issue as well. Have you found a solution?

m1rn avatar Dec 09 '24 17:12 m1rn

I've run into this today but I think I have a workaround, at least so far everything seems to be working fine:

Add a Custom NSIS script that looks like:

!macro customHeader
  ; Workaround for https://github.com/electron-userland/electron-builder/issues/4803
  !define /redef APP_FILENAME "YourProductName"
!macroend

p2004a avatar May 29 '25 23:05 p2004a

This issue is stale because it has been open for 30 days with no activity. Remove stale label or comment, or this will be closed in 30 days.

github-actions[bot] avatar Aug 19 '25 00:08 github-actions[bot]

This issue should not be closed until it has been fixed.

tom-leamon avatar Aug 19 '25 17:08 tom-leamon

This issue is stale because it has been open for 30 days with no activity. Remove stale label or comment, or this will be closed in 30 days.

github-actions[bot] avatar Sep 19 '25 00:09 github-actions[bot]

This issue should not be closed until it has been fixed.

p2004a avatar Sep 19 '25 10:09 p2004a

Community PRs are always welcome. I'm personally not the most familiar with Windows, so a community assistance on this would be greatly appreciated

mmaietta avatar Sep 22 '25 15:09 mmaietta