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

JACK Autobuild fails during installation of JACK #2992

Closed
ann0see opened this issue Jan 14, 2023 · 14 comments · Fixed by #3003
Closed

JACK Autobuild fails during installation of JACK #2992

ann0see opened this issue Jan 14, 2023 · 14 comments · Fixed by #3003
Assignees
Labels
bug Something isn't working

Comments

@ann0see
Copy link
Member

ann0see commented Jan 14, 2023

Describe the bug

#2991 shows that the JACK build fails - most likely - due a permission error https://github.com/jamulussoftware/jamulus/actions/runs/3861264997/jobs/6701549076#step:8:167

I guess this has to do something with a GH runner update.

To Reproduce

Run the JACK Autobuild via GitHub Actions

Expected behavior

JACk should install without error on GH actions.

Screenshots

/
Operating system

Windows

Version of Jamulus

/
Additional context

I think we might fix it by allowing choco to write to C:\ProgramData\chocolatey\lib-bad\jack maybe there's also a better way?

Tagging @henkdegroot as he knows a bit more about the JACK build.

@ann0see ann0see added the bug Something isn't working label Jan 14, 2023
@ann0see ann0see added this to the Release 3.10.0 milestone Jan 14, 2023
@ann0see ann0see changed the title JACK Autobuild fails during installation JACK Autobuild fails during installation of JACK Jan 14, 2023
@henkdegroot
Copy link
Contributor

henkdegroot commented Jan 15, 2023

Going to check this. While the log does indicate a permission error, I don't think that will be the issue as I believe the account used in the github action is an full admin account (to prevent issues like this). It might be some file lock due to another process not releasing the file or something like that.

GH info: Windows virtual machines are configured to run as administrators with User Account Control (UAC) disabled.

@henkdegroot
Copy link
Contributor

The JACK installation is failing due to an issue with Autohotkey v2 being used now, while previous v1 was used. Because of this the autohotkey script fails. Going to see if I can provide a fix for that and get that updated in the choco package library.

@ann0see
Copy link
Member Author

ann0see commented Jan 15, 2023

Same procedure as last time…

Who builds the installer? I suppose there’s a git repo for JACK on windows somewhere. It might be worth raising an issue requesting a real silent installer.

@henkdegroot
Copy link
Contributor

Posted a PR to get the issue resolved. chtof/chocolatey-packages#82

Might be a good improvement to JACK to get a silent installer: https://github.com/jackaudio/jack2

@henkdegroot
Copy link
Contributor

henkdegroot commented Jan 15, 2023

A work around for us can be to manual install the autohotkey,portable package version 1.1.36.02 before installing JACK.

With this option we don't have to wait for the main package to be updated.

@ann0see, what do you think about this?

@ann0see
Copy link
Member Author

ann0see commented Jan 15, 2023

Actually, I googled a bit and the installer does support a silent installer with the /SILENT or /VERYSILENT parameters!

@henkdegroot
Copy link
Contributor

So that would mean we need to implement our own download and install of JACK? Instead of using choco for the installation.

@ann0see
Copy link
Member Author

ann0see commented Jan 15, 2023

I think choco could/should use the SILENT parameters, but OFC, we could also download it manually - but I'd rather prefer not to as choco seems less work.

Another option would to change to winget. If JACK is on winget already.

@ann0see
Copy link
Member Author

ann0see commented Jan 15, 2023

I've just opened issues on WinGet and the JACK repo. Personally, I'd prefer WinGet as I think it will become a standard in future.

@henkdegroot
Copy link
Contributor

Seems WinGet isn't really supported to run on a Windows Server environment....not as straight forward install as Windows 10/11.
It might become the standard one day....

@ann0see
Copy link
Member Author

ann0see commented Jan 16, 2023

Just read about it. Server 22 might work. It seems that they're aware of the issue and a lot of people would like to have it on servers too.

@ann0see
Copy link
Member Author

ann0see commented Jan 25, 2023

We might need to really install JACK by hand if nothing happens on the choco side.

@henkdegroot
Copy link
Contributor

Maybe not a bad solution, it is sourced from a git url and not often updated. Only downside seems that you can "see" if and when the jack install completed. Guess a check can be done to check a folder has been created or not before continue to the next step.

Using /VERYSILENT /SUPPRESSMSGBOXES /NORESTART for 32bit and 64bit jack installer seems to work fine.

@ann0see
Copy link
Member Author

ann0see commented Jan 29, 2023

I think we could use gh to download the JACK binary directly: https://cli.github.com/manual/gh_release_download

ann0see added a commit to ann0see/jamulus that referenced this issue Jan 30, 2023
ann0see added a commit to ann0see/jamulus that referenced this issue Jan 31, 2023
ann0see added a commit to ann0see/jamulus that referenced this issue Jan 31, 2023
ann0see added a commit to ann0see/jamulus that referenced this issue Jan 31, 2023
ann0see added a commit to ann0see/jamulus that referenced this issue Feb 1, 2023
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
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants