Danek Duvall wrote:
On Thu, Mar 12, 2009 at 08:31:11PM -0600, Clay Baenziger wrote:
Is anyone else seeing this failure after moving past 940 or did I miss
something? I think I followed the instructions for the flag day with
regards to removing the index directory, however, I don't have any idea why
my import would have broken. Any ideas or suggestions how to debug this?
Shawn just ran into this as well. It looks like it's being triggered by
creating a file action that has two paths. The file action init code now
assumes that the path attribute is a string -- ie, has only one value --
and keels over. The change Brock made there is obviously where it's dying,
and it probably should be more robust.
Yeah, it should probably be more robust, but was I wrong in assuming
that a) file/directory/link actions only are allowed to have 1 path and
b) that path should be a string?
Just trying to figure out whether the problem with my change is that
it's not robust to bad input, or if it's removed functionality we
actually want.
Thanks,
Brock
However, I'm perplexed as to how we're getting a second path attribute in
the first place. Perhaps this is due to chattr?
If you print out self.attrs["path"] in the file action __init__() method,
what do you see? You might need to do this:
if "path" in self.attrs and isinstance(self.attrs["path"], list):
print self.attrs["path"]
so as not to see a whole bunch of garbage. Or just wrap the line with the
lstrip() with a try/except AttributeError clause, and print out the
attribute (and re-raise).
Danek
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
