Skip to content

Conversation

@yihuang
Copy link
Contributor

@yihuang yihuang commented May 4, 2024

One golang standalone module can exists inside another module, when we replace such module to local directory, we'll try to create symbolic link inside the source directory and ln complains permission denied.

For example, both cosmossdk.io/x/accounts and cosmossdk.io/x/accounts/defaults/lockup are standalone modules, if we do replace cosmossdk.io/x/accounts => ./x/accounts, we'll try to create two symbolic links like this:

$ ln -s ./x/accounts vendor/cosmossdk.io/x/accounts
$ ln -s ./x/accounts/defaults/lockup vendor/cosmossdk.io/x/accounts/defaults/lockup

And the second one is inside source directory and fails.

The temporary solution here is to simply ignore the symbolic link error, not ideal, but fix the issue at hand, and should have no side effect on existing projects.

yihuang and others added 2 commits October 20, 2024 11:38
One golang standalone module can exists inside another module, when we
replace such module to local directory, we'll try to create symbolic link
inside the source directory and `ln` complains permission denied.

For example, both `cosmossdk.io/x/accounts` and `cosmossdk.io/x/accounts/defaults/lockup`
are standalone modules, if we do `replace cosmossdk.io/x/accounts => ./x/accounts`,
we'll try to create two symbolic links like this:

```
$ ln -s ./x/accounts vendor/cosmossdk.io/x/accounts
$ ln -s ./x/accounts/defaults/lockup vendor/cosmossdk.io/x/accounts/defaults/lockup
```

And the second one is inside source directory and fails.

The temporary solution here is to simply ignore the symbolic link error,
not ideal, but fix the issue at hand, and should have not side effect on existing projects.
anthr76 added a commit to anthr76/gomod2nix that referenced this pull request Nov 5, 2024
anthr76 added a commit to anthr76/gomod2nix that referenced this pull request Aug 12, 2025
@jwtocadero
Copy link

I just ran into issues when trying to have a monorepo with gomod2nix. It looks like the same issue. Is there any work on fixing this going on?

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

Successfully merging this pull request may close these issues.

2 participants