<div dir="ltr">If you upload a set of packages for 1 Solaris release only, all is good. If you upload with something like "csw-upload-pkg * " and in your set of packages there is a mix of 5.9 and 5.10 packages, the uploader will doubly assign packages to 5.10 and 5.11 catalogs.<div>

<br></div><div>This logic needs to be corrected. This link points to the specific place of code with a comment describing the issue and suggests which way to go:</div><div><br></div><div><a href="https://buildfarm.opencsw.org/source/xref/opencsw/csw/mgar/gar/v2/lib/python/csw_upload_pkg.py#163">https://buildfarm.opencsw.org/source/xref/opencsw/csw/mgar/gar/v2/lib/python/csw_upload_pkg.py#163</a><br>

</div><div><br></div><div><div>There is also a set of tests:</div><div><a href="https://buildfarm.opencsw.org/source/xref/opencsw/csw/mgar/gar/v2/lib/python/csw_upload_pkg_test.py">https://buildfarm.opencsw.org/source/xref/opencsw/csw/mgar/gar/v2/lib/python/csw_upload_pkg_test.py</a></div>

</div><div><br></div><div>As usual, I'm offering advice and code reviews; if you mail me your patch before committing, I'll reply with a review.</div><div><br></div><div>It's easy to test your changes, you just modify the file in your buildtree (~/opencsw/.buildsys) and run it.</div>

<div><br></div><div>Or you can rewrite the whole thing in Perl. ;-)</div><div><br></div><div>Any takers?</div><div><div><br></div><div>Maciej</div></div></div>