HPCC-11492 Refactor / Cleanup AddPackageMap

Signed-off-by: Anthony Fishbeck <anthony.fishbeck@lexisnexis.com>
By default he ids have the target name pre-pended as a scope modifier... see onAddPackage  So there would be no sharing of actual packagemap instances.  The user can force sharing of instances, but then the same id is expected to have entries in multiple packageSets... and the user has to be wary of over writing shared information.