Add cargo and rustc completion. by bheesham · Pull Request #2409 · fish-shell/fish-shell

bheesham · 2015-09-19T19:06:55Z

I've added completion for Rust and Cargo.

faho · 2015-09-19T21:05:40Z

For the rustc completion: You can add descriptions to any completion option by separating the option and it's description with a tab character and the next option with a newline.

I.e. something like this should give properly described options:

printf "%s\t%s\n" box-pointers "some text that describes box-pointers" fat-ptr-transmutes "some description for fat-ptr-transmutes"

(Note that I'm not a rust user, but to me some of these seem rather esoteric, so descriptions seem useful)

bheesham · 2015-09-22T02:46:50Z

Descriptions for flags have now been added. I ended up writing a Python script to help me out since I'm not entirely sure how to play with strings using Fish.

faho · 2015-09-22T08:15:32Z

Did you mean to commit and push that?

I ended up writing a Python script

Can't say I'm a massive fan. Push it and I'll take a look.

bheesham · 2015-09-22T16:58:41Z

No I didn't since this was only for completions. The script is over here: https://github.com/bheesham/rust.fish/blob/master/gen.py

faho · 2015-09-22T18:23:26Z

No I didn't since this was only for completions.

Ah okay, it's for generating completion scripts. I thought you had written a helper that is called at runtime. This is kinda neat, but I'd still like it more if we could generate the completions on the user's machine since they then perfectly match the rust version (since that's apparently possible).

What you basically do here is call rust -C help and rust -W help and bang the output into the shape we want with some regex-magic, right? If so, we usually use sed for that (and I don't like the idea of asking for other utilities unrelated to the command to be installed, even something as ubiquitous as python), but we now also have, hot off the presses, our new cool string tool, with its subcommand string match (basically grep) and string replace (basically sed's "s/PATTERN/REPLACEMENT/").

I could also take a look, but I'll have to download rust first and my current internet connection won't allow for that, so I'll try to get to it over the weekend.

bheesham · 2015-09-23T02:30:45Z

Yep, that's exactly what the script does.

Ah yeah. I saw the pull request for string. I'll refactor some of the code to start using it when I get a chance to.

faho · 2015-10-05T13:26:12Z

@bheesham: Any progress?

bheesham · 2015-10-12T18:01:28Z

Ah crap, nope. This completely escaped my mind for a bit.

I'll pick this issue back up around Thursday.

bheesham · 2015-10-17T04:45:13Z

@faho , I'm now using the string commands. All I'm using the Python script for now is to not have to write complete -c rustc and complete -c cargo.

I've been able to bring the filesize of rustc.fish down from 28.3kb to 2.66kb!

faho · 2015-10-20T15:16:54Z

How well does this perform? It does quite a lot of math calls.

bheesham · 2015-10-21T06:51:26Z

Not too well I think.

The current version (run the rustc -{Z,W,C} help command, parse output, iterate) sources in around 390ms on my laptop. The previous version which used Python to generate the completions sources in about 9ms.

I guess my next challenge is to find a way to generate completions without heavily relying on math.

bheesham · 2015-10-24T14:59:21Z

Cool. The time it takes to source is now significantly less than what it was before.

It's around 88ms for me. Still not as fast as file Python generates, but hey, it's a major improvement.

bheesham · 2015-10-24T15:02:50Z

I did run into a strange problem though.

rustc -W help | string replace -r -i '(\s+)(.+)(\s+)(allow|warn|deny|forbid)(\s+){2}([^\n]+)' '$2 $6' outputs the wrong thing. It keeps in the header and footer of the help message, which is weird.

Output:


Available lint options:
    -W <foo>           Warn about <foo>
    -A <foo>           Allow <foo>
    -D <foo>           Deny <foo>
    -F <foo>           Forbid <foo> (deny, and deny all overrides)


Lint checks provided by rustc:

                             name  default  meaning
                             ----  -------  -------
box-pointers  use of owned (Box type) heap memory
fat-ptr-transmutes  detects transmutes of fat pointers
missing-copy-implementations  detects potentially-forgotten implementations of `Copy`
missing-debug-implementations  detects missing implementations of fmt::Debug
missing-docs  detects missing documentation for public members
trivial-casts  detects trivial casts which could be removed
trivial-numeric-casts  detects trivial casts of numeric types which could be removed
unsafe-code  usage of `unsafe` code
unstable-features  enabling unstable features (deprecated. do not use)
unused-extern-crates  extern crates that are never used
unused-import-braces  unnecessary braces around an imported item
unused-qualifications  detects unnecessarily qualified names
unused-results  unused result of an expression in a statement
variant-size-differences  detects enums with widely varying variant sizes
dead-code  detect unused, unexported items
deprecated  detects use of #[deprecated] items
drop-with-repr-extern  use of #[repr(C)] on a type that implements Drop
improper-ctypes  proper use of libc types in foreign modules
non-camel-case-types  types, variants, traits and type parameters should have camel case names
non-shorthand-field-patterns  using `Struct { x: x }` instead of `Struct { x }`
non-snake-case  methods, functions, lifetime parameters and modules should have snake case names
non-upper-case-globals  static constants should have uppercase identifiers
overflowing-literals  literal out of range for its type
path-statements  path statements with no effect
plugin-as-library  compiler plugin used as ordinary library in non-plugin crate
private-no-mangle-fns  functions marked #[no_mangle] should be exported
private-no-mangle-statics  statics marked #[no_mangle] should be exported
raw-pointer-derive  uses of #[derive] with raw pointers are rarely correct
stable-features  stable features found in #[feature] directive
unconditional-recursion  functions that cannot return without calling themselves
unknown-lints  unrecognized lint attribute
unreachable-code  detects unreachable code paths
unused-allocation  detects unnecessary allocations that can be eliminated
unused-assignments  detect assignments that will never be read
unused-attributes  detects attributes that were not used by the compiler
unused-comparisons  comparisons made useless by limits of the types involved
unused-features  unused or unknown features found in crate-level #[feature] directives
unused-imports  imports that are never used
unused-must-use  unused result of a type flagged as #[must_use]
unused-mut  detect mut variables which don't need to be mutable
unused-parens  `if`, `match`, `while` and `return` do not need parentheses
unused-unsafe  unnecessary use of an `unsafe` block
unused-variables  detect variables which are not used in any way
warnings  mass-change the level for lints which produce warnings
while-true  suggest using `loop { }` instead of `while true { }`
exceeding-bitshifts  shift exceeds the type's number of bits
mutable-transmutes  mutating transmuted &mut T from &T may cause undefined behavior
no-mangle-const-items  const items will not have their symbols exported
unknown-crate-types  unknown crate type found in #[crate_type] directive


Lint groups provided by rustc:

         name  sub-lints
         ----  ---------
    bad-style  non-camel-case-types, non-snake-case, non-upper-case-globals
       unused  unused-imports, unused-variables, unused-assignments, dead-code, unused-mut, unreachable-code, unused-must-use, unused-unsafe, path-statements


Compiler plugins can provide additional lints and lint groups. To see a listing of these, re-run `rustc -W help` with a crate filename.

Any ideas? I'm using fish, version 2.2.0-378-g573d972.

faho · 2015-10-24T15:13:35Z

outputs the wrong thing. It keeps in the header and footer of the help message, which is weird.

Well, it looks like it just doesn't match on them, so it replaces nothing - the lines appear unchanged.

faho · 2015-10-24T15:14:19Z

8000

It's around 88ms for me. Still not as fast as file Python generates, but hey, it's a major improvement.

That probably warrants the technical term "good enough" - unless you're running a monster rig.

faho · 2015-10-24T16:34:24Z

Okay, I tested it a bit and played a bit around, and while there's still plenty of micro-optimizations possible (e.g. the recurring | egrep -v '^$' | egrep -v ':$' can be replaced by | string match -r '^.*[^:]$' - which saves a total of 6 grep calls when setting up the completion), but for me it's basically instantaneous anyway.

The semantics of egrep and string (i.e. pcre2) seem to be different enough that the egrep | string replace in the rust -W help part can't be replaced with string match -r directly, which is unfortunate, but in the end it doesn't really matter (we've tried so hard, and got so far).

So unless you think there's anything missing, this seems about ready to merge. Okay?

rustc.fish makes heavy use of the `string` commands.

bheesham · 2015-10-27T01:40:11Z

Sounds like a plan. :-)

Thanks for all the pointers!

faho · 2015-10-27T22:25:06Z

Merged, thanks!

faho self-assigned this Sep 26, 2015

Add cargo and rustc completion.

46fbfaa

rustc.fish makes heavy use of the `string` commands.

faho merged commit 46fbfaa into fish-shell:master Oct 27, 2015

zanchey added this to the next-2.x milestone Oct 30, 2015

zanchey added the completions label Oct 30, 2015

github-actions bot locked as resolved and limited conversation to collaborators Apr 17, 2020

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Add cargo and rustc completion.#2409

Add cargo and rustc completion.#2409
faho merged 1 commit intofish-shell:masterfrom
bheesham:rustc-cargo-completion

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Uh oh!

Conversation

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants