All (especially Clay and Ethan),

Here is another webrev that I need reviewed:

        http://cr.opensolaris.org/~johnfisc/12724-manifest-names

This is for:

     http://defect.opensolaris.org/bz/show_bug.cgi?id=12724

The AI Database code stores manifest names as filenames (with the
extension .xml).  These change adds an optional parameter
(return_fnames) to the getManNames function in the AI_database.py
with the default value of True.  Thus the code generates the original
returned filenames (manifest-name.xml).  The installadm list
subcommand code sets this optional parameter to False and gets the
.xml extension stripped off (the manifest name instead of the
filename).

I believe this approach is better then changing the internal database
storage of the manifests as the rest of the webserver code assumes the
manifest filename and not the manifest name.

Thanks,

John

Reply via email to