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

module declares its path as: github.com/sirupsen/logrus but was required as: github.com/Sirupsen/logrus #1041

Closed
feng-crazy opened this issue Oct 15, 2019 · 5 comments

Comments

@feng-crazy
Copy link

    github.com/Sirupsen/logrus: github.com/Sirupsen/[email protected]: parsing go.mod:
    module declares its path as: github.com/sirupsen/logrus
            but was required as: github.com/Sirupsen/logrus
@freeformz
Copy link
Collaborator

github.com/Sirupsen/logrus is the old path.

Anything using it should be updated to import and require github.com/sirupsen/logrus.

If this is in another open source project I'd be happy to PR an update to it for you. Otherwise it's something you/the developers of the software will need to do.

Thanks

@dolmen
Copy link
Contributor

dolmen commented Feb 5, 2020

Unfortunately my PR #859 that could help to diagnose those issues is for now ignored.

@markphelps markphelps unpinned this issue Feb 6, 2020
rfloriano added a commit to rfloriano/go-clients that referenced this issue Mar 12, 2020
rfloriano added a commit to vtex/go-io that referenced this issue Mar 12, 2020
igorframos pushed a commit to vtex/go-io that referenced this issue Mar 27, 2020
whit3rabbit pushed a commit to whit3rabbit/slurp that referenced this issue Aug 8, 2020
Add go module support
Fix logrus not working for modules:
sirupsen/logrus#1041 (comment)
waiariki-koia added a commit to uswitch/vault-creds that referenced this issue Oct 22, 2020
Update from github.com/hashicorp/vault/helper/certutil to github.com/hashicorp/vault/sdk/helper/certutil"
waiariki-koia added a commit to uswitch/vault-creds that referenced this issue Oct 22, 2020
Change from Sirupsen to sirupsen. Ref: sirupsen/logrus#1041
wcarmon added a commit to wcarmon/logrus-gce that referenced this issue Nov 20, 2020
@baha-ai
Copy link

baha-ai commented Jan 30, 2021

It seems this error is back from the dead somehow, I get this error when doing go mod tidy on my module (using go 1.15):

...
        github.com/opencontainers/runc/libcontainer/cgroups imports
        github.com/opencontainers/runc/libcontainer/configs imports
        github.com/Sirupsen/logrus: github.com/Sirupsen/[email protected]: parsing go.mod:
        module declares its path as: github.com/sirupsen/logrus
                but was required as: github.com/Sirupsen/logrus

I stumbled on this issue from pingcap while googling the error.

I did look up github.com/opencontainers/runc/libcontainer/configs source code on GitHub, and they're using sirupsen import correctly
in both configs imports and go.mod.

I search the whole project for Sirupsen, there was nothing found in the project. They only use sirupsen with small s everywhere. I wonder why go mod is looking for capital S?

Anything changed to the library recently?

@yasushi-saito
Copy link

As for github.com/opencontainers/runc, the last published version, v0.1.1, refers to Sirupsen. Try a newer version.

go get github.com/opencontainers/runc/libcontainer/[email protected]

@mjaggard
Copy link

@freeformz Thanks for your kind offer. Could you provide the required PR(s?) for this error please?

go: github.com/rancher/migration-tools/cmd imports
	github.com/kubernetes/kompose/pkg/app imports
	github.com/kubernetes/kompose/pkg/transformer/openshift imports
	github.com/openshift/origin/pkg/image/api imports
	github.com/docker/distribution/manifest/schema1 imports
	github.com/Sirupsen/logrus: github.com/Sirupsen/[email protected]: parsing go.mod:
	module declares its path as: github.com/sirupsen/logrus
	        but was required as: github.com/Sirupsen/logrus

asobrien added a commit to asobrien/ghtrim that referenced this issue Feb 28, 2023
This updates dependencies. In the process, the following changes were
required:

1. logrus path has cnahed from Sirupsen/logrus -> sirupsen/logrus.
   Import paths have been updated to reflect this (cf.
   sirupsen/logrus#1041).
2. Github client requires context as first parameter. Github client
   usage has been updated to reflect this.
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

6 participants