ok setup_logger really does belong in common