I noticed there's a new proc, "site_node::closest_ancestor_package". Should I go ahead and modify that one instead, and mark the old one as deprecated? If so, while I'm at it, I think I should make the new proc's -url parameter default to [ad_conn url] instead of "/", no?
