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

[Bug]: clink-completions have multiple version loaded by clink #6576

Open
3 tasks done
narnaud opened this issue Feb 25, 2025 · 1 comment · May be fixed by #6577
Open
3 tasks done

[Bug]: clink-completions have multiple version loaded by clink #6576

narnaud opened this issue Feb 25, 2025 · 1 comment · May be fixed by #6577
Labels
bug Something isn't working

Comments

@narnaud
Copy link
Contributor

narnaud commented Feb 25, 2025

Prerequisites

  • I have written a descriptive issue title.
  • I have searched all issues/PRs to ensure it has not already been reported or fixed.
  • I have verified that I am using the latest version of Scoop and corresponding bucket.

Package Name

clink-completions

Expected/Current Behaviour

Every time the package is updated, it's adding the path to clink via clink installscripts.
Unfortunately, it means that multiple version of this scripts are passed to clink, showing an error like that whenever clink is loaded:

...sers\nicolas\scoop\apps\clink-completions\0.6.1\yarn.lua:86: setting 'color.yarn.module' already exists

Steps to Reproduce

scoop install clink-completions
Installing 'clink-completions' (0.6.2) [64bit] from 'main' bucket
Loading v0.6.2.zip from cache
Checking hash of v0.6.2.zip ... ok.
Extracting v0.6.2.zip ... done.
Running installer script...Script path 'C:\Users\nicolas\scoop\apps\clink-completions\0.6.2' installed.
done.
Linking ~\scoop\apps\clink-completions\current => ~\scoop\apps\clink-completions\0.6.2
'clink-completions' (0.6.2) was installed successfully!

...sers\nicolas\scoop\apps\clink-completions\0.6.2\yarn.lua:86: setting 'color.yarn.module' already exists

Possible Solution

There are 2 solutions IMO:

  1. don't do clink installscripts step
  2. pass the canonical path to the current directory, this way it won't be duplicated when updating
    I would be in favor of 2)

Scoop and Buckets Version

scoop --version
Current Scoop version:
859d1db5 (HEAD -> master, tag: v0.5.2, origin/master, origin/HEAD) chore(release): Bump to version 0.5.2 (#6080)

'main' bucket:
52b38850b (HEAD -> master, origin/master, origin/HEAD) typos: Update to version 1.29.10

Scoop Config

scoop config

last_update         scoop_repo                              scoop_branch use_sqlite_cache
-----------         ----------                              ------------ ----------------
25/02/2025 19:45:41 https://github.com/ScoopInstaller/Scoop master                   True

PowerShell Version

$PSVersionTable

Name                           Value
----                           -----
PSVersion                      7.4.6
PSEdition                      Core
GitCommitId                    7.4.6
OS                             Microsoft Windows 10.0.26100
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           1.1.0.1
WSManStackVersion              3.0

Additional Softwares

No response

@narnaud narnaud added the bug Something isn't working label Feb 25, 2025
@narnaud
Copy link
Contributor Author

narnaud commented Feb 25, 2025

For the record, I have a patch implementing solution 2)

@narnaud narnaud linked a pull request Feb 25, 2025 that will close this issue
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant