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.

Sign up for GitHub

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

Jump to bottom

[bug] conan remove does not find recipe #5297

Closed
Johnnyxy opened this issue Jun 5, 2019 · 0 comments · Fixed by #5346
Closed

[bug] conan remove does not find recipe #5297

Johnnyxy opened this issue Jun 5, 2019 · 0 comments · Fixed by #5346
Assignees
@uilianries
Labels
complex: low priority: low type: bug
Milestone
1.17

Comments

@Johnnyxy
Copy link

Johnnyxy commented Jun 5, 2019

Hi there,
I tried to delete every version of a recipe in a certain user/channel but Conan stopped working with the error that it could not find the recipe:

conan remove cygwin_installer/*@user/channel
ERROR: Recipe not found: 'cygwin_installer/*@user/channel'

But when I modify the recipe name and append an asterix it works:

conan remove cygwin_installer*/*@user/channel
Are you sure you want to delete from 'cygwin_installer/2.9.0@user/channel' (yes/no):

Is this intentional?
@rweickelt rweickelt mentioned this issue Jun 6, 2019
Closed
uilianries added a commit to uilianries/conan that referenced this issue Jun 12, 2019
@uilianries
conan-io#5297 Add test that reproduces the error
39bbda9 
Signed-off-by: Uilian Ries <uilianries@gmail.com>
uilianries added a commit to uilianries/conan that referenced this issue Jun 12, 2019
@uilianries
conan-io#5297 Remove all when version is asterisk
631d5cf 
Signed-off-by: Uilian Ries <uilianries@gmail.com>
@uilianries uilianries mentioned this issue Jun 12, 2019
Merged
5 tasks
uilianries added a commit to uilianries/conan that referenced this issue Jun 12, 2019
@uilianries
conan-io#5297 Fix remove rule when using asterisk
2c78ea7 
- Not only version, but also channel and user can
  contain asterisk

Signed-off-by: Uilian Ries <uilianries@gmail.com>
uilianries added a commit to uilianries/conan that referenced this issue Jun 12, 2019
@uilianries
conan-io#5297 Validate channel when removing
94af85a 
Signed-off-by: Uilian Ries <uilianries@gmail.com>
uilianries added a commit to uilianries/conan that referenced this issue Jun 12, 2019
@uilianries
conan-io#5297 Check valid reference
3b96ee0 
Signed-off-by: Uilian Ries <uilianries@gmail.com>
danimtb pushed a commit that referenced this issue Jun 17, 2019
@uilianries @danimtb
Remove packages when version is asterisk (#5346)
cad452a 
* #5297 Add test that reproduces the error

Signed-off-by: Uilian Ries <uilianries@gmail.com>

* #5297 Remove all when version is asterisk

Signed-off-by: Uilian Ries <uilianries@gmail.com>

* #5297 Fix remove rule when using asterisk

- Not only version, but also channel and user can
  contain asterisk

Signed-off-by: Uilian Ries <uilianries@gmail.com>

* #5297 Validate channel when removing

Signed-off-by: Uilian Ries <uilianries@gmail.com>

* #5297 Check valid reference

Signed-off-by: Uilian Ries <uilianries@gmail.com>
@memsharded memsharded added this to the 1.17 milestone Jun 24, 2019
@rst-ct rst-ct mentioned this issue Feb 20, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@uilianries uilianries

Labels
complex: low priority: low type: bug
Projects
None yet
Milestone
1.17
Development

Successfully merging a pull request may close this issue.

Remove packages when version is asterisk uilianries/conan
5 participants
@jgsogo @uilianries @Johnnyxy @danimtb @memsharded