cargo icon indicating copy to clipboard operation
cargo copied to clipboard

Warn or document conflicts between strip and debug option

Open SUPERCILEX opened this issue 3 years ago • 2 comments

If a user has strip enabled for their release builds and temporary needs to add debug info to those builds, there's no indication that strip will just remove anything added by debug = true. Should this be documented or should a warning be generated if strip and debug are mixed? If a warning, split_debuginfo needs to be handled.

See https://github.com/rust-lang/rust/pull/95964#issuecomment-1097278317

SUPERCILEX avatar Apr 15 '22 00:04 SUPERCILEX

Hi, and sorry for letting you confused 🙇🏾. My initial intent is like "adding both warning and documentation looks good to me".

Anyway, just like what ehuss said, the cargo team agrees with adding a warning. He also wrote a guide you can take as a reference: https://github.com/rust-lang/rust/pull/95964#issuecomment-1097278317 . Feel free to ask if you have any question :)

weihanglo avatar Apr 15 '22 03:04 weihanglo

adding both warning and documentation looks good to me

That's clear, thanks! :) Hopefully I'll have some time next week to work on this.

SUPERCILEX avatar Apr 15 '22 18:04 SUPERCILEX