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

cpack error when generating Windows installer #235

Closed
tijcolem opened this issue Oct 27, 2022 · 6 comments · Fixed by #236
Closed

cpack error when generating Windows installer #235

tijcolem opened this issue Oct 27, 2022 · 6 comments · Fixed by #236
Assignees

Comments

@tijcolem
Copy link
Collaborator

@bstein Looks like this is still an issue when generating windows packages. Maybe it was when the CI runner was upgraded from 2019 to 2022? Take a look at this line here: https://github.com/NREL/OpenStudio-PAT/actions/runs/3340691973/jobs/5531082030#step:6:160. It's annoying this doesn't throw an error in the build. Maybe I can add a second step to verify the contents after.

I think this was related: #221

@tijcolem
Copy link
Collaborator Author

2022 did work. This was a working build looking back in the history. https://github.com/NREL/OpenStudio-PAT/actions/runs/2862855047/jobs/4539303058

@tijcolem
Copy link
Collaborator Author

@tijcolem
Copy link
Collaborator Author

A little more info. This commit here 70bce19 built fine on Aug 19 in 35m 15s. I replayed this exact same commit just now using the same workflow and it failed. Smells like a version change caused something to break.

Succeeded ( Aug 19 in 35m 15s): https://github.com/NREL/OpenStudio-PAT/actions/runs/2891652196/jobs/4598886496

Failed ( Oct 27 in 41m 45s) https://github.com/tijcolem/OpenStudio-PAT/actions/runs/3342398966/jobs/5534583265
Succeed:

@bstein
Copy link
Collaborator

bstein commented Oct 28, 2022

@tijcolem I created the branch fix-cpack-error-windows to work on this

@tijcolem
Copy link
Collaborator Author

Thanks @bstein for looking into this!

@bstein
Copy link
Collaborator

bstein commented Oct 28, 2022

@tijcolem Of course! Fortunately, this was a straightforward issue where Windows wouldn't run the ./node_modules/.bin/gulp executable. The workaround is to use npx gulp instead, which is functionally the same.

I have opened a PR for the fix: #236

@bstein bstein linked a pull request Oct 28, 2022 that will close this issue
tijcolem added a commit that referenced this issue Oct 28, 2022
#235: Use "npx gulp" instead of relying on executable script
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

Successfully merging a pull request may close this issue.

2 participants