Bart Smaalders wrote:
in manifest.py you're renaming actions named "authority" to
"publisher"....

This won't do what you want, I think; did you want to redo actions
from this:

set name=authority value=contrib.opensolaris.org

to

set name=publisher value=contrib.opensolaris.org

?

Doh!

This should work better:

diff -r f97bd576aebb src/modules/manifest.py
--- a/src/modules/manifest.py   Thu Mar 05 19:57:19 2009 -0600
+++ b/src/modules/manifest.py   Thu Mar 05 22:34:21 2009 -0600
@@ -319,9 +319,10 @@
                                 e.fmri = self.fmri
                                 raise

-                        if action.name == "authority":
+                        if action.name == "set" and \
+                            action.attrs["name"] == "authority":
                                 # Translate old action to new.
-                                action.name = "publisher"
+                                action.attrs["name"] = "publisher"

                         if action.attrs.has_key("path"):
np = action.attrs["path"].lstrip(os.path.sep)

...and valid_manifest doesn't complain, so I assume this is correct. I have no idea how to test for this (i.e. ensuring that the client doesn't trigger valid_manifest failure case).

More tomorrow...

Thank you,
--
Shawn Walker
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

Reply via email to