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

Add value OUT_DIR to build-script-executed JSON message #7622

Merged
merged 2 commits into from
Nov 23, 2019

Conversation

matklad
Copy link
Member

@matklad matklad commented Nov 22, 2019

The target audience here is IDE authors, who can use this feature to
better support crates which generate code to OUT_DIR

@matklad
Copy link
Member Author

matklad commented Nov 22, 2019

r? @ehuss cc @vlad20012

@rust-highfive
Copy link

r? @alexcrichton

(rust_highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive assigned ehuss and unassigned alexcrichton Nov 22, 2019
@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Nov 22, 2019
@matklad
Copy link
Member Author

matklad commented Nov 22, 2019

r? @ehuss

@ehuss
Copy link
Contributor

ehuss commented Nov 22, 2019

Looks good. Can you also update the documentation at https://github.com/rust-lang/cargo/blob/master/src/doc/src/reference/external-tools.md#build-script-output?

The target audience here is IDE authors, who can use this feature to
better support crates which generate code to OUT_DIR
@matklad
Copy link
Member Author

matklad commented Nov 22, 2019

Added docs!

@ehuss
Copy link
Contributor

ehuss commented Nov 23, 2019

Thanks!

@bors r+

@bors
Copy link
Collaborator

bors commented Nov 23, 2019

📌 Commit a87aceb has been approved by ehuss

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Nov 23, 2019
@bors
Copy link
Collaborator

bors commented Nov 23, 2019

⌛ Testing commit a87aceb with merge b505e8a...

bors added a commit that referenced this pull request Nov 23, 2019
Add value OUT_DIR to build-script-executed JSON message

The target audience here is IDE authors, who can use this feature to
better support crates which generate code to OUT_DIR
@bors
Copy link
Collaborator

bors commented Nov 23, 2019

☀️ Test successful - checks-azure
Approved by: ehuss
Pushing b505e8a to master...

@bors bors merged commit a87aceb into rust-lang:master Nov 23, 2019
bors added a commit to rust-lang/rust that referenced this pull request Nov 25, 2019
Update cargo, rls, books.

## nomicon

1 commits in 58e36e0e08dec5a379ac568827c058e25990d6cd..041c46e692a2592853aeca132c8dfe8eb5a79a9e
2019-10-30 08:14:24 -0500 to 2019-11-20 16:46:45 +0100
- Update unsafe-code-guidelines link (rust-lang/nomicon#175)

## cargo

15 commits in 8280633db680dec5bfe1de25156d1a1d53e6d190..750cb1482e4d0e74822cded7ab8b3c677ed8b041
2019-11-11 23:17:05 +0000 to 2019-11-23 23:06:36 +0000
- Some random comments and docstrings. (rust-lang/cargo#7625)
- Add value OUT_DIR to build-script-executed JSON message (rust-lang/cargo#7622)
- Update documentation for custom target dependencies. (rust-lang/cargo#7623)
- Document private items for binary crates by default (rust-lang/cargo#7593)
- Extend documentation on security concerns of crate names in a registry. (rust-lang/cargo#7616)
- Stabilize install-upgrade. (rust-lang/cargo#7560)
- Turn the new lock file format on by default (rust-lang/cargo#7579)
- bump im-rc version (rust-lang/cargo#7609)
- Ignore file lock errors if unsupported, on Windows (rust-lang/cargo#7602)
- Add hack for fwdansi change. (rust-lang/cargo#7607)
- Document Cargo's JSON output. (rust-lang/cargo#7595)
- Remove "cargo login" from user input when asking for login token. (rust-lang/cargo#7588)
- Fix all Clippy suggestions (but not add it to CI 🙃) (rust-lang/cargo#7574)
- Add kind/platform info to `cargo metadata` (rust-lang/cargo#7132)
- Update core-foundation requirement from 0.6.0 to 0.7.0 (rust-lang/cargo#7585)

## reference

2 commits in 45558c4..9e843ae
2019-11-08 14:47:35 +0100 to 2019-11-24 17:44:04 +0100
- Minor never type additions. (rust-lang/reference#723)
- Update associated-items.md.  "it"->is (rust-lang/reference#721)

## book

3 commits in e79dd62aa63396714278d484d91d48826737f47f..81ebaa2a3f88d4d106516c489682e64cacba4f60
2019-10-30 07:33:12 -0500 to 2019-11-15 08:30:04 -0800
- small fix ch04-03 & code block typo ch07-02 (rust-lang/book#2138)
- Adapt content of Chapter 16.3 in order to be consistent with improved compiler message (rust-lang/book#1779)
- [Rust 1.35] Remove FnBox and use builtin impl FnOnce for Box<FnOnce()> instead. (rust-lang/book#1906)

## rls

3 commits in 5db91c7b94ca81eead6b25bcf6196b869a44ece0..9ec2b8cb57c87517bcb506ac302eae339ffa2025
2019-10-30 16:04:39 +0100 to 2019-11-24 23:16:11 +0100
- Fix test for latest nightly. (rust-lang/rls#1595)
- doc: contributing: Remove outdated LSP extension (rust-lang/rls#1594)
- Update cargo. (rust-lang/rls#1591)

## rust-by-example

1 commits in dcee312c66267eb5a2f6f1561354003950e29105..4835e025826729827a94fdeb7cb85fed288d08bb
2019-10-31 11:26:53 -0300 to 2019-11-14 09:20:43 -0300
- crates: fix suggested value for --crate-type flag (rust-lang/rust-by-example#1292)

## edition-guide

1 commits in f553fb26c60c4623ea88a1cfe731eafe0643ce34..6601cab4666596494a569f94aa63b7b3230e9769
2019-10-30 08:27:42 -0500 to 2019-11-22 12:08:58 -0500
- Remove final nursery reference
@matklad matklad deleted the out-dir branch November 25, 2019 16:28
Undin added a commit to intellij-rust/intellij-rust that referenced this pull request Dec 6, 2019
Build scripts can generate some compile time items, like `cfg` options, environment variables, etc.
These things can be used by the plugin to provide proper code insight features where these options is used:
conditional compilation, include macro, etc.
In addition, since rust-lang/cargo#7622 build script message contains info about output directory for generated sources.

Under the hood, `cargo check` is used to get the corresponding build script output.
Unfortunately, it compiles whole project instead of build scripts.
So, at this moment it's under `org.rust.cargo.evaluate.build.scripts` experimental feature and disabled by default
Undin added a commit to intellij-rust/intellij-rust that referenced this pull request Dec 6, 2019
Build scripts can generate some compile time items, like `cfg` options, environment variables, etc.
These things can be used by the plugin to provide proper code insight features where these options is used:
conditional compilation, include macro, etc.
In addition, since rust-lang/cargo#7622 build script message contains info about output directory for generated sources.

Under the hood, `cargo check` is used to get the corresponding build script output.
Unfortunately, it compiles whole project instead of build scripts.
So, at this moment it's under `org.rust.cargo.evaluate.build.scripts` experimental feature and disabled by default
Undin added a commit to intellij-rust/intellij-rust that referenced this pull request Jan 2, 2020
Build scripts can generate some compile time items, like `cfg` options, environment variables, etc.
These things can be used by the plugin to provide proper code insight features where these options is used:
conditional compilation, include macro, etc.
In addition, since rust-lang/cargo#7622 build script message contains info about output directory for generated sources.

Under the hood, `cargo check` is used to get the corresponding build script output.
Unfortunately, it compiles whole project instead of build scripts.
So, at this moment it's under `org.rust.cargo.evaluate.build.scripts` experimental feature and disabled by default
Undin added a commit to intellij-rust/intellij-rust that referenced this pull request Jan 29, 2020
Build scripts can generate some compile time items, like `cfg` options, environment variables, etc.
These things can be used by the plugin to provide proper code insight features where these options is used:
conditional compilation, include macro, etc.
In addition, since rust-lang/cargo#7622 build script message contains info about output directory for generated sources.

Under the hood, `cargo check` is used to get the corresponding build script output.
Unfortunately, it compiles whole project instead of build scripts.
So, at this moment it's under `org.rust.cargo.evaluate.build.scripts` experimental feature and disabled by default
@ehuss ehuss added this to the 1.41.0 milestone Feb 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants