Michal Pryc wrote:
Padraig,

repository.py:
I thought that the convention for statements which exceed one line was that each continuation line was indented 4 spaces from original line. This is not so at lines 87, 102, 106, 108, 110, 112, 127, 129, 131, 133, 164, 167, 170, 173, 176, 179, 195, 197.
Fixed, also there were 8 spaces in few other places such as dictionary with signals.
Is there a bug logged for reference to BUG at line 431?
Thanks for loggin one: http://defect.opensolaris.org/bz/show_bug.cgi?id=11715

Is there a bug or RFE logged about XXX at line 655?
Because we don't have publisher API yet, I had to copy this line from

No, we have a publisher API.  You're using it.

the client.py. This information was there. I have changed it now to:
# This part is copied from "def publisher_set(img, args)" # from the client.py as the publisher API is not ready yet.

Are you fine with this?

No, this isn't "copied"; that is the publisher API. The only thing that is missing is the client configuration supporting storage of cert information at the origin / mirror level. So, the client has to workaround some things right now to deal with that.

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

Reply via email to