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

Unable to initialize UI5 CLI #833

Closed
ArnaudBuchholz opened this issue May 29, 2023 · 4 comments
Closed

Unable to initialize UI5 CLI #833

ArnaudBuchholz opened this issue May 29, 2023 · 4 comments
Labels
bug Something isn't working

Comments

@ArnaudBuchholz
Copy link

I made a fresh install of an app based on @ui5/cli and I cannot execute ui5

Expected Behavior

I can serve my app

Current Behavior

ui5 fails with message

Fatal Error: Unable to initialize UI5 CLI

Steps to Reproduce the Issue

Because of the stack trace, I suspect this is related to the missing APDDATA environment variable.

image

Context

  • @ui5/cli 3.1.2
  • Node.js Version: 18.16.0
  • npm Version: 9.5.1
  • OS/Platform: Windows 11
  • Browser (if relevant): N/A
  • Other information regarding your environment (optional): VS Studio Code terminal

Log Output / Stack Trace

See screenshot

@ArnaudBuchholz ArnaudBuchholz added bug Something isn't working needs triage Needs to be investigated and confirmed as a valid issue that is not a duplicate labels May 29, 2023
@ArnaudBuchholz
Copy link
Author

ArnaudBuchholz commented May 29, 2023

Important info : I confirm the behavior is specifc to the VS Studio Code terminal that does not define APPDATA
Having a fallback mechanism could help (and a better error message too) ;-)

@RandomByte
Copy link
Member

Thanks for reporting. This looks like a regression in @pnpm/npm-conf, which is a transitive dependency of update-notifier, which we where considering to remove/replace before.

It's a fork of another project, where exactly this problem seems to have been fixed a while ago: kevva/npm-conf#7

But the issue got reintroduced in the fork later on: pnpm/npm-conf@ccf74c4#diff-b34f2f53ab94368c86775969fb604e8375abe03b6a378bdd09896fd91ac0a0d2R30

Since the fork has not enabled issues, we should report it to their main project in the coming days: https://github.com/pnpm/pnpm/issues

@RandomByte
Copy link
Member

Reported via pnpm/pnpm#6659

I'll close this issue for the time being. Please let us know if this becomes more critical.

@RandomByte RandomByte removed the needs triage Needs to be investigated and confirmed as a valid issue that is not a duplicate label Jun 9, 2023
@RandomByte
Copy link
Member

pnpm/pnpm#6659 got fixed and @ui5/cli main has been updated.

Your issue should be resolved with the next release of @ui5/cli

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants