crate name in log map is taken from name of output binary, not the crate name

[jesse99]

I can only get error level logging using the project below, I've tried a bunch of versions of RUST_LOG that I think should work and none do (which is complicated by the fact that rustc --ls is still broken). I've tried:

RUST_LOG=logging2::crate::logging2=3
RUST_LOG=::logging2=3
RUST_LOG=3

and more.

Makefile:

RUSTC ?= rustc

dummy1 := $(shell mkdir bin 2> /dev/null)

check:
    $(RUSTC) -o bin/test-logging2 --test crate.rc
    export RUST_LOG=logging2::crate::logging2=3 && ./bin/test-logging2

clean:
    rm -rf bin

crate.rc:

use std;

#[link(name = "logging2",
       vers = "0.1",
       uuid = "91E62841-B9F3-4721-97C6-621BEEE2DA1B")]

mod logging2;

logging2.rs:

#[test]
fn test_logging()
{
    error!("error level");
    warn!("warn level");
    info!("info level");
}

This was with a rustc from master on Sep 1, 2012. Note that I was able to get logging working when compiling a stand-alone rs file.

[brson]

This works with RUST_LOG=test-logging2 (the name of the output binary). I'd say this is a bug and we should be using the actual name of the crate.

[jesse99]

Note that 10.0.4 of the reference manual says that the crate name is from the link attribute.

[graydon]

non-critical for 0.6, de-milestoning

[catamorphism]

Nominating for milestone 5, production-ready. With the new version of rustpkg, the crate name doesn't have to be explicitly specified in a link attribute, so that should be taken into account too (in the absence of an explicit crate name, the crate name for logging purposes should be what-rustpkg-would-infer).

[graydon]

sub-bug of #3309

[graydon]

accepted for production-ready milestone

[pnkfelix]

visiting for triage, email from 2013 sep 23.

[alexcrichton]

This is actually currently implemented. The test case above has the attribute not as a crate attribute. The #[link] above is placed on the mod instead of on the crate because of the missing semicolon at the end, and it wouldn't be a crate attribute if it were after the use std; anyway.

Regardless, we do actually use the crate name specified in link attributes as the prefix for logging (maybe this changed between then and now). Closing.