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

[WIP] remove "Replace" calls to make go get work on Packer #8909

Merged
merged 2 commits into from
Jun 12, 2020
Merged

Conversation

SwampDragons
Copy link
Contributor

@SwampDragons SwampDragons commented Mar 18, 2020

Still needs work to fix the flock library

helps with #8846

@SwampDragons SwampDragons requested a review from a team as a code owner March 18, 2020 23:14
@codecov-io
Copy link

codecov-io commented Mar 18, 2020

Codecov Report

Merging #8909 into master will decrease coverage by 0.00%.
The diff coverage is n/a.

Impacted Files Coverage Δ
packer/communicator.go 75.53% <0.00%> (-1.07%) ⬇️
packer/plugin/client.go 79.91% <0.00%> (-0.45%) ⬇️

@jhawk28
Copy link
Contributor

jhawk28 commented Mar 19, 2020

looks like @azr just needs to update some errors to get his patch merged...

@azr
Copy link
Contributor

azr commented Mar 19, 2020

Heh I hadn't realized the reviewed mesages on my PR, 🤦‍♂ I blame lack of sleep. I updated it now.

@SwampDragons
Copy link
Contributor Author

Waiting on merge of gofrs/flock#43

@jhawk28
Copy link
Contributor

jhawk28 commented Mar 27, 2020

The flock library doesn't have a go.mod file so I think you can just switch to the azr/flock repo. You can just update the package from the gofrs/flock to azr/flock here: https://github.com/hashicorp/packer/blob/master/common/filelock/filelock.go
and then update the go.mod from:

github.com/gofrs/flock v0.7.1

to

github.com/azr/flock v0.0.0-20190823144736-958d66434653

@azr
Copy link
Contributor

azr commented Jun 12, 2020

Howdy ! I just bumped my flock PR ( link: gofrs/flock#43), there's also a WIP to make flock support public at the go level, I just bumped it too ( link: golang/go#33974) , but right now I think this PR will be blocked on those for a while. I already think that removing that apache thrift replace call is nice so I think we should merge this until one of the aforementioned issues gets bumped 🙂 . ( I edited the 'close #8846' to not close the issue ).

@azr azr merged commit fcf7e63 into master Jun 12, 2020
@azr azr deleted the fix_imports branch June 12, 2020 11:04
@SwampDragons SwampDragons added the tech-debt Issues and pull requests related to addressing technical debt or improving the codebase label Jun 15, 2020
@SwampDragons SwampDragons added this to the 1.6.1 milestone Jun 15, 2020
@ghost
Copy link

ghost commented Jul 13, 2020

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@ghost ghost locked and limited conversation to collaborators Jul 13, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
tech-debt Issues and pull requests related to addressing technical debt or improving the codebase track-internal
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants