Hi all,
Can I get a review of the following webrev:
http://cr.opensolaris.org/~kemitche/6999599/
Which addresses CRs:
6999699 <http://bugs.opensolaris.org/bugdatabase/view_bug.do?bug_id=6999699>
Missing shebang lines in python modules leads to incorrect dependency generation
6988338 <http://bugs.opensolaris.org/bugdatabase/view_bug.do?bug_id=6988338>
install/installadm has duplicate dependencies on library/python-2/lxml-26
Since a lot of files were touched, I'll summarize the changes - all
python files in our gate that do not start with a shebang
("#!/usr/bin/python2.6") have had the shebang added. While this change
is in conflict with
https://defect.opensolaris.org/bz/show_bug.cgi?id=11380, it is necessary
to ensure proper dependency generation via pkgdepend. If at some point
in time, pkgdepend does not rely on the shebang to understand that a
file is a Python module, then the shebangs could be removed for our
python files that are non-executable. (No other changes to .py files
were made in this changeset). Note that once this goes back, slim_source
devs will need to ensure that any new Python files they add to the gate
also have a shebang (I will send a heads-up to this effect separately,
once this fix goes back).
The two *.mf files that were modified address the second CR (and related
issues); I removed explicit dependencies from manifests where I found
that pkgdepend was able to automatically generate them.
Thanks,
Keith
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
