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

search: include version for matches too! #2312

Merged

Conversation

dtzWill
Copy link
Member

@dtzWill dtzWill commented Jul 27, 2018

Since the matching isn't done against fullName
the match suffix doesn't include version
for "good" derivations that are parsed properly.

Searching without a match string always shows
the full name, with this the full name
is always shown.

Really helps when you just wanna see what version
is available currently! :)

@edolstra
Copy link
Member

edolstra commented Aug 3, 2018

I don't understand what this does. What is the output after this change?

@dtzWill
Copy link
Member Author

dtzWill commented Aug 6, 2018

Currently if a search matches the attribute name, the version portion of the attribute name is not shown.

This is most easily seen by looking at the output of "nix search". The portion in parenthesis contains just the name not the version as well--indeed the version will only be present for derivations that parseDrvName can't handle.

@dtzWill
Copy link
Member Author

dtzWill commented Aug 6, 2018

After this change the portion in parens contains the full name--including version-- which is what is currently shown when for entries that don't match via attribute name (try nix search shell for a bit of both).

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.

None yet

2 participants