Skip to content

workspaces: All building all members of the workspace #3491

Closed

Description

@sdroege

Something like "cargo build" in the root of the workspace would seem useful, which would then build all members of the workspace. Similar to how "cargo clean" currently cleans the workspace.

Activity

  1. steveklabnik commented on Jan 4, 2017

    @steveklabnik
    Contributor

    This should be --all, in my understanding

  2. alexcrichton commented on Jan 6, 2017

    @alexcrichton
    Member

    Yeah we've got cargo test --all now on Nightly so it should be easy enough to add cargo build --all!

  3. sdroege commented on Jan 6, 2017

    @sdroege
    ContributorAuthor

    Basically something similar to 62c0497 then, just doing the same for build? Are there any other commands that could use some --all'ification?
    I might take a look into that later, would be nice to replace my Makefile doing just that with a cargo command

  4. alexcrichton commented on Jan 6, 2017

    @alexcrichton
    Member

    Now that all of the support was landed with cargo test --all it may just involve copying these lines into build.rs. We could add cargo doc as well, although I'd want to just tread purposefully here to ensure we don't expand it too quickly

  5. sdroege commented on Jan 6, 2017

    @sdroege
    ContributorAuthor

    Thanks, it was almost just that (I've submitted a pull request). For "cargo doc" a little bit more is needed, but if you think that's useful to have (it seems useful to me), I can prepare a pull request for that too.

  6. alexcrichton commented on Jan 7, 2017

    @alexcrichton
    Member

    cargo doc --all sounds great to me! I'm sure it'll be requested sooner rather than later :)

  7. sdroege commented on Jan 7, 2017

    @sdroege
    ContributorAuthor

    Ok, I'll get to that once "build --all" is ready to be merged

  8. added a commit that references this issue on Jan 9, 2017
  9. frewsxcv commented on Jan 9, 2017

    @frewsxcv
    Contributor

    Can this be closed now?

  10. frewsxcv commented on Jan 9, 2017

    @frewsxcv
    Contributor

    Should the default Travis CI Rust script use --all by default once this lands in stable?

  11. sdroege commented on Jan 9, 2017

    @sdroege
    ContributorAuthor

    I would wait until #3515 landed before closing

  12. added a commit that references this issue on Jan 9, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    E-easyExperience: Easy

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions