People
Activity
  • Apr 04 21:28
    lavifb commented #7
  • Apr 04 21:15
    soc commented #7
  • Apr 04 21:08
    lavifb commented #7
  • Apr 04 20:56
    soc commented #7
  • Apr 04 20:56
    soc commented #7
  • Apr 04 20:56
    soc commented #7
  • Apr 04 20:55
    soc commented #7
  • Apr 04 20:54
    soc commented #7
  • Apr 04 20:53
    lavifb commented #7
  • Apr 04 20:14
    sharkdp commented #7
  • Apr 04 20:14
    sharkdp commented #7
  • Apr 04 20:12
    sharkdp commented #7
  • Apr 04 12:26

    killercup on gh-pages

    Deploy rust-lang-nursery/cli-wg… (compare)

  • Apr 04 12:26
    spacekookie commented #124
  • Apr 04 12:23
    Dylan-DPC synchronize #58
  • Apr 04 12:23

    Dylan-DPC on master

    Updated chat with us link to in… Merge pull request #124 from de… (compare)

  • Apr 04 12:23
    Dylan-DPC closed #124
  • Apr 04 12:22
    Dylan-DPC commented #124
  • Mar 31 12:46
    spacekookie commented #125
  • Mar 31 12:36
    soc commented #7
Screwtapello
@Screwtapello
Think of the default impls as more like std::io::Error—yes, it would be very limiting if you had to write your whole program using that as your only error representation, but it's only designed for a narrow class of errors and the expectation is that non-trivial programs will write their own anyway.
Pascal Hertleif
@killercup
Dylan DPC
@Dylan-DPC
:+1:
Bastien Orivel
@Eijebong
They should be small, portable, and quick to run
I'm not sure about that :p
The quick to run part
Also ffmpeg is neither small nor fast but I'd still consider it a CLI
Kevin K.
@kbknapp
I'd tend to agree with @Eijebong :P there are quite a few CLIs that are both large and slowish becoming popular (git, docker, lxc, ffmpeg, I would even venture to say cargo and rustup edging on the larger side)
I love how the book tutorial is looking though!
Pascal Hertleif
@killercup

quick to run

oh, yeah, i meant "can in theory start and finish everything in less than 50ms", so more like "doesn't take 5s to start up an interpreter"

Pascal Hertleif
@killercup
@epage great docs you added to assert_cmd!
(i was really surprised to see this philosophical line, though! but you are right, the assert_cli we have now is not the assert_cli we had before ;))
Pascal Hertleif
@killercup
anyone knows what these so-called signals are and how to "control C" (i only know rust)? made a issue to start a draft of a signals chapter to collect some knowledge and best practices. anyone want to start this? doesn't have to be much at first rust-lang-nursery/cli-wg#50
Ed Page
@epage
Oh that is a fun typo
Pascal Hertleif
@killercup
will rebase and comment in bit, need to go to the airport first
Katharina
@spacekookie
Hey btw @kbknapp seeing how there's a clap-rs organisation now I've been thinking about moving thunder away from my personal account and to a group. Wondering if rust-clique or clap-rs would be more appropriate there
Dylan DPC
@Dylan-DPC
i feel rust-clique is better
Kevin K.
@kbknapp
@spacekookie I'm fine either way. I've sent out the invites to the org, and all who have accepted should have rights to create and transfer repos there
Pascal Hertleif
@killercup
@kbknapp i have the same question but for the clap-{verbosity,port}-flag crates :)
Kevin K.
@kbknapp
I'm more than happy to have them in the org! Obviously I can't make anyone transfer a personal project, but if the project is meant to be officially used with clap I'd like it if they are co-located for ease :)
Pascal Hertleif
@killercup
great! @yoshuawuyts what do you think?
Yoshua Wuyts
@yoshuawuyts
killercup: fine with me
Pascal Hertleif
@killercup
@kbknapp do you think we can manage to get a clap 3 preview in 2 days?
for the big rust 2018 preview release post?
Kevin K.
@kbknapp
I was hoping so, but I don't know. clap v3 builds, and even works with the structopt import (well after a quick fix I push tonight), but there is one major change left that I'm in the middle of doing which is probably the biggest API change (moving away from the stringly typed args) that isn't ready yet. So we could but people would need to note that major change still coming down the pipe
Pascal Hertleif
@killercup
@kbknapp i think that'd be okay for an alpha release -- unless you think it will require very many non-mechanical changes for people?
Kevin K.
@kbknapp
The goal is it's still almost compatible with v2
However if you return an Arg from a function the signature goes from Arg<'_, '_> to Arg<'_, T>
Pascal Hertleif
@killercup
@kbknapp okay, that's not too bad, is it? i'd say "go for it" and make an alpha release, and get all the feedback and publicity from the 2018 edition announcements
Pascal Hertleif
@killercup
Pascal Hertleif
@killercup
any want to review draft for chapters on exit codes rust-lang-nursery/cli-wg#61 and using human panic? rust-lang-nursery/cli-wg#60
Dylan DPC
@Dylan-DPC
@killercup left a review :)
Pascal Hertleif
@killercup
Thanks!
Kevin K.
@kbknapp
clap has been moved to the clap-rs org, and is ready for the alpha.1 release
I still need to write a blog about what all has changed, how to test, and what's still left to do...but at least this part is done
It's late here and I have a Rust Reach meeting early in the morning, so more to follow on this topic tomorrow :wink:
Pascal Hertleif
@killercup
@kbknapp Awesome! Let me know if there's any last minute testing or blog post reviewing to be done!
I'll
Pascal Hertleif
@killercup
@epage I'll have a look at the currently open assert-rs PRs once I get to the office. Maybe we can write some kind of announcement for the release, too.
I'm not sure when the Edition Preview announcement will go out… but I have some time to merge/review/write about stuff this morning :)
Ed Page
@epage

Hey, anyone up for some API brainstorming?

assert_cmd has an API for running Rust bins, see docs

  • People want to expand it to pass features, see assert-rs/assert_cmd#34
  • There is a tension in whether to pass --target between hurting the performance of a naive case or hurting the performance of a non-naive case and misleadingly reducing their coverage, see assert-rs/assert_cmd#4

I like how simple the current API is and would love to maintain it while adding features but it seems unlikely to happen. Instead, the best I can figure is we need to change the return type to some kind of builder that gives the user control over this stuff.

Pascal Hertleif
@killercup
@epage hm, building with specific features enabled could be quite a common thing. any chance to an easy api with escargot? i'd rather not add that to assert_cmd as it's quite cargo specific
Ed Page
@epage
Understandable but escargot provides an API that hides Command like cargo_metadata
Pascal Hertleif
@killercup
i was thinking Command::new(cargo::bin("foo").features(&["extras"]).build_and_get_path()?)
actually there might be another problem -- unless we do some really clever stuff, the randomized test order might require us to recompile this for every test
Ed Page
@epage
Won't it only be a one-time compile for every unique configuration because the artfiacts are kept in hashed directories?
Pascal Hertleif
@killercup
@epage if that behavior is ensured and cargo isn't allowed to clean up stuff, yes
Ed Page
@epage
@killercup spent some time on thinking on what you said and have written up a proposal, see assert-rs/assert_cmd#44
Pascal Hertleif
@killercup
ICYMI: I opened a bunch of of issues labeled 'book' to add/improve sections (most are the todos I added while writing the tutorial). feel free to discuss concrete content ideas and open new ones for sections you'd like to see: https://github.com/rust-lang-nursery/cli-wg/issues?q=is%3Aissue+is%3Aopen+label%3Abook
basically, the idea is to have these issues be concrete action items extracted from the tracking issues