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

github.com/goreleaser/goreleaser not working under Windows #498

Closed
fiftin opened this issue Mar 15, 2018 · 12 comments
Closed

github.com/goreleaser/goreleaser not working under Windows #498

fiftin opened this issue Mar 15, 2018 · 12 comments
Assignees

Comments

@fiftin
Copy link
Collaborator

fiftin commented Mar 15, 2018

I have next error when trying run task deps:tools under Windows 10 (go version 1.10):

go/src/github.com/goreleaser/goreleaser/internal/client/github.go:120:22: cannot use release.GetID() (type int64) as type int in return argument
go/src/github.com/goreleaser/goreleaser/internal/client/github.go:132:28: cannot use releaseID (type int) as type int64 in argument to c.client.Repositories.UploadReleaseAsset

I found similar issue: goreleaser/goreleaser#527

@twhiston
Copy link
Contributor

I think we need to switch downloading the binary for goreleaser. Go getting it is clearly too tempremental across systems. But that sucks because of different binaries for different platforms 😞 if anyone has an elegant solution feel free to propose it.

@twhiston
Copy link
Contributor

I guess we can use something similar (or exactly the same) as the script they provide for ci downloads
https://raw.githubusercontent.com/goreleaser/get/master/get

@twhiston
Copy link
Contributor

@strangeman - do you have capacity to take this in the next day or 2? Should be a reasonably quick fix, but i've got really limited time now until the start of next week.

@strangeman
Copy link
Contributor

Yeah, I'll check it out tomorrow.

@strangeman strangeman self-assigned this Mar 15, 2018
@strangeman
Copy link
Contributor

I guess we can use something similar (or exactly the same) as the script they provide for ci downloads
https://raw.githubusercontent.com/goreleaser/get/master/get

That's good for Linux, but not solving the problem with Windows.

@strangeman
Copy link
Contributor

I can't see the way, how we can properly automate goreleaser installation in different OSes. My suggestion: remove it from task deps:tools and add a note about manual installation to Contributing.md.

@twhiston
Copy link
Contributor

We can have os specific taskfiles using a suffix.
https://github.com/go-task/task/blob/master/README.md#os-specific-task

I think we would only ever need a windows one as well as Linux and osx can always use the same taskfile in my experience.

I wanted to avoid this but if needs must I'd rather do that than have special case installs that we must do by hand as it is error prone and goes against my devops principles. Automate everything is my work motto 😉

@strangeman
Copy link
Contributor

strangeman commented Mar 17, 2018

I hate Windows automation. Ugly Powershell syntax, different PS versions on different Windows releases, ugh.

In my team, we solved 'Windows problem' radically - all Windows users have WSL installed and got the same automation scripts as Linux/Mac users.

@strangeman
Copy link
Contributor

strangeman commented Mar 17, 2018

I tested godownloader (https://github.com/goreleaser/godownloader) on Win10 - it creates only bash script :(

Need to use something like that: https://gist.github.com/MarkTiedemann/c0adc1701f3f5c215fc2c2d5b1d5efd3

@twhiston
Copy link
Contributor

I just generally hate windows 😜 I feel really lucky that at work our solution to the windows problem is just 'we dont support it at all' so I havn't used a Windows machine in about 10 years!

I wonder if we can solve this in the taskfile with the os templating options to pick the apprporiate extension and curling the current release version (https://github.com/go-task/task#gos-template-engine), maybe we can even avoid the windows specific taskfile then?

@twhiston
Copy link
Contributor

I gave it a shot at solving this, though I dont love the windows solution that much, because it blocks the install in circleci v2, which is needed for the dockerfile PR

@twhiston
Copy link
Contributor

closing this as #505 was merged

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

No branches or pull requests

3 participants