Move overrides for registrations to debug

Review Request #2023 — Created April 2, 2015 and submitted

stuhood
pants
14b9e75...
pants-reviews
areitz, patricklaw, zundel

Move overrides for registrations to debug.

These are only useful at INFO if you're worried about accidentally overwriting existing aliases... but even in that case, it's unlikely for overwriting two similar-named but differently-typed to "just work". You kind of have to do it intentionally.

Internally we're overriding both maven_layout and jar... I'd like to upstream the latter soon, but the former has a lot of deps that can't be upstreamed.



PA
  1. Ship It!
  2. 
      
ST
Review request changed

Status: Closed (submitted)

AR
  1. Ship It!
  2. 
      
ZU
  1. I'm concerned that the info messages could be of utility if someone is doing this accidentally. Could there be an override=True or something like that on the second registration where we could indicate that we expect an alias to already exist?

    1. I like override=True, so either you know you're overriding and you force it, or you get an immediate error. Probably we should have something similar for build file aliases, but I think that would complicate the interface quite a bit.

  2. 
      
Loading...