On 04/29/10 10:54 AM, johan...@opensolaris.org wrote:
Hi Shawn,
Thanks for looking at this.
On Thu, Apr 29, 2010 at 12:32:22AM -0500, Shawn Walker wrote:
On 04/26/10 07:36 PM, johan...@opensolaris.org wrote:
Webrev is available here:
http://cr.opensolaris.org/~johansen/webrev-6957/
src/depot.py:
line 149, 219-221, 303-304, 703:
nit; options should be in asciibetical order :)
Ok, fixed. In some places these options were not in alphabetical order
to begin with, which is why I assumed it was okay to add them wherever I
pleased. I've tried to put those places back into complete order.
Perhaps we could enforce this consistently to avoid this kind of
confusion in the future?
I try my best to be certain that these are in a consistent order as it
benefits the user, especially in documentation. However, like most
things, bitrot occurs ;)
...
What happens when you try to access the BUI index page for a
repository that has no cfg_cache file or that doesn't provide
a publisher prefix and is running in --mirror or --llmirror mode?
Right now I get:
404 Not Found
The path '/' was not found.
Is there a way to disable the BUI for this mode? There's not really
anything to display to human users in the mirror case.
No; the 404 is purposefully raised to actual users to prevent accidental
exposure of paths or other information we wouldn't want just anyone to
see. I think there's an RFE to add a debug option though to make
development a bit more convenient.
With that said; you should see something in the error logs (on the
console) where you started the depot server.
Cheers,
-Shawn
_______________________________________________
pkg-discuss mailing list
pkg-discuss@opensolaris.org
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
