Skip to content
This repository has been archived by the owner on Jan 14, 2022. It is now read-only.

feat: add github.com/twpayne/chezmoi #111

Merged
merged 1 commit into from
Apr 30, 2019
Merged

feat: add github.com/twpayne/chezmoi #111

merged 1 commit into from
Apr 30, 2019

Conversation

twpayne
Copy link
Contributor

@twpayne twpayne commented Apr 29, 2019

No description provided.

@twpayne
Copy link
Contributor Author

twpayne commented Apr 29, 2019

It looks like the CI failure is unrelated to this PR:

   • reading https://raw.githubusercontent.com/goreleaser/goreleaser/master/.goreleaser.yml
   ⨯ treewalker failed         error=unable to parse: yaml: unmarshal errors:
  line 64: field archives not found in type config.Project
make: *** [build] Error 1
The command "make ci" exited with 2.

@caarlos0
Copy link
Member

yes, this is broken :/

@twpayne
Copy link
Contributor Author

twpayne commented Apr 29, 2019

Do you have a rough idea of how much knowledge is needed to fix it? Is it a reasonable external contribution, or better done by someone very familiar with the two projects?

@caarlos0
Copy link
Member

I rather not fix this right now... I'll be working on finishing adding multiple archives to goreleaser, which will break this a ton... so maybe its easier to just fix/redo it later.

btw cc/ @client9 do you still think in maintaining this?

@twpayne
Copy link
Contributor Author

twpayne commented Apr 29, 2019

OK, thanks for the update. I'm happy to wait on this so you can fix it properly. Goreleaser is awesome - thanks!

@caarlos0 caarlos0 merged commit bf17a75 into goreleaser:master Apr 30, 2019
@caarlos0
Copy link
Member

I just updated it because more things were broken :/

@caarlos0
Copy link
Member

⨯ treewalker failed error=unable to parse: failed to set defaults: found 2 builds with the ID 'chezmoi', please fix your config

can you fix this though?

seems like you have 2 build with the same id, if you run it against the latest goreleaser it will fail...

caarlos0 added a commit that referenced this pull request Apr 30, 2019
@caarlos0
Copy link
Member

check c5edc58 @twpayne

@client9
Copy link
Collaborator

client9 commented Apr 30, 2019 via email

@twpayne
Copy link
Contributor Author

twpayne commented May 1, 2019

seems like you have 2 build with the same id, if you run it against the latest goreleaser it will fail...

Yes, https://github.com/twpayne/chezmoi has two builds (one CGO, one without CGO) to work around Go's inconsistent behavior. User bug reports are twpayne/chezmoi#125 and twpayne/chezmoi#65.

It sounds like I need to update chezmoi to use the latest goreleaser. Please don't spend time investigating before then.

GitHub
Manage your dotfiles across multiple machines, securely. - twpayne/chezmoi

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 this pull request may close these issues.

3 participants