'sub new' constructor in Source and Dest p4.pm fixed so parse_p4_repo_spec
only called when a $spec is provided to the constructor. parse_p4_repo_spec
now also sets the repo_id. parse_repo_spec (TODO item) no longer returns
a hash value of the values parsed, it only sets fields in $self. Fixed a few
places where that return hash was used.