Add $CARGO_PKG_AUTHORZ that separates the entries with \x01 instead of :

[nabijaczleweli]

What does this PR try to resolve?

Valid and reasonable authors entries like

authors = ["наб <[email protected]>",
           "nevsal",
           "Lynnesbian <https://fedi.lynnesbian.space/@lynnesbian>"]

become

наб <[email protected]>
nevsal
Lynnesbian <https
//fedi.lynnesbian.space/@lynnesbian>

with the standard .replace(':', "\n") treatment.

There's no reason to use : instead of something that's guaranteed to never be in the string. \0 is unusable, so use \1.

How should we test and review this PR?

$ cat src/main.rs
#[macro_use]
extern crate clap;
fn main() {
    println!("{}", env!("CARGO_PKG_AUTHORS"));
    println!("{}", env!("CARGO_PKG_AUTHORZ"));
    println!("{}", crate_authors!("\n"));
    println!("{}", env!("CARGO_PKG_AUTHORZ").replace('\x01', "\n"));
}

$ cat Cargo.toml
[package]
name = "b"
version = "0.1.0"
authors = ["наб <[email protected]>",
           "nevsal",
           "Lynnesbian <https://fedi.lynnesbian.space/@lynnesbian>"]

[dependencies.clap]
version = "3.2"
features = ["cargo"]

$ ../cargo/target/debug/cargo run
наб <[email protected]>:nevsal:Lynnesbian <https://fedi.lynnesbian.space/@lynnesbian>
наб <[email protected]>nevsalLynnesbian <https://fedi.lynnesbian.space/@lynnesbian>
наб <[email protected]>
nevsal
Lynnesbian <https
//fedi.lynnesbian.space/@lynnesbian>
наб <[email protected]>
nevsal
Lynnesbian <https://fedi.lynnesbian.space/@lynnesbian>

$ ../cargo/target/debug/cargo run | cat -A
warning: no edition set: defaulting to the 2015 edition while the latest is 2024
    Finished `dev` profile [unoptimized + debuginfo] target(s) in 0.21s
     Running `target/debug/b`
M-PM-=M-PM-0M-PM-1 <[email protected]>:nevsal:Lynnesbian <https://fedi.lynnesbian.space/@lynnesbian>$
M-PM-=M-PM-0M-PM-1 <[email protected]>^Anevsal^ALynnesbian <https://fedi.lynnesbian.space/@lynnesbian>$
M-PM-=M-PM-0M-PM-1 <[email protected]>$
nevsal$
Lynnesbian <https$
//fedi.lynnesbian.space/@lynnesbian>$
M-PM-=M-PM-0M-PM-1 <[email protected]>$
nevsal$
Lynnesbian <https://fedi.lynnesbian.space/@lynnesbian>$

Additional information

Downstream report: nabijaczleweli/cargo-update#294

[rustbot]

r? @weihanglo

rustbot has assigned @weihanglo.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

[epage]

Note that the package.authors field is deprecated as of 1.86 in #15068.

In light of that, I would be inclined to not add new features to better support this (in fact existing feature requests helped raised the visibility for the deprecation discussion).