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

vendor directory #5037

Closed
pfi79 opened this issue Oct 18, 2024 · 2 comments
Closed

vendor directory #5037

pfi79 opened this issue Oct 18, 2024 · 2 comments
Labels

Comments

@pfi79
Copy link
Contributor

pfi79 commented Oct 18, 2024

Current Status

Currently there is a vendor folder

I have a question: why is it needed?

It is written in the documentation for vendoring:

Vendoring may be used to allow interoperation with older versions of Go, or to ensure that all files used for a build are stored in a single file tree.

After trying to update github.com/prometheus/client_golang, the peer and orderer programs in docker stopped building. However, if the vendor folder is removed, everything builds fine.

Goal

It seems to me that this is already redundant in the fabric project.
And it can be deleted.

Solution

Try deleting the vendor folder and check that no functionality has fallen off.

@pfi79 pfi79 added the workitem label Oct 18, 2024
@C0rWin
Copy link
Contributor

C0rWin commented Oct 20, 2024

After trying to update github.com/prometheus/client_golang, the peer and orderer programs in docker stopped building. However, if the vendor folder is removed, everything builds fine.

Can you put here an error?

@pfi79
Copy link
Contributor Author

pfi79 commented Oct 21, 2024

After trying to update github.com/prometheus/client_golang, the peer and orderer programs in docker stopped building. However, if the vendor folder is removed, everything builds fine.

Can you put here an error?

#5032

#4052

vendor/github.com/prometheus/client_golang/prometheus/registry.go:62:15: undefined: NewGoCollector

a command that causes an error

make docker

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants