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

#3008 breaks windows build #3010

Closed
1 of 13 tasks
Grimonja opened this issue Feb 12, 2024 · 3 comments · Fixed by #3011
Closed
1 of 13 tasks

#3008 breaks windows build #3010

Grimonja opened this issue Feb 12, 2024 · 3 comments · Fixed by #3011

Comments

@Grimonja
Copy link

Which version are you using?

latest-9eb97ad3a0e1bf3e6b5f4db461c2934f0b17edf3

Which operating system are you using?

  • Linux amd64 standard
  • Linux amd64 Docker
  • Linux arm64 standard
  • Linux arm64 Docker
  • Linux arm7 standard
  • Linux arm7 Docker
  • Linux arm6 standard
  • Linux arm6 Docker
  • Windows amd64 standard
  • Windows amd64 Docker (WSL backend)
  • macOS amd64 standard
  • macOS amd64 Docker
  • Other (please describe)

Describe the issue

Build on Windows fails with invalid control character in URL when downloading hls.js during go generate.

2024/02/12 13:40:44 downloading hls.js... 2024/02/12 13:40:44 ERR: parse "https://cdn.jsdelivr.net/npm/[email protected]\r/dist/hls.min.js": net/url: invalid control character in URL exit status 1 internal\servers\hls\http_server.go:26: running "go": exit status 1

There is a rogue \r inserted in the URL due to carriage return differences between Linux and Windows.

Fix that would work for both Windows and Linux is to remove carriage return from the VERSION file and not truncating the last character in the download process

Describe how to replicate the issue

Try to build on Windows.

Did you attach the server logs?

no

Did you attach a network dump?

no

@aler9
Copy link
Member

aler9 commented Feb 12, 2024

Thanks for reporting the issue, this is fixed by #3011

Copy link
Contributor

github-actions bot commented Mar 4, 2024

This issue is mentioned in release v1.6.0 🚀
Check out the entire changelog by clicking here

Copy link
Contributor

github-actions bot commented Sep 6, 2024

This issue is being locked automatically because it has been closed for more than 6 months.
Please open a new issue in case you encounter a similar problem.

@github-actions github-actions bot locked and limited conversation to collaborators Sep 6, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants